Re: [Mesa-dev] [Mesa-announce] Mesa 10.3 release candidate 1

2014-11-04 Thread Matt Turner
On Tue, Nov 4, 2014 at 8:35 AM, Ausmus, James wrote: > On Mon, Nov 3, 2014 at 8:12 PM, Matt Turner wrote: >> On Mon, Nov 3, 2014 at 7:35 PM, Ausmus, James >> wrote: >> > I am able to reproduce this consistently with -j40 - it bisects to: >> >> Thanks. Maybe you could give a little more informati

Re: [Mesa-dev] [PATCH 0/2] Disable the EGL state tracker for Linux/DRI builds

2014-11-04 Thread Emil Velikov
On 04/11/14 22:42, Marek Olšák wrote: > Hi everybody, > > I'm about to address this long-standing issue: The EGL state tracker is > redundant. It duplicates what st/dri does and it also duplicates what > the common loader egl_dri2 does, which is used by all classic drivers > and even works better

Re: [Mesa-dev] [PATCH 00/10] glUniform* micro-optimizations, part 1

2014-11-04 Thread Ian Romanick
On 11/03/2014 04:36 PM, Brian Paul wrote: > On 11/03/2014 05:22 PM, Ian Romanick wrote: >> This is the first, and more minor, batch of micro-optimizations for the >> glUniform* paths. Other than patch 8, these probably aren't going to >> make a lot of difference, even on CPU limited applications.

[Mesa-dev] [PATCH 1/2] configure.ac: remove enable flags for EGL and GBM Gallium state trackers

2014-11-04 Thread Marek Olšák
From: Marek Olšák --- configure.ac| 74 +++-- docs/egl.html | 7 - src/gallium/Makefile.am | 8 -- 3 files changed, 4 insertions(+), 85 deletions(-) diff --git a/configure.ac b/configure.ac index fc7d372..91e111b 100644

[Mesa-dev] [PATCH 0/2] Disable the EGL state tracker for Linux/DRI builds

2014-11-04 Thread Marek Olšák
Hi everybody, I'm about to address this long-standing issue: The EGL state tracker is redundant. It duplicates what st/dri does and it also duplicates what the common loader egl_dri2 does, which is used by all classic drivers and even works better with gallium drivers. Let's compare EGL extension

[Mesa-dev] [PATCH 2/2] egl: remove egl_gallium from the loader

2014-11-04 Thread Marek Olšák
From: Marek Olšák --- src/egl/main/egldriver.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/src/egl/main/egldriver.c b/src/egl/main/egldriver.c index 78d8130..8cf777d 100644 --- a/src/egl/main/egldriver.c +++ b/src/egl/main/egldriver.c @@ -518,19 +518,6 @@ _eglAddUserDriver

Re: [Mesa-dev] [PATCH 2/3][RFC v2] mesa/main/x86: Add sse2 streaming clamping

2014-11-04 Thread Roland Scheidegger
Am 04.11.2014 um 13:05 schrieb Juha-Pekka Heikkila: > Signed-off-by: Juha-Pekka Heikkila > --- > src/mesa/Makefile.am | 8 +++ > src/mesa/main/x86/sse2_clamping.c | 103 > ++ > src/mesa/main/x86/sse2_clamping.h | 49 ++ > 3 file

Re: [Mesa-dev] [PATCH 2/3][RFC v2] mesa/main/x86: Add sse2 streaming clamping

2014-11-04 Thread Patrick Baggett
On Tue, Nov 4, 2014 at 6:05 AM, Juha-Pekka Heikkila < juhapekka.heikk...@gmail.com> wrote: > Signed-off-by: Juha-Pekka Heikkila > --- > src/mesa/Makefile.am | 8 +++ > src/mesa/main/x86/sse2_clamping.c | 103 > ++ > src/mesa/main/x86/sse2_clampi

Re: [Mesa-dev] [PATCH 2/3][RFC v2] mesa/main/x86: Add sse2 streaming clamping

