mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	drm/amdgpu: don't use BACO for reset in S3
Seems to cause a reboots or hangs on some systems.
Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1924
Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1953
Fixes: daf8de0874 ("drm/amdgpu: always reset the asic in suspend (v2)")
Reviewed-by: Lijo Lazar <lijo.lazar@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
			
			
This commit is contained in:
		
							parent
							
								
									7e97de3e7f
								
							
						
					
					
						commit
						34452ac303
					
				
					 1 changed files with 11 additions and 0 deletions
				
			
		| 
						 | 
				
			
			@ -173,6 +173,17 @@ bool amdgpu_dpm_is_baco_supported(struct amdgpu_device *adev)
 | 
			
		|||
 | 
			
		||||
	if (!pp_funcs || !pp_funcs->get_asic_baco_capability)
 | 
			
		||||
		return false;
 | 
			
		||||
	/* Don't use baco for reset in S3.
 | 
			
		||||
	 * This is a workaround for some platforms
 | 
			
		||||
	 * where entering BACO during suspend
 | 
			
		||||
	 * seems to cause reboots or hangs.
 | 
			
		||||
	 * This might be related to the fact that BACO controls
 | 
			
		||||
	 * power to the whole GPU including devices like audio and USB.
 | 
			
		||||
	 * Powering down/up everything may adversely affect these other
 | 
			
		||||
	 * devices.  Needs more investigation.
 | 
			
		||||
	 */
 | 
			
		||||
	if (adev->in_s3)
 | 
			
		||||
		return false;
 | 
			
		||||
 | 
			
		||||
	mutex_lock(&adev->pm.mutex);
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue