On 9/26/2017 7:13 PM, Chris Wilson wrote:
Quoting Sagar Arun Kamble (2017-09-26 14:24:40)
@@ -2077,7 +2079,7 @@ int i915_gem_runtime_suspend(struct drm_i915_private *dev_priv)
reg->dirty = true;
}
- return 0;
+ return ret;
OI! These are written as return 0 because you are very much not meant to
reach this point with an error.
-Chris
What if GuC suspend fail? Do we still want to return 0. It might cause
issues as GuC will be active when PCI Device is set to D3.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
