Ilia Mirkin writes:
> Signed-off-by: Ilia Mirkin
> Cc:
Reviewed-by: Francisco Jerez
> ---
>
> This fixes a whole bunch of piglit tests, as one might imagine.
>
> src/mesa/drivers/dri/nouveau/nouveau_state.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/nouvea
Signed-off-by: Ilia Mirkin
Cc:
---
This fixes a whole bunch of piglit tests, as one might imagine.
src/mesa/drivers/dri/nouveau/nouveau_state.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/nouveau/nouveau_state.c
b/src/mesa/drivers/dri/nouveau/nouveau_state.c
index
https://bugs.freedesktop.org/show_bug.cgi?id=82472
Kenneth Graunke changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|mesa-dev@l
Using alternatives with multiple characters in a lookahead pattern
appears to cause Flex to write past the end of an internal state array.
We don't actually need to do that - we can just make our empty pragma
rule eat the NEWLINE and return it, rather than ensuring one exists but
leaving it as the
https://bugs.freedesktop.org/show_bug.cgi?id=82472
Kenneth Graunke changed:
What|Removed |Added
CC||huax...@intel.com
--- Comment #1 from
Am 16.08.2014 01:59, schrieb Tobias Klausmann:
> with this we determine if the driver wants to enable
> GL_ARB_conditional_render_inverted
>
> Signed-off-by: Tobias Klausmann
> ---
> src/gallium/docs/source/screen.rst | 2 ++
> src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
This is similar to ir_validate.cpp.
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_validate.c | 798
3 files changed, 801 insertions(+)
create mode 100644 src/glsl/nir/nir_v
This is similar to ir_print_visitor.cpp.
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h| 2 +
src/glsl/nir/nir_print.c | 916 ++
3 files changed, 919 insertions(+)
create mode 100644 src/glsl/nir/nir_
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources |1 +
src/glsl/nir/nir.h|4 +
src/glsl/nir/nir_lower_variables_scalar.c | 1241 +
3 files changed, 1246 insertions(+)
create mode 100644 src/glsl/nir/nir_lower_v
These include functions for adding and removing various bits of IR and
helpers for iterating over all the sources and destinations of an
instruction. This is similar to ir.cpp.
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources |1 +
src/glsl/nir/nir.c| 1713 +
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources| 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_lower_atomics.c | 127 +++
3 files changed, 130 insertions(+)
create mode 100644 src/glsl/nir/nir_lower_atomics.c
diff --git
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources | 3 +-
src/glsl/nir/nir.h | 5 ++
src/glsl/nir/nir_lower_samplers.cpp | 170
3 files changed, 177 insertions(+), 1 deletion(-)
create mode 100644 src/glsl/nir/nir_lower
After we lower variables, we want to delete them in order to free up
some memory.
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources| 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_remove_dead_variables.c | 138 +++
3
Signed-off-by: Connor Abbott
---
src/glsl/nir/nir.c| 4
src/glsl/nir/nir.h| 6 ++
src/glsl/nir/nir_lower_variables_scalar.c | 10 ++
3 files changed, 16 insertions(+), 4 deletions(-)
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir
After linking and inlining, this allows us to convert these registers
into SSA values and optimise more code.
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_opt_global_to_local.c | 103
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources | 6 +-
src/glsl/nir/nir_types.cpp | 155 +
src/glsl/nir/nir_types.h | 78 +++
3 files changed, 238 insertions(+), 1 deletion(-)
create mode 100644 src/glsl/nir/nir_ty
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources|3 +-
src/glsl/nir/glsl_to_nir.cpp | 1759 ++
src/glsl/nir/glsl_to_nir.h | 40 +
3 files changed, 1801 insertions(+), 1 deletion(-)
create mode 100644 src/glsl/nir/glsl_to_nir.cpp
crea
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_lower_system_values.c | 106 +
3 files changed, 109 insertions(+)
create mode 100644 src/glsl/nir/nir_lower_system_val
Signed-off-by: Connor Abbott
---
src/glsl/list.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/list.h b/src/glsl/list.h
index 3ee6cda..c402f48 100644
--- a/src/glsl/list.h
+++ b/src/glsl/list.h
@@ -644,6 +644,12 @@ inline void exec_node::insert_before(exec_list *before)
This flag was set to true for the atomic counter intrinsics, but it
never got plumbed through the linker, so by the time it got to the
backends it would always be set to the false. The current i965 backend
code doesn't use is_intrinsic, so this should not change any existing
code, but it's useful f
This includes all the instructions, ifs, loops, functions, etc. This is
similar to the information in ir.h.
Signed-off-by: Connor Abbott
---
src/glsl/Makefile.sources |2 +
src/glsl/nir/nir.h| 1150 +
src/glsl/nir/nir_intrinsics.c | 4
I know what you might be thinking right now. "Wait, *another* IR? Don't
we already have like 5 of those, not counting all the driver-specific
ones? Isn't this stuff complicated enough already?" Well, there are some
pretty good reasons to start afresh (again...). In the years we've been
using GLSL I
Signed-off-by: Connor Abbott
---
src/glsl/nir/README | 118
1 file changed, 118 insertions(+)
create mode 100644 src/glsl/nir/README
diff --git a/src/glsl/nir/README b/src/glsl/nir/README
new file mode 100644
index 000..2c81db9
--- /dev/n
As I pointed out on IRC, it looks like you forgot to send the patch
that modifies _mesa_BeginConditionalRender. (Your tests couldn't have
worked without that... if they did, there's something very wrong with
the tests...)
On Fri, Aug 15, 2014 at 7:59 PM, Tobias Klausmann
wrote:
> This patch serie
Signed-off-by: Tobias Klausmann
---
docs/GL3.txt| 2 +-
docs/relnotes/10.3.html | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index b38e42c..bbd1838 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -186,7 +186,7 @@ GL 4.5, GLSL 4.50
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
index a391a5f..85f8d1e 100644
--- a/src/gallium
Signed-off-by: Tobias Klausmann
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index c5bd7b3..553c01e 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensi
with this we determine if the driver wants to enable
GL_ARB_conditional_render_inverted
Signed-off-by: Tobias Klausmann
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/i915/i915_screen.c | 1 +
src
This is used to tell the drivers if to invert the rendering, used by
ARB_conditional_render_inverted
Signed-off-by: Tobias Klausmann
---
src/mesa/state_tracker/st_cb_condrender.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb
Signed-off-by: Tobias Klausmann
---
src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_query.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_query.c
index 50cef1e..71d48f2 100644
--- a/src/gallium
This patch series adds support for ARB_conditional_render_inverted to nvc0.
Tobias Klausmann (6):
gallium/st: use condition flag in *render_condition()
gallium/st: Add PIPE_CAP_CONDITIONAL_RENDER_INVERTED
mesa: add ARB_conditional_render_inverted extension bit
nvc0: Handle condition whethe
https://bugs.freedesktop.org/show_bug.cgi?id=82628
--- Comment #1 from Christoph Haag ---
It's now reverted but for the record so that it's not forgotten:
Whether it makes sense or not, the GPU does not hang if I change the count from
one to two:
diff --git a/src/gallium/auxiliary/hud/hud_conte
On Fri, Aug 15, 2014 at 3:26 PM, Anuj Phogat wrote:
> With comment on patch 3 addressed:
> Patches 1-7 are: Acked-by: Anuj Phogat
Thanks for looking over the patches!
Acked-by is used in the kernel by a maintainer to acknowledge the
changes to his particular subsystem in a patch that affects ma
On Fri, Aug 15, 2014 at 4:04 PM, Brian Paul wrote:
> The function should return GLboolean, not GLenum.
> If we detect invalid compressed pixel storage parameters, we should
> return GL_TRUE, not GL_FALSE so that the function is no-op'd.
> An update to the piglit s3tc-errors test will check this.
On Fri, Aug 15, 2014 at 4:04 PM, Brian Paul wrote:
> to pixelstore.c, add const qualifier to the 'packing' parameter.
> Add comments.
> ---
> src/mesa/main/pixelstore.c | 42 +
> src/mesa/main/pixelstore.h |8
> src/mesa/main/texgetimage.c
On Monday, July 21, 2014 11:00:50 PM Jordan Justen wrote:
> Today we allocate a miptree's for the hiz buffer. We needed this in
> the past because we would point the hardware at offsets of the hiz
> buffer. Since the hiz format is not documented, this is not a good
> idea.
>
> Since moving to supp
to pixelstore.c, add const qualifier to the 'packing' parameter.
Add comments.
---
src/mesa/main/pixelstore.c | 42 +
src/mesa/main/pixelstore.h |8
src/mesa/main/texgetimage.c |7 ---
src/mesa/main/teximage.c| 44 +++--
The function should return GLboolean, not GLenum.
If we detect invalid compressed pixel storage parameters, we should
return GL_TRUE, not GL_FALSE so that the function is no-op'd.
An update to the piglit s3tc-errors test will check this.
---
src/mesa/main/teximage.c |6 +++---
1 file changed,
Replace the gl_texture_image parameter with mesa_format since we only
used the image's format.
Add some comments.
---
src/mesa/main/texgetimage.c |2 +-
src/mesa/main/texstore.c| 24 +---
src/mesa/main/texstore.h|9 +
3 files changed, 23 insertions(+)
On Wed, Jun 25, 2014 at 10:59 AM, Neil Roberts wrote:
> Hi,
>
> I've been looking into this bug a bit more. I don't think disabling the
> blorp path on Sandybridge is the right thing to do. The only notable
> difference between the blorp code path and the meta code path seems to
> be that the blo
On Sun, Aug 10, 2014 at 4:18 PM, Matt Turner wrote:
> total instructions in shared programs: 4288650 -> 4282838 (-0.14%)
> instructions in affected programs: 595018 -> 589206 (-0.98%)
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 21 +++--
> 1 file changed, 15 insertions(+),
---
src/gallium/state_trackers/dri/Makefile.am | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/state_trackers/dri/Makefile.am
b/src/gallium/state_trackers/dri/Makefile.am
index bda75c3..bcbd081 100644
--- a/src/gallium/state_trackers/dri/Makefile.am
+++ b/src/gallium/state_tr
Hi,
This adds support for dma_buf fds to kms_swrast. This is especially interesting
for drm capable drivers like qxl or udl. The former recently gained prime
support. The second part adds a few defines that werent set anywhere else, but
are necessary for dri_kms_init_screen to be not empty.
r
Allows using prime fds as display target and from display target.
Test for PRIME capability after initializing kms_swrast screen.
Signed-off-by: Andreas Pokorny
---
src/gallium/state_trackers/dri/dri2.c | 8 ++
src/gallium/winsys/sw/kms-dri/kms_dri_sw_winsys.c | 91 +
Emil Velikov writes:
> Cc: Eric Anholt
> Suggested-by: Eric Anholt
> Signed-off-by: Emil Velikov
> ---
>
> Hi Eric,
>
> I fear I was bit too trigger happy with the series :\ Are you sure that
> you want to to the softpipe route here ? I would assume that adding some
> heuristics would be a lo
Cc: Eric Anholt
Suggested-by: Eric Anholt
Signed-off-by: Emil Velikov
---
Hi Eric,
I fear I was bit too trigger happy with the series :\ Are you sure that
you want to to the softpipe route here ? I would assume that adding some
heuristics would be a lot better.
-Emil
src/gallium/drivers/vc
On 14/08/14 10:59, Christian König wrote:
> From: Christian König
>
> Correctly handle that the source_surface is only optional.
>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=80561
Cc: mesa-sta...@lists.freedesktop.org
Would be nice to get this in stable :)
> Signed-off-by: Christia
The next patches are going to combine some of the mapi subdirectories'
Makefiles into a single Makefile, giving better build parallelism.
lib_LTLIBRARIES will be set to something like
lib_LTLIBRARIES = shared-glapi/libglapi.la es2api/libGLESv2.la
and the current code in install-lib-links.mk s
---
configure.ac | 1 -
src/mapi/Makefile.am | 44 ---
src/mapi/shared-glapi/Makefile.am | 34 --
src/mesa/Makefile.sources | 3 ---
4 files changed, 41 insertions(+), 41 deletions(-)
delet
---
configure.ac| 1 -
src/mapi/Makefile.am| 15 ---
src/mapi/shared-glapi/tests/Makefile.am | 19 ---
3 files changed, 12 insertions(+), 23 deletions(-)
delete mode 100644 src/mapi/shared-glapi/tests/Makefile.am
diff -
---
configure.ac| 1 -
src/mapi/Makefile.am| 40 +++-
src/mapi/es1api/Makefile.am | 64 -
3 files changed, 39 insertions(+), 66 deletions(-)
delete mode 100644 src/mapi/es1api/Makefile.am
diff --git a/co
---
configure.ac| 1 -
src/mapi/Makefile.am| 36 ++-
src/mapi/glapi/Makefile.am | 64 -
src/mapi/glapi/Makefile.sources | 22 +++---
4 files changed, 46 insertions(+), 77 deletions(-)
delete
---
configure.ac| 1 -
src/mapi/Makefile.am| 42 ++-
src/mapi/es2api/Makefile.am | 69 -
3 files changed, 41 insertions(+), 71 deletions(-)
delete mode 100644 src/mapi/es2api/Makefile.am
diff --git a/con
---
configure.ac | 1 -
src/mapi/Makefile.am | 15 +--
src/mapi/glapi/tests/Makefile.am | 20
3 files changed, 13 insertions(+), 23 deletions(-)
delete mode 100644 src/mapi/glapi/tests/Makefile.am
diff --git a/configure.ac b/confi
Emil Velikov writes:
> Cc: Eric Anholt
> Signed-off-by: Emil Velikov
> ---
> src/gallium/drivers/vc4/vc4_screen.c | 12
> 1 file changed, 12 insertions(+)
>
> diff --git a/src/gallium/drivers/vc4/vc4_screen.c
> b/src/gallium/drivers/vc4/vc4_screen.c
> index c044c8e..937ed63 10064
https://bugs.freedesktop.org/show_bug.cgi?id=50754
--- Comment #35 from Emil Velikov ---
(In reply to comment #34)
> (In reply to comment #33)
> > (In reply to comment #29)
> > > and documented (docs/autoconf.html) a reasonable approach towards
> > > multilib/cross-compile builds.
> >
> > --
On Thu, Aug 14, 2014 at 6:18 PM, Emil Velikov wrote:
> Provide the real vendor and and hardcode the device id as
> 0x as the devices currently using freedreno are non-pci.
> The device features UMA.
>
> Cc: Rob Clark
> Signed-off-by: Emil Velikov
> ---
> src/gallium/drivers/freedreno/fr
https://bugs.freedesktop.org/show_bug.cgi?id=82668
--- Comment #2 from Neil Roberts ---
(In reply to comment #0)
> On 32-bit builds you can't set int attributes using glUniformI*
Err, I meant glVertexAttribI*
--
You are receiving this mail because:
You are the assignee for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=82668
--- Comment #1 from Neil Roberts ---
Created attachment 104683
--> https://bugs.freedesktop.org/attachment.cgi?id=104683&action=edit
Piglit test case to demonstrate the problem
--
You are receiving this mail because:
You are the assignee for
https://bugs.freedesktop.org/show_bug.cgi?id=82668
Priority: medium
Bug ID: 82668
Assignee: mesa-dev@lists.freedesktop.org
Summary: Can't set int attributes to certain values on 32-bit
Severity: normal
Classification: Unclassified
Reviewed-by: Jason Ekstrand
On Aug 15, 2014 6:43 AM, "Brian Paul" wrote:
> Generate a GL error and return rather than crashing on a null
> ctx->Driver.CopyImageSubData pointer (gallium). This allows apitraces
> with glCopyImageSubData() calls to continue rather than crash.
>
> Plus, fix a comme
Hi Marek,
I've created a new patch which allows drivers to set the limit. I'll
leave the r600g patch up to someone who knows gallium better than me and
can actually test it.
Tim
On Fri, 2014-08-15 at 00:57 +0200, Marek Olšák wrote:
> Hi Timothy,
>
> The maximum stride is 2047 with r600g and 204
Generate a GL error and return rather than crashing on a null
ctx->Driver.CopyImageSubData pointer (gallium). This allows apitraces
with glCopyImageSubData() calls to continue rather than crash.
Plus, fix a comment typo.
---
src/mesa/main/copyimage.c |8 +++-
1 file changed, 7 insertions
V2: moved test for the VertexAttrib*Pointer() functions
to update_array(), and made constant available for drivers to set
Signed-off-by: Timothy Arceri
---
Although 4.4 is a while away GL_MAX_VERTEX_ATTRIB_STRIDE is used in
the ARB_direct_state_access spec so it seemed worth while adding this
On Fri, Aug 15, 2014 at 11:57 AM, Ville Syrjälä
wrote:
> On Fri, Aug 15, 2014 at 10:52:50AM +0200, Erik Faye-Lund wrote:
>> On Thu, Aug 7, 2014 at 10:31 AM, wrote:
>> > diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c
>> > index c61a748..f414ea3 100644
>> > --- a/src/mesa/main/
Pushed, thanks.
Marek
On Fri, Aug 15, 2014 at 10:30 AM, Glenn Kennard wrote:
> Requires Evergreen/Cayman
>
> Signed-off-by: Glenn Kennard
> ---
> Changes since v1:
> Move TGSI_OPCODE_DD[XY]_FINE in opcode tables to reflect
> changed enum values.
>
> All ARB_derivative_control piglit tests pas
On Fri, Aug 15, 2014 at 10:52:50AM +0200, Erik Faye-Lund wrote:
> On Thu, Aug 7, 2014 at 10:31 AM, wrote:
> > diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c
> > index c61a748..f414ea3 100644
> > --- a/src/mesa/main/texformat.c
> > +++ b/src/mesa/main/texformat.c
> > case G
https://bugs.freedesktop.org/show_bug.cgi?id=50754
--- Comment #34 from Emil Velikov ---
(In reply to comment #33)
> (In reply to comment #29)
> > and documented (docs/autoconf.html) a reasonable approach towards
> > multilib/cross-compile builds.
>
> --
> The following should be suffici
https://bugs.freedesktop.org/show_bug.cgi?id=79629
--- Comment #5 from meng ---
(In reply to comment #1)
> Fix you build bot.
Hi, with the patch would fix build issue:
diff --git a/test/dri3.c b/test/dri3.c
index b5eb4ba..45f3285 100644
--- a/test/dri3.c
+++ b/test/dri3.c
@@
On 15/08/14 04:41, Alex Deucher wrote:
> On Thu, Aug 14, 2014 at 6:18 PM, Emil Velikov
> wrote:
>> Signed-off-by: Emil Velikov
>
> Might make sense to set PIPE_CAP_UMA for the the IGP chips and APUs.
> I guess it depends how applications use that info.
>
At first we need to get this out, in o
On Thu, Aug 7, 2014 at 10:31 AM, wrote:
> diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c
> index c61a748..f414ea3 100644
> --- a/src/mesa/main/texformat.c
> +++ b/src/mesa/main/texformat.c
> case GL_ALPHA12:
> case GL_ALPHA16:
>RETURN_IF_SUPPORTED(MESA_FORMAT_A
Requires Evergreen/Cayman
Signed-off-by: Glenn Kennard
---
Changes since v1:
Move TGSI_OPCODE_DD[XY]_FINE in opcode tables to reflect
changed enum values.
All ARB_derivative_control piglit tests pass on radeon 6670
docs/GL3.txt | 2 +-
src/gallium/drivers/r600/r600
Yes, that is needed... fixed and landed.
On Fri, Aug 15, 2014 at 7:04 PM, Kenneth Graunke wrote:
> On Sunday, August 10, 2014 02:14:41 PM Chris Forbes wrote:
>> ---
>> src/mesa/drivers/dri/i965/brw_vec4.h | 2 +-
>> src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 53
>> +
https://bugs.freedesktop.org/show_bug.cgi?id=50754
--- Comment #33 from Eero Tamminen ---
(In reply to comment #29)
> and documented (docs/autoconf.html) a reasonable approach towards
> multilib/cross-compile builds.
--
The following should be sufficient to configure multilib Mesa
./con
On Sunday, August 10, 2014 02:14:41 PM Chris Forbes wrote:
> ---
> src/mesa/drivers/dri/i965/brw_vec4.h | 2 +-
> src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 53
> +++---
> 2 files changed, 49 insertions(+), 6 deletions(-)
>
> diff --git a/src/mesa/drivers/dri
75 matches
Mail list logo