On Saturday, October 7, 2017 6:30:38 AM PDT Ilia Mirkin wrote: > Hi Ken, > > Have a look at > > https://patchwork.freedesktop.org/patch/134865/ > https://patchwork.freedesktop.org/patch/135410/ > > No idea what the outcome of that was. I guess some code reorg has > happened since too. If, after reading through those, you think this is > right - go ahead :) I've lost all context on this. > > -ilia
I went back through the history and discovered that you, Antia, and I all independently wrote basically the same patch. Nicolai pointed out that the GL 4.6 specification now has a quote making this clearly the right behavior, so it's no longer ambiguous. I think this was added after Antia noticed that ES 3.2 had text for this and filed a bug about GL 4.5 with Khronos. Antia decided at one point that using ImmutableLevels was the right thing to do, which I agree with. The new spec makes this very clear. So, time to finally land the fix. I just pushed a patch. I kept my code (as it applies to today's Mesa), but added the spec quote from Nicolai, updated it to reuse some of your commit message, and added Antia's R-b from a previous iteration of this patch. Thank you all for your hard work tracking this down over the last 10 months :)
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
