drm/amdgpu: Use amdgpu by default on SI dedicated GPUs (v2)

Now that the DC analog connector support and VCE1 support landed,
amdgpu is at feature parity with the old radeon driver
on SI dGPUs.

Enabling the amdgpu driver by default for SI dGPUs has the
following benefits:

- More stable OpenGL support through RadeonSI
- Vulkan support through RADV
- Improved performance
- Better display features through DC

Users who want to keep using the old driver can do so using:
amdgpu.si_support=0 radeon.si_support=1

v2:
- Update documentation in Kconfig file

Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/amdgpu/Kconfig b/drivers/gpu/drm/amd/amdgpu/Kconfig
index 883f32428871356a8896b78a5fb2cbadde498f68..7f515be5185dcff6d919e89c56dd26fb666d6957 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/Kconfig
+++ b/drivers/gpu/drm/amd/amdgpu/Kconfig
@@ -43,14 +43,16 @@ config DRM_AMDGPU_SI
        bool "Enable amdgpu support for SI parts"
        depends on DRM_AMDGPU
        help
-         Choose this option if you want to enable experimental support
+         Choose this option if you want to enable support
          for SI (Southern Islands) asics.
 
-         SI is already supported in radeon. Experimental support for SI
-         in amdgpu will be disabled by default and is still provided by
-         radeon. Use module options to override this:
+         SI (Southern Islands) are first generation GCN GPUs,
+         supported by both drivers: radeon (old) and amdgpu (new).
+         By default, SI dedicated GPUs are supported by amdgpu.
 
-         radeon.si_support=0 amdgpu.si_support=1
+         Use module options to override this:
+         To use radeon for SI,
+         radeon.si_support=1 amdgpu.si_support=0
 
 config DRM_AMDGPU_CIK
        bool "Enable amdgpu support for CIK parts"

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
index 09d3b080a4daa7397669bad93602cb3812066954..2dfbddcef9ab3c9ee707679d52576b47ad4ecd0f 100644 (file)
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -623,7 +623,7 @@ module_param_named(timeout_period, amdgpu_watchdog_timer.period, uint, 0644);
  * SI (Southern Islands) are first generation GCN GPUs, supported by both
  * drivers: radeon (old) and amdgpu (new). This parameter controls whether
  * amdgpu should support SI.
- * By default, SI chips are supported by radeon (except when radeon is not built).
+ * By default, SI dedicated GPUs are supported by amdgpu.
  * Only relevant when CONFIG_DRM_AMDGPU_SI is enabled to build SI support in amdgpu.
  * See also radeon.si_support which should be disabled when amdgpu.si_support is
  * enabled, and vice versa.
@@ -2326,6 +2326,7 @@ static bool amdgpu_support_enabled(struct device *dev,
                param = "si_support";
                module_param = amdgpu_si_support;
                amdgpu_support_built = IS_ENABLED(CONFIG_DRM_AMDGPU_SI);
+               support_by_default = true;
                break;
 
        case CHIP_BONAIRE:

diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c
index 440145fed625351ca20638c5c2650bd7cfe452b8..87fd6255c114128aca826eb6f50e2591f15fbcb8 100644 (file)
--- a/drivers/gpu/drm/radeon/radeon_drv.c
+++ b/drivers/gpu/drm/radeon/radeon_drv.c
@@ -273,6 +273,7 @@ static bool radeon_support_enabled(struct device *dev,
                gen = "SI";
                module_param = radeon_si_support;
                amdgpu_support_built &= IS_ENABLED(CONFIG_DRM_AMDGPU_SI);
+               support_by_default = false;
                break;
 
        case CHIP_BONAIRE: