openSUSE / SUSEPrime

Provide nvidia-prime like package for openSUSE
64 stars 11 forks source link

Breaks HDMI audio on Dell Alienware 13R3 #60

Closed tvogel closed 3 years ago

tvogel commented 3 years ago

https://github.com/openSUSE/SUSEPrime/blob/2a2e1a346db966b1a67ebe05438a0e24af380269/90-nvidia-udev-pm-G05.rules#L8

"Silently" disabling this feature is a surprising side-effect. I would very much appreciate if you could note that effect in the documentation and give instructions how to re-enable HDMI audio (maybe sacrificing power-management - but in some cases that might be the easier death to die?).

BTW, on my device and even with Linux 5.10, I still needed the nvhda module but this is a different story...

sndirsch commented 3 years ago

Well, actually it's documented https://github.com/openSUSE/SUSEPrime#nvidia-power-off-support-since-435xxx-driver-with-turing-gpu-and-later-g05-driver-packages

[...] Recreate your initrd with some special settings, which are needed to enable DynamicPowerManagement and remove NVIDIA kernel modules from initrd, so some special udev rules can be applied to disable NVIDIA Audio and NVIDIA USB and make runtime PM for NVIDIA GPU active. This is needed as workaround, since NVIDIA Audio/USB currently cannot be enabled at the same time as NVIDIA GPU DynamicPowerManagement. This is easily done with: [...]

Maybe it's missing a big red warning ...

tvogel commented 3 years ago

Hi,

thanks for your reply but even without taking any of the described customizations of the initrd, just installing the package activates the udev rules that remove the nvidia HDMI audio device. I don't find this behavior documented. And I think, it should be optional or at least an opt-out should be documented.

Best regards,

Tilman Vogel

Unterwegs geschrieben

Stefan Dirsch notifications@github.com schrieb am Di., 23. Feb. 2021, 13:56:

Well, actually it's documented

https://github.com/openSUSE/SUSEPrime#nvidia-power-off-support-since-435xxx-driver-with-turing-gpu-and-later-g05-driver-packages

Recreate your initrd with some special settings, which are needed to enable DynamicPowerManagement and remove NVIDIA kernel modules from initrd, so some special udev rules can be applied to disable NVIDIA Audio and NVIDIA USB and make runtime PM for NVIDIA GPU active. This is needed as workaround, since NVIDIA Audio/USB currently cannot be enabled at the same time as NVIDIA GPU DynamicPowerManagement. This is easily done with:

Maybe it's missing a big red warning ...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/openSUSE/SUSEPrime/issues/60#issuecomment-784182113, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACVI57Z6IJW6GNYIGRU4N3TAOQZLANCNFSM4YBK73LQ .

sndirsch commented 3 years ago

In case you're talking about suse-prime package this documentation is in /usr/share/doc/packages/suse-prime/README.md

tvogel commented 3 years ago

Hi!

I also checked /usr/share/doc/packages/suse-prime/README.md from suse-prime-0.7.15-1.2.noarch :

That document as well es the current README.md from the repository, list the steps to enable blacklisting and disabling HDMI audio as extra steps (copy conf and rules files and then run mkinitrd), the user has to take in order to enable power-off support. However, in contrast, at least the package already places the conf and rules files at the active locations, disabling HDMI audio automatically just by installing the package. That's where my confusion comes from and why I suggest that this shall (again?) be a manual step because, I don't think, one can expect that everybody who installs suse-prime wants to sacrifice HDMI audio support. Of course it's a pity that I cannot benefit from full power saving if I want to use HDMI audio but if that's the case, I see two options:

1) make it a manual choice (what I am suggesting) 2) make it dependent on HDMI cable attachment and disable power-off mode and enable HDMI audio support in that case (that would be really cool but I guess, it's quite complicated to get right?)

Regards,

Tilman

Am Mi., 24. Feb. 2021 um 02:10 Uhr schrieb Stefan Dirsch < notifications@github.com>:

In case you're talking about suse-prime package this documentation is in /usr/share/doc/packages/suse-prime/README.md

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/openSUSE/SUSEPrime/issues/60#issuecomment-784655724, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACVI53ZMKOKFTKSONSWUDDTARGYPANCNFSM4YBK73LQ .

sndirsch commented 3 years ago

Ok. I've improved the documentation now, made a new realease and updated the package. This should help already.

It's easier to reeanble HDMI Audio sound than to enable DynamicPowerManagement. Therefore I want to stay with the latter as default. Whatever you chose as default it will be wrong anyway ...

tvogel commented 3 years ago

Thanks a lot! I will take look!

Am Mi., 24. Feb. 2021 um 12:01 Uhr schrieb Stefan Dirsch < notifications@github.com>:

Closed #60 https://github.com/openSUSE/SUSEPrime/issues/60.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/openSUSE/SUSEPrime/issues/60#event-4370299753, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACVI57R3XWS6OU7TJODNWTTATL6ZANCNFSM4YBK73LQ .