On 30.03.2017 14:20, Bartosz Tomczyk wrote:
Thanks Nicolai,

Adding Timothy who seems most active on glthread topic.

Guys, do you think we can land this, with above comments addressed?

Well, Edmondo's testing suggests the performance looks good, so I'd say go for it, and we'll see if anything regresses.

Cheers,
Nicolai



On Thu, Mar 30, 2017 at 1:40 PM, Nicolai Hähnle <nhaeh...@gmail.com
<mailto:nhaeh...@gmail.com>> wrote:

    On 30.03.2017 10:30, Bartosz Tomczyk wrote:

        Thank you guys for testing.

        I'll address all issues in next patch if we decide to merge it.

        Nicolai,

        Could you comment on _glapi_{get/set}_dispatch part. I'm not
        familiar with it and I'm not sure if it's correct.


    It looks good to me.

    Cheers,
    Nicolai


        On Thu, Mar 30, 2017 at 5:38 AM, Michel Dänzer
        <mic...@daenzer.net <mailto:mic...@daenzer.net>
        <mailto:mic...@daenzer.net <mailto:mic...@daenzer.net>>> wrote:

            On 30/03/17 02:31 AM, Bartosz Tomczyk wrote:
            > Call it directly when batch queue is empty. This avoids
        costly thread
            > synchronisation. With this fix games that previously regressed
            > with mesa_glthread=true like xonotic or grid autosport.

            The second sentence here is missing a verb (at least).


            --
            Earthling Michel Dänzer               |
         http://www.amd.com
            Libre software enthusiast             |             Mesa and
        X developer




    --
    Lerne, wie die Welt wirklich ist,
    Aber vergiss niemals, wie sie sein sollte.




--
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to