On Mon, 15 Jun 2015, Ville Syrjälä <[email protected]> wrote: > On Mon, Jun 15, 2015 at 12:59:37PM +0300, Jani Nikula wrote: >> This reverts commit 0aedb1626566efd72b369c01992ee7413c82a0c5. >> >> I messed things up while applying [1] to drm-intel-fixes. Rectify. >> >> [1] >> http://mid.gmane.org/[email protected] >> >> Fixes: 0aedb1626566 ("drm/i915: Don't skip request retirement if the active >> list is empty") >> Cc: [email protected] >> Signed-off-by: Jani Nikula <[email protected]> >> > > Acked-by: Ville Syrjälä <[email protected]>
Pushed to drm-intel-fixes. BR, Jani. > > However this leaves us with an early bailout if request_list is empty. > Hopefully that won't cause other issues. I suppose we shouldn't have > stuff on the active_list w/o any pending requests. > >> --- >> >> Note to stable team: please do *not* backport >> >> commit 0aedb1626566efd72b369c01992ee7413c82a0c5 >> Author: Ville Syrjälä <[email protected]> >> Date: Thu May 28 18:32:36 2015 +0300 >> >> drm/i915: Don't skip request retirement if the active list is empty >> >> but if you did, please backport this revert as well. >> >> Thanks, and sorry. >> >> BR, >> Jani. >> --- >> drivers/gpu/drm/i915/i915_gem.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/drivers/gpu/drm/i915/i915_gem.c >> b/drivers/gpu/drm/i915/i915_gem.c >> index c3806c66650a..2d0995e7afc3 100644 >> --- a/drivers/gpu/drm/i915/i915_gem.c >> +++ b/drivers/gpu/drm/i915/i915_gem.c >> @@ -2656,6 +2656,9 @@ void i915_gem_reset(struct drm_device *dev) >> void >> i915_gem_retire_requests_ring(struct intel_engine_cs *ring) >> { >> + if (list_empty(&ring->request_list)) >> + return; >> + >> WARN_ON(i915_verify_lists(ring->dev)); >> >> /* Retire requests first as we use it above for the early return. >> -- >> 2.1.4 > > -- > Ville Syrjälä > Intel OTC -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
