[Mesa-dev] [Bug 79629] [Dri3 bisected] piglit glx_GLX_ARB_create_context_current_with_no_framebuffer fail

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79629 --- Comment #9 from XiongZhang --- Created attachment 113030 --> https://bugs.freedesktop.org/attachment.cgi?id=113030&action=edit it should fix this issue -- You are receiving this mail because: You are the assignee for the bug.

Re: [Mesa-dev] [PATCH 1/2] glsl: Add define for ARB_shader_precision

2015-02-01 Thread Chris Forbes
Glenn, Fixed in the version of this patch which landed (d36fa601913 on master). - Chris On Mon, Feb 2, 2015 at 2:08 PM, Glenn Kennard wrote: > On Wed, 31 Dec 2014 21:43:51 +0100, Micah Fedke > wrote: > >> --- >> src/glsl/glcpp/glcpp-parse.y| 3 +++ >> src/glsl/glsl_parser_extras.cpp | 1 +

Re: [Mesa-dev] [PATCH 1/2] glsl: Add define for ARB_shader_precision

2015-02-01 Thread Glenn Kennard
On Wed, 31 Dec 2014 21:43:51 +0100, Micah Fedke wrote: --- src/glsl/glcpp/glcpp-parse.y| 3 +++ src/glsl/glsl_parser_extras.cpp | 1 + src/glsl/glsl_parser_extras.h | 2 ++ src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 5 files changed, 8 insertions(+) d

Re: [Mesa-dev] [PATCH 0/7] i965 L3 caching and pull constant improvements.

2015-02-01 Thread Syrja, Harri
Ok I see. I am not sure if it was good choice from VPG not to support dedicated region for constant cache, but if that is what it is there is little we can do for it. Thank, Harri -Original Message- From: Kenneth Graunke [mailto:kenn...@whitecape.org] Sent: Wednesday, January 28, 2015

[Mesa-dev] [PATCH] c11/threads: Use PTHREAD_MUTEX_RECURSIVE by default

2015-02-01 Thread Felix Janda
Previously PTHREAD_MUTEX_RECURSIVE_NP had been used on linux for compatibility with old glibc. Now fall back to the nonstandard version only if the standard version is not defined. This makes mesa compile with musl libc. --- include/c11/threads_posix.h | 11 +-- 1 file changed, 5 insertion

