[Mesa-dev] [PATCH 0/4] Add EXT_polygon_offset_clamp support

2014-12-30 Thread Ilia Mirkin
I haven't the faintest clue how to test this, but the implementation seems fairly straightforward. Not sure if I should just change the extension to be dummy_true (for core contexts) or leave it as-is. Ilia Mirkin (4): glapi: add GL_EXT_polygon_offset_clamp mesa: add support for GL_EXT_polygon

[Mesa-dev] [PATCH 3/4] i965/gen6+: enable EXT_polygon_offset_clamp

2014-12-30 Thread Ilia Mirkin
Replace the hard-coded 0's with the context clamp value. Signed-off-by: Ilia Mirkin --- docs/relnotes/10.5.0.html| 1 + src/mesa/drivers/dri/i965/gen6_sf_state.c| 2 +- src/mesa/drivers/dri/i965/gen7_sf_state.c| 2 +- src/mesa/drivers/dri/i965/gen8_sf_state.c| 2 +

[Mesa-dev] [PATCH 1/4] glapi: add GL_EXT_polygon_offset_clamp

2014-12-30 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- src/mapi/glapi/gen/gl_API.xml | 11 +++ src/mesa/main/polygon.c | 6 ++ src/mesa/main/polygon.h | 5 - 3 files changed, 21 insertions(+), 1 deletion(-) diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml index e1

[Mesa-dev] [PATCH 4/4] st/mesa: add EXT_polygon_offset_clamp support

2014-12-30 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- docs/relnotes/10.5.0.html | 2 +- src/mesa/state_tracker/st_atom_rasterizer.c | 1 + src/mesa/state_tracker/st_extensions.c | 5 + 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/docs/relnotes/10.5.0.html b/docs/relnotes/10.5

[Mesa-dev] [PATCH 2/4] mesa: add support for GL_EXT_polygon_offset_clamp

2014-12-30 Thread Ilia Mirkin
Nothing enables the extension yet, but the values are now available. The spec calls for it to only be exposed for GL 3.3+, which is core-only in mesa. Restrict it as such so that drivers enabling the extension don't end up accidentally exposing the function in compat contexts. Signed-off-by: Ilia

Re: [Mesa-dev] [PATCH] u_primconvert: Fix leak of the upload BO on context destroy.

2014-12-30 Thread Dave Airlie
Reviewed-by: Dave Airlie On 31 December 2014 at 10:13, Eric Anholt wrote: > --- > src/gallium/auxiliary/indices/u_primconvert.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/gallium/auxiliary/indices/u_primconvert.c > b/src/gallium/auxiliary/indices/u_primconvert.c > index eba1f

Re: [Mesa-dev] [PATCH 1/2] i965/skl: Always use a header for SIMD4x2 sampler messages

2014-12-30 Thread Matt Turner
Also... :) >> Signed-off-by: Kristian Høgsberg ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 2/2] i965/skl: Report more accurate number of samples for format

2014-12-30 Thread Matt Turner
On Tue, Dec 30, 2014 at 9:24 PM, Kristian Høgsberg wrote: > Signed-off-by: Kristian Høgsberg Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 1/2] i965/skl: Always use a header for SIMD4x2 sampler messages

2014-12-30 Thread Matt Turner
On Tue, Dec 30, 2014 at 9:24 PM, Kristian Høgsberg wrote: > SKL+ overloads the SIMD4x2 SIMD mode to mean either SIMD8D or SIMD4x2 > depending on bit 22 in the message header. If the bit is 0 or there is > no header we get SIMD8D. We always wand SIMD4x2 in vec4 and for fs pull > constants, so alw

[Mesa-dev] [PATCH 1/2] i965/skl: Always use a header for SIMD4x2 sampler messages

2014-12-30 Thread Kristian Høgsberg
SKL+ overloads the SIMD4x2 SIMD mode to mean either SIMD8D or SIMD4x2 depending on bit 22 in the message header. If the bit is 0 or there is no header we get SIMD8D. We always wand SIMD4x2 in vec4 and for fs pull constants, so always use a message header and set bit 22 there. Signed-off-by: Kris

[Mesa-dev] [PATCH 2/2] i965/skl: Report more accurate number of samples for format

2014-12-30 Thread Kristian Høgsberg
Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/dri/i965/brw_context.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c index 860ee22d..e8ecbfe 100644 --- a/src/mesa/drivers/dri/i965/brw_context.c +++ b/

[Mesa-dev] [PATCH] nv50,nvc0: set vertex id base to index_bias

2014-12-30 Thread Ilia Mirkin
Fixes the piglits which check that gl_VertexID includes the base vertex offset: arb_draw_indirect-vertexid elements gl-3.2-basevertex-vertexid Note that this leaves out the original G80, for which this will continue to fail. It could be fixed by passing a driver constbuf value in, but that's b

[Mesa-dev] [PATCH 1/2] nv50: regenerate rnndb headers

