From: Ian Romanick Date: Mon, 21 Jan 2013 01:39:54 +0000 (-0500) Subject: intel: Don't expose XRGB8888 visuals any more X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=75b7e1df139676f2456fea4d3a57cf0044d8409e;p=mesa.git intel: Don't expose XRGB8888 visuals any more There really isn't any point. There is no resource savings, and we have to do gymnastics in the driver to make it work. There are also bad interactions with multisampling and OpenGL ES 3.0. In ES3, a multisample-to-singlesample blit must have identical source and destination format. This means a multisample RGBA8 to singlesample RGB8 (window) blit will generate an error. Also in ES3, RGB8 is not a renderable format. This means that the application CANNOT make an RGB8 multisample renderbuffer. As a result, if an application gets an RGB8 window and wants to do multisample FBO rendering, it will probably break. "Fixes" gles3conform framebuffer_blit_functionality_multisampled_to_singlesampled_blit test on RGB8 visuals. v2: Fix 'formats' array size. Suggested by Ken. Signed-off-by: Ian Romanick Reviewed-by: Kenneth Graunke Acked-by: Eric Anholt --- diff --git a/src/mesa/drivers/dri/intel/intel_screen.c b/src/mesa/drivers/dri/intel/intel_screen.c index 750cb78832c..98dd2f3115d 100644 --- a/src/mesa/drivers/dri/intel/intel_screen.c +++ b/src/mesa/drivers/dri/intel/intel_screen.c @@ -904,9 +904,8 @@ intel_detect_swizzling(struct intel_screen *screen) static __DRIconfig** intel_screen_make_configs(__DRIscreen *dri_screen) { - static const gl_format formats[3] = { + static const gl_format formats[] = { MESA_FORMAT_RGB565, - MESA_FORMAT_XRGB8888, MESA_FORMAT_ARGB8888 };