https://bugs.freedesktop.org/show_bug.cgi?id=76188
--- Comment #5 from Pekka Paalanen ppaala...@gmail.com ---
Patches are now in Piglit and Mesa:
http://cgit.freedesktop.org/piglit/commit/?id=f51d966f0610721a0947619950f43ddd4d29d42d
On Sunday, August 03, 2014 12:36:31 PM Chris Forbes wrote:
We're about to be using this infrastructure to build descriptors in
src1 of non-send instructions, when preparing to do an indirect send.
Don't accidentally clobber the conditionalmod field of those
instructions with SFID bits, which
https://bugs.freedesktop.org/show_bug.cgi?id=79629
meng mengmeng.m...@intel.com changed:
What|Removed |Added
CC||mengmeng.m...@intel.com
Thanks, landed.
On Fri, Aug 15, 2014 at 6:44 PM, Kenneth Graunke kenn...@whitecape.org wrote:
On Sunday, August 03, 2014 12:36:31 PM Chris Forbes wrote:
We're about to be using this infrastructure to build descriptors in
src1 of non-send instructions, when preparing to do an indirect send.
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
https://bugs.freedesktop.org/show_bug.cgi?id=50754
--- Comment #33 from Eero Tamminen eero.t.tammi...@intel.com ---
(In reply to comment #29)
and documented (docs/autoconf.html) a reasonable approach towards
multilib/cross-compile builds.
--
pThe following should be sufficient to
Yes, that is needed... fixed and landed.
On Fri, Aug 15, 2014 at 7:04 PM, Kenneth Graunke kenn...@whitecape.org 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
Requires Evergreen/Cayman
Signed-off-by: Glenn Kennard glenn.kenn...@gmail.com
---
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 +-
On Thu, Aug 7, 2014 at 10:31 AM, ville.syrj...@linux.intel.com 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:
On 15/08/14 04:41, Alex Deucher wrote:
On Thu, Aug 14, 2014 at 6:18 PM, Emil Velikov emil.l.veli...@gmail.com
wrote:
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
Might make sense to set PIPE_CAP_UMA for the the IGP chips and APUs.
I guess it depends how applications use that info.
https://bugs.freedesktop.org/show_bug.cgi?id=79629
--- Comment #5 from meng mengmeng.m...@intel.com ---
(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
---
https://bugs.freedesktop.org/show_bug.cgi?id=50754
--- Comment #34 from Emil Velikov emil.l.veli...@gmail.com ---
(In reply to comment #33)
(In reply to comment #29)
and documented (docs/autoconf.html) a reasonable approach towards
multilib/cross-compile builds.
--
pThe following
On Fri, Aug 15, 2014 at 10:52:50AM +0200, Erik Faye-Lund wrote:
On Thu, Aug 7, 2014 at 10:31 AM, ville.syrj...@linux.intel.com 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
+++
Pushed, thanks.
Marek
On Fri, Aug 15, 2014 at 10:30 AM, Glenn Kennard glenn.kenn...@gmail.com wrote:
Requires Evergreen/Cayman
Signed-off-by: Glenn Kennard glenn.kenn...@gmail.com
---
Changes since v1:
Move TGSI_OPCODE_DD[XY]_FINE in opcode tables to reflect
changed enum values.
All
On Fri, Aug 15, 2014 at 11:57 AM, Ville Syrjälä
ville.syrj...@linux.intel.com wrote:
On Fri, Aug 15, 2014 at 10:52:50AM +0200, Erik Faye-Lund wrote:
On Thu, Aug 7, 2014 at 10:31 AM, ville.syrj...@linux.intel.com wrote:
diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c
index
V2: moved test for the VertexAttrib*Pointer() functions
to update_array(), and made constant available for drivers to set
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
Although 4.4 is a while away GL_MAX_VERTEX_ATTRIB_STRIDE is used in
the ARB_direct_state_access spec so it seemed
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
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 2048
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
On Aug 15, 2014 6:43 AM, Brian Paul bri...@vmware.com 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
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
https://bugs.freedesktop.org/show_bug.cgi?id=82668
--- Comment #1 from Neil Roberts n...@linux.intel.com ---
Created attachment 104683
-- https://bugs.freedesktop.org/attachment.cgi?id=104683action=edit
Piglit test case to demonstrate the problem
--
You are receiving this mail because:
You
https://bugs.freedesktop.org/show_bug.cgi?id=82668
--- Comment #2 from Neil Roberts n...@linux.intel.com ---
(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
On Thu, Aug 14, 2014 at 6:18 PM, Emil Velikov emil.l.veli...@gmail.com 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 robcl...@freedesktop.org
Signed-off-by: Emil Velikov
https://bugs.freedesktop.org/show_bug.cgi?id=50754
--- Comment #35 from Emil Velikov emil.l.veli...@gmail.com ---
(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.
Emil Velikov emil.l.veli...@gmail.com writes:
Cc: Eric Anholt e...@anholt.net
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/gallium/drivers/vc4/vc4_screen.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/gallium/drivers/vc4/vc4_screen.c
---
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
---
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
---
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
---
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 | 44 ---
src/mapi/shared-glapi/Makefile.am | 34 --
src/mesa/Makefile.sources | 3 ---
4 files changed, 41 insertions(+), 41 deletions(-)
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
On 14/08/14 10:59, Christian König wrote:
From: Christian König christian.koe...@amd.com
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 :)
Cc: Eric Anholt e...@anholt.net
Suggested-by: Eric Anholt e...@anholt.net
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
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
Emil Velikov emil.l.veli...@gmail.com writes:
Cc: Eric Anholt e...@anholt.net
Suggested-by: Eric Anholt e...@anholt.net
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
Hi Eric,
I fear I was bit too trigger happy with the series :\ Are you sure that
you want to to the softpipe
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 andreas.poko...@canonical.com
---
src/gallium/state_trackers/dri/dri2.c | 8 ++
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.
---
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
+++
On Sun, Aug 10, 2014 at 4:18 PM, Matt Turner matts...@gmail.com 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
On Wed, Jun 25, 2014 at 10:59 AM, Neil Roberts n...@linux.intel.com 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
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
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
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 support
On Fri, Aug 15, 2014 at 4:04 PM, Brian Paul bri...@vmware.com 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
On Fri, Aug 15, 2014 at 4:04 PM, Brian Paul bri...@vmware.com 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
On Fri, Aug 15, 2014 at 3:26 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
With comment on patch 3 addressed:
Patches 1-7 are: Acked-by: Anuj Phogat anuj.pho...@gmail.com
Thanks for looking over the patches!
Acked-by is used in the kernel by a maintainer to acknowledge the
changes to his
https://bugs.freedesktop.org/show_bug.cgi?id=82628
--- Comment #1 from Christoph Haag haa...@frickel.club ---
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
This is used to tell the drivers if to invert the rendering, used by
ARB_conditional_render_inverted
Signed-off-by: Tobias Klausmann tobias.johannes.klausm...@mni.thm.de
---
src/mesa/state_tracker/st_cb_condrender.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff
Signed-off-by: Tobias Klausmann tobias.johannes.klausm...@mni.thm.de
---
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
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
Signed-off-by: Tobias Klausmann tobias.johannes.klausm...@mni.thm.de
---
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
---
with this we determine if the driver wants to enable
GL_ARB_conditional_render_inverted
Signed-off-by: Tobias Klausmann tobias.johannes.klausm...@mni.thm.de
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
Signed-off-by: Tobias Klausmann tobias.johannes.klausm...@mni.thm.de
---
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
Signed-off-by: Tobias Klausmann tobias.johannes.klausm...@mni.thm.de
---
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
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
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
This includes all the instructions, ifs, loops, functions, etc. This is
similar to the information in ir.h.
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
src/glsl/Makefile.sources |2 +
src/glsl/nir/nir.h| 1150 +
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
After we lower variables, we want to delete them in order to free up
some memory.
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
src/glsl/Makefile.sources| 1 +
src/glsl/nir/nir.h | 2 +
src/glsl/nir/nir_remove_dead_variables.c | 138
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
After linking and inlining, this allows us to convert these registers
into SSA values and optimise more code.
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h | 2 +
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 connor.abb...@intel.com
---
src/glsl/Makefile.sources |1 +
src/glsl/nir/nir.c
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
This is similar to ir_print_visitor.cpp.
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
This is similar to ir_validate.cpp.
Signed-off-by: Connor Abbott connor.abb...@intel.com
---
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
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 tobias.johannes.klausm...@mni.thm.de
---
src/gallium/docs/source/screen.rst | 2 ++
https://bugs.freedesktop.org/show_bug.cgi?id=82472
Kenneth Graunke kenn...@whitecape.org changed:
What|Removed |Added
CC||huax...@intel.com
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
https://bugs.freedesktop.org/show_bug.cgi?id=82472
Kenneth Graunke kenn...@whitecape.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
Cc: mesa-sta...@lists.freedesktop.org
---
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
78 matches
Mail list logo