2014-11-04 Thread Siavash Eliasi
Hello. I'd get rid of "_mm_set1_ps" inside "_mesa_clamp_float_rgba" by passing _m128 version of min/max directly, so "_mm_set1_ps" will be moved out of the for loop. I'd also unroll the "_mesa_streaming_clamp_float_rgba" loop to minimize the loop overhead (and utilize out of order execution as

Re: [Mesa-dev] [PATCH 0/4] i965: Totally legit line width patches

2014-11-04 Thread Kristian Høgsberg
On Mon, Nov 3, 2014 at 6:42 PM, Kenneth Graunke wrote: > Here are some totally legit line width patches. I noticed that Cherryview > was setting line width in DW2 of 3DSTATE_SF, when it actually moved to DW1 > at a different bit location. While fixing that, I figured I should update > the clamp

[Mesa-dev] [Bug 84566] Unify the format conversion code

2014-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84566 --- Comment #48 from Samuel Iglesias --- (In reply to Jason Ekstrand from comment #34) > (In reply to Samuel Iglesias from comment #33) > > Jason, I would like to know your opinion about the integer RGBA clamping > > done in pack.c (_mesa_pack_rg

Re: [Mesa-dev] Conditional rendering on i965

2014-11-04 Thread Kenneth Graunke
On Tuesday, November 04, 2014 05:31:55 PM Neil Roberts wrote: > Hi, > > I was thinking of taking a look at implementing proper conditional > rendering for i965. Currently it looks like we cheat to implement this > and just stall the GPU to wait for the result before deciding whether to > issue the

Re: [Mesa-dev] [PATCH] i965: Fix sampler state pointer adjustment for nonconst samplers

2014-11-04 Thread Kenneth Graunke
On Wednesday, November 05, 2014 06:41:13 AM Chris Forbes wrote: > This started hitting an assertion recently. Only affects Haswell > (Ivybridge doesn't support this meddling with the sampler state pointer, > and ARB_gpu_shader5 is not enabled yet on Broadwell) > > 14 Piglits crash->pass. > > Sign

[Mesa-dev] [PATCH] i965: Fix sampler state pointer adjustment for nonconst samplers

2014-11-04 Thread Chris Forbes
This started hitting an assertion recently. Only affects Haswell (Ivybridge doesn't support this meddling with the sampler state pointer, and ARB_gpu_shader5 is not enabled yet on Broadwell) 14 Piglits crash->pass. Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 2 +-

[Mesa-dev] [Bug 85869] Offscreen rendering to pixmap EGLSurface is odd

2014-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=85869 Rinat changed: What|Removed |Added CC||ibragimovri...@mail.ru -- You are receiving thi

[Mesa-dev] [Bug 85869] Offscreen rendering to pixmap EGLSurface is odd

2014-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=85869 Bug ID: 85869 Summary: Offscreen rendering to pixmap EGLSurface is odd Product: Mesa Version: unspecified Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

[Mesa-dev] Conditional rendering on i965

2014-11-04 Thread Neil Roberts
Hi, I was thinking of taking a look at implementing proper conditional rendering for i965. Currently it looks like we cheat to implement this and just stall the GPU to wait for the result before deciding whether to issue the 3DPRIMITIVE command. I think Gen7+ can support it in hardware with the MI

[Mesa-dev] [PATCH] i965/blorp: Fix hiz ops on MSAA surfaces

2014-11-04 Thread Chris Forbes
Two things were broken here: - The depth/stencil surface dimensions were broken for MSAA. - Sample count was programmed incorrectly. Result was the depth resolve didn't work correctly on MSAA surfaces, and so sampling the surface later produced garbage. Fixes the new piglit test arb_texture_multi

Re: [Mesa-dev] [Mesa-announce] Mesa 10.3 release candidate 1

2014-11-04 Thread Ausmus, James
On Mon, Nov 3, 2014 at 8:12 PM, Matt Turner wrote: > > On Mon, Nov 3, 2014 at 7:35 PM, Ausmus, James wrote: > > I am able to reproduce this consistently with -j40 - it bisects to: > > Thanks. Maybe you could give a little more information, like an error > message or something? Same error as Thie

[Mesa-dev] [Bug 85608] Account request for David Heidelberg

2014-11-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=85608 Brian Paul changed: What|Removed |Added Component|Other |New Accounts Assignee|mesa-dev@li

Re: [Mesa-dev] [PATCH] util: Implement unreachable for MSVC using __assume

2014-11-04 Thread Brian Paul
On 11/03/2014 04:43 PM, Ian Romanick wrote: From: Ian Romanick Based on the description of __assume at: http://msdn.microsoft.com/en-us/library/1b3fsfxw.aspx Signed-off-by: Ian Romanick Cc: Brian Paul --- src/util/macros.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/uti

Re: [Mesa-dev] [PATCH 00/10] glUniform* micro-optimizations, part 1

2014-11-04 Thread Tapani
Reviewed-by: Tapani Pälli On 11/04/2014 02:22 AM, Ian Romanick wrote: This is the first, and more minor, batch of micro-optimizations for the glUniform* paths. Other than patch 8, these probably aren't going to make a lot of difference, even on CPU limited applications. The next batch, which

[Mesa-dev] [PATCH] glsl: throw error when using invariant(all) in a fragment shader

2014-11-04 Thread Tapani Pälli
Note that some of the GLSL specifications explicitly state this as compile error, some simply state that 'it is an error'. Signed-off-by: Tapani Pälli --- src/glsl/glsl_parser.yy | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/src/glsl/glsl_parser.yy b/src/glsl/

[Mesa-dev] [PATCH 0/3][RFC v2] Clamp rgba floats with sse

2014-11-04 Thread Juha-Pekka Heikkila
Here is new version of sse2 clamping, one patch grew into small set. Now sse2 stuff is separated into its own object which will get -msse2 compile flag. I did sse'ize also rest of _mesa_map_rgba function. As previously there are ifdefs which I don't think look nice but I tried to keep looks of the

[Mesa-dev] [PATCH 1/3][RFC v2] configure.ac: Add detection for sse2 compilation support

2014-11-04 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- configure.ac | 7 +++ 1 file changed, 7 insertions(+) diff --git a/configure.ac b/configure.ac index fc7d372..a01e605 100644 --- a/configure.ac +++ b/configure.ac @@ -258,6 +258,13 @@ if test "x$SSE41_SUPPORTED" = x1; then fi AM_CONDITIONAL([SSE41_SUP

[Mesa-dev] [PATCH 2/3][RFC v2] mesa/main/x86: Add sse2 streaming clamping

2014-11-04 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/mesa/Makefile.am | 8 +++ src/mesa/main/x86/sse2_clamping.c | 103 ++ src/mesa/main/x86/sse2_clamping.h | 49 ++ 3 files changed, 160 insertions(+) create mode 100644 src/mesa/main/x86/

[Mesa-dev] [PATCH 3/3][RFC v2] mesa/main: Clamp rgba with streamed sse

2014-11-04 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/mesa/main/pixeltransfer.c | 62 ++- 1 file changed, 43 insertions(+), 19 deletions(-) diff --git a/src/mesa/main/pixeltransfer.c b/src/mesa/main/pixeltransfer.c index 8bbeeb8..99eed38 100644 --- a/src/mesa/main/pi

Re: [Mesa-dev] [PATCH] mesa: Silence unused parameter warning in check_context_limits in non-debug builds

2014-11-04 Thread Kenneth Graunke
On Monday, November 03, 2014 03:43:40 PM Ian Romanick wrote: > From: Ian Romanick > > ../../src/mesa/main/context.c: In function 'check_context_limits': > ../../src/mesa/main/context.c:733:41: warning: unused parameter 'ctx' [- Wunused-parameter] > > Signed-off-by: Ian Romanick > --- > src/mes