From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 00/10] [RFC v2] quick dump Date: Sun, 3 Feb 2013 12:22:25 +0000 Message-ID: <20130203122225.GC9892@cantiga.alporthouse.com> References: <1359850082-14793-1-git-send-email-ben@bwidawsk.net> <20130203102915.44288d05@jbarnes-t420> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga03.intel.com (mga03.intel.com [143.182.124.21]) by gabe.freedesktop.org (Postfix) with ESMTP id F3313E5CD4 for ; Sun, 3 Feb 2013 04:22:31 -0800 (PST) Content-Disposition: inline In-Reply-To: <20130203102915.44288d05@jbarnes-t420> 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: Jesse Barnes Cc: Ben Widawsky , vincent.beng.keat.cheah@intel.com, intel-gfx@lists.freedesktop.org, alan.previn.teres.alexis@intel.com List-Id: intel-gfx@lists.freedesktop.org On Sun, Feb 03, 2013 at 10:29:15AM +0100, Jesse Barnes wrote: > On Sat, 2 Feb 2013 16:07:52 -0800 > Ben Widawsky wrote: > > > This is my second attempt at winning approval for the series. First one > > was: https://patchwork.kernel.org/patch/1493131/ > > > > In spending the time to rework this tool, I've begun to lose my belief > > in some of the original motivations I had. Even if you don't want to > > review, but just like (or dislike) what you see, I'd appreciate such > > comments. > > I'd like to see it land in i-g-t. Having the regs defined in a text or > xml file is an improvement over what we have today, and is easier to > extend. At first the advantage of reg_dumper was that it parsed out > the bitfields of the various regs. But we didn't keep up with that, > and also haven't kept up with the regs on new platforms as well as we > could. Text files would make that easier, and xml files might bring > back the bit field parsing, which would be extra nice. Completely agree. For me the big improvement would be being able to use the bspec register names or our internal approximation thereof rather than having to loop up the actual addresses every time. Having the name database available in python should make building integrated little snippets to parse traces which are also python accessible. -Chris -- Chris Wilson, Intel Open Source Technology Centre