[Mesa-dev] Mesa 8.0 regression - two bugs

2012-07-12 Thread Deve
Hi! I'm writing about two bugs which I see on 8.0 - now 8.0.4 version of mesa. It was OK on mesa 7.11.2, after upgrade to 8.0 is problem. First is starting movie in Sims 3 run by Wine. It runs VERY slow. The later part of game runs ok. Example: http://imageshack.us/clip/my-videos/444/f5f.mp4

Re: [Mesa-dev] [PATCH 00/11] Add YUYV format support of dri image

2012-07-12 Thread Gwenole Beauchesne
Hi Halley, 2012/7/12 Zhao, Halley : > There is no other comments for these patches, could you help me commit it to > git tree? Please rebase against current git master tree. YUV support, including YUYV was added. The remaining patchset should reduce to the GBM bits. Thanks, Gwenole. __

[Mesa-dev] [Bug 51972] Compilation error on x86-64 with --enable-32-bit option

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51972 Michel Dänzer changed: What|Removed |Added Attachment #64118|text/x-log |text/plain mime type|

[Mesa-dev] [Bug 51972] Compilation error on x86-64 with --enable-32-bit option

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51972 --- Comment #3 from Michel Dänzer 2012-07-12 08:04:47 PDT --- The below looks like your system environment for 32 bit compilation is broken. configure:20439: checking expat.h usability configure:20439: gcc -c -g -O2 -Wall -std=c99 -Werror=impli

[Mesa-dev] [PATCH] mesa, st/mesa: implement GL_RGB565 from ARB_ES2_compatibility

2012-07-12 Thread Marek Olšák
This was not implemented, because the spec was changed just recently. Everything has been in place already. Gallium has PIPE_FORMAT_B5G6R5_UNORM, while Mesa has MESA_FORMAT_RGB565. --- src/mesa/main/fbobject.c |3 +++ src/mesa/main/image.c |1 + src/m

Re: [Mesa-dev] [PATCH 5/5] i965/fs: Make register spill/unspill only do the regs for that instruction.

2012-07-12 Thread Kenneth Graunke
On 07/09/2012 03:40 PM, Eric Anholt wrote: Previously, if we were spilling the result of a texture call, we would store all 4 regs, then for each use of one of those regs as the source of an instruction, we would unspill all 4 regs even though only one was needed. In an app we're looking at, one

Re: [Mesa-dev] [PATCH] mesa, st/mesa: implement GL_RGB565 from ARB_ES2_compatibility

2012-07-12 Thread Kenneth Graunke
On 07/12/2012 05:25 AM, Marek Olšák wrote: This was not implemented, because the spec was changed just recently. Everything has been in place already. Gallium has PIPE_FORMAT_B5G6R5_UNORM, while Mesa has MESA_FORMAT_RGB565. Oh wow, I didn't realize this was missing. These changes look good t

Re: [Mesa-dev] [PATCH 00/11] Add YUYV format support of dri image

2012-07-12 Thread Ian Romanick
On 07/11/2012 10:47 PM, Zhao, Halley wrote: Hi Ian: There is no other comments for these patches, could you help me commit it to git tree? I haven't seen any Reviewed-by or Acked-by messages. This is a requirement. Thanks. -Original Message- From: Zhao, Halley Sent: Monday, July

[Mesa-dev] [PATCH 0/7] i965/blorp: Improvements to the blorp engine for Gen7.

2012-07-12 Thread Paul Berry
This patch series makes three improvements to the blorp engine (which does MSAA resolves and other blits) for Gen7: Patches 1-3 fix downsampling of integer format framebuffers on Gen7, by using the "AVG" instruction to average the samples, rather than adding the samples and performing a division.

[Mesa-dev] [PATCH 1/7] i965: Add support for AVG instruction.

2012-07-12 Thread Paul Berry
>From the Ivy Bridge PRM, Vol4 Part3 p152: "The avg instruction performs component-wise integer average of src0 and src1 and stores the results in dst. An integer average uses integer upward rounding. It is equivalent to increment one to the addition of src0 and src1 and then apply

[Mesa-dev] [PATCH 2/7] i965/blorp: Modify manual_blend() to avoid unnecessary loss of precision.

2012-07-12 Thread Paul Berry
When downsampling from an MSAA image to a single-sampled image, it is inevitable that some loss of numerical precision will occur, since we have to use 32-bit floating point registers to hold the intermediate results while blending. However, it seems reasonable to expect that when all samples corr

[Mesa-dev] [PATCH 3/7] i965/blorp: Fix integer downsampling on Gen7.

2012-07-12 Thread Paul Berry
When downsampling an integer-format buffer on Gen7, we need to use the "avg" instruction rather than the "add" instruction, to ensure that we don't overflow the range of 32-bit integers. Also, we need to use the proper register type (BRW_REGISTER_TYPE_D or BRW_REGISTER_TYPE_UD) for intermediate co

[Mesa-dev] [PATCH 4/7] i965/blorp: Optimize manual_blend() for compressed multisampled surfaces.

2012-07-12 Thread Paul Berry
When downsampling a compressed multisampled surface, we can take a shortcut to downsample any pixels that were completely covered by a single primitive. In this case, the first color value we fetch is the correct final color for the downsampled pixel, so we can skip the rest of the blending operat

[Mesa-dev] [PATCH 5/7] i965/blorp: Configure SURFACE_STATE correctly for IMS surfaces.

2012-07-12 Thread Paul Berry
This patch modifies gen7_set_surface_num_multisamples() to set up the SURFACE_STATE appropriately for texturing from IMS format MSAA surfaces (which are only used on Gen7 for depth and stencil buffers). Since the function now sets more than just the number of multisamples, it's been renamed to gen7

[Mesa-dev] [PATCH 6/7] i965/blorp: Loosen assertions in compute_msaa_layout_for_pipeline.

2012-07-12 Thread Paul Berry
Previously, on Gen7, compute_msaa_layout_for_pipeline() would verify that IMS layout is not used. However, now that we configure SURFACE_STATE correctly for IMS surfaces, IMS layout is available. --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp |9 ++--- 1 files changed, 2 insertions(+),

[Mesa-dev] [PATCH 7/7] i965/blorp: Use IMS layout when texturing from depth/stencil surfaces.

2012-07-12 Thread Paul Berry
Previously, on Gen7, when texturing from a depth or stencil surface, the blorp engine would configure the 3D pipeline as though the input surface was non-multisampled, and perform the necessary coordinate transformations in the fragment shader to account for the IMS layout. This meant outputting a

[Mesa-dev] [Bug 51972] Compilation error on x86-64 with --enable-32-bit option

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51972 --- Comment #4 from virgile.bes...@free.fr 2012-07-12 17:47:07 PDT --- Created attachment 64145 --> https://bugs.freedesktop.org/attachment.cgi?id=64145 config.log -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email

Re: [Mesa-dev] [PATCH 1/7] i965: Add support for AVG instruction.

2012-07-12 Thread Kenneth Graunke
On 07/12/2012 10:43 AM, Paul Berry wrote: > From the Ivy Bridge PRM, Vol4 Part3 p152: > > "The avg instruction performs component-wise integer average of > src0 and src1 and stores the results in dst. An integer average > uses integer upward rounding. It is equivalent to increment one

Re: [Mesa-dev] Mesa 8.0 regression - two bugs

2012-07-12 Thread Sven Arvidsson
On Wed, 2012-07-11 at 21:25 +0200, Deve wrote: > Hi! > > I'm writing about two bugs which I see on 8.0 - now 8.0.4 version of > mesa. It was OK on mesa 7.11.2, after upgrade to 8.0 is problem. > > First is starting movie in Sims 3 run by Wine. It runs VERY slow. The > later part of game runs ok

Re: [Mesa-dev] [PATCH] mesa, st/mesa: implement GL_RGB565 from ARB_ES2_compatibility

2012-07-12 Thread Marek Olšák
On Thu, Jul 12, 2012 at 6:42 PM, Kenneth Graunke wrote: > On 07/12/2012 05:25 AM, Marek Olšák wrote: >> >> This was not implemented, because the spec was changed just recently. >> >> Everything has been in place already. >> >> Gallium has PIPE_FORMAT_B5G6R5_UNORM, while Mesa has MESA_FORMAT_RGB565

[Mesa-dev] [Bug 51972] Compilation error on x86-64 with --enable-32-bit option

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51972 --- Comment #5 from virgile.bes...@free.fr 2012-07-12 18:25:31 PDT --- (In reply to comment #3) > The below looks like your system environment for 32 bit compilation is broken. > > configure:20439: checking expat.h usability > configure:20439: gc

Re: [Mesa-dev] [PATCH 2/9] automake: convert libglsl

2012-07-12 Thread Matt Turner
On Wed, Jul 11, 2012 at 11:58 AM, Jon TURNEY wrote: > v2: Use AM_V_GEN to silence generated code rules. Add BUILT_SOURCES to > CLEANFILES > v3: > - Fix an accidental // in a path > - Use automake make rules for lex/yacc rather than writing our own > - Update .gitignore appropriately > - Build a l

