The series looks good to me. One formatting nit: in one of the patches
there's unneeded space around the conditional in some the "if (" code.
Reviewed-by: Brian Paul <bri...@vmware.com>
On Wed, Aug 9, 2017 at 3:53 AM, Thomas Hellstrom <thellst...@vmware.com>
> The current implementation was suffering from the following problems:
> 1) The driGetConfigAttribIndex function was not returning any value for
> the driconfig swapMethod.
> 2) The X server GLX code is using the value obtained from
> the AIGLX dri driver driGetConfigAttribIndex to populate its GLX fbconfig.
> 3) The client side GLX code was assuming fbconfig and driconfig match
> regardless of whether there actually was a swapMethod match.
> 4) We were using GLX tokens in the dri code.
> 5) dri3 does currently not support GLX_SWAP_COPY_OML, even if that's
> advertized by the gallium dri2 state tracker.
> Attempt to fix 1-4. Some highlights:
> Because of 1) in combination with 2), if the X server uses an AIGLX dri
> that doesn't have this fix, the GLX transmitted fbconfig swapMethod value
> be bogus. So if we, on the client side detect a bogus value, change it to
> v2: Split the single patch into four patches.
> mesa-dev mailing list
mesa-dev mailing list