From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Cc: Jani Nikula <jani.nikula@intel.com>, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] drm/i915: Mark up early pre-production Kabylakes
Date: Wed, 28 Nov 2018 08:55:50 -0800 [thread overview]
Message-ID: <20181128165550.GB5161@intel.com> (raw)
In-Reply-To: <87a7ltfgn8.fsf@gaia.fi.intel.com>
On Wed, Nov 28, 2018 at 03:58:19PM +0200, Mika Kuoppala wrote:
> Chris Wilson <chris@chris-wilson.co.uk> writes:
>
> > Mark A0 as the one and only pre-production variant of Kabylake and
> > remove its couple of workarounds, consigning them to the annals of
> > history.
> >
> > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> > Cc: Jani Nikula <jani.nikula@intel.com>
> > Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
>
> My two patches are the same so in favour of compactness: :)
>
> Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
>
> > ---
> > drivers/gpu/drm/i915/i915_drv.c | 1 +
> > drivers/gpu/drm/i915/intel_lrc.c | 12 ------------
> > drivers/gpu/drm/i915/intel_workarounds.c | 5 -----
> > 3 files changed, 1 insertion(+), 17 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> > index b1d23c73c147..e39016713464 100644
> > --- a/drivers/gpu/drm/i915/i915_drv.c
> > +++ b/drivers/gpu/drm/i915/i915_drv.c
> > @@ -868,6 +868,7 @@ static void intel_detect_preproduction_hw(struct drm_i915_private *dev_priv)
> > pre |= IS_HSW_EARLY_SDV(dev_priv);
> > pre |= IS_SKL_REVID(dev_priv, 0, SKL_REVID_F0);
> > pre |= IS_BXT_REVID(dev_priv, 0, BXT_REVID_B_LAST);
> > + pre |= IS_KBL_REVID(dev_priv, 0, KBL_REVID_A0);
> >
> > if (pre) {
> > DRM_ERROR("This is a pre-production stepping. "
> > diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
> > index 08fd9b12e4d7..11f4e6148557 100644
> > --- a/drivers/gpu/drm/i915/intel_lrc.c
> > +++ b/drivers/gpu/drm/i915/intel_lrc.c
> > @@ -1401,18 +1401,6 @@ static u32 *gen9_init_indirectctx_bb(struct intel_engine_cs *engine, u32 *batch)
> >
> > batch = emit_lri(batch, lri, ARRAY_SIZE(lri));
> >
> > - /* WaClearSlmSpaceAtContextSwitch:kbl */
> > - /* Actual scratch location is at 128 bytes offset */
> > - if (IS_KBL_REVID(engine->i915, 0, KBL_REVID_A0)) {
> > - batch = gen8_emit_pipe_control(batch,
> > - PIPE_CONTROL_FLUSH_L3 |
> > - PIPE_CONTROL_GLOBAL_GTT_IVB |
> > - PIPE_CONTROL_CS_STALL |
> > - PIPE_CONTROL_QW_WRITE,
> > - i915_ggtt_offset(engine->scratch)
> > - + 2 * CACHELINE_BYTES);
> > - }
> > -
> > /* WaMediaPoolStateCmdInWABB:bxt,glk */
> > if (HAS_POOLED_EU(engine->i915)) {
> > /*
> > diff --git a/drivers/gpu/drm/i915/intel_workarounds.c b/drivers/gpu/drm/i915/intel_workarounds.c
> > index ca1f78a42b17..39cd1f823ea9 100644
> > --- a/drivers/gpu/drm/i915/intel_workarounds.c
> > +++ b/drivers/gpu/drm/i915/intel_workarounds.c
> > @@ -384,11 +384,6 @@ static int kbl_ctx_workarounds_init(struct drm_i915_private *dev_priv)
> > if (ret)
> > return ret;
> >
> > - /* WaDisableFenceDestinationToSLM:kbl (pre-prod) */
> > - if (IS_KBL_REVID(dev_priv, KBL_REVID_A0, KBL_REVID_A0))
> > - WA_SET_BIT_MASKED(HDC_CHICKEN0,
> > - HDC_FENCE_DEST_SLM_DISABLE);
> > -
> > /* WaToEnableHwFixForPushConstHWBug:kbl */
> > if (IS_KBL_REVID(dev_priv, KBL_REVID_C0, REVID_FOREVER))
> > WA_SET_BIT_MASKED(COMMON_SLICE_CHICKEN2,
> > --
> > 2.20.0.rc1
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2018-11-28 16:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-28 13:53 [PATCH] drm/i915: Mark up early pre-production Kabylakes Chris Wilson
2018-11-28 13:58 ` Mika Kuoppala
2018-11-28 16:55 ` Rodrigo Vivi [this message]
2018-11-28 15:03 ` ✓ Fi.CI.BAT: success for " Patchwork
2018-11-28 21:21 ` ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181128165550.GB5161@intel.com \
--to=rodrigo.vivi@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@intel.com \
--cc=mika.kuoppala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).