Re: [Mesa-dev] [Nouveau] [PATCH] nv50/ir: only enable mul saturate on G200+

Reviewed-by: Tobias Klausmann On 09.05.2015 09:31, Ilia Mirkin wrote: Commit 44673512a84 enabled support for saturating fmul. However experimentally this does not seem to work on the older chips. Restrict the feature to G200 (NVA0) and later. Reported-by: Pierre Moreau Bugzilla: https

Re: [Mesa-dev] Statically linking libstdc++ and libgcc

On 11.03.2015 17:01, Francisco Jerez wrote: Vivek Dasmohapatra writes: Hi - Hi, As you probably already know, there can only be one version of libstdc++.so in your runtime link chain That's a common misconception, in principle several versions of libstdc++.so with different DT_SONAME (i.

Re: [Mesa-dev] [PATCH 1/2] gallium: include util/macros.h

On 24.02.2015 14:57, Mark Janes wrote: Hi Tobias, FYI, This commit failed build-test after it was pushed. Whoops, did not build that driver, right! Sorry for that! Eric fixed it up in 1d1e820. -Mark Thanks for the quick fix :) Greetings, Tobias Tobias Klausmann writes: snip

Re: [Mesa-dev] [PATCH 1/2] gallium: include util/macros.h

Can somebody push this please?! Thanks, Tobias On 12.02.2015 23:04, Jose Fonseca wrote: LGTM. On 12/02/15 17:31, Tobias Klausmann wrote: The most common macros are defined there, no use to duplicate these Clean up the already redefinded macros Signed-off-by: Tobias Klausmann --- src

[Mesa-dev] [PATCH 1/2] gallium: include util/macros.h

The most common macros are defined there, no use to duplicate these Clean up the already redefinded macros Signed-off-by: Tobias Klausmann --- src/gallium/include/pipe/p_compiler.h | 57 ++- 1 file changed, 2 insertions(+), 55 deletions(-) diff --git a/src

[Mesa-dev] [PATCH 2/2] st/nine: Mark end of non-void function unreachable

Signed-off-by: Tobias Klausmann --- src/gallium/state_trackers/nine/nine_pipe.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_pipe.h b/src/gallium/state_trackers/nine/nine_pipe.h index 17844d5..b8e728e 100644 --- a/src/gallium

Re: [Mesa-dev] [PATCH 1/2] gallium: define unreachable macro

SERT and likely/unlikely in p_compiler.h We could clean these up there and include macros.h there... On Wed, Feb 11, 2015 at 10:47 AM, Tobias Klausmann wrote: This is done the same way for glsl et al. already Signed-off-by: Tobias Klausmann --- src/gallium/include/pipe/p_

[Mesa-dev] [PATCH 2/2] st/nine: Mark end of non-void function unreachable

Signed-off-by: Tobias Klausmann --- src/gallium/state_trackers/nine/nine_pipe.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_pipe.h b/src/gallium/state_trackers/nine/nine_pipe.h index 17844d5..b8e728e 100644 --- a/src/gallium

[Mesa-dev] [PATCH 1/2] gallium: define unreachable macro

This is done the same way for glsl et al. already Signed-off-by: Tobias Klausmann --- src/gallium/include/pipe/p_compiler.h | 23 +++ 1 file changed, 23 insertions(+) diff --git a/src/gallium/include/pipe/p_compiler.h b/src/gallium/include/pipe/p_compiler.h index fb018bf

[Mesa-dev] Re: [PATCH] st/nine: Mark end of non-void function unreachable

Right, the function isn't even defined in that scope, tired me shouldn't even try to write code ;-) On 10.02.2015 04:48, Jan Vesely wrote: On Tue, 2015-02-10 at 04:43 +0100, Tobias Klausmann wrote: Signed-off-by: Tobias Klausmann --- src/gallium/state_trackers/nine/nine_pipe.h

[Mesa-dev] [PATCH] st/nine: Mark end of non-void function unreachable

Signed-off-by: Tobias Klausmann --- src/gallium/state_trackers/nine/nine_pipe.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_pipe.h b/src/gallium/state_trackers/nine/nine_pipe.h index 17844d5..3480aad 100644 --- a/src/gallium

Re: [Mesa-dev] [PATCH] glsl/parser: (trivial) fix indentation in one function

On 21.01.2015 22:22, Matt Turner wrote: On Wed, Jan 21, 2015 at 1:12 PM, Tobias Klausmann wrote: (snip) We're messing up the indentation here. But really we want to get rid of tabs, not add more. glcpp is unfortunately full of tabs. Alright ignore this one then. Thanks, T

[Mesa-dev] [PATCH] glsl/parser: (trivial) fix indentation in one function

Signed-off-by: Tobias Klausmann --- src/glsl/glcpp/glcpp-parse.y | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y index e5bebe5..8b545da 100644 --- a/src/glsl/glcpp/glcpp-parse.y +++ b/src/glsl

[Mesa-dev] [PATCH] mesa: change assert to unreachable in two format functions

This fixes two problems reported by osc: I: Program returns random data in a function E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/format_utils.c:180 E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/glformats.c:2714 Signed-off-by: Tobias Klausmann --- src/mesa/main

Re: [Mesa-dev] [PATCH] mesa: fix no return in two format functions

On 19.01.2015 19:54, Matt Turner wrote: On Thu, Jan 15, 2015 at 10:26 AM, Tobias Klausmann wrote: I: Program returns random data in a function E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/format_utils.c:180 E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/glformats.c

Re: [Mesa-dev] [PATCH] mesa: fix no return in two format functions

On 16.01.2015 15:05, Jan Vesely wrote: On Thu, 2015-01-15 at 20:49 +0100, Tobias Klausmann wrote: On 15.01.2015 19:35, Jan Vesely wrote: On Thu, 2015-01-15 at 19:26 +0100, Tobias Klausmann wrote: I: Program returns random data in a function E: Mesa no-return-in-nonvoid-function ../../src

Re: [Mesa-dev] [PATCH] mesa: fix no return in two format functions

On 15.01.2015 19:35, Jan Vesely wrote: On Thu, 2015-01-15 at 19:26 +0100, Tobias Klausmann wrote: I: Program returns random data in a function E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/format_utils.c:180 E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/glformats.c:2714

[Mesa-dev] [PATCH] mesa: fix no return in two format functions

I: Program returns random data in a function E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/format_utils.c:180 E: Mesa no-return-in-nonvoid-function ../../src/mesa/main/glformats.c:2714 --- src/mesa/main/format_utils.c | 1 + src/mesa/main/glformats.c| 1 + 2 files changed, 2 insert

[Mesa-dev] Re: [Nouveau] [RFC] mesa/st: Avoid passing a NULL buffer to the drivers

On 12.01.2015 01:57, Ilia Mirkin wrote: On Sun, Jan 11, 2015 at 7:43 PM, Tobias Klausmann wrote: On 11.01.2015 06:05, Ilia Mirkin wrote: Can you elaborate a bit as to why that's the right thing to do? On Wed, Jan 7, 2015 at 1:52 PM, Tobias Klausmann wrote: If we capture tran

[Mesa-dev] Re: [Nouveau] [RFC] mesa/st: Avoid passing a NULL buffer to the drivers

On 11.01.2015 06:05, Ilia Mirkin wrote: Can you elaborate a bit as to why that's the right thing to do? On Wed, Jan 7, 2015 at 1:52 PM, Tobias Klausmann wrote: If we capture transform feedback from n stream in (n-1) buffers we face a NULL buffer, use the buffer (n-1) to capture the o

[Mesa-dev] [RFC] mesa/st: Avoid passing a NULL buffer to the drivers

If we capture transform feedback from n stream in (n-1) buffers we face a NULL buffer, use the buffer (n-1) to capture the output of stream n. This fixes one piglit test with nvc0: arb_gpu_shader5-xfb-streams-without-invocations Signed-off-by: Tobias Klausmann --- src/mesa/state_tracker

[Mesa-dev] [PATCH RESEND] nv50/ir: use unordered_set instead of list to keep track of var defs

The set of variable defs does not need to be ordered in any way, and removing/adding elements is a fairly common operation in various optimization passes. This shortens runtime of piglit test fp-long-alu to ~11s from ~22s No piglit regressions observed on nvc0! Signed-off-by: Tobias Klausmann

[Mesa-dev] [PATCH v2] nv50: Handle ARB_conditional_render_inverted and enable it

Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.4.html | 1 + src/gallium/drivers/nouveau/nv50/nv50_context.h | 3 +- src/gallium/drivers/nouveau/nv50/nv50_query.c | 56 + src

[Mesa-dev] [PATCH] nv50/ir: use unordered_set instead of list to keep track of var defs

The set of variable defs does not need to be ordered in any way, and removing/adding elements is a fairly common operation in various optimization passes. This shortens runtime of piglit test fp-long-alu to ~11s from ~22s No piglit regressions observed on nvc0! Signed-off-by: Tobias Klausmann

[Mesa-dev] [PATCH] nv50: Handle ARB_conditional_render_inverted and enable it

Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.3.html | 2 +- src/gallium/drivers/nouveau/nv50/nv50_context.h | 3 +- src/gallium/drivers/nouveau/nv50/nv50_query.c | 59 - src

[Mesa-dev] [PATCH v4 6/6] docs: Update status of ARB_conditional_render_inverted

Done for: nvc0, softpipe and llvmpipe Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.3.html | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index b38e42c..5549b30 100644 --- a/docs/GL3.txt +++ b/docs/GL3

[Mesa-dev] [PATCH v4 5/6] llvmpipe/softpipe: enable ARB_conditional_render_inverted

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/llvmpipe/lp_screen.c | 3 ++- src/gallium/drivers/softpipe/sp_screen.c | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c b/src/gallium/drivers/llvmpipe/lp_screen.c index 08597fa

[Mesa-dev] [PATCH v4 0/6] Implement ARB_conditional_render_inverted

: - Only allow the new modes if the extension is enabled - Merge several patches to always have a working tree V4: - Reorder patches - Fix nvc0 blit and inverted rendering once more Tobias Klausmann (6): mesa: add ARB_conditional_render_inverted flags gallium: Add and handle

[Mesa-dev] [PATCH v4 2/6] gallium: Add and handle PIPE_CAP_CONDITIONAL_RENDER_INVERTED

Signed-off-by: Tobias Klausmann --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/drivers/freedreno/freedreno_screen.c | 1 + src/gallium/drivers/i915/i915_screen.c | 1 + src/gallium/drivers/ilo/ilo_screen.c | 1 + src/gallium/drivers/llvmpipe

[Mesa-dev] [PATCH v4 3/6] mesa/st: Support ARB_conditional_render_inverted modes

Signed-off-by: Tobias Klausmann --- src/mesa/state_tracker/st_cb_condrender.c | 20 +++- src/mesa/state_tracker/st_extensions.c| 1 + 2 files changed, 20 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_cb_condrender.c b/src/mesa/state_tracker

[Mesa-dev] [PATCH v4 4/6] nvc0: Handle ARB_conditional_render_inverted and enable it

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 3 +- src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 61 + src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 +- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 3 +- 4 files

[Mesa-dev] [PATCH v4 1/6] mesa: add ARB_conditional_render_inverted flags

Also add an extension bit so we can safely enable Signed-off-by: Tobias Klausmann --- src/mesa/main/condrender.c | 10 -- src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/condrender.c b

[Mesa-dev] [PATCH v3 4/6] nvc0: Handle ARB_conditional_render_inverted and enable it

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 5 ++--- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 +-- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 4 +++- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers

[Mesa-dev] [PATCH v3 2/6] mesa/st: Support ARB_conditional_render_inverted modes

Signed-off-by: Tobias Klausmann --- src/mesa/state_tracker/st_cb_condrender.c | 20 +++- src/mesa/state_tracker/st_extensions.c| 1 + 2 files changed, 20 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_cb_condrender.c b/src/mesa/state_tracker

[Mesa-dev] [PATCH v3 6/6] docs: Update status of ARB_conditional_render_inverted

Done for: nvc0, softpipe and llvmpipe Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.3.html | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index b38e42c..5549b30 100644 --- a/docs/GL3.txt +++ b/docs/GL3

[Mesa-dev] [PATCH v3 5/6] llvmpipe/softpipe: enable ARB_conditional_render_inverted

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/llvmpipe/lp_screen.c | 3 ++- src/gallium/drivers/softpipe/sp_screen.c | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c b/src/gallium/drivers/llvmpipe/lp_screen.c index 08597fa

[Mesa-dev] [PATCH v3 3/6] gallium: Add and handle PIPE_CAP_CONDITIONAL_RENDER_INVERTED

Signed-off-by: Tobias Klausmann --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/drivers/freedreno/freedreno_screen.c | 1 + src/gallium/drivers/i915/i915_screen.c | 1 + src/gallium/drivers/ilo/ilo_screen.c | 1 + src/gallium/drivers/llvmpipe

[Mesa-dev] [PATCH v3 1/6] mesa: add ARB_conditional_render_inverted flags

Also add an extension bit so we can safely enable Signed-off-by: Tobias Klausmann --- src/mesa/main/condrender.c | 10 -- src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/condrender.c b

[Mesa-dev] [PATCH v3 0/6] Implement ARB_conditional_render_inverted

: - Only allow the new modes if the extension is enabled - Merge several patches to always have a working tree Tobias Klausmann (6): mesa: add ARB_conditional_render_inverted flags mesa/st: Support ARB_conditional_render_inverted modes gallium: Add and handle PIPE_CAP_CONDITIONAL_RENDER_INVERTED

[Mesa-dev] [PATCH v2 8/8] docs: Update status of ARB_conditional_render_inverted

Done for: nvc0, softpipe and llvmpipe Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.3.html | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index b38e42c..5549b30 100644 --- a/docs/GL3.txt +++ b/docs/GL3

[Mesa-dev] [PATCH v2 3/8] gallium/st: Add PIPE_CAP_CONDITIONAL_RENDER_INVERTED

with this we determine if the driver wants to enable GL_ARB_conditional_render_inverted Signed-off-by: Tobias Klausmann --- src/gallium/docs/source/screen.rst| 2 ++ src/gallium/include/pipe/p_defines.h | 1 + src/mesa/state_tracker/st_cb_condrender.c | 20

[Mesa-dev] [PATCH v2 7/8] llvmpipe/softpipe: enable ARB_conditional_render_inverted

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/llvmpipe/lp_screen.c | 3 ++- src/gallium/drivers/softpipe/sp_screen.c | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c b/src/gallium/drivers/llvmpipe/lp_screen.c index 08597fa

[Mesa-dev] [PATCH v2 5/8] nvc0: Handle ARB_conditional_render_inverted and enable it

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 5 ++--- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 +-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_query.c b/src/gallium/drivers/nouveau/nvc0

[Mesa-dev] [PATCH v2 2/8] mesa: add ARB_conditional_render_inverted extension bit

Signed-off-by: Tobias Klausmann --- src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 2 files changed, 2 insertions(+) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index c5bd7b3..553c01e 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main

[Mesa-dev] [PATCH v2 4/8] gallium/drivers: Handle PIPE_CAP_CONDITIONAL_RENDER_INVERTED

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/freedreno/freedreno_screen.c | 1 + src/gallium/drivers/i915/i915_screen.c | 1 + src/gallium/drivers/ilo/ilo_screen.c | 1 + src/gallium/drivers/llvmpipe/lp_screen.c | 1 + src/gallium/drivers/nouveau/nv30

[Mesa-dev] [PATCH v2 0/8] Implement ARB_conditional_render_inverted

This patch series adds support for ARB_conditional_render_inverted to nvc0, softpipe and llvmpipe. V2: - Add missing _mesa_BeginConditionalRender() parts to series - Fix nvc0 blit and inverted rendering - Fix relnotes - Enable for softpipe and llvmpipe - Rebase on top of current HEAD Tobias

[Mesa-dev] [PATCH v2 1/8] mesa: add ARB_conditional_render_inverted flags

Signed-off-by: Tobias Klausmann --- src/mesa/main/condrender.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/main/condrender.c b/src/mesa/main/condrender.c index 0ad1e5c2..65bdaaf 100644 --- a/src/mesa/main/condrender.c +++ b/src/mesa/main/condrender.c @@ -77,6 +77,10

[Mesa-dev] [PATCH v2 6/8] nvc0: Fix blits with inverted conditional rendering

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c b/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c index a29f0cc..622193b 100644 --- a/src

[Mesa-dev] [PATCH 6/6] docs: Mark off ARB_conditional_render_inverted for nvc0

Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.3.html | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index b38e42c..bbd1838 100644 --- a/docs/GL3.txt +++ b/docs/GL3.txt @@ -186,7 +186,7 @@ GL 4.5, GLSL

[Mesa-dev] [PATCH 5/6] nvc0: Enable ARB_conditional_render_inverted

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c index a391a5f..85f8d1e 100644 --- a/src

[Mesa-dev] [PATCH 3/6] mesa: add ARB_conditional_render_inverted extension bit

Signed-off-by: Tobias Klausmann --- src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 2 files changed, 2 insertions(+) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index c5bd7b3..553c01e 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main

[Mesa-dev] [PATCH 2/6] gallium/st: Add PIPE_CAP_CONDITIONAL_RENDER_INVERTED

with this we determine if the driver wants to enable GL_ARB_conditional_render_inverted Signed-off-by: Tobias Klausmann --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/drivers/freedreno/freedreno_screen.c | 1 + src/gallium/drivers/i915/i915_screen.c | 1

[Mesa-dev] [PATCH 1/6] gallium/st: use condition flag in *render_condition()

This is used to tell the drivers if to invert the rendering, used by ARB_conditional_render_inverted Signed-off-by: Tobias Klausmann --- src/mesa/state_tracker/st_cb_condrender.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker

[Mesa-dev] [PATCH 4/6] nvc0: Handle condition whether to invert if we render or not

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_query.c b/src/gallium/drivers/nouveau/nvc0/nvc0_query.c index 50cef1e..71d48f2 100644 --- a/src

[Mesa-dev] [PATCH 0/6] Implement ARB_conditional_render_inverted

This patch series adds support for ARB_conditional_render_inverted to nvc0. Tobias Klausmann (6): gallium/st: use condition flag in *render_condition() gallium/st: Add PIPE_CAP_CONDITIONAL_RENDER_INVERTED mesa: add ARB_conditional_render_inverted extension bit nvc0: Handle condition

Re: [Mesa-dev] [RFC] gallium/st/drivers: Extend render_condition() Add a new boolean flag: inverted Adopt all places where we make use of this This will be used to implement ARB_conditional_render_inv

5.08.2014 00:53, schrieb Tobias Klausmann: Signed-off-by: Tobias Klausmann --- src/gallium/auxiliary/util/u_blitter.c | 5 +++-- src/gallium/drivers/galahad/glhd_context.c | 5 +++-- src/gallium/drivers/ilo/ilo_3d.c| 3 ++- src/gallium/drivers/llvmpipe/lp_cont

[Mesa-dev] [RFC] gallium/st/drivers: Extend render_condition() Add a new boolean flag: inverted Adopt all places where we make use of this This will be used to implement ARB_conditional_render_inverte

Signed-off-by: Tobias Klausmann --- src/gallium/auxiliary/util/u_blitter.c | 5 +++-- src/gallium/drivers/galahad/glhd_context.c | 5 +++-- src/gallium/drivers/ilo/ilo_3d.c| 3 ++- src/gallium/drivers/llvmpipe/lp_context.c | 3 ++- src/gallium/drivers/nouveau

[Mesa-dev] [PATCH v3 1/2] nv50/ir: Add support for the double Type to BuildUtil

Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_build_util.cpp | 17 + .../drivers/nouveau/codegen/nv50_ir_build_util.h| 2 ++ 2 files changed, 19 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_build_util.cpp b/src

[Mesa-dev] [PATCH v3 2/2] nv50/ir: Handle OP_CVT when folding constant expressions

Folding for conversions: F32/64->(U16/32, S16/32) and (U16/32, S16/32)->F32 No piglit regressions observed on nv50 and nvc0! Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_peephole.cpp | 74 ++ 1 file changed, 74 insertions(+) diff --git

[Mesa-dev] [PATCH] nv50/ir: Handle OP_CVT when folding constant expressions

Folding for conversions: F32/64->(U16/32, S16/32) and (U16/32, S16/32)->F32 No piglit regressions observed! Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_peephole.cpp | 74 ++ 1 file changed, 74 insertions(+) diff --git a/src/gallium/d

[Mesa-dev] [PATCH v3] nv50/ir: make ARB_viewport_array behave like it does with other drivers

ter on each emit. This fixes the remaining piglit tests in ARB_viewport_array for nvc0. Note: Not tested on nv50 Signed-off-by: Tobias Klausmann --- V2: provide description fix naming of vars V3: spare a function call, use the idx we already have second arg to mkSymbol changed to 0,

[Mesa-dev] [PATCH v2] nv50/ir: make ARB_viewport_array behave like it does with other drivers

ter on each emit. This fixes the remaining piglit tests in ARB_viewport_array for nvc0. Note: Not tested on nv50 Signed-off-by: Tobias Klausmann --- V2: provide description fix naming of vars .../drivers/nouveau/codegen/nv50_ir_driver.h | 1 + .../drivers/nouveau/code

[Mesa-dev] [PATCH] nv50/ir: make ARB_viewport_array behave like it does with other drivers

Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_driver.h | 1 + .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 27 -- 2 files changed, 26 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h b

[Mesa-dev] [PATCH v2 1/3] nvc0: implement multiple viewports/scissors, enable ARB_viewport_array

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_context.h| 7 +- src/gallium/drivers/nouveau/nvc0/nvc0_program.c| 2 +- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 20 ++-- src/gallium/drivers/nouveau/nvc0/nvc0_screen.h | 3 + src/gallium

[Mesa-dev] [PATCH v2 2/3] docs: update GL3.txt, relnotes: mark GL_ARB_viewport_array as done for nvc0

Signed-off-by: Tobias Klausmann --- docs/GL3.txt| 2 +- docs/relnotes/10.3.html | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index f5f940c..a392b92 100644 --- a/docs/GL3.txt +++ b/docs/GL3.txt @@ -132,7 +132,7 @@ GL 4.1

[Mesa-dev] [PATCH v2 0/3] ARB_viewport_array for nvc0

This patch-series implements the ARB_viewport_array for nvc0 and does a little house-cleanig afterwords. V2: Add Release-Notes, mark this in GL3 as done for nvc0 Don't mark the scissors dirty when we don't need to do that Tobias Klausmann (3): nvc0: implement multiple viewport

[Mesa-dev] [PATCH v2 3/3] nv50/ir: Remove NV50_SEMANTIC_VIEWPORTINDEX

Use TGSI_SEMANTIC_VIEWPORT_INDEX for the last consumer. Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h| 1 - src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src

[Mesa-dev] [PATCH 3/3] nv50/ir: Remove NV50_SEMANTIC_VIEWPORTINDEX and its last consumer

We use TGSI_SEMANTIC_VIEWPORT_INDEX for nvc0 now as well. Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h| 1 - src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp | 1 - 2 files changed, 2 deletions(-) diff --git a/src/gallium/drivers

[Mesa-dev] [PATCH 0/3] ARB_viewport_array for nvc0

This patch-series implements the ARB_viewport_array for nvc0 and does a little house-cleanig afterwords. Tobias Klausmann (3): nvc0: implement multiple viewports/scissors, enable ARB_viewport_array nvc0: mark scissor in nvc0_clear_{} nv50/ir: Remove NV50_SEMANTIC_VIEWPORTINDEX and its last

[Mesa-dev] [PATCH 2/3] nvc0: mark scissor in nvc0_clear_{}

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c b/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c index c28ec6d..72227b8 100644 --- a/src/gallium/drivers

[Mesa-dev] [PATCH 1/3] nvc0: implement multiple viewports/scissors, enable ARB_viewport_array

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_context.h| 7 +- src/gallium/drivers/nouveau/nvc0/nvc0_program.c| 2 +- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 20 ++-- src/gallium/drivers/nouveau/nvc0/nvc0_screen.h | 3 + src/gallium

[Mesa-dev] [PATCH v3 3/4] nvc0/ir: Handle OP_BFIND when folding constant expressions

Signed-off-by: Tobias Klausmann --- V3: Use BuildUtil for the Immediate instead of a type conversion .../drivers/nouveau/codegen/nv50_ir_peephole.cpp| 17 + 1 file changed, 17 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src

[Mesa-dev] [PATCH v3 2/4] nvc0/ir: Handle reverse subop for OP_EXTBF when folding constant expressions

Signed-off-by: Tobias Klausmann --- V2: Handle the instruction right (shift after reverse) V3: Reverse once, not independently for every TYPE src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH v3 4/4] nvc0/ir: Handle OP_POPCNT when folding constant expressions

Signed-off-by: Tobias Klausmann --- V2: Add support for a single-argument version of POPCNT for Maxwell (SM5) V3: Clean up a bit more src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen

[Mesa-dev] [PATCH v3 1/4] nvc0/ir: clear subop when folding constant expressions

Some operations (e.g. OP_MUL/OP_MAD/OP_EXTBF might have a subop set. After folding, make sure that it is cleared Signed-off-by: Tobias Klausmann Reviewed-by: Ilia Mirkin --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium

[Mesa-dev] [PATCH v3 0/4] Constant folding of new Instructions

Yet another try for constant folding of Instructions for nvc0. Please Review this again! (Hopefully the last time ;-) ) Tobias Klausmann (4): nvc0/ir: clear subop when folding constant expressions nvc0/ir: Handle reverse subop for OP_EXTBF when folding constant expressions nvc0/ir

[Mesa-dev] [PATCH v2 2/4] nvc0/ir: Handle reverse subop for OP_EXTBF when folding constant expressions

V2: Handle the instruction right (shift after reverse) Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b

[Mesa-dev] [PATCH v2 4/4] nvc0/ir: Handle OP_POPCNT when folding constant expressions

V2: Add support for a single-argument version of POPCNT for Maxwell (SM5) Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 15 +++ 1 file changed, 15 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b

[Mesa-dev] [PATCH v2 1/4] nvc0/ir: clear subop when folding constant expressions

Some operations (e.g. OP_MUL/OP_MAD/OP_EXTBF might have a subop set. After folding, make sure that it is cleared Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nouveau/codegen

[Mesa-dev] [PATCH v2 3/4] nvc0/ir: Handle OP_BFIND when folding constant expressions

Signed-off-by: Tobias Klausmann --- .../drivers/nouveau/codegen/nv50_ir_peephole.cpp | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp index a214ffc

[Mesa-dev] [PATCH v2 0/4] Constant folding of new Instructions

And another try for constant folding of Instructions for nvc0. Please Review this! Thanks, Tobias Klausmann Tobias Klausmann (4): nvc0/ir: clear subop when folding constant expressions nvc0/ir: Handle reverse subop for OP_EXTBF when folding constant expressions nvc0/ir: Handle

Re: [Mesa-dev] [PATCH 2/4] nvc0/ir: Handle reverse subop for OP_EXTBF when folding constant expressions

ify that you tested how the HW handles this, as well as exactly how you did it (i.e. how did you modify the code + piglit test, what the results were, etc) On Thu, May 29, 2014 at 3:43 PM, Tobias Klausmann wrote: Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/code

[Mesa-dev] [PATCH 3/4] nvc0/ir: Handle OP_POPCNT when folding constant expressions

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp index 93f7c2a..68b9a6d

[Mesa-dev] [PATCH 4/4] nvc0/ir: Handle OP_BFIND when folding constant expressions

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp index 68b9a6d

[Mesa-dev] [PATCH 2/4] nvc0/ir: Handle reverse subop for OP_EXTBF when folding constant expressions

Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp

[Mesa-dev] [PATCH 1/4] nvc0/ir: clear subop when folding constant expressions

Some operations (e.g. OP_MUL/OP_MAD/OP_EXTBF might have a subop set. After folding, make sure that it is cleared Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nouveau/codegen

[Mesa-dev] Add constant folding for new opcodes

when folding constant expressions src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 39 +-- 1 file changed, 37 insertions(+), 2 deletions(-) Thanks, Tobias Klausmann ___ mesa-dev mailing list mesa-dev

[Mesa-dev] [PATCH V3] nvc0: implement clear_buffer

- don't mark the framebuffer dirty for if we don't touch the GPU Signed-off-by: Tobias Klausmann --- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 151 1 file changed, 151 insertions(+) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c

[Mesa-dev] [PATCH V2] nvc0: implement clear_buffer

v2: change patch according to Ilia Mirkins review --- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 151 1 file changed, 151 insertions(+) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c b/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c index 6b7c30c..24

[Mesa-dev] Implement buffer_clear for nvc0

Hi, please review the following patch! Thanks, Tobias Klausmann ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] nvc0: Implement buffer_clear for this type of hardware

--- src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 171 1 file changed, 171 insertions(+) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c b/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c index 6b7c30c..987b6c4 100644 --- a/src/gallium/drivers/nouveau/nvc

<    1   2