2014-12-30 Thread Ilia Mirkin
The headers hadn't been regenerated in a long time, and there were a few minor divergences. Among other things, rnndb has changed naming to G80/etc, for now I've not tackled switching that over and manually replaced the nvidia codenames back to the chip ids. However no other modifications of the he

Re: [Mesa-dev] [PATCH 19/41] main: Added entry point for glTextureParameterfv.

2014-12-30 Thread Anuj Phogat
On Tue, Dec 16, 2014 at 11:54 PM, Laura Ekstrand wrote: > No. I have a 78 column line highlight in my editor. > It does fit when you remove the whitespaces in function braces. > On Tue, Dec 16, 2014 at 7:46 AM, Brian Paul wrote: >> >> On 12/15/2014 06:22 PM, Laura Ekstrand wrote: >>> >>> --- >>>

[Mesa-dev] [Bug 87886] constant fps drops with Intel and Radeon on Source games

2014-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=87886 Jason Ekstrand changed: What|Removed |Added CC||ja...@jlekstrand.net -- You are receiv

Re: [Mesa-dev] [PATCH 18/41] main: Added entry point for glTextureParameterf.

2014-12-30 Thread Anuj Phogat
On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand wrote: > --- > src/mapi/glapi/gen/ARB_direct_state_access.xml | 6 +++ > src/mesa/main/texparam.c | 52 > +- > src/mesa/main/texparam.h | 20 ++ > 3 files changed, 68 inse

[Mesa-dev] [Bug 87886] constant fps drops with Intel and Radeon on Source games

2014-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=87886 Bug ID: 87886 Summary: constant fps drops with Intel and Radeon on Source games Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All)

Re: [Mesa-dev] [PATCH 18/41] main: Added entry point for glTextureParameterf.

2014-12-30 Thread Laura Ekstrand
On Tue, Dec 16, 2014 at 7:45 AM, Brian Paul wrote: > On 12/15/2014 06:22 PM, Laura Ekstrand wrote: > >> --- >> src/mapi/glapi/gen/ARB_direct_state_access.xml | 6 +++ >> src/mesa/main/texparam.c | 52 >> +- >> src/mesa/main/texparam.h

Re: [Mesa-dev] [PATCH 00/41] ARB_direct_state_access texture functions

2014-12-30 Thread Laura Ekstrand
That is now fixed. Each added entry point in my history passes make check now. Thanks. Laura On Tue, Dec 30, 2014 at 2:21 PM, Anuj Phogat wrote: > Your adsa-textures branch on fdo fails make check. > > On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand > wrote: > > This implements all of the te

[Mesa-dev] [PATCH] u_primconvert: Fix leak of the upload BO on context destroy.

2014-12-30 Thread Eric Anholt
--- src/gallium/auxiliary/indices/u_primconvert.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/auxiliary/indices/u_primconvert.c b/src/gallium/auxiliary/indices/u_primconvert.c index eba1f9e..11fceaf 100644 --- a/src/gallium/auxiliary/indices/u_primconvert.c +++ b/src/gallium/a

Re: [Mesa-dev] [PATCH 17/41] main: Added get_texobj_by_name in texparam.c.

2014-12-30 Thread Anuj Phogat
On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand wrote: > This is a convenience function for *Texture*Parameter functions. > --- > src/mesa/main/texparam.c | 38 ++ > 1 file changed, 38 insertions(+) > > diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texp

Re: [Mesa-dev] [PATCH 15/41] main: set_tex_parameteri now handles errors according to the OpenGL 4.5 Specification.

2014-12-30 Thread Anuj Phogat
On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand wrote: > Beginning in the OpenGL 4.3 core specification, some error handling has > changed. As an example, changing sampler states with a multisample target > throws INVALID_ENUM rather than INVALID_OPERATION. Patch also changes error handling for GL

Re: [Mesa-dev] [PATCH] Avoid mixing #ifdef USE_SSE41 and control flow

2014-12-30 Thread Matt Turner
On Tue, Dec 30, 2014 at 3:04 PM, Kristian Høgsberg wrote: > diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > index f815fbe..c471a76 100644 > --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > +++ b/src/mesa/drivers/dri/i965/intel_mi

Re: [Mesa-dev] [PATCH 14/41] main: Added entry point for BindTextureUnit.

2014-12-30 Thread Anuj Phogat
On Tue, Dec 16, 2014 at 7:46 AM, Brian Paul wrote: > On 12/15/2014 06:22 PM, Laura Ekstrand wrote: >> >> The following preparations were made in texstate.c and texstate.h to >> better facilitate the BindTextureUnit function: >> >> Dylan Noblesmith: >> mesa: add _mesa_get_tex_unit() >> mesa: factor

Re: [Mesa-dev] [PATCH] Avoid mixing #ifdef USE_SSE41 and control flow

