drm/amdgpu: expand runpm parameter
authorAlex Deucher <alexander.deucher@amd.com>
Wed, 7 Jun 2023 16:56:18 +0000 (12:56 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 15 Aug 2023 22:08:29 +0000 (18:08 -0400)
Allow the user to specify -2 as auto enabled with displays.

By default we don't enter runtime suspend when there are
displays attached because it does not work well in some
desktop environments due to the driver sending hotplug
events on resume in case any new displays were attached
while the GPU was powered down.  Some users still want
this functionality though, so this lets you enable it.

Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2428
Reviewed-by: Guchun Chen <guchun.chen@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c

index 892ecb5..81edf66 100644 (file)
@@ -347,8 +347,9 @@ module_param_named(aspm, amdgpu_aspm, int, 0444);
  * Override for runtime power management control for dGPUs. The amdgpu driver can dynamically power down
  * the dGPUs when they are idle if supported. The default is -1 (auto enable).
  * Setting the value to 0 disables this functionality.
+ * Setting the value to -2 is auto enabled with power down when displays are attached.
  */
-MODULE_PARM_DESC(runpm, "PX runtime pm (2 = force enable with BAMACO, 1 = force enable with BACO, 0 = disable, -1 = auto)");
+MODULE_PARM_DESC(runpm, "PX runtime pm (2 = force enable with BAMACO, 1 = force enable with BACO, 0 = disable, -1 = auto, -2 = autowith displays)");
 module_param_named(runpm, amdgpu_runtime_pm, int, 0444);
 
 /**
@@ -2496,24 +2497,26 @@ static int amdgpu_runtime_idle_check_display(struct device *dev)
                struct drm_connector_list_iter iter;
                int ret = 0;
 
-               /* XXX: Return busy if any displays are connected to avoid
-                * possible display wakeups after runtime resume due to
-                * hotplug events in case any displays were connected while
-                * the GPU was in suspend.  Remove this once that is fixed.
-                */
-               mutex_lock(&drm_dev->mode_config.mutex);
-               drm_connector_list_iter_begin(drm_dev, &iter);
-               drm_for_each_connector_iter(list_connector, &iter) {
-                       if (list_connector->status == connector_status_connected) {
-                               ret = -EBUSY;
-                               break;
+               if (amdgpu_runtime_pm != -2) {
+                       /* XXX: Return busy if any displays are connected to avoid
+                        * possible display wakeups after runtime resume due to
+                        * hotplug events in case any displays were connected while
+                        * the GPU was in suspend.  Remove this once that is fixed.
+                        */
+                       mutex_lock(&drm_dev->mode_config.mutex);
+                       drm_connector_list_iter_begin(drm_dev, &iter);
+                       drm_for_each_connector_iter(list_connector, &iter) {
+                               if (list_connector->status == connector_status_connected) {
+                                       ret = -EBUSY;
+                                       break;
+                               }
                        }
-               }
-               drm_connector_list_iter_end(&iter);
-               mutex_unlock(&drm_dev->mode_config.mutex);
+                       drm_connector_list_iter_end(&iter);
+                       mutex_unlock(&drm_dev->mode_config.mutex);
 
-               if (ret)
-                       return ret;
+                       if (ret)
+                               return ret;
+               }
 
                if (adev->dc_enabled) {
                        struct drm_crtc *crtc;