[Mesa-dev] Any updates on threaded GL dispatch?

[Gregory Hainaut]

Hello Tim,

>  Ignoring the attributes in XML files during generation just seems hacky to me. Are you able to take you reg expression and apply it to sed in some way to actually update the XML?

Ok. I will update the XML files. I will try to send you a patch Thursday.

Cheers,
Gregory

On 3/6/17, Timothy Arceri <tarceri at itsqueeze.com> wrote:
>
>
> On 04/03/17 04:41, gregory hainaut wrote:
>> On Fri, 3 Mar 2017 16:46:24 +0100
>> Marek Olšák <maraeo at gmail.com> wrote:
>>
>>> On Fri, Mar 3, 2017 at 10:19 AM, Timothy Arceri <tarceri at itsqueeze.com>
>>> wrote:
>>>> On 02/03/17 22:18, Marek Olšák wrote:
>>>>>
>>>>> The bad news is my involvement is currently on hold due to other
>>>>> projects and responsibilities.
>>>>
>>>>
>>>> I can probably spend some time on this. Seems like Gregory has taken
>>>> care of
>>>> most of the problems and it just needs someone to push it over the
>>>> line.
>>>
>>> There are also plenty of unresolved review comments from Emil and
>>> others.
>
> I've addressed all but the no trivial comments in patch 6 [1]. And
> pushed to a glthread branch in my repo[2]. I've also sent out some of
> the trivial standalone patches to the list that I will push with my
> review in a couple of days.
>
> I'll try address the rest of the issue in patch 6 tomorrow and then I'll
> resend the series.
>
>
> [1] https://patchwork.freedesktop.org/patch/137755/
> [2] https://github.com/tarceri/Mesa/compare/glthread
>
>>>
>>> Marek
>>
>> Yes I only fixed the piglit bad/crash regression on Nouveau. I added
>> also the perf optimization for PCSX2 ;) I did a basic test of EGL
>> (i.e. PCSX2) and it seems to work.
>>
>> By the way, I don't know how costly is the remaining synchronization
>> but there are 2 potential optimizations
>> * glUniform for double seems to sync whereas standard float
>>   are asynchronous. Maybe it misses the scale parameter in XML.
>>   I guess double was added after glthread. It might worth to check
>>   the behavior of glProgramUniform (introduce by SSO) too.
>> * GL3 glClearBuffer functions are synchronous due to the pointer to
>>   the single pixel value. However there are maybe tricks to find the
>>   correct size of the pixel.
>>
>> Cheers,
>> Gregory
>>
>
> Hi Gregory,
>
> Just a comment on your patch:
>
>   mesa glthread: allow asynchronous pixel transfer operation when a
> buffer is bound
>
> Ignoring the attributes in XML files during generation just seems hacky
> to me. Are you able to take you reg expression and apply it to sed in
> some way to actually update the XML?
>
> Thanks,
> Tim
>