2014-12-30 Thread Jason Ekstrand
Yeah, I like this; it's much cleaner. #ifdef's are always liable to break the build with different compile options. I'm CC'ing matt because I'd like his quick eye as he knows far more about the build system than I do. As far as I'm concerned, it looks good. Reviewed-by: Jason Ekstrand On Tue,

[Mesa-dev] [PATCH] Avoid mixing #ifdef USE_SSE41 and control flow

2014-12-30 Thread Kristian Høgsberg
We avoid using the USE_SSE41 #ifdef by #defining cpu_has_sse4_1 to 0 when USE_SSE41 is undefined. This way, we only need to test if cpu_has_sse4_1 is true before calling into SSE 4.1 code. If USE_SSE41 is undefined, we typically end up with if (0), and the compiler will optimize the SSE4.1 code a

[Mesa-dev] [Bug 86651] [i965 Bisected]Piglit glx_glx-pixmap-multi fails, with DRI3 disabled

2014-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=86651 Chris Wilson changed: What|Removed |Added Component|Driver/intel|GLX Assignee|ch...@chris-wilson

Re: [Mesa-dev] [PATCH 00/41] ARB_direct_state_access texture functions

2014-12-30 Thread Anuj Phogat
Your adsa-textures branch on fdo fails make check. On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand wrote: > This implements all of the texture functions for ARB_direct_state_access, > with the exception of glTextureBufferRange. There is an outstanding bug on > the specification for glTextureBu

Re: [Mesa-dev] [PATCH 01/41] glapi: Added ARB_direct_state_access.xml file.

2014-12-30 Thread Laura Ekstrand
To run this partial implementation, export MESA_EXTENSION_OVERRIDE=+GL_ARB_direct_state_access I just did that and my dsa piglit tests ran fine. Laura On Mon, Dec 29, 2014 at 3:25 PM, Emil Velikov wrote: > On 16/12/14 01:22, Laura Ekstrand wrote: > > diff --git a/src/mesa/main/extensions.c b/

Re: [Mesa-dev] [PATCH 12/41] main: Added entry points for glTextureSubImage*D.

2014-12-30 Thread Laura Ekstrand
On Mon, Dec 29, 2014 at 3:02 PM, Anuj Phogat wrote: > On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand > wrote: > > --- > > src/mapi/glapi/gen/ARB_direct_state_access.xml | 36 +++ > > src/mesa/main/teximage.c | 343 > +++-- > > src/mesa/main/teximage.h

[Mesa-dev] [Bug 86837] kodi segfault since auxiliary/vl: rework the build of the VL code

2014-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=86837 --- Comment #14 from smoki --- (In reply to Emil Velikov from comment #12) > Seems like Christian dropped the link with the tentative fix. > http://patchwork.freedesktop.org/patch/39400/ > > Guys can you test this please ? > Works OK for me to

Re: [Mesa-dev] [PATCH] i965: Use WARN_ONCE for the single-primitive-exceeded-aperture message.

2014-12-30 Thread Jason Ekstrand
Fine by me Reviewed-by: Jason Ekstrand On Tue, Dec 30, 2014 at 12:23 PM, Kenneth Graunke wrote: > This makes it show up via ARB_debug_output and is also less code. > > Signed-off-by: Kenneth Graunke > --- > src/mesa/drivers/dri/i965/brw_draw.c | 13 - > 1 file changed, 4 insertio

[Mesa-dev] [PATCH] i965: Use WARN_ONCE for the single-primitive-exceeded-aperture message.

2014-12-30 Thread Kenneth Graunke
This makes it show up via ARB_debug_output and is also less code. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_draw.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_draw.c b/src/mesa/drivers/dri/i965/brw_draw.c

[Mesa-dev] [Bug 87658] [llvmpipe] SEGV in sse2_has_daz on ancient Pentium4-M

2014-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=87658 --- Comment #20 from Chris Paulson-Ellis --- (In reply to Roland Scheidegger from comment #19) > Maybe using > PIPE_ALIGN_STACK static INLINE boolean sse2_has_daz(void) > would do the trick. I guess the PIPE_ALIGN_VAR on a stack'ed variable isn't

[Mesa-dev] [Bug 86837] kodi segfault since auxiliary/vl: rework the build of the VL code

2014-12-30 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=86837 --- Comment #13 from Andy Furniss --- (In reply to Emil Velikov from comment #12) > Seems like Christian dropped the link with the tentative fix. > http://patchwork.freedesktop.org/patch/39400/ > > Guys can you test this please ? Works OK for m

Re: [Mesa-dev] [PATCH] auxiliary/vl: move most sources back into the aux library

2014-12-30 Thread Christian König
Am 20.12.2014 um 12:34 schrieb Emil Velikov: On 19/12/14 14:14, Christian König wrote: Am 19.12.2014 um 15:04 schrieb Emil Velikov: Due to the current implementation of the gl-vdpau interop, we depend on the dri module having a working set of util VL functions. Otherwise we will end up calling