Re: [PATCH 9/9] drm/vc4: Add an interface for capturing the GPU state after a hang.

From: Eric Anholt <eric@anholt.net>
To: Emil Velikov <emil.l.velikov@gmail.com>
Cc: ML dri-devel <dri-devel@lists.freedesktop.org>,
	"Linux-Kernel\@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 9/9] drm/vc4: Add an interface for capturing the GPU state after a hang.
Date: Wed, 02 Dec 2015 11:35:16 -0800	[thread overview]
Message-ID: <87r3j48wzv.fsf@eliezer.anholt.net> (raw)
In-Reply-To: <CACvgo52FXmpJYtjT1nfmOrQZFB3CVTF=J2Zurpw=SULHSU-aAQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1529 bytes --]

Emil Velikov <emil.l.velikov@gmail.com> writes:

> On 1 December 2015 at 20:35, Eric Anholt <eric@anholt.net> wrote:
>> This can be parsed with vc4-gpu-tools tools for trying to figure out
>> what was going on.
>>
> I might be pushing my luck here ... have you thought about basing
> (forking) vc4-gpu-tools of intel-gpu-tools ? I'd imagine that the
> macros and helpers will come in handy, despite that some are quite
> intel specific.
>
> On a related note - with the above project in place I'd imagine you
> have (re)considered about having libdrm-vc4 ? Copying hunks around
> might lead to interesting moments (as you have already noticed :-P)
>
> On that note I'll stop now with beating the libdrm drum :-)

The headers and code that I copy between the two userspace locations
will go in libdrm when I have a kernel ABI, but vc4_drm.h can't go in
until merging to the kernel, and there's not a whole lot of point
without that.

Yes, I have thought about basing vc4-gpu-tools off of intel-gpu-tools.
I've actually tried to build and use the kms testing stuff on vc4, and
it was a total bust.  Someone needs to do a lot of work to make igt
useful for non-intel.  If you'd like me to build my vc4 testing inside
of igt, I'd someone to demo one of my tests building inside of igt, with
the test runner working and none of the intel-specific tests reporting
failure, and get me permission to just push code to that repository
(It's hard enough getting piglit tests reviewed, vc4-specific tests and
tools would never get review).

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 818 bytes --]