Re: [PATCH 00/16] fix VLV S4 suspend/resume, unify S3/S4 handlers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 10, 2014 at 06:16:53PM +0300, Imre Deak wrote:
> The first part of the patchset (1-6) fixes an S4 bug on VLV introduced
> recently. The rest unifies the various S3/S4 handlers, which are in
> practice the same. The only real difference - besides some unused code -
> is that during S3 suspend we disable the PCI device whereas across an S4
> freeze/thaw we keep it enabled. Given that we disable everything else
> anyway, we can just as well disable the PCI device there too. Doing so
> allows us to handle S3 suspend/resume and S4 freeze/thaw/restore/
> power-off the same way, simplifying/clarifying things quite a bit.
> 
> I smoke tested this on BDW, HSW, VLV (BYT-M), IVB, GEN45.
> 
> Imre Deak (16):
>   drm/i915: vlv: fix gunit HW state corruption during S4 suspend
>   drm/i915: remove dead code from legacy suspend handler
>   drm/i915: factor out i915_drm_suspend_late
>   drm/i915: unify legacy S3 suspend and S4 freeze handlers
>   drm/i915: propagate error from legacy resume handler
>   drm/i915: vlv: fix switcheroo/legacy suspend/resume
>   drm/i915: fix S4 suspend while switcheroo state is off
>   drm/i915: remove unused restore_gtt_mappings optimization during
>     suspend
>   drm/i915: check for GT faults during S3 resume and S4 restore too
>   drm/i915: enable output polling during S4 thaw
>   drm/i915: disable/re-enable PCI device around S4 freeze/thaw
>   drm/i915: unify S3 and S4 suspend/resume handlers
>   drm/i915: sanitize suspend/resume helper function names
>   drm/i915: add poweroff_late handler
>   drm/i915: unify switcheroo and legacy suspend/resume handlers
>   drm/i915: add comments on what stage a given PM handler is called

It all looks pretty decent to me. You can slap

Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

onto everything except 09/16. I was a bit lazy and didn't look at that
one since Chris has already checked it.

> 
>  drivers/gpu/drm/i915/i915_dma.c |   4 +-
>  drivers/gpu/drm/i915/i915_drv.c | 185 +++++++++++++++-------------------------
>  drivers/gpu/drm/i915/i915_drv.h |   4 +-
>  3 files changed, 74 insertions(+), 119 deletions(-)
> 
> -- 
> 1.8.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux