gpu-manager should also support using custom xorg.conf files for Nvidia

This bug was fixed in the package ubuntu-drivers-common - 1:0.4.23

---------------
ubuntu-drivers-common (1:0.4.23) artful; urgency=medium

  [ Alberto Milone ]
  * gpu-manager.{c|py}:
    - Disable persistence mode before powering down the dGPU.
      If persistence mode is not disabled, gpu-manager will not be
      able to unload the nvidia module.
    - Add support for using custom xorg.confs with the nvidia
      driver (LP: #1667198).
      Custom xorg files can be named "non-hybrid" (for non hybrid
      systems), "hybrid-performance", and "hybrid-power-saving",
      and will have to placed in the /usr/share/gpu-manager.d
      directory.
      The directory can be overridden by passing another directory
      along with the "--custom-xorg-conf-path" parameter.
    - Add tests for the custom xorg.confs code and for amdgpu-pro
      hybrid support.
  [ Kai-Heng Feng ]
  * debian/control:
    - Add build dependency on libkmod-dev.
  * gpu-manager.c:
    - Detect and handle switchable graphics with the amdgpu-pro driver.
      AMDGPU-Pro has its own script to switch between powersaving and
      performance mode.
      Some machines allow enabling switchable graphics, or disabling
      the iGPU completely, thus leaving only the dGPU on. This new code
      simply detects the different profiles supported by AMD, and
      enables powersaving mode when possible.

 -- Alberto Milone <email address hidden> Thu, 11 May 2017 11:05:40 +0200

Please accept ubuntu-drivers-common 1:0.4.17.3 in xenial-proposed.

An upload of ubuntu-drivers-common to xenial-proposed has been rejected from the upload queue for the following reason: "pycache cruft left on the tree".

Hello Alex, or anyone else affected,

Accepted ubuntu-drivers-common into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-drivers-common/1:0.4.17.3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

with ubuntu-drivers-common 1:0.4.17.3,
verified /usr/share/gpu-manager.d/hybrid-performance on "Inspiron 15 7000 Gaming" which has Nvidia VGA controller [10de:1c8c], the /etc/X11/xorg.conf file was replaced by /usr/share/gpu-manager.d/hybrid-performance as expected.

Has this feature been added to Zesty which is / will be a supported upgrade path from Xenial?

hi Brian,
Could you please give some more detail about question in #6?

from bug description, this change has been landed to Artful.
So, what do we need to let this change be landed to Xenial also?

When Ubuntu 16.10 becomes end of life next week, people will be able to upgrade from Ubuntu 16.04 to Ubuntu 17.04. I gather that this feature exists in Ubuntu 17.10 (artful) and that you want to add it to Ubuntu 16.04 (xenial), however if someone were to upgrade from Ubuntu 16.04 to Ubuntu 17.04 they would they experience a regression because the feature does not exist in Ubuntu 17.04. Given that I think the feature should also be added to Ubuntu 17.04 (zesty).

I'm unsetting verification-done-xenial for now, since based on Brian's comment this is not yet ready to release to Xenial. Let's get it out of the report, so when the action is taken we actually notice.

@Alberto
Based on #8, could you please also land this package to 17.04 (zesty)?
So that, we can finish this task. :)

yes, sorry but I have been busy with other work. I plan to take care of this soon.

Please accept ubuntu-drivers-common (1:0.4.22.1) into zesty-proposed.

Hello Alex, or anyone else affected,

Accepted ubuntu-drivers-common into zesty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-drivers-common/1:0.4.22.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

just checked it on zesty + ubuntu-drivers-common 1:0.4.22.1, it just works as the behavior like #5 said.

This bug was fixed in the package ubuntu-drivers-common - 1:0.4.22.1

---------------
ubuntu-drivers-common (1:0.4.22.1) zesty-proposed; urgency=medium

  * gpu-manager.{c|py}:
    - Add support for using custom xorg.confs with the nvidia
      driver (LP: #1667198).
      Custom xorg files can be named "non-hybrid" (for non hybrid
      systems), "hybrid-performance", and "hybrid-power-saving",
      and will have to placed in the /usr/share/gpu-manager.d
      directory.
      The directory can be overridden by passing another directory
      along with the "--custom-xorg-conf-path" parameter.
    - Add tests for the custom xorg.confs code and for amdgpu-pro
      hybrid support.

 -- Alberto Milone <email address hidden> Wed, 02 Aug 2017 17:17:13 +0200

The verification of the Stable Release Update for ubuntu-drivers-common has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

This bug was fixed in the package ubuntu-drivers-common - 1:0.4.17.3

---------------
ubuntu-drivers-common (1:0.4.17.3) xenial-proposed; urgency=medium

  * gpu-manager.{c|py}:
    - Add support for using custom xorg.confs with the nvidia
      driver (LP: #1667198).
      Custom xorg files can be named "non-hybrid" (for non hybrid
      systems), "hybrid-performance", and "hybrid-power-saving",
      and will have to placed in the /usr/share/gpu-manager.d
      directory.
      The directory can be overridden by passing another directory
      along with the "--custom-xorg-conf-path" parameter.
    - Add tests for the custom xorg.confs code and for amdgpu-pro
      hybrid support.

 -- Alberto Milone <email address hidden> Mon, 19 Jun 2017 17:27:33 +0200

This is no longer found in Bionic, since ubuntu-drivers-common 1:0.5 (see [1]). Was this feature intentionally removed?

[1]: https://git.launchpad.net/ubuntu/+source/ubuntu-drivers-common/commit/share/hybrid/gpu-manager.c?id=fb450c8eddf9290225e198ce92123e4fff346589

@Josh: we don't overwrite the xorg.conf any more. We simply create configuration files in /usr/share/X11/xorg.conf.d/ . This is why we dropped the feature.

@Alberto: would you be open to adding it again? I need to enable TearFree and select the Intel driver when using my integrated GPU (but if I have `Driver "intel"` in my config when using the Nvidia GPU, I get a black screen on login), so it would be nice to have a way to automatically put a file in xorg.conf.d when switching the GPU.