<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 14 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> From f93a54e7447a6f30c68d31ac82637e724c3953eb Mon Sep 17 00:00:00 2001<o:p></o:p> From: Daniel Czarnowski <daniel.czarnowski@intel.com><o:p></o:p> Date: Wed, 16 Oct 2013 13:36:33 +0200<o:p></o:p> Subject: [PATCH] Support of GLX_RGBA*_FLOAT_BIT*, and correct setting of the <o:p></o:p> flags. Also commented each renderType use with information which (fbconfig<o:p></o:p> or context) RENDER_TYPE it is. Changes in DMX component.<o:p></o:p> <o:p> </o:p> ---<o:p></o:p> hw/dmx/dmx_glxvisuals.c | 4 +++-<o:p></o:p> hw/dmx/glxProxy/glxcmds.c | 36 +++++++++++++++++++++++++++++-------<o:p></o:p> 2 files changed, 32 insertions(+), 8 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/hw/dmx/dmx_glxvisuals.c b/hw/dmx/dmx_glxvisuals.c<o:p></o:p> index 56bd67b..0cd5f21 100644<o:p></o:p> --- a/hw/dmx/dmx_glxvisuals.c<o:p></o:p> +++ b/hw/dmx/dmx_glxvisuals.c<o:p></o:p> @@ -448,7 +448,9 @@ GetGLXFBConfigs(Display * dpy, int glxMajorOpcode, int *nconfigs)<o:p></o:p> /* Fill in derived values */<o:p></o:p> config->screen = screen;<o:p></o:p> <o:p></o:p> - config->rgbMode = config->renderType & GLX_RGBA_BIT;<o:p></o:p> + /* The rgbMode should be true for any mode which has distinguishible R, G and B components */<o:p></o:p> + config->rgbMode = (config->renderType & (GLX_RGBA_BIT |<o:p></o:p> + GLX_RGBA_FLOAT_BIT_ARB | GLX_RGBA_UNSIGNED_FLOAT_BIT_EXT)) != 0;<o:p></o:p> config->colorIndexMode = !config->rgbMode;<o:p></o:p> <o:p></o:p> config->haveAccumBuffer =<o:p></o:p> diff --git a/hw/dmx/glxProxy/glxcmds.c b/hw/dmx/glxProxy/glxcmds.c<o:p></o:p> index 8cdb25e..fb56017 100644<o:p></o:p> --- a/hw/dmx/glxProxy/glxcmds.c<o:p></o:p> +++ b/hw/dmx/glxProxy/glxcmds.c<o:p></o:p> @@ -123,6 +123,21 @@ GetBackEndDisplay(__GLXclientState * cl, int s)<o:p></o:p> return cl->be_displays[s];<o:p></o:p> }<o:p></o:p> <o:p></o:p> +/* Convert the render type bits from fbconfig into context render type. */<o:p></o:p> +static int convFBconfRenderTypeBits2CtxRenderType(int fbRenderType)<o:p></o:p> +{<o:p></o:p> + if (fbRenderType & GLX_RGBA_BIT)<o:p></o:p> + return GLX_RGBA_TYPE;<o:p></o:p> + if (fbRenderType & GLX_COLOR_INDEX_BIT)<o:p></o:p> + return GLX_COLOR_INDEX_TYPE;<o:p></o:p> + if (fbRenderType & GLX_RGBA_FLOAT_BIT_ARB)<o:p></o:p> + return GLX_RGBA_FLOAT_TYPE_ARB;<o:p></o:p> + if (fbRenderType & GLX_RGBA_UNSIGNED_FLOAT_BIT_EXT)<o:p></o:p> + return GLX_RGBA_UNSIGNED_FLOAT_TYPE_EXT;<o:p></o:p> + /* There's no recognized renderType in the config */<o:p></o:p> + return GLX_RGBA_TYPE;<o:p></o:p> +}<o:p></o:p> +<o:p></o:p> /*<o:p></o:p> ** Create a GL context with the given properties.<o:p></o:p> */<o:p></o:p> @@ -308,12 +323,13 @@ CreateContext(__GLXclientState * cl,<o:p></o:p> /* send the create context request to the back-end server */<o:p></o:p> dpy = GetBackEndDisplay(cl, screen);<o:p></o:p> if (glxc->pFBConfig) {<o:p></o:p> - /*Since for a certain visual both RGB and COLOR INDEX<o:p></o:p> - *can be on then the only parmeter to choose the renderType<o:p></o:p> - * should be the class of the colormap since all 4 first <o:p></o:p> - * classes does not support RGB mode only COLOR INDEX ,<o:p></o:p> - * and so TrueColor and DirectColor does not support COLOR INDEX*/<o:p></o:p> - int renderType = glxc->pFBConfig->renderType;<o:p></o:p> + /* For a specific visual, multiple render types (ie. both RGB and COLOR INDEX)<o:p></o:p> + * can be accessible. The only parameter to choose the renderType<o:p></o:p> + * should be the class of the colormap, since all 4 first classes<o:p></o:p> + * does not support RGB mode only COLOR INDEX,<o:p></o:p> + * and so TrueColor and DirectColor does not support COLOR INDEX. <o:p></o:p> + */<o:p></o:p> + int renderType = GLX_RGBA_TYPE;<o:p></o:p> <o:p></o:p> if (pVisual) {<o:p></o:p> switch (pVisual->class) {<o:p></o:p> @@ -329,7 +345,10 @@ CreateContext(__GLXclientState * cl,<o:p></o:p> renderType = GLX_RGBA_TYPE;<o:p></o:p> break;<o:p></o:p> }<o:p></o:p> + } else {<o:p></o:p> + renderType = convFBconfRenderTypeBits2CtxRenderType(glxc->pFBConfig->renderType);<o:p></o:p> }<o:p></o:p> +<o:p></o:p> if (__GLX_IS_VERSION_SUPPORTED(1, 3)) {<o:p></o:p> LockDisplay(dpy);<o:p></o:p> GetReq(GLXCreateNewContext, be_new_req);<o:p></o:p> @@ -3185,6 +3204,7 @@ __glXQueryContext(__GLXclientState * cl, GLbyte * pc)<o:p></o:p> __GLXcontext *ctx;<o:p></o:p> xGLXQueryContextReq *req;<o:p></o:p> xGLXQueryContextReply reply;<o:p></o:p> + int renderType;<o:p></o:p> int nProps;<o:p></o:p> int *sendBuf, *pSendBuf;<o:p></o:p> int nReplyBytes;<o:p></o:p> @@ -3197,6 +3217,8 @@ __glXQueryContext(__GLXclientState * cl, GLbyte * pc)<o:p></o:p> return __glXBadContext;<o:p></o:p> }<o:p></o:p> <o:p></o:p> + renderType = convFBconfRenderTypeBits2CtxRenderType(ctx->pFBConfig->renderType);<o:p></o:p> +<o:p></o:p> nProps = 3;<o:p></o:p> <o:p></o:p> reply = (xGLXQueryContextReply) {<o:p></o:p> @@ -3212,7 +3234,7 @@ __glXQueryContext(__GLXclientState * cl, GLbyte * pc)<o:p></o:p> *pSendBuf++ = GLX_FBCONFIG_ID;<o:p></o:p> *pSendBuf++ = (int) (ctx->pFBConfig->id);<o:p></o:p> *pSendBuf++ = GLX_RENDER_TYPE;<o:p></o:p> - *pSendBuf++ = (int) (ctx->pFBConfig->renderType);<o:p></o:p> + *pSendBuf++ = renderType; /* context render type (one of GLX_*_TYPE values) */<o:p></o:p> *pSendBuf++ = GLX_SCREEN;<o:p></o:p> *pSendBuf++ = (int) (ctx->pScreen->myNum);<o:p></o:p> <o:p></o:p> -- <o:p></o:p> 1.8.1.2<o:p></o:p> </div> --------------------------------------------------------------------- Intel Technology Poland sp. z o.o. ul. S&#322owackiego 173 | 80-298 Gda&#324sk | S&#261d Rejonowy Gda&#324sk P&#243&#322noc | VII Wydzia&#322 Gospodarczy Krajowego Rejestru S&#261dowego - KRS 101882 | NIP 957-07-52-316 | Kapita&#322 zak&#322adowy 200.000 PLN. <span style="font-size:8.0pt;font-family:"Arial Narrow","sans-serif"; mso-fareast-font-family:"Times New Roman";mso-bidi-font-family:Arial; color:#595959;mso-ansi-language:EN-US;mso-fareast-language:EN-US;mso-bidi-language: AR-SA">Ta wiadomo&#347&#263 wraz z za&#322&#261cznikami jest przeznaczona dla okre&#347lonego adresata i mo&#380e zawiera&#263 informacje poufne. W razie przypadkowego otrzymania tej wiadomo&#347ci, prosimy o powiadomienie nadawcy oraz trwa&#324e jej usuni&#281cie; jakiekolwiek przegl&#261danie lub rozpowszechnianie jest zabronione. This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.<o:p></o:p> </body> </html>