From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH] drm/i915: Initialize ring->obj last and use it as a marker Date: Sat, 14 Apr 2012 23:27:39 +0100 Message-ID: <1334442471_862@CP5-2952> References: <1334403584-14653-1-git-send-email-chris@chris-wilson.co.uk> <20120414221524.GF4215@phenom.ffwll.local> 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 998DD9EF3E for ; Sat, 14 Apr 2012 15:27:54 -0700 (PDT) In-Reply-To: <20120414221524.GF4215@phenom.ffwll.local> 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: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Sun, 15 Apr 2012 00:15:24 +0200, Daniel Vetter wrote: > On Sat, Apr 14, 2012 at 12:39:44PM +0100, Chris Wilson wrote: > > Use the assignment of ring->obj as a marker that the ring is active, and > > so be careful not to initialise that value too early in case we need to > > perform some workarounds that would ordinarily require touching the ring > > whilst prepping the object. > > > > Signed-off-by: Chris Wilson > > I guess we don't need this with the actual workaround that needs this > patch (i.e. that try-to-make-i845 patch)? Or is other stuff in the works? It's relevant for the quirk where we might need to idle the gpu whilst manipulating the GATT, but I'm still chasing the magic to make it work for 845g. I also thought it relevant in the light of your other ringbuffer cleanups. There's probably a few other places where we could make the code more architecture agnostic and just operate on what has been setup. -Chris -- Chris Wilson, Intel Open Source Technology Centre