Comment 117 for bug 1061073

Revision history for this message
In , Kenxeth (kenxeth) wrote :

(In reply to comment #25)
> I've pushed a patch to KWin that should fix the problem. The patch is in
> both master and the 4.9 branch, so you should see it in 4.9.3. That release
> will be tagged on Thursday.

Thanks Fredrik! Based on that, I'm closing this as RESOLVED/NOTOURBUG.

> @Rune: The GLX backend in KWin uses glXGetFBConfigs(), so the ordering rules
> in the spec don't apply here. I've thought about rewriting that code to use
> glXChooseFBConfig(), but I'm reluctant to touch working code without a good
> reason. This bug might be enough of a reason to do that though.

@Rune: Feel free to double check by running glxinfo, but I believe the ordering is correct: multisample configs are always sorted later, as required.

Using glXChooseFBConfig() does seem like a good idea...could guard against future problems (though I don't know what), and should simplify the code a fair bit. Also, KWin's EGL backend uses eglChooseConfig() and it never had this problem. That would also make the GLX and EGL backends more similar.

Thanks again.