On 30 December 2016 at 09:48, Ilia Mirkin <imir...@alum.mit.edu> wrote: > On Thu, Dec 29, 2016 at 5:54 PM, Thomas Helland > <thomashellan...@gmail.com> wrote: >> Apart from that it's basically a case of looking at a patch series >> implementing >> some other fairly trivial extension to get an idea of what needs changing >> apart >> from the functionality of the extension itself. > > There are a few ways to implement the ext. You could have the trivial > implementation which just makes it not report errors. However there's > a bunch of validation that gets done in various cases, which can be > costly. I believe the point of the extension is to get rid of that > extra costly validation. > > You're going to have to track down the specifics, but one thing that > comes off the top of my head is the validation that's done for ES > interface matching. There's probably 30 other things too though, that > one is just one that I happen to remember at this moment. I think > sampler/texture validation is another, probably stuff with uniforms, > etc. > > So the idea would be for those validation functions to just do if > (ctxflags & NO_ERROR) { return; } somewhere near the top. (For > example.)
In theory for a lot of the API you could split the API into _mesa_API_nocheck and _mesa_API { do error checks call _mesa_API_nocheck } variants, then have a separate dispatch table that goes straight to nocheck for a bunch of the API. Then you'd fixed up the other callsites on a case by case basis. Dave. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev