Re: [Mesa-dev] [PATCH 00/16 v2] i965: Enable winsys MSAA on gen >= 6

2012-08-02 Thread Chad Versace
By the way, this series is availabe on my winsys-msaa-v22 branch. -Chad ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 00/16 v2] i965: Enable winsys MSAA on gen >= 6

2012-08-02 Thread Chad Versace
For test results, see the commit message in patch 16. The summary is that piglit, oglconform, and xonotic are all happy. It was nice to play xonotic without seeing ugly, jagged edges. Many of these patches have already been reviewed. I'm not seeking additional review for those, although additional

[Mesa-dev] [PATCH 15/16] intel: Clarify intel_screen_make_configs

2012-08-02 Thread Chad Versace
This function felt sloppy, so this patch cleans it up a little bit. - Rename `color` to `i`. It is not a color value, only an iterator int. - Move `depth_bits[0] = 0` into the non-accum loop because that is where it used. The accum loop later overwrites depth_bits[0]. - Redefine `msaa_samples_ar

[Mesa-dev] [PATCH 16/16] intel: Advertise multisample DRI2 configs on gen >= 6

2012-08-02 Thread Chad Versace
This turns on window system MSAA. This patch changes the id of many GLX visuals and configs, but that couldn't be prevented. I attempted to preserve the id's of extant configs by appending the multisample configs to the end of the extant ones. But somewhere, perhaps in the X server, the configs ar

[Mesa-dev] [PATCH 14/16] dri: Simplify use of driConcatConfigs

2012-08-02 Thread Chad Versace
If either argument to driConcatConfigs(a, b) is null or the empty list, then simply return the other argument as the resultant list. All callers were accomplishing that same behavior anyway. And each caller accopmplished it with the same pattern. So this patch moves that external pattern into the

[Mesa-dev] [PATCH 13/16] intel: Refactor creation of DRI2 configs

2012-08-02 Thread Chad Versace
DRI2 configs were constructed in intelInitScreen2. That function already does too much, so move verbatim the code for creating configs to a new function, intel_screen_make_configs. CC: Eric Anholt Reviewed-by: Paul Berry Signed-off-by: Chad Versace --- src/mesa/drivers/dri/intel/intel_screen.c

[Mesa-dev] [PATCH 12/16] intel: Downsample on DRI2 flush

2012-08-02 Thread Chad Versace
CC: Eric Anholt Reviewed-by: Paul Berry Signed-off-by: Chad Versace --- src/mesa/drivers/dri/intel/intel_screen.c | 31 +++ 1 file changed, 31 insertions(+) diff --git a/src/mesa/drivers/dri/intel/intel_screen.c b/src/mesa/drivers/dri/intel/intel_screen.c index 9ac

[Mesa-dev] [PATCH 11/16] intel: Support mapping multisample miptrees

2012-08-02 Thread Chad Versace
Add two new functions: intel_miptree_{map,unmap}_multisample, to which intel_miptree_{map,unmap} dispatch. Only mapping flat, renderbuffer-like miptrees are supported. CC: Eric Anholt CC: Paul Berry Signed-off-by: Chad Versace --- src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 108 ++

[Mesa-dev] [PATCH 10/16] intel: Refactor use of intel_miptree_map

2012-08-02 Thread Chad Versace
Move the opencoded construction and destruction of intel_miptree_map into new functions, intel_miptree_attach_map and intel_miptree_release_map. This patch prevents code duplication in a future commit that adds support for mapping multisample miptrees. CC: Eric Anholt CC: Paul Berry Signed-off-b

[Mesa-dev] [PATCH 09/16] intel: Refactor intel_miptree_map/unmap

2012-08-02 Thread Chad Versace
Move the body of intel_miptree_map into a new function, intel_miptree_map_singlesample. Now intel_miptree_map dispatches to the new function. A future commit adds a multisample variant. Ditto for intel_miptree_unmap. CC: Paul Berry CC: Eric Anholt Signed-off-by: Chad Versace --- src/mesa/driv

[Mesa-dev] [PATCH 08/16] i965: Mark needed downsamples for msaa winsys buffers

2012-08-02 Thread Chad Versace
Do this at two locations: - Immediately after drawing is complete. - After blitting. v2: - Mark the front buffer too, per Paul. - Mark after blitting, per Paul. CC: Eric Anholt CC: Paul Berry Signed-off-by: Chad Versace --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 2 +-

[Mesa-dev] [PATCH 06/16] intel: Allocate miptree for multisample DRI2 buffers (v2)

2012-08-02 Thread Chad Versace
Immediately after obtaining, with DRI2GetBuffersWithFormat, the DRM buffer handle for a DRI2 buffer, we wrap that DRM buffer handle with a region and a miptree. This patch additionally allocates an accompanying multisample miptree if the DRI2 buffer is multisampled. Since we do not yet advertise m

[Mesa-dev] [PATCH 07/16] intel: Add function for marking needed downsample

2012-08-02 Thread Chad Versace
Add function intel_renderbuffer_set_needs_downsample. It is a no-op except on multisample winsys buffers shared with DRI2. Reviewed-by: Paul Berry Signed-off-by: Chad Versace --- src/mesa/drivers/dri/intel/intel_fbo.c | 10 ++ src/mesa/drivers/dri/intel/intel_fbo.h | 3 +++ 2 files cha

[Mesa-dev] [PATCH 04/16] intel: Refactor creation of hiz and mcs miptrees

2012-08-02 Thread Chad Versace
Move the logic for creating the ancillary hiz and mcs miptress for winsys and non-texture renderbuffers from intel_alloc_renderbuffer_storage to intel_miptree_create_for_renderbuffer. Let's try to isolate complex miptree logic to intel_mipmap_tree.c. Without this refactor, code duplication would b

[Mesa-dev] [PATCH 05/16] i965: Add functions up/downsampling on miptrees (v2)

2012-08-02 Thread Chad Versace
These functions do an up or downsample between mt and mt->singlesample_mt. Conceptually, these functions belong in intel_mipmap_tree.c. However, they needs to interact with blorp, which is C++. So I created a new file, brw_blorp_orphans.cpp, for these and other functions that fall into the same ca

[Mesa-dev] [PATCH 02/16] intel: Set num samples for winsys renderbuffers (v3)

2012-08-02 Thread Chad Versace
Add a new param, num_samples, to intel_create_renderbuffer and intel_create_private_renderbuffer. No multisample GL config is yet advertised, so the value of num_samples is currently 0. For server-owned winsys buffers, gl_renderbuffer::NumSamples is not yet used. v2: Quantize num_samples just on

[Mesa-dev] [PATCH 03/16] intel: Add singlesample fields to intel_mipmap_tree

2012-08-02 Thread Chad Versace
Add the following fields: singlesamle_width0 singlesample_height0 singlesample_mt needs_downsample These will all be used by later commits that implement MSAA for window system buffers. CC: Paul Berry Signed-off-by: Chad Versace --- src/mesa/drivers/dri/intel/intel_mipmap_tree.

[Mesa-dev] [PATCH 01/16] intel: Refactor quantize_num_samples (v1)

2012-08-02 Thread Chad Versace
Rename quantize_num_samples to intel_quantize_num_samples and change the first param from struct intel_context* to struct intel_screen*. The function will later be used by intelCreateBuffer, which is not bound to any context but is bound to a screen. v2: Keep the function in intel_fbo.c, per Anhol

Re: [Mesa-dev] [PATCH] gallium: consolidate CSO sampler and sampler_view functions

2012-08-02 Thread Marek Olšák
I just skimmed through it and couldn't find anything wrong. Reviewed-by: Marek Olšák Someday we should definitely do something like this in pipe_context too. Marek On Thu, Aug 2, 2012 at 9:16 PM, Brian Paul wrote: > Merge the vertex/fragment versions of the cso_set/save/restore_samplers() > f

Re: [Mesa-dev] [PATCH] r600g: atomize sampler state

2012-08-02 Thread Marek Olšák
This generally looks good. I have tested it on R700 and there are 2 regressions. I'll try to take a look at them tomorrow. 1) The test arb_seamless_cubemap is failing. I think the only way to make that feature work correctly is to add a separate atom for it. 2) spec/glsl-1.30/execution/vs-texture

Re: [Mesa-dev] [PATCH 0/4] Convert Nouveau gallium drivers to automake

2012-08-02 Thread Tom Stellard
On Fri, Aug 03, 2012 at 01:53:06AM +0200, Lucas Stach wrote: > Trivial series to convert Nouveau gallium drivers to automake > in order to generate Makefiles. > > This is just the general direction in which the MESA buildsystem > is moving. Aside from this it makes the "make" output a lot more > r

[Mesa-dev] [PATCH 4/4] nvc0: use automake to generate Makefile

2012-08-02 Thread Lucas Stach
--- configure.ac | 1 + src/gallium/drivers/nvc0/Makefile| 12 src/gallium/drivers/nvc0/Makefile.am | 25 + 3 Dateien ge??ndert, 26 Zeilen hinzugef??gt(+), 12 Zeilen entfernt(-) delete mode 100644 src/gallium/drivers/nvc0/Makefile

[Mesa-dev] [PATCH 2/4] nv30: use automake to generate Makefile

2012-08-02 Thread Lucas Stach
--- configure.ac | 1 + src/gallium/drivers/nv30/Makefile| 12 src/gallium/drivers/nv30/Makefile.am | 16 3 Dateien ge??ndert, 17 Zeilen hinzugef??gt(+), 12 Zeilen entfernt(-) delete mode 100644 src/gallium/drivers/nv30/Makefile create m

[Mesa-dev] [PATCH 3/4] nv50: use automake to generate Makefile

2012-08-02 Thread Lucas Stach
--- configure.ac | 1 + src/gallium/drivers/nv50/Makefile| 12 src/gallium/drivers/nv50/Makefile.am | 25 + 3 Dateien ge??ndert, 26 Zeilen hinzugef??gt(+), 12 Zeilen entfernt(-) delete mode 100644 src/gallium/drivers/nv50/Makefile

[Mesa-dev] [PATCH 0/4] Convert Nouveau gallium drivers to automake

2012-08-02 Thread Lucas Stach
Trivial series to convert Nouveau gallium drivers to automake in order to generate Makefiles. This is just the general direction in which the MESA buildsystem is moving. Aside from this it makes the "make" output a lot more readable and therefore helps to spot compiler warnings more easily. Lucas

[Mesa-dev] [PATCH 1/4] nouveau: use automake to generate Makefile

2012-08-02 Thread Lucas Stach
--- configure.ac| 1 + src/gallium/drivers/nouveau/Makefile| 16 src/gallium/drivers/nouveau/Makefile.am | 16 3 Dateien ge??ndert, 17 Zeilen hinzugef??gt(+), 16 Zeilen entfernt(-) delete mode 100644 src/gallium/drivers/nouveau/Ma

[Mesa-dev] [PATCH] i965/msaa: Add sample-alpha-to-coverage support for multiple render targets

2012-08-02 Thread Anuj Phogat
Render Target Write message should include source zero alpha value when sample-alpha-to-coverage is enabled for an FBO with multiple render targets. Source zero alpha value is used as fragment coverage for all the render targets. This patch makes piglit tests draw-buffers-alpha-to-coverage and al

Re: [Mesa-dev] [PATCH 2/2] r600g: Fix instruction group merge when there are predicated insts.

2012-08-02 Thread Tom Stellard
On Wed, Aug 1, 2012 at 4:52 PM, Vincent Lejeune wrote: > --- > src/gallium/drivers/r600/r600_asm.c | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/src/gallium/drivers/r600/r600_asm.c > b/src/gallium/drivers/r600/r600_asm.c > index 0f28824..b0194bf 100644 > --- a/src/

[Mesa-dev] [PATCH] gallium: consolidate CSO sampler and sampler_view functions

2012-08-02 Thread Brian Paul
Merge the vertex/fragment versions of the cso_set/save/restore_samplers() functions. Now we pass the shader stage (PIPE_SHADER_x) to the function to indicate vertex/fragment/geometry samplers. For example: cso_single_sampler(cso, PIPE_SHADER_FRAGMENT, unit, sampler); This results in quite a bit

Re: [Mesa-dev] [PATCH 1/2] radeonsi: Handle TGSI DIV opcode.

2012-08-02 Thread Tom Stellard
On Thu, Aug 02, 2012 at 05:37:02PM +0200, Michel Dänzer wrote: > From: Michel Dänzer > > Signed-off-by: Michel Dänzer Reviewed-by: Tom Stellard > --- > src/gallium/drivers/radeon/SIInstructions.td |5 + > 1 file changed, 5 insertions(+) > > diff --git a/src/gallium/drivers/radeon/SIIn

Re: [Mesa-dev] [PATCH 2/2] radeonsi: Handle TGSI TXP opcode.

2012-08-02 Thread Tom Stellard
On Thu, Aug 02, 2012 at 05:37:03PM +0200, Michel Dänzer wrote: > From: Michel Dänzer > > Signed-off-by: Michel Dänzer Reviewed-by: Tom Stellard > --- > src/gallium/drivers/radeonsi/radeonsi_shader.c | 26 > ++-- > 1 file changed, 24 insertions(+), 2 deletions(-) > > dif

[Mesa-dev] [PATCH 1/2] radeonsi: Handle TGSI DIV opcode.

2012-08-02 Thread Michel Dänzer
From: Michel Dänzer Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeon/SIInstructions.td |5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/drivers/radeon/SIInstructions.td b/src/gallium/drivers/radeon/SIInstructions.td index 94748b6..31b9e5e 100644 --- a/src/gall

[Mesa-dev] [PATCH 2/2] radeonsi: Handle TGSI TXP opcode.

2012-08-02 Thread Michel Dänzer
From: Michel Dänzer Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/radeonsi_shader.c | 26 ++-- 1 file changed, 24 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c b/src/gallium/drivers/radeonsi/radeonsi_shader.c in

Re: [Mesa-dev] [PATCH] st/mesa: Ensure dst in compile_instruction is initialized.

2012-08-02 Thread Brian Paul
On 08/02/2012 12:26 AM, Vinson Lee wrote: Fixes uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee --- src/mesa/state_tracker/st_mesa_to_tgsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_mesa_to_tgsi.c b/src

Re: [Mesa-dev] [PATCH 01/18] gallium: rename cso_save/restore_samplers -> cso_save/restore_fragment_samplers

2012-08-02 Thread Brian Paul
On 08/02/2012 07:14 AM, Marek Olšák wrote: --- src/gallium/auxiliary/cso_cache/cso_context.c |4 ++-- src/gallium/auxiliary/cso_cache/cso_context.h |4 ++-- src/gallium/auxiliary/util/u_blit.c |8 src/gallium/auxiliary/util/u_gen_mipmap.c |4 ++-- src/

Re: [Mesa-dev] [PATCH 13/18] gallium/u_blitter: add a query for checking whether copying is supported

2012-08-02 Thread Brian Paul
On 08/02/2012 07:14 AM, Marek Olšák wrote: Cc: Stephane Marchesin --- src/gallium/auxiliary/util/u_blitter.c | 86 ++ src/gallium/auxiliary/util/u_blitter.h |4 + src/gallium/drivers/i915/i915_resource_texture.c |7 +- src/gallium/drivers/i91

Re: [Mesa-dev] [PATCH 3/5] egl_dri2: Refactor dereference of dri2_ctx_shared

2012-08-02 Thread Kristian Høgsberg
On Wed, Aug 01, 2012 at 05:21:36PM -0700, Ian Romanick wrote: > From: Ian Romanick > > Signed-off-by: Ian Romanick > Cc: Kristian Høgsberg > Cc: Matt Turner > --- > src/egl/drivers/dri2/egl_dri2.c | 11 +-- > 1 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/src/egl

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Tom Stellard
On Thu, Aug 02, 2012 at 11:33:43AM +0200, Christian König wrote: > On 02.08.2012 11:21, Michel Dänzer wrote: > >On Don, 2012-08-02 at 11:05 +0200, Christian König wrote: > >>On 02.08.2012 07:51, Michel Dänzer wrote: > >>>On Mit, 2012-08-01 at 23:28 +0200, Christian König wrote: > Using the writ

[Mesa-dev] [PATCH 18/18] r600g: precompute color buffer state in pipe_surface and reuse it

2012-08-02 Thread Marek Olšák
--- .../drivers/r600/evergreen_compute_internal.c | 22 ++- src/gallium/drivers/r600/evergreen_state.c | 143 ++-- src/gallium/drivers/r600/r600_pipe.h |6 +- src/gallium/drivers/r600/r600_resource.h | 17 +++ src/gallium/drivers/r600/r

[Mesa-dev] [PATCH 17/18] r600g: precompute depth buffer state in pipe_surface and reuse it

2012-08-02 Thread Marek Olšák
This is done on-demand, because we don't know in advance if a zbuffer will be bound as depth or color. --- src/gallium/drivers/r600/evergreen_state.c | 104 ++-- src/gallium/drivers/r600/r600_resource.h | 12 src/gallium/drivers/r600/r600_state.c | 53 +

[Mesa-dev] [PATCH 16/18] r600g: simplify create_surface

2012-08-02 Thread Marek Olšák
--- src/gallium/drivers/r600/r600_texture.c | 28 1 file changed, 8 insertions(+), 20 deletions(-) diff --git a/src/gallium/drivers/r600/r600_texture.c b/src/gallium/drivers/r600/r600_texture.c index 81b5777..507dc16 100644 --- a/src/gallium/drivers/r600/r600_textu

[Mesa-dev] [PATCH 15/18] r600g: drop the old texture allocation code

2012-08-02 Thread Marek Olšák
Made obsolete by the libdrm surface allocator. --- .../drivers/r600/evergreen_compute_internal.c |2 +- src/gallium/drivers/r600/evergreen_state.c | 322 src/gallium/drivers/r600/r600_pipe.c |2 - src/gallium/drivers/r600/r600_pipe.h

[Mesa-dev] [PATCH 14/18] r600g: make sure copying of all texture formats is accelerated

2012-08-02 Thread Marek Olšák
--- src/gallium/drivers/r600/r600_blit.c| 60 +-- src/gallium/drivers/r600/r600_texture.c | 46 +++- 2 files changed, 54 insertions(+), 52 deletions(-) diff --git a/src/gallium/drivers/r600/r600_blit.c b/src/gallium/drivers/r600/r600_blit.c

[Mesa-dev] [PATCH 13/18] gallium/u_blitter: add a query for checking whether copying is supported

2012-08-02 Thread Marek Olšák
Cc: Stephane Marchesin --- src/gallium/auxiliary/util/u_blitter.c | 86 ++ src/gallium/auxiliary/util/u_blitter.h |4 + src/gallium/drivers/i915/i915_resource_texture.c |7 +- src/gallium/drivers/i915/i915_surface.c |7 ++ 4 files cha

[Mesa-dev] [PATCH 12/18] r600g: don't decompress depth or stencil if there isn't any

2012-08-02 Thread Marek Olšák
--- src/gallium/drivers/r600/evergreen_state.c |7 +-- src/gallium/drivers/r600/r600_blit.c | 12 +++- src/gallium/drivers/r600/r600_pipe.h |1 + src/gallium/drivers/r600/r600_state.c |6 -- 4 files changed, 17 insertions(+), 9 deletions(-) diff --gi

[Mesa-dev] [PATCH 11/18] r600g: correct texture memory size for Z32F_S8X24 on evergreen

2012-08-02 Thread Marek Olšák
--- src/gallium/drivers/r600/r600_texture.c | 23 --- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/src/gallium/drivers/r600/r600_texture.c b/src/gallium/drivers/r600/r600_texture.c index 4cf9cb8..5e99a04 100644 --- a/src/gallium/drivers/r600/r600_texture.c

[Mesa-dev] [PATCH 10/18] gallium/u_blitter: remove fallback for stencil copy that all drivers skipped

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blitter.c |6 ++ src/gallium/auxiliary/util/u_blitter.h | 10 +- src/gallium/drivers/i915/i915_surface.c |2 +- src/gallium/drivers/r600/r600_blit.c|2 +- 4 files changed, 5 insertions(+), 15 deletions(-) diff --git a/src/gallium/au

[Mesa-dev] [PATCH 09/18] gallium/u_blitter: add ability to blit only depth or only stencil

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blitter.c | 26 ++ src/gallium/auxiliary/util/u_blitter.h |3 ++- src/gallium/drivers/r300/r300_blit.c |2 +- 3 files changed, 17 insertions(+), 14 deletions(-) diff --git a/src/gallium/auxiliary/util/u_blitter.c b/src/gallium/

[Mesa-dev] [PATCH 08/18] gallium: define PIPE_MASK_RGBAZS

2012-08-02 Thread Marek Olšák
I need this and it seems like it could be useful. --- src/gallium/include/pipe/p_defines.h |1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/include/pipe/p_defines.h b/src/gallium/include/pipe/p_defines.h index 8d787ec..59c6382 100644 --- a/src/gallium/include/pipe/p_defines.h ++

[Mesa-dev] [PATCH 07/18] gallium/u_blitter: minor cleanup

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blitter.c | 12 ++-- src/gallium/auxiliary/util/u_blitter.h |4 ++-- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/auxiliary/util/u_blitter.c b/src/gallium/auxiliary/util/u_blitter.c index 1ad6c14..6015ad6 100644 --- a/sr

[Mesa-dev] [PATCH 06/18] gallium/tgsi: fixup texture name strings

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/tgsi/tgsi_strings.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_strings.c b/src/gallium/auxiliary/tgsi/tgsi_strings.c index b5fd1fc..5f8f4be 100644 --- a/src/gallium/auxiliary/tgsi/tgsi_strings.c +++ b/sr

[Mesa-dev] [PATCH 05/18] gallium/u_blitter: set sample mask to ~0

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blitter.c | 10 ++ src/gallium/auxiliary/util/u_blitter.h | 10 ++ 2 files changed, 20 insertions(+) diff --git a/src/gallium/auxiliary/util/u_blitter.c b/src/gallium/auxiliary/util/u_blitter.c index b31ac2d..1ad6c14 100644 --- a/src/gallium/a

[Mesa-dev] [PATCH 04/18] gallium/u_blit: bail out if src is a multisample texture

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blit.c |6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/auxiliary/util/u_blit.c b/src/gallium/auxiliary/util/u_blit.c index 1beca0f..3992f71 100644 --- a/src/gallium/auxiliary/util/u_blit.c +++ b/src/gallium/auxiliary/util/u_blit.c @@ -507,

[Mesa-dev] [PATCH 03/18] gallium/u_blit: check nr_samples before using resource_copy_region

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blit.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util/u_blit.c b/src/gallium/auxiliary/util/u_blit.c index 84af2b4..1beca0f 100644 --- a/src/gallium/auxiliary/util/u_blit.c +++ b/src/gallium/auxiliary/util/u_bli

[Mesa-dev] [PATCH 02/18] gallium: set sample mask to ~0 for clear, blit and gen_mipmap

2012-08-02 Thread Marek Olšák
The sample mask affects single-sampled rendering too (it's orthogonal to the color mask). --- src/gallium/auxiliary/cso_cache/cso_context.c | 13 - src/gallium/auxiliary/cso_cache/cso_context.h |2 ++ src/gallium/auxiliary/util/u_blit.c |6 ++ src/gallium/auxili

[Mesa-dev] [PATCH 01/18] gallium: rename cso_save/restore_samplers -> cso_save/restore_fragment_samplers

2012-08-02 Thread Marek Olšák
--- src/gallium/auxiliary/cso_cache/cso_context.c |4 ++-- src/gallium/auxiliary/cso_cache/cso_context.h |4 ++-- src/gallium/auxiliary/util/u_blit.c |8 src/gallium/auxiliary/util/u_gen_mipmap.c |4 ++-- src/gallium/state_trackers/vega/renderer.c| 12 +

Re: [Mesa-dev] [PATCH 3/6] radeon/llvm: fix fp immediates on SI

2012-08-02 Thread Tom Stellard
On Wed, Aug 01, 2012 at 11:28:24PM +0200, Christian König wrote: > I don't know if this is a good idea, but it > fixes the problem at hand. > > Signed-off-by: Christian König Reviewed-by: Tom Stellard > --- > src/gallium/drivers/radeon/SICodeEmitter.cpp | 27 > +++--- >

Re: [Mesa-dev] [PATCH 4/5] egl: Replace KHR_surfaceless_* extensions with KHR_surfaceless_context

2012-08-02 Thread Kristian Høgsberg
On Wed, Aug 1, 2012 at 8:21 PM, Ian Romanick wrote: > From: Ian Romanick > > KHR extension name is reserved for Khronos ratified extensions, and there is > no such thing as EGL_KHR_surfaceless_{gles1,gles2,opengl}. Replace these > three extensions with EGL_KHR_surfaceless_context since that exte

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Michel Dänzer
On Don, 2012-08-02 at 11:57 +0200, Christian König wrote: > On 02.08.2012 11:46, Michel Dänzer wrote: > > On Don, 2012-08-02 at 11:33 +0200, Christian König wrote: > >> On 02.08.2012 11:21, Michel Dänzer wrote: > >>> On Don, 2012-08-02 at 11:05 +0200, Christian König wrote: > On 02.08.2012 07

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Christian König
On 02.08.2012 11:46, Michel Dänzer wrote: On Don, 2012-08-02 at 11:33 +0200, Christian König wrote: On 02.08.2012 11:21, Michel Dänzer wrote: On Don, 2012-08-02 at 11:05 +0200, Christian König wrote: On 02.08.2012 07:51, Michel Dänzer wrote: Couldn't this incorrectly clobber components of the

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Michel Dänzer
On Don, 2012-08-02 at 11:33 +0200, Christian König wrote: > On 02.08.2012 11:21, Michel Dänzer wrote: > > On Don, 2012-08-02 at 11:05 +0200, Christian König wrote: > >> On 02.08.2012 07:51, Michel Dänzer wrote: > > > >>> Couldn't this incorrectly clobber components of the destination which > >>> w

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Christian König
On 02.08.2012 11:21, Michel Dänzer wrote: On Don, 2012-08-02 at 11:05 +0200, Christian König wrote: On 02.08.2012 07:51, Michel Dänzer wrote: On Mit, 2012-08-01 at 23:28 +0200, Christian König wrote: Using the writemask in the sampler results in packet VGPRS. What does that mean? The instruc

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Michel Dänzer
On Don, 2012-08-02 at 11:05 +0200, Christian König wrote: > On 02.08.2012 07:51, Michel Dänzer wrote: > > On Mit, 2012-08-01 at 23:28 +0200, Christian König wrote: > >> Using the writemask in the sampler results in packet > >> VGPRS. > > What does that mean? > > The instructions with a destinatio

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Christian König
On 02.08.2012 07:51, Michel Dänzer wrote: On Mit, 2012-08-01 at 23:28 +0200, Christian König wrote: Using the writemask in the sampler results in packet VGPRS. What does that mean? The instructions with a destination mask are packing their results, e.g. when you sample RGBA you get: R in VG

[Mesa-dev] [PATCH] st/mesa: Ensure dst in compile_instruction is initialized.

2012-08-02 Thread Vinson Lee
Fixes uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee --- src/mesa/state_tracker/st_mesa_to_tgsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_mesa_to_tgsi.c b/src/mesa/state_tracker/st_mesa_to_tgsi.c index 9

Re: [Mesa-dev] [PATCH 4/6] radeonsi: fix shader param and color count

2012-08-02 Thread Michel Dänzer
On Mit, 2012-08-01 at 23:28 +0200, Christian König wrote: > Signed-off-by: Christian König > --- > src/gallium/drivers/radeonsi/radeonsi_shader.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c > b/src/galliu

Re: [Mesa-dev] [PATCH 5/6] radeonsi: fix TEX writemask

2012-08-02 Thread Michel Dänzer
On Mit, 2012-08-01 at 23:28 +0200, Christian König wrote: > Using the writemask in the sampler results in packet > VGPRS. What does that mean? > For now just sample all components and let llvm chose the right one. > > Signed-off-by: Christian König > --- > src/gallium/drivers/radeonsi/radeon