From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: Add means to apply WA conditionally Date: Thu, 23 Oct 2014 17:51:25 +0200 Message-ID: <20141023155125.GR26941@phenom.ffwll.local> References: <1414078170-29469-1-git-send-email-arun.siluvery@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f182.google.com (mail-wi0-f182.google.com [209.85.212.182]) by gabe.freedesktop.org (Postfix) with ESMTP id 49BB26E4A6 for ; Thu, 23 Oct 2014 08:51:17 -0700 (PDT) Received: by mail-wi0-f182.google.com with SMTP id bs8so2394736wib.3 for ; Thu, 23 Oct 2014 08:51:16 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1414078170-29469-1-git-send-email-arun.siluvery@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Arun Siluvery Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Oct 23, 2014 at 04:29:30PM +0100, Arun Siluvery wrote: > We would want to apply some of the workarounds based on a condition to a > particular platform or Gen but we may not know all possible controlling > parameters in advance hence allow to define open conditions; a WA makes > it to the list only if the condition is true. > > With the appropriate conditions we can combine all of the workarounds > and apply them from a single place irrespective of platform instead of > having them in separate functions. > > For: VIZ-4090 > Signed-off-by: Arun Siluvery Imo we should just pull the condition out into proper control flow. Hiding it like that in the macro doesn't seem to buy us anything at all, but obfuscates the code. -Daniel > --- > drivers/gpu/drm/i915/intel_ringbuffer.c | 35 +++++++++++++++++++++++++++++++++ > 1 file changed, 35 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c > index 497b836..0525a5d 100644 > --- a/drivers/gpu/drm/i915/intel_ringbuffer.c > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c > @@ -736,6 +736,41 @@ static int wa_add(struct drm_i915_private *dev_priv, > > #define WA_WRITE(addr, val) WA_REG(addr, val, 0xffffffff) > > +#define WA_SET_BIT_MASKED_IF(cond, addr, mask) \ > + do { \ > + if (cond) { \ > + WA_SET_BIT_MASKED(addr, mask); \ > + } \ > + } while(0) > + > +#define WA_CLR_BIT_MASKED_IF(cond, addr, mask) \ > + do { \ > + if (cond) { \ > + WA_CLR_BIT_MASKED(addr, mask); \ > + } \ > + } while(0) > + > +#define WA_SET_BIT_IF(cond, addr, mask) \ > + do { \ > + if (cond) { \ > + WA_SET_BIT(addr, mask); \ > + } \ > + } while(0) > + > +#define WA_CLR_BIT_IF(cond, addr, mask) \ > + do { \ > + if (cond) { \ > + WA_CLR_BIT(addr, mask); \ > + } \ > + } while(0) > + > +#define WA_WRITE_IF(cond, addr, val) \ > + do { \ > + if (cond) { \ > + WA_WRITE(addr, val); \ > + } \ > + } while(0) > + > static int bdw_init_workarounds(struct intel_engine_cs *ring) > { > struct drm_device *dev = ring->dev; > -- > 2.1.2 > > _______________________________________________ > 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