[Intel-gfx] [PATCH] drm/i915: Split out parking from the idle worker for reuse

[Chris Wilson]

Quoting Michal Wajdeczko (2018-04-06 16:18:53)
> On Fri, 06 Apr 2018 16:33:39 +0200, Chris Wilson  
> <chris at chris-wilson.co.uk> wrote:
> 
> > We will want to park GEM before disengaging the drive^W^W^W unwedging.
> > Since we already do the work for idling, expose the guts as a new
> > function that we can then reuse.
> >
> > v2: Just skip if already parked; makes it more forgiving to use by
> > future callers.
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Michal Wajdeczko <michal.wajdeczko at intel.com>
> > Cc: Sagar Arun Kamble <sagar.a.kamble at intel.com>
> > Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> > Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> > Reviewed-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> > ---
> > Even with the follow up patch on hold, I think this will be useful when
> > we figure out the right order of operations in reset and stands by itself
> > as an improvement.
> >
> > Any objections to pushing this by itself?
> > -Chris
> 
> I would only suggest to make this new function more symmetrical to
> "mark_busy" from i915_request.c both in naming and location ;)

Fine, we'll pull back unpark and export that as well!
-Chris