[Mesa-dev] [PATCH 1/2] r600g: Unify 3D and compute vertex buffer emission

2012-07-12 Thread Tom Stellard
--- src/gallium/drivers/r600/evergreen_compute.c | 47 +--- .../drivers/r600/evergreen_compute_internal.c | 81 .../drivers/r600/evergreen_compute_internal.h |1 - src/gallium/drivers/r600/evergreen_state.c | 30 +-- src/gallium/dr

[Mesa-dev] [PATCH 2/2] r600g: Emit vertex buffers using the same method as constant buffers

2012-07-12 Thread Tom Stellard
--- src/gallium/drivers/r600/evergreen_compute.c |8 +++-- src/gallium/drivers/r600/evergreen_state.c | 45 - src/gallium/drivers/r600/r600_buffer.c |5 ++- src/gallium/drivers/r600/r600_hw_context.c |2 +- src/gallium/drivers/r600/r600_pipe.h

[Mesa-dev] [PATCH 1/2] draw,gallivm: Fix draw_get_shader_param.

2012-07-12 Thread jfonseca
From: José Fonseca - Use LLVM limits when LLVM is being used, instead of TGSI limits - Provide draw_get_shader_param_no_llvm for when llvm is never used (softpipe) - Eliminate several of the hacks around draw shader caps in several drivers Unfortunately the hack for PIPE_MAX_VERTEX_SAMPLERS is s

[Mesa-dev] [PATCH 2/2] mesa/st: Generates TGSI that always recognizes INSTANCEID/VERTEXID as integers.

2012-07-12 Thread jfonseca
From: José Fonseca Tested by running piglit draw-instanced, and by forcing llvmpipe advertise no native integer support, which now produces: VERT DCL IN[0] DCL SV[0], INSTANCEID DCL OUT[0], POSITION DCL OUT[1], COLOR DCL CONST[0..19] DCL TEMP[0], LOCAL DCL TEMP[1], LOCAL DCL TEMP[2], LOCAL DCL

Re: [Mesa-dev] [PATCH] mesa, st/mesa: implement GL_RGB565 from ARB_ES2_compatibility

2012-07-12 Thread Kenneth Graunke
On 07/12/2012 11:12 AM, Marek Olšák wrote: On Thu, Jul 12, 2012 at 6:42 PM, Kenneth Graunke wrote: On 07/12/2012 05:25 AM, Marek Olšák wrote: This was not implemented, because the spec was changed just recently. Everything has been in place already. Gallium has PIPE_FORMAT_B5G6R5_UNORM, whi

Re: [Mesa-dev] [PATCH 1/2] r600g: Unify 3D and compute vertex buffer emission

2012-07-12 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Marek On Thu, Jul 12, 2012 at 9:50 PM, Tom Stellard wrote: > --- > src/gallium/drivers/r600/evergreen_compute.c | 47 +--- > .../drivers/r600/evergreen_compute_internal.c | 81 > > .../drivers/r600/evergreen

Re: [Mesa-dev] [PATCH 1/2] draw, gallivm: Fix draw_get_shader_param.

2012-07-12 Thread Marek Olšák
On Thu, Jul 12, 2012 at 9:50 PM, wrote: > From: José Fonseca > > - Use LLVM limits when LLVM is being used, instead of TGSI limits > - Provide draw_get_shader_param_no_llvm for when llvm is never used (softpipe) > - Eliminate several of the hacks around draw shader caps in several drivers > > Un

[Mesa-dev] [Bug 52033] New: [swrast] swrast/s_texfetch.c:1156: set_fetch_functions: Assertion `texImage->FetchTexel' failed.

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=52033 Bug #: 52033 Summary: [swrast] swrast/s_texfetch.c:1156: set_fetch_functions: Assertion `texImage->FetchTexel' failed. Classification: Unclassified Product: Mesa

[Mesa-dev] [Bug 52034] New: [llvmpipe] piglit depthstencil-default_fb-blit regression

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=52034 Bug #: 52034 Summary: [llvmpipe] piglit depthstencil-default_fb-blit regression Classification: Unclassified Product: Mesa Version: git Platform: x86-64 (AMD64)

[Mesa-dev] [Bug 52034] [llvmpipe] piglit depthstencil-default_fb-blit regression

2012-07-12 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=52034 --- Comment #1 from Marek Olšák 2012-07-13 04:11:50 PDT --- This is not a regression. The stencil blit has never been implemented for llvmpipe. The test used to pass incorrectly. -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?