Re: [Mesa-dev] [PATCH 4/9] i965: remove unused members in blorp clear program

On Thu, Nov 28, 2013 at 11:50:24PM -0800, Kenneth Graunke wrote: On 11/28/2013 11:44 PM, Kenneth Graunke wrote: On 11/27/2013 01:13 PM, Topi Pohjolainen wrote: Documentation for R0 and R1 is taken from fs_visitor::setup_payload_gen6(). Signed-off-by: Topi Pohjolainen

[Mesa-dev] [PATCH 0/6] Add support for 'sample' qualifier

This series adds the mesa and glsl compiler support for the new 'sample in' and 'sample out' qualifiers from GLSL 4.0 / ARB_gpu_shader5. Driver support (beyond triggering per-sample fragment shader evaluation) is not yet implemented. ___ mesa-dev

[Mesa-dev] [PATCH 4/6] mesa: add IsSample bitfield to gl_fragment_program

Drivers will need to look at this to decide if they need to do per-sample fragment shader dispatch. Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/main/mtypes.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index

[Mesa-dev] [PATCH 3/6] glsl: Put `sample`-qualified varyings in their own packing classes

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/glsl/link_varyings.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp index d2a4fc8..097cee5 100644 --- a/src/glsl/link_varyings.cpp +++ b/src/glsl/link_varyings.cpp

[Mesa-dev] [PATCH 5/6] glsl: Populate gl_fragment_program::IsSample bitfield

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/glsl/ir_set_program_inouts.cpp | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/glsl/ir_set_program_inouts.cpp b/src/glsl/ir_set_program_inouts.cpp index ab23538..1a36527 100644 ---

[Mesa-dev] [PATCH 1/6] glsl: Add frontend support for `sample` auxiliary storage qualifier

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/glsl/ast.h | 1 + src/glsl/ast_type.cpp | 3 ++- src/glsl/glsl_lexer.ll | 2 +- src/glsl/glsl_parser.yy | 9 +++-- src/glsl/glsl_parser_extras.cpp | 2 ++ 5 files changed, 13 insertions(+), 4

[Mesa-dev] [PATCH 6/6] mesa: Require per-sample shading if the `sample` qualifier is used.

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/program/program.c | 8 1 file changed, 8 insertions(+) diff --git a/src/mesa/program/program.c b/src/mesa/program/program.c index 01f8c6f..cdf1c03 100644 --- a/src/mesa/program/program.c +++ b/src/mesa/program/program.c @@

[Mesa-dev] [PATCH 2/6] glsl: Add ir support for `sample` qualifier; adjust compiler and linker

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/glsl/ast_to_hir.cpp | 15 +++ src/glsl/builtin_variables.cpp| 2 ++ src/glsl/glsl_types.cpp | 5 + src/glsl/glsl_types.h | 6 ++ src/glsl/ir.cpp

[Mesa-dev] [PATCH] i965: Don't flag gather quirks for Gen8+

My understanding is that Broadwell retains the same SCS mechanism that Haswell has, so even if the underlying issue with this format is not fixed, the w/a will be applied in SCS rather than needing shader code. Signed-off-by: Chris Forbes chr...@ijw.co.nz Cc: Kenneth Graunke kenn...@whitecape.org

[Mesa-dev] [PATCH 1/2] nv50: fix a small leak on context destroy

Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 9.2 10.0 mesa-sta...@lists.freedesktop.org --- Found with valgrind. src/gallium/drivers/nouveau/nv50/nv50_context.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_context.c

[Mesa-dev] [PATCH 2/2] mesa: don't leak performance monitors on context destroy

Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 10.0 mesa-sta...@lists.freedesktop.org --- Found with valgrind. Don't have the hardware to test a real implementation, but with nv50 it seemed to work in that valgrind was no longer marking the hash table as leaked. src/mesa/main/context.c

Re: [Mesa-dev] Fw: Taking part in MESA development - Dissertation Project

On Fri, 2013-11-29 at 09:36 +0200, Petri Latvala wrote: On 11/28/2013 11:15 PM, Timothy Arceri wrote: Hi guys, I received the following submitted as an Issue on my github account. Maybe someone here has a project they can suggest. Is NewbieProjects too newbie for this?

[Mesa-dev] [PATCH] egl: add HAVE_LIBDRM define, fix EGL X11 platform

Commit a594cec broke EGL X11 backend by adding dependency between X11 and DRM backends requiring HAVE_EGL_PLATFORM_DRM defined for X11. This patch fixes the issue by adding additional define for libdrm detection independent of which backend is being compiled. Tested by compiling Mesa with

Re: [Mesa-dev] Fw: Taking part in MESA development - Dissertation Project

The tesselation shaders and the compute shader are definitely not trivial. Marek On Fri, Nov 29, 2013 at 12:01 PM, Timothy Arceri t_arc...@yahoo.com.au wrote: On Fri, 2013-11-29 at 09:36 +0200, Petri Latvala wrote: On 11/28/2013 11:15 PM, Timothy Arceri wrote: Hi guys, I received the

Re: [Mesa-dev] Fw: Taking part in MESA development - Dissertation Project

They are not a newbie task, too. On Fri, Nov 29, 2013 at 3:34 PM, Marek Olšák mar...@gmail.com wrote: The tesselation shaders and the compute shader are definitely not trivial. Marek On Fri, Nov 29, 2013 at 12:01 PM, Timothy Arceri t_arc...@yahoo.com.au wrote: On Fri, 2013-11-29 at 09:36

[Mesa-dev] [PATCH][9.2] st/xorg: Handle new DamageUnregister API which has only one argument

This fixes building against the new API in X server 1.15 Taken from xf86-video-modesetting beca4dfb0e4d11d3729214967a1fe56ee5669831 from Keith Packard --- src/gallium/state_trackers/xorg/xorg_driver.c | 4 1 file changed, 4 insertions(+) diff --git

[Mesa-dev] [PATCH 01/10] radeon: move some functions to r600_buffer.c

From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeon/Makefile.sources | 1 + src/gallium/drivers/radeon/r600_buffer.c | 133 ++ src/gallium/drivers/radeon/r600_pipe_common.c | 106 3 files changed, 134 insertions(+), 106

[Mesa-dev] [PATCH 00/10] Sharing r600g glMapBuffer optimizations with radeonsi

This series moves the r600_buffer.c files from both drivers to the shared directory gallium/drivers/radeon, and implements what's missing for radeonsi to make sharing the code possible. This improves Valve's Team Fortress 2 performance by 75%. Before: 20 fps After: 35 fps Please review.

[Mesa-dev] [PATCH 03/10] radeonsi: implement accelerated buffer copying

From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeonsi/r600_blit.c | 7 ++-- src/gallium/drivers/radeonsi/si_descriptors.c | 58 +++ src/gallium/drivers/radeonsi/si_state.h | 3 ++ 3 files changed, 64 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH 05/10] radeonsi: handle PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE

From: Marek Olšák marek.ol...@amd.com which can come from glBufferData and glMapBufferRange. --- src/gallium/drivers/radeonsi/r600_buffer.c| 11 +++ src/gallium/drivers/radeonsi/si_descriptors.c | 123 ++ src/gallium/drivers/radeonsi/si_state.h | 1 + 3 files

[Mesa-dev] [PATCH 06/10] r600g, radeonsi: share flags has_cp_dma and has_streamout

From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/r600/evergreen_hw_context.c | 2 +- src/gallium/drivers/r600/evergreen_state.c | 4 ++-- src/gallium/drivers/r600/r600_blit.c| 8 src/gallium/drivers/r600/r600_buffer.c | 4 ++--

[Mesa-dev] [PATCH 07/10] r600g: refactor out code for buffer invalidation

From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/r600/r600_buffer.c | 56 +--- src/gallium/drivers/r600/r600_pipe.h | 1 + src/gallium/drivers/r600/r600_state_common.c | 55 +++ 3 files changed, 57 insertions(+), 55

[Mesa-dev] [PATCH 09/10] r600g, radeonsi: add common interface for buffer invalidation

From: Marek Olšák marek.ol...@amd.com This will be used by common code in the next commit. --- src/gallium/drivers/r600/r600_buffer.c| 2 +- src/gallium/drivers/r600/r600_pipe.h | 1 - src/gallium/drivers/r600/r600_state_common.c | 3 ++-

[Mesa-dev] [PATCH 10/10] r600g, radeonsi: consolidate buffer code, add handling of DISCARD_RANGE for SI

From: Marek Olšák marek.ol...@amd.com This adds 2 optimizations for radeonsi: - handling of DISCARD_RANGE - mapping an uninitialized buffer range is automatically UNSYNCHRONIZED --- src/gallium/drivers/r600/Makefile.sources | 1 - src/gallium/drivers/r600/r600_buffer.c| 202

[Mesa-dev] [PATCH 04/10] radeon: squash with buffer.c

From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeon/r600_buffer.c | 14 ++ src/gallium/drivers/radeon/r600_pipe_common.c | 14 -- src/gallium/drivers/radeon/r600_pipe_common.h | 22 -- 3 files changed, 26 insertions(+), 24

[Mesa-dev] [PATCH 08/10] r600g, radeonsi: consolidate some debug flags

From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/r600/r600_pipe.c | 3 --- src/gallium/drivers/r600/r600_pipe.h | 1 - src/gallium/drivers/radeon/r600_pipe_common.c | 4 src/gallium/drivers/radeon/r600_pipe_common.h | 1 + 4 files changed, 5 insertions(+), 4

[Mesa-dev] [PATCH 02/10] r600g: use common interfaces in buffer_transfer_unmap

From: Marek Olšák marek.ol...@amd.com i.e. dma_copy and resource_copy_region. --- src/gallium/drivers/r600/evergreen_state.c | 6 ++ src/gallium/drivers/r600/r600_blit.c | 4 ++-- src/gallium/drivers/r600/r600_buffer.c | 18 -- src/gallium/drivers/r600/r600_pipe.h

Re: [Mesa-dev] [PATCH 00/10] Sharing r600g glMapBuffer optimizations with radeonsi

BTW the improvement only applies to the radeonsi driver. Marek On Fri, Nov 29, 2013 at 6:55 PM, Marek Olšák mar...@gmail.com wrote: This series moves the r600_buffer.c files from both drivers to the shared directory gallium/drivers/radeon, and implements what's missing for radeonsi to make

Re: [Mesa-dev] [PATCH 04/10] radeon: squash with buffer.c

Sorry, I forgot to merge this patch with the first one (which is what the commit message says). I'll do so before committing. Marek On Fri, Nov 29, 2013 at 6:55 PM, Marek Olšák mar...@gmail.com wrote: From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeon/r600_buffer.c |

[Mesa-dev] [PATCH] winsys/radeon: set/get the scanout flag with the tiling ioctls

From: Marek Olšák marek.ol...@amd.com If we assume that all buffers allocated by the DDX are scanout, a new flag that says this is not scanout has to be added to support the non-scanout buffers and maintain backward compatibility. This fixes bad rendering on Wayland. The flag is defined as:

Re: [Mesa-dev] [PATCH 01/10] radeon: move some functions to r600_buffer.c

Reviewed-by: Christoph Brill egore...@gmail.com 2013/11/29 Marek Olšák mar...@gmail.com From: Marek Olšák marek.ol...@amd.com --- src/gallium/drivers/radeon/Makefile.sources | 1 + src/gallium/drivers/radeon/r600_buffer.c | 133 ++

[Mesa-dev] [PATCH 1/2 v2] nv50: fix a small leak on context destroy

Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 9.2 10.0 mesa-sta...@lists.freedesktop.org --- v2: use FREE instead of free. src/gallium/drivers/nouveau/nv50/nv50_context.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_context.c

[Mesa-dev] [PATCH 5/6] i965/vs: Sample from MCS surface when required

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/brw_vec4.h | 1 + src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 41 +- 2 files changed, 35 insertions(+), 7 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h

[Mesa-dev] [PATCH 1/6] i965: make compute_msaa_layout() nonstatic, add intel_ prefix

We're about to need this in the computation of the sampler key. Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 +++--- src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 0/6] i965: Enable CMS layout for multisample textures

This series enables the compressed multisample layout for multisample textures. Previously we would only use CMS for renderbuffers, since our texelFetch() implementation didn't understand it. Known to break the sample-mask-execution -tex test, but not sure why yet -- it doesn't use texelFetch().

[Mesa-dev] [PATCH 6/6] i965/Gen7: Allow CMS layout for multisample textures

Now that all the pieces are in place, this should provide a nice performance boost for apps using multisample textures. Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff

[Mesa-dev] [PATCH 3/6] i965: Add shader opcode for sampling MCS surface

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/brw_defines.h | 1 + src/mesa/drivers/dri/i965/brw_fs.cpp | 1 + src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 5 + src/mesa/drivers/dri/i965/brw_shader.cpp | 3 +++

[Mesa-dev] [PATCH 4/6] i965/fs: Sample from MCS surface when required

Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/brw_fs.h | 3 +- src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 2 +- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 46 +++- 3 files changed, 41 insertions(+), 10 deletions(-) diff

[Mesa-dev] [PATCH 2/6] i965/Gen7: Include bitfield in the sampler key for CMS layout

We need to emit extra shader code in this case to sample the MCS surface first; we can't just blindly do this all the time since IVB will sometimes try to access the MCS surface even if disabled. Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/brw_program.h | 5 +

[Mesa-dev] [PATCH 2/2] nv50: wait on the buf's fence before sticking it into pushbuf

This resolves some rendering issues in source games. See https://bugs.freedesktop.org/show_bug.cgi?id=64323 Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 9.2 10.0 mesa-sta...@lists.freedesktop.org --- Doing a nouveau_bo_wait works as well, but I got a slightly higher framerate from

[Mesa-dev] [PATCH 1/2] nouveau: avoid leaking fences while waiting

This fixes a memory leak in some situations. Also avoids emitting an extra fence if the kick handler does the call to nouveau_fence_next itself. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 9.2 10.0 mesa-sta...@lists.freedesktop.org --- TBH I'm pretty confused by the whole fence

Re: [Mesa-dev] [PATCH 2/4] glsl: Fix loop analysis of nested loops.

On 28 November 2013 11:45, Chris Forbes chr...@ijw.co.nz wrote: + /* The assignmnet to the variable in the loop must be unconditional and + * not inside a nested loop. */ s/assignmnet/assignment/ Fixed, thanks. ___ mesa-dev