Re: [PATCH V5 RESEND 5/5] venus: pm_helpers: Use dev_pm_genpd_set_hwmode to switch GDSC mode on V6

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 17/06/2024 03:31, Jagadeesh Kona wrote:


On 5/31/2024 5:26 PM, Jagadeesh Kona wrote:


On 5/10/2024 6:31 PM, Bryan O'Donoghue wrote:
On 01/05/2024 10:14, Bryan O'Donoghue wrote:
On 30/04/2024 21:01, Konrad Dybcio wrote:
On 24.04.2024 11:50 AM, Bryan O'Donoghue wrote:
On 24/04/2024 10:45, Jagadeesh Kona wrote:

Thanks Bryan for testing this series. Can you please confirm if this issue is observed in every run or only seen during the first run? Also please let me know on which platform this issue is observed?

Thanks,
Jagadeesh

rb5/sm8250

My observation was on a previous _boot_ the stuttering was worse. There is in the video capture three times that I count where the video halts briefly, I guess we need to vote or set an OPP so the firmware knows not to power-collapse quite so aggressively.

We seem to be having some qualcomm-wide variance on perf/pwr usage on some odd boots.. Any chance you could try like 5 times and see if it was a fluke?

Konrad

Sure.

The first time I tried it, it was much worse.

The second time, captured in the video is only noticeable because I was *looking* for this specific error i.e. I don't think I would have noticed the error on the second run, had I not seen the first run.

I'll find some time to do 5x with and 5x without.

---
bod

ping bod please remember to do this thanks


Hi Bryan, Could you please let me know if you got a chance to check the above? Thank you!


Hi Bryan, Kindly can you please help confirm if this is a real issue or observed as a fluke? so we can go ahead and mainline these changes.

Thanks,
Jagadeesh

So I'm happier with this patchset when I run gstreamer instead of ffmpeg.

There doesn't appear to be a discernable difference between before/after on framerate or subjective UX with/without this set.

gst-launch-1.0 -vvv -e filesrc location=sample-5s.mp4 ! qtdemux ! parsebin ! v4l2h264dec ! autovideosink

Tested-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux