Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

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>

authored by

Timur Kristóf and committed by
Alex Deucher
ccd3b4c7 c0344266

+10 -6
+7 -5
drivers/gpu/drm/amd/amdgpu/Kconfig
··· 43 43 bool "Enable amdgpu support for SI parts" 44 44 depends on DRM_AMDGPU 45 45 help 46 - Choose this option if you want to enable experimental support 46 + Choose this option if you want to enable support 47 47 for SI (Southern Islands) asics. 48 48 49 - SI is already supported in radeon. Experimental support for SI 50 - in amdgpu will be disabled by default and is still provided by 51 - radeon. Use module options to override this: 49 + SI (Southern Islands) are first generation GCN GPUs, 50 + supported by both drivers: radeon (old) and amdgpu (new). 51 + By default, SI dedicated GPUs are supported by amdgpu. 52 52 53 - radeon.si_support=0 amdgpu.si_support=1 53 + Use module options to override this: 54 + To use radeon for SI, 55 + radeon.si_support=1 amdgpu.si_support=0 54 56 55 57 config DRM_AMDGPU_CIK 56 58 bool "Enable amdgpu support for CIK parts"
+2 -1
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
··· 623 623 * SI (Southern Islands) are first generation GCN GPUs, supported by both 624 624 * drivers: radeon (old) and amdgpu (new). This parameter controls whether 625 625 * amdgpu should support SI. 626 - * By default, SI chips are supported by radeon (except when radeon is not built). 626 + * By default, SI dedicated GPUs are supported by amdgpu. 627 627 * Only relevant when CONFIG_DRM_AMDGPU_SI is enabled to build SI support in amdgpu. 628 628 * See also radeon.si_support which should be disabled when amdgpu.si_support is 629 629 * enabled, and vice versa. ··· 2326 2326 param = "si_support"; 2327 2327 module_param = amdgpu_si_support; 2328 2328 amdgpu_support_built = IS_ENABLED(CONFIG_DRM_AMDGPU_SI); 2329 + support_by_default = true; 2329 2330 break; 2330 2331 2331 2332 case CHIP_BONAIRE:
+1
drivers/gpu/drm/radeon/radeon_drv.c
··· 273 273 gen = "SI"; 274 274 module_param = radeon_si_support; 275 275 amdgpu_support_built &= IS_ENABLED(CONFIG_DRM_AMDGPU_SI); 276 + support_by_default = false; 276 277 break; 277 278 278 279 case CHIP_BONAIRE: