mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	drm/i915: Move GEM sanitize from resume_early to resume
This should be a no-op in terms of our control flow, we move the sanitization (GPU reset) from the bottom of the early resume phase to the top of the next. However, following hibernation debug, the power code skips the early resume phase, but as we are about to completely restore the GTT mappings, we first need to stop the GPU using them i.e. perform a GPU reset (i915_gem_sanitize()). Testcase: igt/gem_exec_suspend/basic-S4-devices Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Imre Deak <imre.deak@intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20180614094103.18025-1-chris@chris-wilson.co.uk Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
This commit is contained in:
		
							parent
							
								
									2e932b9cde
								
							
						
					
					
						commit
						1288786b18
					
				
					 1 changed files with 2 additions and 2 deletions
				
			
		| 
						 | 
				
			
			@ -1698,6 +1698,8 @@ static int i915_drm_resume(struct drm_device *dev)
 | 
			
		|||
	disable_rpm_wakeref_asserts(dev_priv);
 | 
			
		||||
	intel_sanitize_gt_powersave(dev_priv);
 | 
			
		||||
 | 
			
		||||
	i915_gem_sanitize(dev_priv);
 | 
			
		||||
 | 
			
		||||
	ret = i915_ggtt_enable_hw(dev_priv);
 | 
			
		||||
	if (ret)
 | 
			
		||||
		DRM_ERROR("failed to re-enable GGTT\n");
 | 
			
		||||
| 
						 | 
				
			
			@ -1839,8 +1841,6 @@ static int i915_drm_resume_early(struct drm_device *dev)
 | 
			
		|||
	else
 | 
			
		||||
		intel_display_set_init_power(dev_priv, true);
 | 
			
		||||
 | 
			
		||||
	i915_gem_sanitize(dev_priv);
 | 
			
		||||
 | 
			
		||||
	enable_rpm_wakeref_asserts(dev_priv);
 | 
			
		||||
 | 
			
		||||
out:
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue