From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH] drm/i915: Disable the BLT on pre-production SNB hardware Date: Thu, 05 Jul 2012 23:27:28 +0100 Message-ID: <1341527310_183440@CP5-2952> References: <1341518509-30562-1-git-send-email-chris@chris-wilson.co.uk> <87sjd5swfe.fsf@eliezer.anholt.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from fireflyinternet.com (smtp.fireflyinternet.com [109.228.6.236]) by gabe.freedesktop.org (Postfix) with ESMTP id 25052A0EF8 for ; Thu, 5 Jul 2012 15:28:36 -0700 (PDT) In-Reply-To: <87sjd5swfe.fsf@eliezer.anholt.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: Eric Anholt , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, 05 Jul 2012 14:57:41 -0700, Eric Anholt wrote: > Chris Wilson writes: > > > It never quite worked despite the numerous workarounds, yet I still see > > people trying to use this hardware and filing bug reports. As we no > > longer even try to implement the workarounds, since 6a233c78878 > > (drm/i915/ringbuffer: kill snb blt workaround), simply disable the ring. > > Do we have something elsewhere that's very verbose telling them to get > real hardware? Userland isn't going to do very well when the ring > doesn't work. The DDX already checks for the pre-production hardware and forces the shadow buffer if it detects the broken blitter. We can easily add a message there to remind them to return their SDV and get themselves a real computer. So I had actually forgotten about that check. The only question is then whether that is better in the kernel so that all clients are protected from pathological users, or left in the DDX? -Chris -- Chris Wilson, Intel Open Source Technology Centre