On Fri, Aug 05, 2016 at 08:56:57PM +0200, Daniel Vetter wrote:
> On Fri, Aug 05, 2016 at 10:06:04AM +0100, Chris Wilson wrote:
> > Our error states are quickly growing, pinning kernel memory with them.
> > The majority of the space is taken up by the error objects. These
> > compress well using zlib and without decode are mostly meaningless, so
> > encoding them does not hinder quickly parsing the error state for
> > familiarity.
> >
> > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
>
> Seems to also contain a wholesale rework of the capture logic using a ggtt
> mappable entry. Would explain why the missing clflush isn't an issue for
> you.
No. clflush was replaced by wbinvd, which is dropped here because it is
no longer required.
> Imo best if that part is reordered as the first patch (or at least
> before stop_machine, which requires the removal of cflush), and then the
> zlib on top.
>
> On the idea itself, since I have no clue: How do we uncompress these
> again? Patched intel_error_decode, or can zlib deal with in-line streams?
intel_error_decode was updated to handle the compressed streams.
commit d4c3cd4d04ab7f317d3429708d19cd5fc4d0f5fa
Author: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
AuthorDate: Fri Oct 31 11:27:21 2014 +0000
Commit: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
CommitDate: Thu Dec 31 22:10:11 2015 +0000
intel_error_decode: Inflate compressed error state
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
