[Intel-gfx] [PATCH v2 1/1] drm/i915: Split i915_gem_suspend into user quiescing and HW cleanup/suspend

[Sagar Arun Kamble]

On 10/12/2017 2:43 PM, Joonas Lahtinen wrote:
> On Fri, 2017-10-06 at 13:39 +0530, Sagar Arun Kamble wrote:
>> On 10/6/2017 12:07 PM, Sagar Arun Kamble wrote:
>>> GTT mappings are to be suspended post i915_gem_suspend and display
>> I think I should say this "post idling GPU and display suspend". Will
>> update in next rev.
>>> suspend. GPU is reset in i915_gem_suspend. This separated the HW suspend
>>> across reset. To achieve the complete HW cleanup prior to GPU reset we
>>> need to separate i915_gem_suspend functionality into user quiescing
>>> and HW cleanup. With this patch new function i915_gem_quiescent
>> And this should have been named "i915_gem_quiesce"?
>> Chris, Joonas, does i915_gem_quiesce  for user quiescing and
>> i915_gem_suspend for HW suspend look good?
> The BAT results seemed quite negative.
Yes. Addressed as part of GEM/GuC fixes series.
>
> Regards, Joonas