On 05/24/2015 06:58 AM, Ilia Mirkin wrote:
nv30_validate_clip depends on the rasterizer state. Also we should
upload all the new clip planes on change since next time the plane data
won't have changed, but the enables might.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/nv30/nv
https://bugs.freedesktop.org/show_bug.cgi?id=90612
Bug ID: 90612
Summary: Request commit access to mesa
Product: freedesktop.org
Version: unspecified
Hardware: Other
OS: All
Status: NEW
Severity: normal
https://bugs.freedesktop.org/show_bug.cgi?id=90612
--- Comment #1 from Samuel Pitoiset ---
Created attachment 116003
--> https://bugs.freedesktop.org/attachment.cgi?id=116003&action=edit
SSH public key
--
You are receiving this mail because:
You are on the CC list for the bug.
___
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_text.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_text.c
b/src/gallium/auxiliary/tgsi/tgsi_text.c
index a9734db..a6675c5 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_text.c
+++ b/src/gallium/auxil
From: Marek Olšák
I forgot to do this when pushing the interface changes.
---
src/gallium/auxiliary/tgsi/tgsi_dump.c | 6 +++---
src/gallium/auxiliary/tgsi/tgsi_info.c | 4 ++--
src/gallium/auxiliary/tgsi/tgsi_sanity.c | 12 ++--
src/gallium/auxiliary/tgsi/tgsi_scan.c |
From: Marek Olšák
This will be needed by radeonsi.
---
These 5 patches are picked from my tessellation branch.
src/gallium/auxiliary/util/u_math.h | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_math.h
b/src/gallium/auxiliary/util/u_math.h
From: Marek Olšák
---
src/gallium/drivers/trace/tr_context.c | 4 ++--
src/gallium/include/pipe/p_context.h | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/trace/tr_context.c
b/src/gallium/drivers/trace/tr_context.c
index a1aeebd..0013c96 100644
--
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
b/src/gallium/auxiliary/tgsi/tgsi_ureg.c
index 7a8bf54..037d31a 100644
--- a/src/gallium/auxiliary/tgsi/tgs
From: Marek Olšák
Cc: 10.6
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
index 25811ab..610af94 100644
--- a/s
From: Marek Olšák
Without it, texcoords are mapped to GENERIC[0..7], PointCoord is mapped to
GENERIC[8], and user-defined varyings start from GENERIC[9]. Since texcoords
can only be used between VS and PS, and PointCoord is PS-only, it's silly to
always start from GENERIC[9] in all other shaders
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 25 -
1 file changed, 16 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
index 610af94..e037ce4 100644
--
From: Marek Olšák
These can't occur between VS and GS, because GS is only supported
in the core profile.
---
src/gallium/drivers/radeonsi/si_shader.c | 17 +++--
1 file changed, 3 insertions(+), 14 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/dri
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 19 +--
1 file changed, 13 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
index e037ce4..208c852 100644
--- a/sr
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
index 5974e77..25811ab 100644
--- a/src/gallium/drivers/radeonsi/si_st
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Mesa 10.5.6 is now available.
This time around we have fixes from all over mesa - core mesa, driver
specific patches (i965, nouveau, freedreno), reentrancy improvements
in Clover (OpenCL), GLX and EGL loader patches, and Darwin build fixes.
Alex Deu
From: Marek Olšák
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 73 +-
1 file changed, 12 insertions(+), 61 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 5432520..d0880cd 100644
--- a/src/m
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c| 46 +--
src/gallium/auxiliary/tgsi/tgsi_ureg.h| 29 +
src/gallium/state_trackers/nine/nine_shader.c | 2 +-
src/mesa/state_tracker/st_glsl_to_tgsi.cpp| 2 +-
4 files change
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 84 +-
1 file changed, 33 insertions(+), 51 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
b/src/gallium/auxiliary/tgsi/tgsi_ureg.c
index a8f0867..ad13f64 100644
--- a/src/gallium/a
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 11 +--
src/gallium/auxiliary/tgsi/tgsi_scan.h | 2 ++
2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index d821072..cd7eb9e 1
Hi,
The reason I add this is that TGSI doesn't allow indirect indexing of inputs
and outputs. Consider this:
MOV OUT[ADDR[0]-1000], IMM[0]
There is no way to know where the output array starts here. It could be for
example OUT[6]=GENERIC4 or anything else. The problem is some outputs are
phys
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c| 52 ---
src/gallium/auxiliary/tgsi/tgsi_ureg.h| 38 ++--
src/gallium/state_trackers/nine/nine_ff.c | 6 ++--
src/gallium/state_trackers/nine/nine_shader.c | 13 ---
4 fi
From: Marek Olšák
---
src/gallium/drivers/radeon/radeon_llvm.h| 3 +--
src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 4 ++--
src/gallium/include/pipe/p_state.h | 1 +
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 11 +++
4 files changed, 7
From: Marek Olšák
There is nothing special about it and it's used for tessellation shaders
too.
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 31 +++---
src/gallium/auxiliary/tgsi/tgsi_ureg.h | 7 +++
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 7 +++
sr
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 37 --
src/gallium/auxiliary/tgsi/tgsi_ureg.h | 4 +++-
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +-
src/mesa/state_tracker/st_mesa_to_tgsi.c | 2 +-
4 files changed, 30 insertions(+), 1
From: Marek Olšák
My editor thinks "emit" is a keyword, which breaks code indexing.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 428 ++---
1 file changed, 214 insertions(+), 214 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_t
From: Marek Olšák
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 215 ++---
src/mesa/state_tracker/st_glsl_to_tgsi.h | 1 +
src/mesa/state_tracker/st_program.c| 9 ++
3 files changed, 202 insertions(+), 23 deletions(-)
diff --git a/src/mesa/state_tracker/
From: Marek Olšák
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 136 -
src/mesa/state_tracker/st_glsl_to_tgsi.h | 1 +
src/mesa/state_tracker/st_program.c| 7 ++
src/mesa/state_tracker/st_program.h| 1 +
4 files changed, 104 insertions(+), 4
https://bugs.freedesktop.org/show_bug.cgi?id=86701
nerdopol...@verizon.net changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Sun, May 24, 2015 at 5:47 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> This will be needed by radeonsi.
Then it should probably be included in a patch series that touches radeonsi...
Reviewed-by: Ilia Mirkin
but I'd prefer it if you could push this along with radeonsi changes,
not just t
Series is: Reviewed-by: Ilia Mirkin
On Sun, May 24, 2015 at 5:47 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/auxiliary/tgsi/tgsi_ureg.c | 18 ++
> 1 file changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
> b/
While I'm all for doing this, won't this break every driver if it no
longer has all the decl's? It'll take special logic to convert
DECL IN[0..5], GENERIC[0]
into
DECL IN[0], GENERIC[0]
DECL IN[1], GENERIC[1]
etc
Perhaps this should be guarded by a cap? Or an audit of all drivers
should be done
Drivers that only use tgsi_shader_info won't break.
Drivers that process tgsi_full_declaration manually and interpret
Range.First .. Range.Last correctly won't break either.
A driver can only break if it doesn't handle Range.Last correctly. If
that's the case, the driver should be fixed, because
Right, but you're changing what the driver receives, so we should make
sure they all handle it correctly. Or let the major driver authors
know what's going on so they can test it out and fix their driver
accordingly. Perhaps find a couple of piglit tests that exercise the
functionality. On that not
This enables ARB_cull_distance.
Signed-off-by: Tobias Klausmann
---
docs/GL3.txt | 2 +-
docs/relnotes/10.7.0.html | 4 +++-
src/gallium/drivers/nouveau/nvc0/nvc0_program.c| 6 +-
src/gallium/drivers/nouveau/nvc0/
This patch series adds the needed support for this extension to the various
parts of mesa to finally enable it for nvc0.
Dave Airlie (1):
glsl: lower cull_distance into cull_distance_mesa
Tobias Klausmann (10):
glapi: add GL_ARB_cull_distance
mesa/main: add support for GL_ARB_cull_distance
Signed-off-by: Tobias Klausmann
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/get.c | 26 ++
src/mesa/main/get_hash_params.py | 4
src/mesa/main/mtypes.h | 22 +-
src/mesa/main/shaderapi.c
Signed-off-by: Tobias Klausmann
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_gs.c | 2 +-
src/mesa/drivers/dri/i965/brw_vec4.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vs.c
Add another pipe cap so we can savely enable of disable this extension
Signed-off-by: Tobias Klausmann
---
src/gallium/auxiliary/cso_cache/cso_context.c| 3 +++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/i915/i915_screen.c | 1 +
src/gallium/drivers
Signed-off-by: Tobias Klausmann
---
src/glsl/glsl_types.cpp | 8
src/glsl/glsl_types.h | 10 --
src/glsl/linker.cpp | 2 +-
3 files changed, 13 insertions(+), 7 deletions(-)
diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cpp
index f675e90..4bc7324 100644
---
Signed-off-by: Tobias Klausmann
---
src/mesa/state_tracker/st_extensions.c | 4
src/mesa/state_tracker/st_program.c| 34 ++
2 files changed, 38 insertions(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
in
Signed-off-by: Tobias Klausmann
---
src/glsl/ast_to_hir.cpp | 14 +
src/glsl/builtin_variables.cpp | 13 +++-
src/glsl/glcpp/glcpp-parse.y| 3 +
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 3 +
src/glsl/link_varyings.cpp
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
index ecd115f..381a958 1006
Signed-off-by: Tobias Klausmann
---
src/mesa/program/prog_print.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/program/prog_print.c b/src/mesa/program/prog_print.c
index d588d07..e8855cd 100644
--- a/src/mesa/program/prog_print.c
+++ b/src/mesa/program/prog_print.c
@@ -147,6 +
From: Dave Airlie
Signed-off-by: Tobias Klausmann
---
src/glsl/Makefile.sources| 1 +
src/glsl/ir_optimization.h | 1 +
src/glsl/link_varyings.cpp | 15 +-
src/glsl/link_varyings.h | 3 +-
src/glsl/linker.cpp | 1 +
src/glsl/lower_cull_distance.
Signed-off-by: Tobias Klausmann
---
src/mapi/glapi/gen/gl_API.xml | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index 3090b9f..a792056 100644
--- a/src/mapi/glapi/gen/gl_API.xml
+++ b/src/mapi/glapi/gen/gl_A
Testing is really easy. Just run all piglit variable-indexing tests.
Also, drivers don't have to do anything for outputs yet, because those
are always moved to temps by lower_output_reads.
Marek
On Sun, May 24, 2015 at 6:22 PM, Ilia Mirkin wrote:
> Right, but you're changing what the driver rece
Would it be possible to modify lower_clip_distance to add support for
cull distances instead of duplicating it?
Marek
On Sun, May 24, 2015 at 7:58 PM, Tobias Klausmann
wrote:
> From: Dave Airlie
>
> Signed-off-by: Tobias Klausmann
> ---
> src/glsl/Makefile.sources| 1 +
> src/glsl/i
On Sun, May 24, 2015 at 1:58 PM, Tobias Klausmann
wrote:
> Signed-off-by: Tobias Klausmann
> ---
> src/mesa/main/extensions.c | 1 +
> src/mesa/main/get.c | 26 ++
> src/mesa/main/get_hash_params.py | 4
> src/mesa/main/mtypes.h
On Sun, May 24, 2015 at 7:58 PM, Tobias Klausmann
wrote:
> Signed-off-by: Tobias Klausmann
> ---
> src/mesa/state_tracker/st_extensions.c | 4
> src/mesa/state_tracker/st_program.c| 34
> ++
> 2 files changed, 38 insertions(+)
>
> diff --git a/src/mesa/
I'm having a bit of trouble tracing through this. What happens if I
have a shader that just does:
gl_ClipDistance[0] = 1;
gl_CullDistance[0] = 1;
what does the resulting TGSI look like? (Assuming that clip plane 0 is
enabled.) What about the generated nvc0 code (for the vertex shader)?
On Sun, M
https://bugs.freedesktop.org/show_bug.cgi?id=90600
Pali Rohár changed:
What|Removed |Added
CC||pali.ro...@gmail.com
--- Comment #7 from Pa
For when you resend, I tend to name things like this "nv50/ir" or
"nvc0/ir". In this case, I would have chosen "nv50/ir" since it
applies to both nvc0 and nv50. When it's a driver change, I just call
it "nv50: " or "nvc0: " (even if it might include small codegen
changes).
On Sun, May 24, 2015 at
https://bugs.freedesktop.org/show_bug.cgi?id=90600
--- Comment #8 from Ilia Mirkin ---
I tried actually tracing some of the code in the backtraces, but my line
numbers don't match up. The big one that I don't find in argparse.py at all but
is there in the trace is:
default = self._get_value(
https://bugs.freedesktop.org/show_bug.cgi?id=90621
Bug ID: 90621
Summary: Mesa fail to build from source
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Priority:
https://bugs.freedesktop.org/show_bug.cgi?id=90621
Lorenzo Bona changed:
What|Removed |Added
Summary|Mesa fail to build from |Mesa fail to build from git
On Sun, 2015-05-24 at 19:58 +0200, Tobias Klausmann wrote:
> Signed-off-by: Tobias Klausmann
> ---
> src/glsl/ast_to_hir.cpp | 14 +
> src/glsl/builtin_variables.cpp | 13 +++-
> src/glsl/glcpp/glcpp-parse.y| 3 +
> src/glsl/glsl_parser_extras.cpp | 1 +
> s
hi,
replay inline.
On 25.05.2015 00:34, Timothy Arceri wrote:
On Sun, 2015-05-24 at 19:58 +0200, Tobias Klausmann wrote:
Signed-off-by: Tobias Klausmann
---
src/glsl/ast_to_hir.cpp | 14 +
src/glsl/builtin_variables.cpp | 13 +++-
src/glsl/glcpp/glcpp-parse.y
On Mon, 2015-05-25 at 00:46 +0200, Tobias Klausmann wrote:
> hi,
> replay inline.
>
> On 25.05.2015 00:34, Timothy Arceri wrote:
> > On Sun, 2015-05-24 at 19:58 +0200, Tobias Klausmann wrote:
> >> Signed-off-by: Tobias Klausmann
> >> ---
> >> src/glsl/ast_to_hir.cpp | 14 +
> >>
On 25.05.2015 01:19, Timothy Arceri wrote:
On Mon, 2015-05-25 at 00:46 +0200, Tobias Klausmann wrote:
hi,
replay inline.
On 25.05.2015 00:34, Timothy Arceri wrote:
On Sun, 2015-05-24 at 19:58 +0200, Tobias Klausmann wrote:
Signed-off-by: Tobias Klausmann
---
src/glsl/ast_to_hir.cpp
On 24.05.2015 18:52, Marek Olšák wrote:
> From: Marek Olšák
>
> These can't occur between VS and GS, because GS is only supported
> in the core profile.
Could these be used by other state trackers, though? If not, this patch
and either way, the rest of this series is
Reviewed-by: Michel Dänzer
https://bugs.freedesktop.org/show_bug.cgi?id=90621
Vinson Lee changed:
What|Removed |Added
URL||https://llvm.org/bugs/show_
From mesa-10.5.6 compilation.
Is this an oversight ?
lp_query.c: In function..llvmpipe_check_render_cond..:
lp_query.c:317:23: warning: logical not is only applied
to the left hand side of comparison [-Wlogical-not-parentheses]
return (!result == lp->render_cond_cond);
62 matches
Mail list logo