From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [RFC 02/21] drm/i915: Remove redundant parameter to i915_gem_object_wait_rendering__tail() Date: Sun, 19 Oct 2014 14:25:17 +0200 Message-ID: <20141019122517.GR26941@phenom.ffwll.local> References: <1412604925-11290-1-git-send-email-John.C.Harrison@Intel.com> <1412604925-11290-2-git-send-email-John.C.Harrison@Intel.com> <1412604925-11290-3-git-send-email-John.C.Harrison@Intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f178.google.com (mail-wi0-f178.google.com [209.85.212.178]) by gabe.freedesktop.org (Postfix) with ESMTP id CA1A56E06E for ; Sun, 19 Oct 2014 05:25:10 -0700 (PDT) Received: by mail-wi0-f178.google.com with SMTP id h11so4705971wiw.17 for ; Sun, 19 Oct 2014 05:25:09 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1412604925-11290-3-git-send-email-John.C.Harrison@Intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: John.C.Harrison@Intel.com Cc: Intel-GFX@Lists.FreeDesktop.Org List-Id: intel-gfx@lists.freedesktop.org On Mon, Oct 06, 2014 at 03:15:06PM +0100, John.C.Harrison@Intel.com wrote: > From: John Harrison Empty commit messages are a bit too thin. E.g. in this case this was an oversight from commit c8725f3dc0911d4354315a65150aecd8b7d0d74a Author: Chris Wilson Date: Mon Mar 17 12:21:55 2014 +0000 drm/i915: Do not call retire_requests from wait_for_rendering which means your patch here really should have Cc'ed patch author and revierer (i.e. Chris & Brad) of the offending patch, too. Especially in GEM where some of the interactions are _really_ tricky you can't just remove code that looks funky. You really have to dig into the history and find solid evidence that this is really just an oversight and not something deeper. > For: VIZ-4377 > Signed-off-by: John.C.Harrison@Intel.com Youre sob line isn't quite up to spec, it should have a full name + mail address (like the From: line git send-email inserted). -Daniel > --- > drivers/gpu/drm/i915/i915_gem.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c > index 8c68219..aa2d882 100644 > --- a/drivers/gpu/drm/i915/i915_gem.c > +++ b/drivers/gpu/drm/i915/i915_gem.c > @@ -1281,8 +1281,7 @@ i915_wait_seqno(struct intel_engine_cs *ring, uint32_t seqno) > } > > static int > -i915_gem_object_wait_rendering__tail(struct drm_i915_gem_object *obj, > - struct intel_engine_cs *ring) > +i915_gem_object_wait_rendering__tail(struct drm_i915_gem_object *obj) > { > if (!obj->active) > return 0; > @@ -1319,7 +1318,7 @@ i915_gem_object_wait_rendering(struct drm_i915_gem_object *obj, > if (ret) > return ret; > > - return i915_gem_object_wait_rendering__tail(obj, ring); > + return i915_gem_object_wait_rendering__tail(obj); > } > > /* A nonblocking variant of the above wait. This is a highly dangerous routine > @@ -1359,7 +1358,7 @@ i915_gem_object_wait_rendering__nonblocking(struct drm_i915_gem_object *obj, > if (ret) > return ret; > > - return i915_gem_object_wait_rendering__tail(obj, ring); > + return i915_gem_object_wait_rendering__tail(obj); > } > > /** > -- > 1.7.9.5 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch