On Mon, Sep 28, 2015 at 1:29 PM, Emil Velikov <[email protected]> wrote: > As the spec allows for {server,client}_wait_sync to be called without > currently bound context, while our implementation requires context > pointer. > > UNTESTED. > > Cc: Chad Versace <[email protected]> > Cc: Marek Olšák <[email protected]> > Cc: Chih-Wei Huang <[email protected]> > Cc: Mauro Rossi <[email protected]> > Cc: 10.6 11.0 <[email protected]> > Signed-off-by: Emil Velikov <[email protected]> > --- > > Upon second thought I'm leaning that we should move this to the API > specific library (i.e. libEGL) rather than keeping it here. It will give > us remove API specific from the DRI extension, plus it'll give us better > mix'n'match (loader+dri module) compatibility. > > How do you guys feel on the topic ?
I don't think there is anything to move into libEGL. st/dri doesn't add a context pointer to the fence, instead it adds a screen pointer, which is a different thing. This i965 fix is probably not thread-safe, because eglClientWaitSync can be called from another thread and contexts typically contain no locks. Marek _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
