<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 20, 2017 at 8:17 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 20 March 2017 at 15:07, Lionel Landwerlin<br>
<span><<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a><wbr>> wrote:<br>
> On 20/03/17 12:14, Emil Velikov wrote:<br>
>><br>
>> On 20 March 2017 at 09:13, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>> wrote:<br>
>>><br>
>>> This should give substantially better decoding, as the public libdrm<br>
>>> decoder hasn't been properly maintained in years.<br>
>>><br>
>>> For now, we reuse the existing state dumping mechanism.  We'll improve<br>
>>> that in the next patch.<br>
>><br>
>> Can you share the size of i965_dri.so before/after this patch/series ?<br>
>> I'd imagine that things will be noticeably larger since we the,<br>
>> thankfully zlib[ed], _xml.h files are within the final binary.<br>
>> That in itself is _not_ a blocker, but good to point out, since it's<br>
>> not obvious.<br>
><br>
><br>
> I measured an 8k increase, which seems fine to me for now.<br>
> Since this will increase for every new generation, I'll investigate a<br>
> mechanism to share the common data we have in different files.<br>
> Maybe moving away from xml to get store the data in our binary.<br></span></blockquote><div><br></div><div>Deduplication?  That sounds like a good job for a compression library. :-)  Maybe we can just store all of the XML together in some sort of a gziped tarball so that zlib can see all of them at the same time and dedup.  I wouldn't spend much time on it though.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
><br>
</span>Nice - that's a lot less than i expected.<br>
<br>
Can we please have some before/after numbers in the commit log ?<br>
Personally I wouldn't bother 'optimising'/reworking it anytime soon,<br>
since zlib [seemingly] does a pretty good job at compressing the XML<br>
files ;-)<br></blockquote><div><br></div><div>+1 for not bothering to make it any better right now.  8KB is tiny in a 12 MB driver. <br></div></div><br></div></div>