-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Prevent the state doesn't be changed issue with NV driver (Bugfix) #1679
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1679 +/- ##
=======================================
Coverage 49.30% 49.30%
=======================================
Files 373 373
Lines 40454 40455 +1
Branches 6834 6834
=======================================
+ Hits 19946 19947 +1
Misses 19783 19783
Partials 725 725
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
810d44b
to
e13bc31
Compare
This review should be quite straightforward. Please merge this branch to main, since the sru-gate validation issue should be solved there. |
Hey @hanhsuan, I'll need you to rebase your branch on main to make it pass CI. |
e13bc31
to
41ebb55
Compare
@p-gentili I've rebased. Thanks for your help. |
The test case is under developing and will be used to replace |
Description
While
cycle
updating the configuration with NVIDIA (Xorg) devices, it might failed withGDBus.Error:org.freedesktop.DBus.Error.AccessDenied: The requested configuration is based on stale information
, however user could use the gnome control center to change the same display configuration without any error. Therefore, get the current state before_apply_monitors_config
in thecycle
to avoid this issue.Resolved issues
Documentation
Tests
Tested with the same script in #1576 on intel, AMD and NVIDIA devices.