Commit 4d6fc55a authored by Alex Deucher's avatar Alex Deucher

drm/amdgpu: expand runpm parameter

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/2428Reviewed-by: default avatarGuchun Chen <guchun.chen@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 47f1724d
...@@ -347,8 +347,9 @@ module_param_named(aspm, amdgpu_aspm, int, 0444); ...@@ -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 * 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). * 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 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); module_param_named(runpm, amdgpu_runtime_pm, int, 0444);
/** /**
...@@ -2496,24 +2497,26 @@ static int amdgpu_runtime_idle_check_display(struct device *dev) ...@@ -2496,24 +2497,26 @@ static int amdgpu_runtime_idle_check_display(struct device *dev)
struct drm_connector_list_iter iter; struct drm_connector_list_iter iter;
int ret = 0; int ret = 0;
/* XXX: Return busy if any displays are connected to avoid if (amdgpu_runtime_pm != -2) {
* possible display wakeups after runtime resume due to /* XXX: Return busy if any displays are connected to avoid
* hotplug events in case any displays were connected while * possible display wakeups after runtime resume due to
* the GPU was in suspend. Remove this once that is fixed. * 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); mutex_lock(&drm_dev->mode_config.mutex);
drm_for_each_connector_iter(list_connector, &iter) { drm_connector_list_iter_begin(drm_dev, &iter);
if (list_connector->status == connector_status_connected) { drm_for_each_connector_iter(list_connector, &iter) {
ret = -EBUSY; if (list_connector->status == connector_status_connected) {
break; ret = -EBUSY;
break;
}
} }
} drm_connector_list_iter_end(&iter);
drm_connector_list_iter_end(&iter); mutex_unlock(&drm_dev->mode_config.mutex);
mutex_unlock(&drm_dev->mode_config.mutex);
if (ret) if (ret)
return ret; return ret;
}
if (adev->dc_enabled) { if (adev->dc_enabled) {
struct drm_crtc *crtc; struct drm_crtc *crtc;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment