Re: [Mesa-dev] [PATCH] radeonsi: implement TGSI_PROPERTY_VS_WINDOW_SPACE_POSITION

2014-11-09 Thread Michel Dänzer
On 10.11.2014 08:09, Marek Olšák wrote: From: Marek Olšák Required by Nine. Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer| http://www.amd.com Libre software enthusiast |Mesa and X developer __

Re: [Mesa-dev] [PATCH V2] mesa: Permanently enable features supported by target CPU at compile time.

2014-11-09 Thread Emil Velikov
On 09/11/14 04:37, Siavash Eliasi wrote: > > On 11/08/2014 09:55 PM, Emil Velikov wrote: [...] >> Can you confirm that it does not cause issues with "interesting" setups >> such as https://bugs.freedesktop.org/show_bug.cgi?id=71547 > > Challenge accepted! What my patch is doing is to check for pr

Re: [Mesa-dev] The final batch of gallium 'make dist'

2014-11-09 Thread Emil Velikov
Ping anyone ? Would love to get gallium ticked off, and continue with the rest of mesa. -Emil On 14/10/14 16:44, Emil Velikov wrote: > Hello list, > > As the title says it - this ought to be the final batch of patches > needed to get 'make dist' working for gallium. With that one out of the > w

Re: [Mesa-dev] [PATCH] gallium: remove unused pipe_viewport_state::translate[3] and scale[3]

2014-11-09 Thread Ilia Mirkin
On Sun, Nov 9, 2014 at 6:39 PM, Marek Olšák wrote: > diff --git a/src/gallium/drivers/nouveau/nv30/nv30_state_validate.c > b/src/gallium/drivers/nouveau/nv30/nv30_state_validate.c > index f227559..6df0c47 100644 > --- a/src/gallium/drivers/nouveau/nv30/nv30_state_validate.c > +++ b/src/gallium/dr

[Mesa-dev] [PATCH] gallium: remove unused pipe_viewport_state::translate[3] and scale[3]

2014-11-09 Thread Marek Olšák
From: Marek Olšák Almost all drivers ignore them. --- src/gallium/auxiliary/draw/draw_context.c | 4 +--- src/gallium/auxiliary/hud/hud_context.c| 2 -- src/gallium/auxiliary/postprocess/pp_init.c| 2 -- src/gallium/auxiliary/util/u_blit.c

[Mesa-dev] [PATCH] radeonsi: implement TGSI_PROPERTY_VS_WINDOW_SPACE_POSITION

2014-11-09 Thread Marek Olšák
From: Marek Olšák Required by Nine. --- src/gallium/drivers/radeonsi/si_pipe.c | 2 +- src/gallium/drivers/radeonsi/si_state.c | 1 - src/gallium/drivers/radeonsi/si_state_draw.c | 16 +++- 3 files changed, 16 insertions(+), 3 deletions(-) diff --git a/src/gallium/drive

[Mesa-dev] [PATCH 4/4] tgsi/ureg: simplify code for declaring properties

2014-11-09 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/tgsi/tgsi_ureg.c| 153 ++ src/gallium/auxiliary/tgsi/tgsi_ureg.h| 35 +- src/gallium/auxiliary/util/u_simple_shaders.c | 2 +- src/mesa/state_tracker/st_glsl_to_tgsi.cpp| 12 +- src/mesa/state_tracker

[Mesa-dev] [PATCH 2/4] gallium/util: add a window_space option to the passthrough vertex shader

2014-11-09 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/postprocess/pp_program.c | 2 +- src/gallium/auxiliary/util/u_blit.c| 2 +- src/gallium/auxiliary/util/u_blitter.c | 5 +++-- src/gallium/auxiliary/util/u_simple_shaders.c | 10 -- src/gallium/auxiliary/util/u_simple_sha

[Mesa-dev] [PATCH 3/4] gallium/util: add a test for TGSI_PROPERTY_VS_WINDOW_SPACE_POSITION

2014-11-09 Thread Marek Olšák
From: Marek Olšák Not testable by OpenGL. Required by Nine. This is an example of how to implement a piglit-like test using gallium only. --- src/gallium/auxiliary/Makefile.sources | 1 + src/gallium/auxiliary/util/u_tests.c | 270 + src/gallium/auxiliary/uti

[Mesa-dev] [PATCH 1/4] tgsi: fixup the string of VS_WINDOW_SPACE_POSITION

2014-11-09 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/tgsi/tgsi_strings.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_strings.c b/src/gallium/auxiliary/tgsi/tgsi_strings.c index f84cd79..01fa5a9 100644 --- a/src/gallium/auxiliary/tgsi/tgsi_strings.c

[Mesa-dev] [Bug 86070] Host application crash on vmware fusion 7 in vmw_swc_flush

2014-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=86070 Bug ID: 86070 Summary: Host application crash on vmware fusion 7 in vmw_swc_flush Product: Mesa Version: 8.0 Hardware: x86-64 (AMD64) OS: Linux (All)

Re: [Mesa-dev] [PATCH v2 2/2] i965: Use the predicate enable bit for conditional rendering without stalling

2014-11-09 Thread Daniel Vetter
On Fri, Nov 07, 2014 at 06:53:00PM +, Neil Roberts wrote: > Previously whenever a primitive is drawn the driver would call > _mesa_check_conditional_render which blocks waiting for the result of the > query to determine whether to render. On Gen7+ there is a bit in the > 3DPRIMITIVE command whi

Re: [Mesa-dev] [PATCH] i965: Use the predicate enable bit for conditional rendering without stalling

2014-11-09 Thread Daniel Vetter
On Fri, Nov 07, 2014 at 03:28:01PM +, Neil Roberts wrote: > Previously whenever a primitive is drawn the driver would call > _mesa_check_conditional_render which blocks waiting for the result of the > query to determine whether to render. On Gen7+ there is a bit in the > 3DPRIMITIVE command whi

Re: [Mesa-dev] [PATCH] r600g: Implement GL_ARB_draw_indirect

2014-11-09 Thread Benjamin Bellec
Hello, You have defined some "define" but you don't use it everywhere, for instance : cs->buf[cs->cdw++] = PKT3(0x24 /* PKT3_DRAW_INDIRECT */, 1, rctx->b.predicate_drawing); instead of simply : cs->buf[cs->cdw++] = PKT3(PKT3_DRAW_INDIRECT, 1, rctx->b.predicate_drawing); There is 5 instances lik

Re: [Mesa-dev] [PATCH] gbm: dlopen libglapi so gbm_create_device works

2014-11-09 Thread Frank Henigman
On Sat, Nov 8, 2014 at 7:13 PM, Emil Velikov wrote: > On 06/11/14 21:29, Frank Henigman wrote: >> From: Frank Henigman >> >> Dri driver libs are not linked to pull in libglapi so gbm_create_device() >> fails when it tries to dlopen them (unless the application is linked >> with something that doe

Re: [Mesa-dev] [PATCH] r600g: Implement GL_ARB_draw_indirect

2014-11-09 Thread Marek Olšák
This might not always work due to these reasons: These variables shouldn't be used anywhere if info.indirect != NULL: - info.start - info.count - info.index_bias For example, the translation of 8-bit indices is broken. The code which uses these variables has no effect if info.indirect != NULL. Fo

[Mesa-dev] [PATCH V4 3/3] i965: add runtime check for SSSE3 rgba8_copy

2014-11-09 Thread Timothy Arceri
Callgrind cpu usage results from pts benchmarks: For ytile_copy_faster() Nexuiz 1.6.1: 2.48% -> 0.97% The folowing are the only discernible results from teximage: Without patch and mesa default build flags - TexSubImage(BGRA/ubyte 256 x 256): 6122.6 images/sec, 1530.6 MB/sec With patch runtime

Re: [Mesa-dev] [PATCH V2] mesa: Permanently enable features supported by target CPU at compile time.

2014-11-09 Thread Bruno Jimenez
On Sun, 2014-11-09 at 14:24 +1100, Timothy Arceri wrote: > On Sat, 2014-11-08 at 18:25 +, Emil Velikov wrote: > > On 08/11/14 08:35, Siavash Eliasi wrote: > > > This will remove the need for unnecessary runtime checks for CPU features > > > if > > > already supported by target CPU, resulting i