intel/tools/error: Drop support for non-ascii85 encoded error states.
authorKenneth Graunke <kenneth@whitecape.org>
Sun, 12 Nov 2017 05:55:27 +0000 (21:55 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 14 Nov 2017 01:10:38 +0000 (17:10 -0800)
commit1898bf11a885c440fd861978f3ee204fe918b159
treed660bc87484ba7d159a02fee406a33022b955d22
parent53586f88d7a3386d3122584b0af9fddc8abc6657
intel/tools/error: Drop support for non-ascii85 encoded error states.

Error state files used to look like:

   render ring --- gtt_offset = 0x0e8f6000
   00000000 :  69040000
   00000004 :  79090000
   ...
   00007ffc :  00000000
   --- ringbuffer = 0x00001000

There were thousands of lines between sections.  The file format changed
with Kernel 4.10, and now has a single ascii85-encoded line following
each section heading.  This is much easier to parse.

There are a bunch of bugs in our handling of the old style format,
where we'd decode the wrong data, at the wrong time.  Fixing all of
these is going to be a giant pain.  It's also a lot of extra code
complexity.  In order to properly decode indirect state, or compute
shaders, we'll also need to parse data in advance of decoding, which
is going to be a giant pain with this ad-hoc "decode everywhere!"
mentality.  So, let's just drop support for the older file format.

This unfortunately requires an error state generated by Kernel 4.10 or
later.  That's probably not the end of the world, as we encourage users
to upgrade to the latest kernel when encountering GPU hangs anyway.  It
might be a giant pain for people with LTS kernels, though...

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
src/intel/tools/aubinator_error_decode.c