[Intel-gfx] [PATCH v12 03/11] drm/i915: Move i915_gem_restore_fences to i915_gem_resume

Joonas Lahtinen joonas.lahtinen at linux.intel.com
Mon Oct 2 08:33:54 UTC 2017


On Fri, 2017-09-29 at 19:29 +0530, Sagar Arun Kamble wrote:
> 
> On 9/29/2017 5:18 PM, Joonas Lahtinen wrote:
> > On Thu, 2017-09-28 at 12:18 +0530, Sagar Arun Kamble wrote:
> > > i915_gem_restore_fences is GEM resumption task hence it is moved to
> > > i915_gem_resume from i915_restore_state.
> > 
> > + Chris
> > 
> > Didn't I just review this patch elsewhere? Other thread explains that
> > the emytology of fixing display checks out, what you suggest is more
> > appropriate. Question is if we should note with a comment?
> > 
> > PS. You seem to have sent this with --suppress-cc, which decreases the
> > chances of getting the reviews :)
> 
> No. this was sent without "--suppress-cc". I actually floated two 
> revisions yesterday so it might have been missed.
> Sorry for the thrash.
> How did you figure out "--suppress-cc"? Don't see in the headers

There were Cc: entries in the text, but as the patch reached me, there
were none.

Could be the mailing list acting weird, too.

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation


More information about the Intel-gfx mailing list