glx: Fix config chooser logic for 'mask' matching
authorKristian Høgsberg <krh@bitplanet.net>
Fri, 9 Apr 2010 02:09:11 +0000 (22:09 -0400)
committerKristian Høgsberg <krh@bitplanet.net>
Fri, 9 Apr 2010 19:24:29 +0000 (15:24 -0400)
commit394c25133e756e9d9580b65e0f1af7d67155e0f7
tree9b459046040faa846958b5d1618b0325dbbebd3f
parentb22a00bff4aadd390dd8af6b5b05bd2833ec7f85
glx: Fix config chooser logic for 'mask' matching

When matching attributes using the 'mask' matching criteria, the spec
says that

  "Only GLXFBConfigs for which the set bits of attribute include all
   the bits that are set in the requested value are
   considered. (Additional bits might be set in the attribute)."

The current test returns true if the two bit masks have bits in
common, specifically it matches even if the requested value has bits
set that are not set in the fbconfig attribute.  For example, an
application asking for

  GLX_DRAWABLE_TYPE, GLX_PIXMAP_BIT | GLX_PBUFFER_BIT,

as glxpbdemo does, will match fbconfigs that don't support pbuffer
rendering, as long as they support pixmap rendering.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/glx/glxcmds.c