search

NVIDIA / open-gpu-kernel-modules

NVIDIA Linux open GPU kernel module source
Other
15.06k stars 1.25k forks source link

Patches for testing r555 stutter issues #658

Open mtijanic opened 3 months ago

mtijanic commented 3 months ago

This is not a pull request.

Just publishing my side branch here to have a centralized place to test and discuss these patches.

We've been having reports of stutter issues in 555 releases related to GSP enablement. On the proprietary driver, NVreg_EnableGpuFirmware=0 makes them go away; on the open driver that's not an option.

So far, we've identified two possible causes here. One is fixed by commit 674c009 below. The other we can't fix/workaround in the kernel modules and requires usermode changes, but commit 8c1c49b should tell us if that path is actually being hit or not.

I've also augmented the logs captured by nvidia-bug-report.sh with some of the info that we found severely lacking in the bug reports so far.

My hope is that folks that have experienced these stutter issues can take these patches, try to reproduce the issue and report back with their findings (and their nvidia-bug-report logs). Many thanks in advance to anyone willing to go the extra mile(s) for us here!

We've unfortunately missed beta2 / 555.52 with this stuff (security fixes can't wait), but here it is early so we don't have to wait on the next release.

CLAassistant commented 3 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

bkdwt commented 2 months ago

@mtijanic Thanks for this patch. There's a possibility to your update your patch for support 550.100 drivers? 555.100 drivers fix a bug that caused OpenGL triple buffering to behave like double buffering.

Thanks in advance! :)

EDIT: 555.58.02 you're welcome too. ^^

Edgars-Cirulis commented 2 weeks ago

Does that patch is needed for 560

ptr1337 commented 2 weeks ago

Does that patch is needed for 560

They are already merged in 560.