<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jul 23, 2014 at 10:31 AM, Marek Olšák <span dir="ltr"><<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Fri, Jul 18, 2014 at 5:32 PM, Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>> wrote:<br>

> On 07/17/2014 12:04 PM, Jason Ekstrand wrote:<br>
>><br>
>> This is the first installment of some work I've been doing over the past<br>
>> couple of weeks to refactor mesa's texture conversion/storage code.  There<br>
>> is more to be done and more that I have done but have not included in this<br>
>> series.  This is the first mailing-list-ready fruits of my efforts.  The<br>
>> important bits here include:<br>
>><br>
>>   1) Using a human-readable CSV file to describe texture formats similar<br>
>> to<br>
>>      the way it is currently don in gallium.  This is much easier to<br>
>>      read/edit than the structure in formats.c.  The guts of formats.c is<br>
>>      then autogenerated from this CSV file.<br>
><br>
><br>
> I'm kind of on the fence about this.  Some of us have been hoping that we'd<br>
> eventually consolidate some of the Mesa and gallium code so we wouldn't have<br>
> duplicated/parallel code.  The format code is a good example.  In theory,<br>
> the MESA_FORMAT_ stuff could be replaced by the gallium PIPE_FORMAT_ code.<br>
<br>
</div>I agree and I think this rework brings us closer to sharing the code,<br>
which is a good thing. In the meantime, it nicely cleans up the code.<br></blockquote><div><br></div><div>In that vein, I've taken a slightly different tack that I think will make the sharing easier.  Right now, I'm working on moving a bunch of code into a new static library called libformat that can be independently developed and *tested*.  One of the problems we had before was that the only format-conversion testing we did was through piglit which can't actually hit all of the code-paths.  Hopefully, splitting it out will  make it easier to test and easier to combine efforts than gallium depending on mesa or the other way around.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
BTW, this series will conflict with the big endian work that Richard<br>
Sandiford is doing.<br></blockquote><div><br></div><div>Yeah, I'm aware of that.<br></div><div><br></div><div>--Jason Ekstrand<br></div><div><br></div></div></div></div>