[Mesa-dev] [PATCH 00/17] dispatch: code generate _mesa_create_exec_table.

[Paul Berry]

On 2 November 2012 09:24, Paul Berry <stereotype441 at gmail.com> wrote:

> On 1 November 2012 21:55, Jordan Justen <jljusten at gmail.com> wrote:
>
>> Excellent Paul! I didn't realize you were so close to having this
>> ready. It makes a good portion of my GL Core dispatch sanity v2 series
>> unnecessary.
>>
>
I don't think it makes your series unnecessary, just...different.  For
instance, your patch "mesa shaderapi: don't enable various functions for GL
CORE" disables three functions for core contexts by adjusting the hand
coded function _mesa_init_shader_dispatch().  After my series lands,
_mesa_init_shader_dispatch() will only used for the "save" dispatch table,
which doesn't apply to core contexts, so it won't be necessary to adjust
_mesa_init_shader_dispatch() anymore.  Instead, we'll need to adjust the
XML so that the generated code will disable these functions appropriately.

I think similar reasoning applies to most (all?) of the other patches in
your series.

As an experiment, I tried rebasing your patch series on top of my
work-in-progress branch, taking these kinds of considerations into
account.  It's on branch "gl-core-sanity" of git://
github.com/stereotype441/mesa.git if you want to look at it.


>
>> Anyway, I rebased my 5 dispatch-sanity changes onto your branch, and
>> it reported that these functions should be nop in GL Core profiles:
>> * ActiveProgramEXT
>> * CreateShaderProgramEXT
>> * UseShaderProgramEXT
>> * StencilFuncSeparateATI
>>
>> The first 3 were changed in my recent v2 03/13 patch.
>>
>
Yeah, adjusting the XML to reproduce the effect of your patch fixed those
three.  As for StencilFuncSeparateATI, I screwed it up in commit a21116f,
which has already landed in master.  I'll put out a patch to fix that ASAP.


>
>> Anyway, I pushed my rebased changes to:
>> git://people.freedesktop.org/~jljusten/mesa
>> branch=code-gen-api-exec+gl-core-sanity
>>
>> Those 5 changes haven't been fully code reviewed yet, but I think they
>> are reasonably close to ready. What would you think about adding them
>> to the end of your series?
>>
>
Given the fact that your series is reasonably close to ready, I would
prefer to let it land first, then adjust mine to incorporate your changes
into the XML.  Is that ok with you?  I'll try to review your series by the
end of the day.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20121102/e3bc3e83/attachment.html>