From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 03/18] drm/i915: Introduce VECS: the 4th ring Date: Wed, 29 May 2013 21:10:08 +0200 Message-ID: <20130529191008.GM15743@phenom.ffwll.local> References: <1367110769-1306-1-git-send-email-ben@bwidawsk.net> <1369794154-1639-1-git-send-email-ben@bwidawsk.net> <1369794154-1639-4-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ea0-f169.google.com (mail-ea0-f169.google.com [209.85.215.169]) by gabe.freedesktop.org (Postfix) with ESMTP id 9A950E5D4A for ; Wed, 29 May 2013 12:10:13 -0700 (PDT) Received: by mail-ea0-f169.google.com with SMTP id m14so5614438eaj.0 for ; Wed, 29 May 2013 12:10:12 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1369794154-1639-4-git-send-email-ben@bwidawsk.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Ben Widawsky Cc: Intel GFX List-Id: intel-gfx@lists.freedesktop.org On Tue, May 28, 2013 at 07:22:19PM -0700, Ben Widawsky wrote: > The video enhancement command streamer is a new ring on HSW which does > what it sounds like it does. This patch provides the most minimal > inception of the ring. > > In order to support a new ring, we need to bump the number. The patch > may look trivial to the untrained eye, but bumping the number of rings > is a bit scary. As such the patch is not terribly useful by itself, but > a pretty nice place to find issues during a bisection. > > Reviewed-by: Damien Lespiau > Signed-off-by: Ben Widawsky > --- > drivers/gpu/drm/i915/intel_ringbuffer.c | 2 ++ > drivers/gpu/drm/i915/intel_ringbuffer.h | 3 ++- > 2 files changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c > index 5df1791..ead979a 100644 > --- a/drivers/gpu/drm/i915/intel_ringbuffer.c > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c > @@ -915,6 +915,8 @@ void intel_ring_setup_status_page(struct intel_ring_buffer *ring) > case VCS: > mmio = BSD_HWS_PGA_GEN7; > break; > + case VECS: > + BUG(); My maintainer script freaked out a bit since you're adding a BUG here without any particular reason that it's strictly better than a WARN. But since it'll disappear in a follow-up patch I'll let it slide. Yes, I've hit one of my own stupid BUG_ONs just recently, so I'm now doubleplusgrumpy about this henceforth ;-) Cheers, Daniel > } > } else if (IS_GEN6(ring->dev)) { > mmio = RING_HWS_PGA_GEN6(ring->mmio_base); > diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.h b/drivers/gpu/drm/i915/intel_ringbuffer.h > index f55d92e..73619cb 100644 > --- a/drivers/gpu/drm/i915/intel_ringbuffer.h > +++ b/drivers/gpu/drm/i915/intel_ringbuffer.h > @@ -47,8 +47,9 @@ struct intel_ring_buffer { > RCS = 0x0, > VCS, > BCS, > + VECS, > } id; > -#define I915_NUM_RINGS 3 > +#define I915_NUM_RINGS 4 > u32 mmio_base; > void __iomem *virtual_start; > struct drm_device *dev; > -- > 1.8.3 > > _______________________________________________ > 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