Re: [Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Connor Abbott
On Sun, Feb 1, 2015 at 6:54 PM, Eric Anholt wrote: > Connor Abbott writes: > >> On Sun, Feb 1, 2015 at 4:17 PM, Eric Anholt wrote: >>> We've probably never seen this ridiculous pattern in the wild, so it >>> didn't matter. >> >> I don't think this pattern is 'ridiculous' at all -- GLSL doesn't h

Re: [Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Eric Anholt
Connor Abbott writes: > On Sun, Feb 1, 2015 at 4:17 PM, Eric Anholt wrote: >> We've probably never seen this ridiculous pattern in the wild, so it >> didn't matter. > > I don't think this pattern is 'ridiculous' at all -- GLSL doesn't have > a sat operation, so shaders wanting to do a saturate h

Re: [Mesa-dev] [PATCH v2 0/2] Fast mutex v2

2015-02-01 Thread Jordan Justen
On 2015-01-29 20:54:27, Kristian Høgsberg wrote: > Here's take two of the fast mutex path. This version has a check for > linux/futex.h and it split into two patches. First patch introduces > full_mtx_t and updates callers. Second patch adds the inline mtx_t. > > Kristian Høgsberg (2): > mesa

Re: [Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Matt Turner
On Sun, Feb 1, 2015 at 2:07 PM, Connor Abbott wrote: > On Sun, Feb 1, 2015 at 4:17 PM, Eric Anholt wrote: >> We've probably never seen this ridiculous pattern in the wild, so it >> didn't matter. > > I don't think this pattern is 'ridiculous' at all -- GLSL doesn't have > a sat operation, so shad

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

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=86837 John changed: What|Removed |Added CC||john.etted...@gmail.com -- You are receiving thi

Re: [Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Jason Ekstrand
On Feb 1, 2015 2:07 PM, "Connor Abbott" wrote: > > On Sun, Feb 1, 2015 at 4:17 PM, Eric Anholt wrote: > > We've probably never seen this ridiculous pattern in the wild, so it > > didn't matter. > > I don't think this pattern is 'ridiculous' at all -- GLSL doesn't have > a sat operation, so shader

Re: [Mesa-dev] [PATCH 04/10] radeonsi: move the buffer descriptor to the end of the image descriptor

2015-02-01 Thread Ilia Mirkin
On Sun, Feb 1, 2015 at 5:06 PM, Marek Olšák wrote: > No. The image descriptor really has 8 dwords, therefore the slot can > contain either a texture or a buffer, but not both. Gotcha. Seemed like a typo, so figured I'd point it out. Sounds like it was intentional though. _

Re: [Mesa-dev] [PATCH 1/9] tgsi: add tgsi_get_processor_type helper from radeon

2015-02-01 Thread Marek Olšák
Yes. I think the tests should segfault on most drivers, because they test something that has never been used before. Marek On Sun, Feb 1, 2015 at 9:10 PM, Glenn Kennard wrote: > On Sun, 01 Feb 2015 18:15:53 +0100, Marek Olšák wrote: > >> From: Marek Olšák >> >> --- >> src/gallium/auxiliary/tg

Re: [Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Connor Abbott
On Sun, Feb 1, 2015 at 4:17 PM, Eric Anholt wrote: > We've probably never seen this ridiculous pattern in the wild, so it > didn't matter. I don't think this pattern is 'ridiculous' at all -- GLSL doesn't have a sat operation, so shaders wanting to do a saturate have to do this already. Rather, w

Re: [Mesa-dev] [PATCH 1/9] nir: Slightly simplify algebraic code generation by reusing a struct.

2015-02-01 Thread Jason Ekstrand
Wow... Reviewed-by: Jason Ekstrand On Feb 1, 2015 1:17 PM, "Eric Anholt" wrote: > --- > src/glsl/nir/nir_algebraic.py | 9 +++-- > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/src/glsl/nir/nir_algebraic.py b/src/glsl/nir/nir_algebraic.py > index ea7f5fc..4929745 100644

Re: [Mesa-dev] [PATCH 04/10] radeonsi: move the buffer descriptor to the end of the image descriptor

2015-02-01 Thread Marek Olšák
No. The image descriptor really has 8 dwords, therefore the slot can contain either a texture or a buffer, but not both. The "NULL" descriptor is different for buffers and textures, therefore it seemed textures and buffers couldn't occupy the same slot if we wanted to have both. Since the last 4 d

Re: [Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Feb 1, 2015 1:17 PM, "Eric Anholt" wrote: > We've probably never seen this ridiculous pattern in the wild, so it > didn't matter. > --- > src/glsl/nir/nir_opt_algebraic.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/glsl/nir/nir_op

[Mesa-dev] [Bug 88885] Transform feedback uses incorrect interleaving if a previous draw did not write gl_Position

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=5 Chris Forbes changed: What|Removed |Added CC||chr...@ijw.co.nz -- You are receiving th

Re: [Mesa-dev] [PATCH 5/9] nir: Add a flag for lowering fneg/ineg.

2015-02-01 Thread Connor Abbott
I think this logic should be flipped around... you should have a flag 'lower_sub' which is false for vc4 but true for i965, and then if the flag is true we lower sub to add+neg and if false we try and reconstruct sub (and do other things like pulling neg's out of multiplies). That way, eventually w

Re: [Mesa-dev] [PATCH 36/43] st/nine: Fix multisampling support detection

2015-02-01 Thread Marek Olšák
I don't think it matters. Generally, all drivers should support mapping all formats, but some drivers can't map certain formats with more than 1 sample. Marek On Sun, Feb 1, 2015 at 8:28 PM, Axel Davy wrote: > I checked, and the application is allowed to lock the backbuffer if it > specified a

[Mesa-dev] Target-aware nir_opt_algebraic.

2015-02-01 Thread Eric Anholt
I'm pretty happy with how it turned out. What I'm not sure on is that I've got a couple of places where I replicated another transformation, like fsat(fsat(a)) -> fsat(a) but with a lowered fsat. I could see potentially having only the canonical form appear in nir_opt_algebraic, and I'd have two

[Mesa-dev] [PATCH 4/9] nir: Add a flag for lowering fsqrt(x) to frcp(frsqrt(x)).

2015-02-01 Thread Eric Anholt
vc4 cse/algebraic-disabled stats: total uniforms in shared programs: 13972 -> 13966 (-0.04%) uniforms in affected programs: 408 -> 402 (-1.47%) total instructions in shared programs: 44973 -> 44911 (-0.14%) instructions in affected programs: 1551 -> 1489 (-4.00%) --- src/gallium/drivers/vc

[Mesa-dev] [PATCH 9/9] nir: Recognize and reduce duplicated fsats.

2015-02-01 Thread Eric Anholt
No effect on vc4 shader-db. --- src/glsl/nir/nir_opt_algebraic.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/glsl/nir/nir_opt_algebraic.py b/src/glsl/nir/nir_opt_algebraic.py index 9542ef3..fc7e47c 100644 --- a/src/glsl/nir/nir_opt_algebraic.py +++ b/src/glsl/nir/nir_opt_algebraic.

[Mesa-dev] [PATCH 7/9] nir: Fix broken fsat recognizer.

2015-02-01 Thread Eric Anholt
We've probably never seen this ridiculous pattern in the wild, so it didn't matter. --- src/glsl/nir/nir_opt_algebraic.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/glsl/nir/nir_opt_algebraic.py b/src/glsl/nir/nir_opt_algebraic.py index 8a14fe0..84c60ec 100644 --- a/s

[Mesa-dev] [PATCH 1/9] nir: Slightly simplify algebraic code generation by reusing a struct.

2015-02-01 Thread Eric Anholt
--- src/glsl/nir/nir_algebraic.py | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/glsl/nir/nir_algebraic.py b/src/glsl/nir/nir_algebraic.py index ea7f5fc..4929745 100644 --- a/src/glsl/nir/nir_algebraic.py +++ b/src/glsl/nir/nir_algebraic.py @@ -179,10 +179,7 @@ st

[Mesa-dev] [PATCH 8/9] nir: Add a flag for lowering fsat.

2015-02-01 Thread Eric Anholt
vc4 cse/algebraic-disabled stats: total instructions in shared programs: 44356 -> 44354 (-0.00%) instructions in affected programs: 55 -> 53 (-3.64%) --- src/gallium/drivers/vc4/vc4_program.c | 1 + src/glsl/nir/nir.h| 1 + src/glsl/nir/nir_opt_algebraic.py | 3 ++- 3 f

[Mesa-dev] [PATCH 6/9] nir: Add a flag for lowering ffma.

2015-02-01 Thread Eric Anholt
vc4 cse/algebraic-disabled stats: total uniforms in shared programs: 13966 -> 13791 (-1.25%) uniforms in affected programs: 435 -> 260 (-40.23%) total instructions in shared programs: 44732 -> 44356 (-0.84%) instructions in affected programs: 9599 -> 9223 (-3.92%) --- src/gallium/drivers/v

[Mesa-dev] [PATCH 5/9] nir: Add a flag for lowering fneg/ineg.

2015-02-01 Thread Eric Anholt
vc4 cse/algebraic-disabled stats: total instructions in shared programs: 44911 -> 44732 (-0.40%) instructions in affected programs: 11371 -> 11192 (-1.57%) --- src/gallium/drivers/vc4/vc4_program.c | 1 + src/glsl/nir/nir.h| 2 ++ src/glsl/nir/nir_opt_algebraic.py | 1

[Mesa-dev] [PATCH 2/9] nir: Conditionalize the POW reconstruction on shader compiler options.

2015-02-01 Thread Eric Anholt
Mesa has a shader compiler struct flagging whether GLSL IR's opt_algebraic and other passes should try and generate certain types of opcodes or patterns. Extend that to NIR by defining our own struct, which is automatically generated from the Mesa struct in glsl_to_nir and provided directly by the

[Mesa-dev] [PATCH 3/9] nir: Add lowering of POW instructions if the lower flag is set.

2015-02-01 Thread Eric Anholt
This could be done in a separate pass like we do in GLSL IR, but it seems to me like having the definitions of the transformations in the two directions next to each other makes a lot of sense. --- src/glsl/nir/nir_opt_algebraic.py | 1 + 1 file changed, 1 insertion(+) diff --git a/src/glsl/nir/n

[Mesa-dev] [Bug 88885] Transform feedback uses incorrect interleaving if a previous draw did not write gl_Position

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=5 Felix Schwarz changed: What|Removed |Added CC||felix.schwarz@oss.schwarz.e

[Mesa-dev] [Bug 88885] Transform feedback uses incorrect interleaving if a previous draw did not write gl_Position

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=5 --- Comment #1 from Felix Schwarz --- This works for me on r600g but I see a crash in your reproducer on Intel (gen4) using both mesa 10.4.2. Problem might be specific to Intel. -- You are receiving this mail because: You are the QA Contact for

Re: [Mesa-dev] [PATCH 1/9] tgsi: add tgsi_get_processor_type helper from radeon

2015-02-01 Thread Glenn Kennard
On Sun, 01 Feb 2015 18:15:53 +0100, Marek Olšák wrote: From: Marek Olšák --- src/gallium/auxiliary/tgsi/tgsi_parse.c | 12 src/gallium/auxiliary/tgsi/tgsi_parse.h | 2 ++ src/gallium/drivers/radeon/r600_pipe_common.c | 11 --- 3 files changed, 14 insertions(

Re: [Mesa-dev] [PATCH 01/10] r600g, radeonsi: don't append to streamout buffers that haven't been used yet

2015-02-01 Thread Glenn Kennard
On Sun, 01 Feb 2015 18:36:58 +0100, Marek Olšák wrote: From: Marek Olšák The FILLED_SIZE counter is uninitialized at the beginning, so we can't use it. Instead, use offset = 0, which is what we always do when not appending. This unexpectedly fixes spec/ARB_texture_multisample/sample-posit

[Mesa-dev] [Bug 87076] Dead Island needs allow_glsl_extension_directive_midshader

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=87076 Laurent carlier changed: What|Removed |Added CC||v10la...@myway.de --- Comment #3 from

Re: [Mesa-dev] [PATCH 36/43] st/nine: Fix multisampling support detection

2015-02-01 Thread Axel Davy
I checked, and the application is allowed to lock the backbuffer if it specified a specific flag. However it seems locking a multisampled buffer is forbidden. So probably here the PIPE_BIND_TRANSFER_* flags could be removed for multisampling, should I add that behaviour ? Does it matter for s

Re: [Mesa-dev] [PATCH 3/7] glsl: Forbid opaque variables as operands of the ternary operator.

2015-02-01 Thread Francisco Jerez
Ian Romanick writes: > On 02/01/2015 01:15 PM, Francisco Jerez wrote: >> Ian Romanick writes: >> >>> On 01/31/2015 09:54 PM, Francisco Jerez wrote: --- src/glsl/ast_to_hir.cpp | 12 1 file changed, 12 insertions(+) diff --git a/src/glsl/ast_to_hir.cpp b/sr

Re: [Mesa-dev] [PATCH 04/10] radeonsi: move the buffer descriptor to the end of the image descriptor

2015-02-01 Thread Ilia Mirkin
On Sun, Feb 1, 2015 at 12:37 PM, Marek Olšák wrote: > From: Marek Olšák > > This will allow supporting NULL textures. > --- > src/gallium/drivers/radeonsi/si_descriptors.c | 2 +- > src/gallium/drivers/radeonsi/si_pipe.h| 2 ++ > src/gallium/drivers/radeonsi/si_shader.c | 4 ++-- >

Re: [Mesa-dev] [PATCH] r600g: add polygon stipple support

2015-02-01 Thread Marek Olšák
One more thing. Polygon stippling should be disabled for lines and points (if a GS is bound, the GS out primitive matters). BTW, I have sent out a patch that adds the ability to specify a fixed texture slot for u_pstipple. Marek On Sat, Jan 31, 2015 at 1:42 AM, Marek Olšák wrote: > Hi Dave, > >

[Mesa-dev] [PATCH 05/10] radeonsi: fix a crash when binding a NULL sampler view list

2015-02-01 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_descriptors.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c b/src/gallium/drivers/radeonsi/si_descriptors.c index 15b4da1..372782f 100644 --- a/src/gallium/drivers/radeonsi/

[Mesa-dev] [PATCH 02/10] radeonsi: fix instanced arrays with non-zero start instance

2015-02-01 Thread Marek Olšák
From: Marek Olšák Fixes piglit ARB_base_instance/arb_base_instance-drawarrays. Cc: 10.3 10.4 --- src/gallium/drivers/radeonsi/si_shader.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader

[Mesa-dev] [PATCH 00/10] RadeonSI polygon stippling, NULL resources, bug fixes

2015-02-01 Thread Marek Olšák
Patch summary: These are bug fixes and one cleanup patch: r600g,radeonsi: don't append to streamout buffers that haven't been used yet radeonsi: fix instanced arrays with non-zero start instance radeonsi: don't use tgsi_parse_context to get processor type We previously supported NULL buffer

[Mesa-dev] [PATCH 06/10] radeonsi: add support for NULL texture sampler views that return (0, 0, 0, 1)

2015-02-01 Thread Marek Olšák
From: Marek Olšák This used to hang. --- src/gallium/drivers/radeonsi/si_descriptors.c | 30 +-- 1 file changed, 28 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c b/src/gallium/drivers/radeonsi/si_descriptors.c index 372782f..81

[Mesa-dev] [PATCH 10/10] radeonsi: implement polygon stippling

2015-02-01 Thread Marek Olšák
From: Marek Olšák The stipple texture is bound to slot 16, so there are 17 sampler states and 34 sampler views now (17 normal slots + 17 fmask slots). --- src/gallium/drivers/radeonsi/si_pipe.c | 2 + src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/

[Mesa-dev] [PATCH 08/10] radeonsi: allow 64 descriptors per array

2015-02-01 Thread Marek Olšák
From: Marek Olšák We need a slot for the stipple texture and the pixel shader already uses 32 textures (16 API slots + 16 FMASK slots). --- src/gallium/drivers/radeonsi/si_descriptors.c | 64 +-- src/gallium/drivers/radeonsi/si_state.h | 4 +- 2 files changed, 34 i

[Mesa-dev] [PATCH 04/10] radeonsi: move the buffer descriptor to the end of the image descriptor

2015-02-01 Thread Marek Olšák
From: Marek Olšák This will allow supporting NULL textures. --- src/gallium/drivers/radeonsi/si_descriptors.c | 2 +- src/gallium/drivers/radeonsi/si_pipe.h| 2 ++ src/gallium/drivers/radeonsi/si_shader.c | 4 ++-- src/gallium/drivers/radeonsi/si_state.c | 8 4 files

[Mesa-dev] [PATCH 07/10] radeonsi: add support for sampler views where resource = NULL

2015-02-01 Thread Marek Olšák
From: Marek Olšák The hardware obeys swizzles even if the resource is NULL. This will be used by set_polygon_stipple. --- src/gallium/drivers/radeonsi/si_descriptors.c | 13 + src/gallium/drivers/radeonsi/si_state.c | 15 +-- 2 files changed, 22 insertions(+), 6 del

[Mesa-dev] [PATCH 09/10] radeonsi: deduce rasterizer primitive type at the beginning of draw_vbo

2015-02-01 Thread Marek Olšák
From: Marek Olšák I will need this for polygon stippling. --- src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_state_draw.c | 29 +++- 2 files changed, 17 insertions(+), 13 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe

[Mesa-dev] [PATCH 01/10] r600g, radeonsi: don't append to streamout buffers that haven't been used yet

2015-02-01 Thread Marek Olšák
From: Marek Olšák The FILLED_SIZE counter is uninitialized at the beginning, so we can't use it. Instead, use offset = 0, which is what we always do when not appending. This unexpectedly fixes spec/ARB_texture_multisample/sample-position/*. Yes, the test does use transform feedback. Cc: 10.3 10

[Mesa-dev] [PATCH 03/10] radeonsi: don't use tgsi_parse_context to get processor type

2015-02-01 Thread Marek Olšák
From: Marek Olšák Also remove unused "tokens". --- src/gallium/drivers/radeonsi/si_shader.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index 4b1279c..52f1fa4 100644 --- a/src/gall

[Mesa-dev] [PATCH 2/9] gallium/util: add u_bit_scan64

2015-02-01 Thread Marek Olšák
From: Marek Olšák Same as u_bit_scan, but for uint64_t. --- src/gallium/auxiliary/util/u_math.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/gallium/auxiliary/util/u_math.h b/src/gallium/auxiliary/util/u_math.h index 19c7343..f5d3487 100644 --- a/src/gallium/auxiliary/util/u_m

[Mesa-dev] [PATCH 4/9] gallium: set PIPE_MAX_SAMPLERS to 18

2015-02-01 Thread Marek Olšák
From: Marek Olšák For drivers that use higher slots not to crash in tgsi_shader_info. --- src/gallium/include/pipe/p_state.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/include/pipe/p_state.h b/src/gallium/include/pipe/p_state.h index 43bc48b..e15860c 100644

[Mesa-dev] [PATCH 3/9] gallium/u_pstipple: add ability to specify a fixed texture unit

2015-02-01 Thread Marek Olšák
From: Marek Olšák E.g. r600g can use slot 17, which is outside of the API range. --- src/gallium/auxiliary/util/u_pstipple.c| 25 ++--- src/gallium/auxiliary/util/u_pstipple.h| 3 ++- src/gallium/drivers/softpipe/sp_state_shader.c | 2 +- 3 files changed, 21

[Mesa-dev] [PATCH 1/9] tgsi: add tgsi_get_processor_type helper from radeon

2015-02-01 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/tgsi/tgsi_parse.c | 12 src/gallium/auxiliary/tgsi/tgsi_parse.h | 2 ++ src/gallium/drivers/radeon/r600_pipe_common.c | 11 --- 3 files changed, 14 insertions(+), 11 deletions(-) diff --git a/src/gallium/auxiliary/tgs

[Mesa-dev] [PATCH 5/9] gallium: run gallium tests if GALLIUM_TESTS=1 is set

2015-02-01 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/target-helpers/inline_debug_helper.h | 4 src/gallium/auxiliary/util/u_tests.c | 9 - src/gallium/auxiliary/util/u_tests.h | 4 ++-- 3 files changed, 14 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 7/9] gallium/u_tests: test a NULL texture sampler view

2015-02-01 Thread Marek Olšák
From: Marek Olšák It shouldn't crash the GPU and it should return (0, 0, 0, 1). This is r300 behavior, so I assume it's also DX9 behavior. Radeonsi can support this easily. --- src/gallium/auxiliary/util/u_tests.c | 39 1 file changed, 39 insertions(+) diff

[Mesa-dev] [PATCH 8/9] gallium/u_tests: test a NULL constant buffer

2015-02-01 Thread Marek Olšák
From: Marek Olšák This expects (0,0,0,0), though it can be changed to something else or allow more than one set of values to be considered correct. This is currently the radeonsi behavior. --- src/gallium/auxiliary/util/u_tests.c | 57 1 file changed, 57 ins

[Mesa-dev] [PATCH 9/9] gallium/u_tests: test a NULL buffer sampler view

2015-02-01 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/util/u_simple_shaders.c | 12 ++--- src/gallium/auxiliary/util/u_tests.c | 37 +-- 2 files changed, 38 insertions(+), 11 deletions(-) diff --git a/src/gallium/auxiliary/util/u_simple_shaders.c b/src/gallium/auxili

[Mesa-dev] [PATCH 6/9] gallium/u_tests: restructure the only test, refactor out reusable code

2015-02-01 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/util/u_tests.c | 110 +-- 1 file changed, 79 insertions(+), 31 deletions(-) diff --git a/src/gallium/auxiliary/util/u_tests.c b/src/gallium/auxiliary/util/u_tests.c index 764c4c3..89ae840 100644 --- a/src/gallium/auxil

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

2015-02-01 Thread Glenn Kennard
On Sun, 01 Feb 2015 16:18:51 +0100, Ilia Mirkin wrote: 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 - src/mesa/main/tests/dispatch_sanity.cpp |

Re: [Mesa-dev] [RFC PATCH 6/6] i965: add EXT_polygon_offset_clamp support to gen4/gen5

2015-02-01 Thread Ilia Mirkin
On Sun, Feb 1, 2015 at 10:18 AM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > > Thoroughly untested, beyond compilation. Don't have the HW, nor am I > particularly familiar with the i965 driver in general. Should be easy enough > to test though for someone with the hw. > > src/mesa/dr

Re: [Mesa-dev] [PATCH v2 2/6] mesa: add support for GL_EXT_polygon_offset_clamp

2015-02-01 Thread Ilia Mirkin
On Sun, Feb 1, 2015 at 10:18 AM, Ilia Mirkin wrote: > diff --git a/src/mesa/main/polygon.c b/src/mesa/main/polygon.c > index e3b9073..2aa8f31 100644 > --- a/src/mesa/main/polygon.c > +++ b/src/mesa/main/polygon.c > @@ -235,25 +235,33 @@ _mesa_GetPolygonStipple( GLubyte *dest ) > _mesa_GetnPoly

[Mesa-dev] [PATCH v2 5/6] st/mesa: add EXT_polygon_offset_clamp support

2015-02-01 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- v1 -> v2: - rely on cap rather than turning it on for everything docs/relnotes/10.5.0.html | 2 +- src/mesa/state_tracker/st_atom_rasterizer.c | 1 + src/mesa/state_tracker/st_extensions.c | 1 + 3 files changed, 3 insertions(+), 1 deletion

[Mesa-dev] [RFC PATCH 6/6] i965: add EXT_polygon_offset_clamp support to gen4/gen5

2015-02-01 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- Thoroughly untested, beyond compilation. Don't have the HW, nor am I particularly familiar with the i965 driver in general. Should be easy enough to test though for someone with the hw. src/mesa/drivers/dri/i965/brw_clip.c | 1 + src/mesa/drivers/dri/i96

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

2015-02-01 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 4/6] gallium: add a cap to determine whether the driver supports offset_clamp

2015-02-01 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- src/gallium/drivers/freedreno/freedreno_screen.c | 1 + src/gallium/drivers/i915/i915_screen.c | 1 + src/gallium/drivers/ilo/ilo_screen.c | 2 ++ src/gallium/drivers/llvmpipe/lp_screen.c | 2 ++ src/gallium/drivers/nouveau/nv30/nv30_sc

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

2015-02-01 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 - src/mesa/main/tests/dispatch_sanity.cpp | 3 +++ 4 files changed, 24 insertions(+), 1 deletion(-) dif

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

2015-02-01 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. Instead we allow any driver to enable it, including in a compat context for any GL version. Signed-off-by: Ilia Mirkin --- v1 -> v2: - removed c

Re: [Mesa-dev] [PATCH 3/7] glsl: Forbid opaque variables as operands of the ternary operator.

2015-02-01 Thread Ian Romanick
On 02/01/2015 01:15 PM, Francisco Jerez wrote: > Ian Romanick writes: > >> On 01/31/2015 09:54 PM, Francisco Jerez wrote: >>> --- >>> src/glsl/ast_to_hir.cpp | 12 >>> 1 file changed, 12 insertions(+) >>> >>> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp >>> index 1

Re: [Mesa-dev] [PATCH 1/7] glsl: Return correct number of coordinate components for cubemap array images.

2015-02-01 Thread Francisco Jerez
Ian Romanick writes: > On 01/31/2015 09:54 PM, Francisco Jerez wrote: >> Cubemap array images are unlike cubemap array samplers in that they don't >> need >> an additional coordinate to index individual cubemaps in the array, instead >> they behave like a 2D array of 6n layers, with n the number

Re: [Mesa-dev] [PATCH 3/7] glsl: Forbid opaque variables as operands of the ternary operator.

2015-02-01 Thread Francisco Jerez
Ian Romanick writes: > On 01/31/2015 09:54 PM, Francisco Jerez wrote: >> --- >> src/glsl/ast_to_hir.cpp | 12 >> 1 file changed, 12 insertions(+) >> >> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp >> index 1ba29f7..783384e 100644 >> --- a/src/glsl/ast_to_hir.cpp >

Re: [Mesa-dev] [PATCH 3/7] glsl: Forbid opaque variables as operands of the ternary operator.

2015-02-01 Thread Ian Romanick
On 01/31/2015 09:54 PM, Francisco Jerez wrote: > --- > src/glsl/ast_to_hir.cpp | 12 > 1 file changed, 12 insertions(+) > > diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp > index 1ba29f7..783384e 100644 > --- a/src/glsl/ast_to_hir.cpp > +++ b/src/glsl/ast_to_hir.cpp >

Re: [Mesa-dev] [PATCH 2/7] glsl: Forbid calling the constructor of any opaque type.

2015-02-01 Thread Ian Romanick
This patch is Reviewed-by: Ian Romanick On 01/31/2015 09:54 PM, Francisco Jerez wrote: > The spec doesn't define any opaque type constructors. > --- > src/glsl/ast_function.cpp | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/glsl/ast_function.cpp b/src/glsl/a

Re: [Mesa-dev] [PATCH 1/7] glsl: Return correct number of coordinate components for cubemap array images.

2015-02-01 Thread Ian Romanick
On 01/31/2015 09:54 PM, Francisco Jerez wrote: > Cubemap array images are unlike cubemap array samplers in that they don't need > an additional coordinate to index individual cubemaps in the array, instead > they behave like a 2D array of 6n layers, with n the number of cubemaps in the > array. Ta

[Mesa-dev] [Bug 84019] Request commit access to piglit and mesa

2015-02-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=84019 Martin Peres changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---