On Wed, May 20, 2015 at 09:32:06AM +0300, Abdiel Janulgue wrote:
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_defines.h | 24
On 05/20/2015 09:47 AM, Pohjolainen, Topi wrote:
On Wed, May 20, 2015 at 09:32:06AM +0300, Abdiel Janulgue wrote:
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_defines.h | 24
Unlike normal software binding tables where the driver has to manually
generate and fill a binding table array which are then uploaded to the
hardware, the resource streamer instead presents the driver with an option
to fill out slots for individual binding table indices. The hardware
accumlates
This patch implements the binding table enable command which is also
used to allocate a binding table pool where where hardware-generated
binding table entries are flushed into. Each binding table offset in
the binding table pool is unique per each shader stage that are
enabled within a batch.
This is passed on the kernel to enable the resource streamer enable bit
on MI_BATCHBUFFER_START
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_context.h | 1 +
src/mesa/drivers/dri/i965/intel_batchbuffer.c | 8 +++-
2 files changed, 8
When hardware-generated binding tables are enabled, use the hw-generated
binding table format when uploading binding table state.
Normally, the CS will will just consume the binding table pointer commands
as pipelined state. When the RS is enabled however, the RS flushes whatever
edited surface
I've decided to split my gather constants work into even smaller chunks
to make it easier for reviewers which in turn makes it easier to merge.
First in this patch series enables hardware-generated binding tables
which is required for gather constants[*] to work.
Chances since previous posting
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_defines.h | 24
src/mesa/drivers/dri/i965/intel_reg.h | 3 +++
2 files changed, 27 insertions(+)
diff --git
On Wed, May 20, 2015 at 09:32:08AM +0300, Abdiel Janulgue wrote:
This patch implements the binding table enable command which is also
used to allocate a binding table pool where where hardware-generated
binding table entries are flushed into. Each binding table offset in
the binding table pool
https://bugs.freedesktop.org/show_bug.cgi?id=28130
Matt Turner matts...@gmail.com changed:
What|Removed |Added
QA Contact|
On Wed, May 20, 2015 at 09:32:09AM +0300, Abdiel Janulgue wrote:
Unlike normal software binding tables where the driver has to manually
generate and fill a binding table array which are then uploaded to the
hardware, the resource streamer instead presents the driver with an option
to fill out
On 05/20/2015 09:54 AM, Pohjolainen, Topi wrote:
On Wed, May 20, 2015 at 09:32:08AM +0300, Abdiel Janulgue wrote:
This patch implements the binding table enable command which is also
used to allocate a binding table pool where where hardware-generated
binding table entries are flushed into.
On Tue, 2015-05-19 at 15:14 -0700, Jordan Justen wrote:
On 2015-05-14 07:06:05, Iago Toral Quiroga wrote:
From: Kristian Høgsberg k...@bitplanet.net
This will be used to identify buffer variables inside shader storage
buffer objects, which are very similar to uniforms except for a few
On Wed, May 20, 2015 at 10:11:36AM +0300, Abdiel Janulgue wrote:
On 05/20/2015 09:54 AM, Pohjolainen, Topi wrote:
On Wed, May 20, 2015 at 09:32:08AM +0300, Abdiel Janulgue wrote:
This patch implements the binding table enable command which is also
used to allocate a binding table pool
https://bugs.freedesktop.org/show_bug.cgi?id=25898
Matt Turner matts...@gmail.com changed:
What|Removed |Added
QA Contact|
Just one issue below, otherwise looks great.
Thanks for doing this, now only the explicit derivs are still
fundamentally broken in softpipe sampling ;-).
Roland
On 05/20/2015 12:48 AM, Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This adds both ARB_texture_gather and the
We now have is_array() and without_array() that make the
code much clearer and remove the need for this.
For all remaining calls to this we already knew that
the type was an array so returning a null wasn't adding any value.
v2: use without_array() in _mesa_ast_array_index_to_hir() and don't use
I've got some nitpicks for the actual math performed for the offsets in
some of the cases, but otherwise looks good. If it's slightly slower
that's ok (initially I think the idea was to special purpose all these
things into separate functions but this got a bit out of control with
more and
On 05/20/2015 06:30 AM, Ilia Mirkin wrote:
+typedef const iterator const_iterator;
+
This at least is wrong. A const iterator allows *iter = val, a
const_iterator doesn't. A const_iterator allows ++, a const iterator
doesn't.
--
Petri Latvala
On 05/20/2015 10:21 AM, Pohjolainen, Topi wrote:
On Wed, May 20, 2015 at 10:11:36AM +0300, Abdiel Janulgue wrote:
On 05/20/2015 09:54 AM, Pohjolainen, Topi wrote:
On Wed, May 20, 2015 at 09:32:08AM +0300, Abdiel Janulgue wrote:
This patch implements the binding table enable command which
Fixes non-determinism in bin/point-sprite rendering, and the stars on
the intro screen to neverball.
Cc: 10.6 mesa-sta...@lists.freedesktop.org
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
src/gallium/drivers/freedreno/a3xx/fd3_program.c | 9 ++---
1 file changed, 6 insertions(+), 3
Ilia Mirkin imir...@alum.mit.edu writes:
Francisco, any opinion on this patch (as the resident C++ expert)? It
seems a little odd. I'd just as soon skip this, and just use using
std::tr1::unordered_set or using std::unordered_set as necessary in
the next patch. But perhaps this is a common
This patch implements the binding table enable command which is also
used to allocate a binding table pool where where hardware-generated
binding table entries are flushed into. Each binding table offset in
the binding table pool is unique per each shader stage that are
enabled within a batch.
Switch off hardware-generated binding tables and gather push
constants in the blorp. Blorp requires only a minimal set of
simple constants. There is no need for the extra complexity
to program a gather table entry into the pipeline.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
This is passed on the kernel to enable the resource streamer enable bit
on MI_BATCHBUFFER_START
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_context.h | 1 +
src/mesa/drivers/dri/i965/intel_batchbuffer.c | 8 +++-
2 files changed, 8
Unlike normal software binding tables where the driver has to manually
generate and fill a binding table array which are then uploaded to the
hardware, the resource streamer instead presents the driver with an option
to fill out slots for individual binding table indices. The hardware
accumulates
Jason Ekstrand ja...@jlekstrand.net writes:
I think *most* of that code *should* already be there. In theory,
it's all keyed off of the block size provided by formats.csv.
However, given some of the rendering errors we're currently seeing, it
looks like it may need a little patching here and
On Tue, May 19, 2015 at 9:35 PM, Jason Ekstrand ja...@jlekstrand.net wrote:
On Tue, May 19, 2015 at 7:08 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
Hi Nanley,
I was expecting to see some code dealing with the fact that the block
sizes are all different. Did I miss it, or is that still on the
To reduce the number of parameters.
---
src/wgl/wglinfo.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/wgl/wglinfo.c b/src/wgl/wglinfo.c
index 42bf3a7..8693450 100644
--- a/src/wgl/wglinfo.c
+++ b/src/wgl/wglinfo.c
@@ -79,8 +79,7 @@ WndProc(HWND hWnd,
On Wed, May 20, 2015 at 9:52 AM, Francisco Jerez curroje...@riseup.net wrote:
Ilia Mirkin imir...@alum.mit.edu writes:
Francisco, any opinion on this patch (as the resident C++ expert)? It
seems a little odd. I'd just as soon skip this, and just use using
std::tr1::unordered_set or using
v2: Simply HW binding table bit definitions and magic constants (Topi)
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_defines.h | 28
When hardware-generated binding tables are enabled, use the hw-generated
binding table format when uploading binding table state.
Normally, the CS will will just consume the binding table pointer commands
as pipelined state. When the RS is enabled however, the RS flushes whatever
edited surface
Based on a patch by Bryan Quigley gquigs+b...@gmail.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90438
---
src/wgl/wglinfo.c | 19 +++---
src/xdemos/glinfo_common.c | 5 -
src/xdemos/glinfo_common.h | 3 ++-
src/xdemos/glxinfo.c | 49
To reduce the number of parameters.
---
src/xdemos/glxinfo.c | 38 +++---
1 file changed, 19 insertions(+), 19 deletions(-)
diff --git a/src/xdemos/glxinfo.c b/src/xdemos/glxinfo.c
index b25186b..0f28647 100644
--- a/src/xdemos/glxinfo.c
+++ b/src/xdemos/glxinfo.c
Hi Ilia,
Thanks for finding these errors. I'll have fixes in v2.
- Nanley
On Tue, May 19, 2015 at 7:07 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Tue, May 19, 2015 at 8:48 PM, Nanley Chery nanleych...@gmail.com
wrote:
From: Nanley Chery nanley.g.ch...@intel.com
Signed-off-by:
On Wed, May 20, 2015 at 11:09 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
Now that we're using NIR by default and everything seems stable there, we
have about 3000 lines of extra code lying around that we don't need. We
might as well delete it. :-)
This series together with one of the
https://bugs.freedesktop.org/show_bug.cgi?id=90542
Bug ID: 90542
Summary: [softpipe] piglit glsl-1.50-gs-mismatch-prim-type
lines_adjacency regression
Product: Mesa
Version: 10.6
Hardware: x86-64 (AMD64)
On Thu, 2015-05-14 at 22:49 +0200, Alejandro Piñeiro wrote:
On 14/05/15 20:38, Ian Romanick wrote:
On 05/14/2015 04:30 AM, Timothy Arceri wrote:
On Wed, 2015-05-13 at 09:58 -0700, Ian Romanick wrote:
On 05/11/2015 03:37 AM, Alejandro Piñeiro wrote:
Since commit c0cd5b var-data.binding was
https://bugs.freedesktop.org/show_bug.cgi?id=25898
Brian Paul bri...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=79706
--- Comment #1 from Matt Turner matts...@gmail.com ---
I don't know what we were thinking by opening a non-versioned regression
tracker bug. It's like a meta-bugzilla. Can we stop this?
--
You are receiving this mail because:
You are the
Hi Jeremy,
On 11 February 2015 at 18:00, Jeremy Huddleston Sequoia
jerem...@apple.com wrote:
On Feb 11, 2015, at 05:02, Emil Velikov emil.l.veli...@gmail.com wrote:
On 11 February 2015 at 10:32, Jeremy Huddleston Sequoia
jerem...@apple.com wrote:
libOSMesa is a library, not a module
Fwiw
https://bugs.freedesktop.org/show_bug.cgi?id=58812
Matt Turner matts...@gmail.com changed:
What|Removed |Added
Status|REOPENED|RESOLVED
On May 20, 2015, at 13:59, Emil Velikov emil.l.veli...@gmail.com wrote:
Hi Jeremy,
On 11 February 2015 at 20:36, Jeremy Huddleston Sequoia
jerem...@freedesktop.org wrote:
Signed-off-by: Jeremy Huddleston Sequoia jerem...@apple.com
---
include/GL/glext.h | 1 +
1 file changed, 1
Hi Jeremy,
On 11 February 2015 at 20:36, Jeremy Huddleston Sequoia
jerem...@freedesktop.org wrote:
Signed-off-by: Jeremy Huddleston Sequoia jerem...@apple.com
---
include/GL/glext.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/GL/glext.h b/include/GL/glext.h
index
On 17 May 2015 at 14:23, Jon TURNEY jon.tur...@dronecode.org.uk wrote:
On 16/05/2015 05:12, Alan Coopersmith wrote:
Ah yes, I see Jons patch now in
https://bugs.freedesktop.org/show_bug.cgi?id=90147#c9
That looks like it should work for Solaris too, so whichever the
Mesa developers
On 05/20/2015 10:55 AM, Ilia Mirkin wrote:
On Wed, May 20, 2015 at 1:14 PM, Brian Paul bri...@vmware.com wrote:
Based on a patch by Bryan Quigley gquigs+b...@gmail.com
Bugzilla:
On Tue, May 19, 2015 at 9:35 PM, Jason Ekstrand ja...@jlekstrand.net
wrote:
On Tue, May 19, 2015 at 7:08 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
Hi Nanley,
I was expecting to see some code dealing with the fact that the block
sizes are all different. Did I miss it, or is that still
Now that we're using NIR by default and everything seems stable there, we
have about 3000 lines of extra code lying around that we don't need. We
might as well delete it. :-)
This series together with one of the patches I sent yesterday can be found
in the review/fs-nir-only branch on my
This is using multiple inheritance in C++. However, ir_visitor is really
just an interface with no data so it shouldn't be so bad.
---
src/mesa/drivers/dri/i965/brw_fs.h | 2 +-
src/mesa/drivers/dri/i965/brw_shader.h | 2 +-
src/mesa/drivers/dri/i965/brw_vec4.h | 2 +-
3 files changed, 3
The backend_shader class really is a representation of a shader. The fact
that it inherits from ir_visitor is somewhat immaterial.
---
src/mesa/drivers/dri/i965/brw_cfg.cpp| 10 +-
src/mesa/drivers/dri/i965/brw_cfg.h | 4 ++--
Now that everything is running through NIR, this is all dead.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 99 --
src/mesa/drivers/dri/i965/brw_fs.h | 50 +-
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2139 +-
3 files changed, 31 insertions(+), 2257
On Tue, May 19, 2015 at 7:22 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Tue, May 19, 2015 at 10:16 PM, Matt Turner matts...@gmail.com wrote:
On Tue, May 19, 2015 at 5:48 PM, Nanley Chery nanleych...@gmail.com wrote:
From: Nanley Chery nanley.g.ch...@intel.com
Signed-off-by: Nanley
On Wed, May 20, 2015 at 1:14 PM, Brian Paul bri...@vmware.com wrote:
Based on a patch by Bryan Quigley gquigs+b...@gmail.com
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=90438
---
src/wgl/wglinfo.c | 19 +++---
src/xdemos/glinfo_common.c | 5 -
On Wed, May 20, 2015 at 12:02 PM, Matt Turner matts...@gmail.com wrote:
On Wed, May 20, 2015 at 11:09 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
Now that we're using NIR by default and everything seems stable there, we
have about 3000 lines of extra code lying around that we don't need. We
I didn't review them very closely, but I like the idea!
Acked-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
---
src/mesa/drivers/dri/i965/brw_context.c | 7 ++-
src/mesa/drivers/dri/i965/brw_fs.cpp| 25 +++--
2 files changed, 5 insertions(+), 27 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index
Now that everything is running through NIR, this is all dead.
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 -
src/mesa/drivers/dri/i965/brw_fs.h | 26 -
src/mesa/drivers/dri/i965/brw_fs_fp.cpp| 742 -
3 files changed, 769 deletions(-)
delete mode
https://bugs.freedesktop.org/show_bug.cgi?id=90539
Bug ID: 90539
Summary: [softpipe] piglit varying-packing-simple dmat3 array
regression
Product: Mesa
Version: 10.6
Hardware: x86-64 (AMD64)
OS: Linux (All)
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Vinson Lee v...@freedesktop.org changed:
What|Removed |Added
Depends on||90539
--
You are
Whether or not to use NIR is now equivalent to brw-scalar_vs. We can
simplify the logic and make it far less confusing.
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 23 +++
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp
From: Ian Romanick ian.d.roman...@intel.com
The extension on which this depends will always be enabled in core
profile, and the extension bit is about to be removed.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: 10.6 mesa-sta...@lists.freedesktop.org
Cc: Fredrik Höglund
On Wed, May 20, 2015 at 6:03 PM, Dylan Baker baker.dyla...@gmail.com wrote:
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_gentable.py | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git
I breezed through the series and pointed out a couple of typos, but it
all looks fine to me. Nice clean ups.
Acked-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_genexec.py | 29 -
1 file changed, 12 insertions(+), 17 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_genexec.py b/src/mapi/glapi/gen/gl_genexec.py
index 4e76fe3..dbaafa7 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_SPARC_asm.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_SPARC_asm.py
b/src/mapi/glapi/gen/gl_SPARC_asm.py
index df7a039..fa6217e 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/glX_proto_size.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/glX_proto_size.py
b/src/mapi/glapi/gen/glX_proto_size.py
index 59f65d4..75fc26f 100644
---
This is roughly equivalent to the original getopt, except that it
removes the '-h' short option, which argparse reserves for
auto-generated help messages. It does retain the long option specified
by the getopt version, and changes the makefile to use that.
Signed-off-by: Dylan Baker
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_x86_asm.py | 39 ++-
1 file changed, 14 insertions(+), 25 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_x86_asm.py b/src/mapi/glapi/gen/gl_x86_asm.py
index f855dba..86d45f2 100644
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/glX_proto_recv.py | 55
1 file changed, 30 insertions(+), 25 deletions(-)
diff --git a/src/mapi/glapi/gen/glX_proto_recv.py
b/src/mapi/glapi/gen/glX_proto_recv.py
index
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_x86-64_asm.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_x86-64_asm.py
b/src/mapi/glapi/gen/gl_x86-64_asm.py
index 0188964..cf42371 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/glX_proto_recv.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/glX_proto_recv.py
b/src/mapi/glapi/gen/glX_proto_recv.py
index dbc26a8..da468dc 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_genexec.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_genexec.py b/src/mapi/glapi/gen/gl_genexec.py
index dbaafa7..0d58a8a 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/glX_proto_send.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/glX_proto_send.py
b/src/mapi/glapi/gen/glX_proto_send.py
index ea766af..2b33030 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/glX_proto_send.py | 59 +---
1 file changed, 28 insertions(+), 31 deletions(-)
diff --git a/src/mapi/glapi/gen/glX_proto_send.py
b/src/mapi/glapi/gen/glX_proto_send.py
index
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/glX_server_table.py | 38 ++
1 file changed, 15 insertions(+), 23 deletions(-)
diff --git a/src/mapi/glapi/gen/glX_server_table.py
b/src/mapi/glapi/gen/glX_server_table.py
index
Also removes the redundant -m argument, which could only be set to
'generic', or it would raise an exception. This option wasn't used in
the make file.
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_x86-64_asm.py | 40 +++--
1 file
Also drop -m switch, which only accepted a single value or raised an
error, and was unused in the makefile.
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_SPARC_asm.py | 38 ++
1 file changed, 14 insertions(+), 24 deletions(-)
glapi: gl_procs.py: Fix a few long hanging style things
s/long/low/ presumably
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Dylan Baker dylanx.c.ba...@intel.com
This makes the tools shut up about a bunch of problems, making them more
useful for catching actual problems.
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/remap_helper.py | 17 +
1 file changed, 13
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_enums.py | 30 ++
1 file changed, 14 insertions(+), 16 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_enums.py b/src/mapi/glapi/gen/gl_enums.py
index f45782d..927a68b 100644
---
From: Dylan Baker dylanx.c.ba...@intel.com
Make the code simpler, cleaner, and easier to work with.
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/remap_helper.py | 46 ++
1 file changed, 22 insertions(+), 24 deletions(-)
diff
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_apitemp.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_apitemp.py b/src/mapi/glapi/gen/gl_apitemp.py
index 7647428..5e985a2 100644
---
From: Dylan Baker dylanx.c.ba...@intel.com
Shuts up analysis tools to make them return actual problems.
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_procs.py | 23 ---
1 file changed, 16 insertions(+), 7 deletions(-)
diff --git
From: Dylan Baker dylanx.c.ba...@intel.com
Making the tools shut up about worthless errors so you can see real ones
is very useful
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_table.py | 42 ++
1 file changed, 26
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_gentable.py | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_gentable.py
b/src/mapi/glapi/gen/gl_gentable.py
index 06a5ebf..f7ffaf0 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_gentable.py | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_gentable.py
b/src/mapi/glapi/gen/gl_gentable.py
index f7ffaf0..32c1b68 100644
---
From: Dylan Baker dylanx.c.ba...@intel.com
This results in slightly less code, but code that is much more readable.
It has the advantage of putting everything together in one place, all of
the code is self documenting, help messages are auto-generated, choices
are automatically enforced, and the
This is a small slice of a larger series I'm working on with the end
goal of using mako, the khronos XML, and being python3 compatible with
our generators. That series is over 100 patches already, and not even
finished with the first step of using mako. I've sent this out to get
some patches
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_enums.py | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_enums.py b/src/mapi/glapi/gen/gl_enums.py
index 927a68b..955f27d 100644
--- a/src/mapi/glapi/gen/gl_enums.py
+++
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_x86_asm.py | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_x86_asm.py b/src/mapi/glapi/gen/gl_x86_asm.py
index 86d45f2..c0c7941 100644
---
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_apitemp.py | 39 +++
1 file changed, 19 insertions(+), 20 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_apitemp.py b/src/mapi/glapi/gen/gl_apitemp.py
index 4157032..7647428 100644
From: Dylan Baker dylanx.c.ba...@intel.com
Signed-off-by: Dylan Baker dylanx.c.ba...@intel.com
---
src/mapi/glapi/gen/gl_procs.py | 42 +++---
1 file changed, 19 insertions(+), 23 deletions(-)
diff --git a/src/mapi/glapi/gen/gl_procs.py
2015-05-20 16:35 GMT+08:00 Petri Latvala petri.latv...@intel.com:
On 05/20/2015 06:30 AM, Ilia Mirkin wrote:
+typedef const iterator const_iterator;
+
This at least is wrong. A const iterator allows *iter = val, a
const_iterator doesn't. A const_iterator allows ++, a const iterator
Starting with GEN8, there is documentation that the multisample state command
must be emitted before the 3DSTATE_WM_HZ_OP command any time the multisample
count changes. The 3DSTATE_WM_HZ_OP packet gets emitted as a result of a
intel_hix_exec(), which is called upon a fast clear and/or a resolve.
From: Dave Airlie airl...@redhat.com
this fixes a build problem found on RHEL s390.
not sure what configure options caused it, I couldn't get it on
x86 here.
Cc: 10.6 mesa-sta...@lists.freedesktop.org
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/auxiliary/util/u_math.h | 1 +
On 05/19/2015 07:29 PM, Ilia Mirkin wrote:
On Tue, May 19, 2015 at 3:54 PM, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Currently on the functions that are exclusive to core-profile are
implemented. The remainder continue to live in the XML.
From: Ian Romanick ian.d.roman...@intel.com
The next patch will add a test for compatibility profile dispatch, and
it seems to make more sense to share the lists.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Ilia Mirkin imir...@alum.mit.edu
Cc: 10.6 mesa-sta...@lists.freedesktop.org
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Ilia Mirkin imir...@alum.mit.edu
Cc: 10.6 mesa-sta...@lists.freedesktop.org
---
src/mesa/main/tests/dispatch_sanity.cpp | 493
1 file changed, 493 insertions(+)
From: Ian Romanick ian.d.roman...@intel.com
GL_ARB_vertex_attrib_64bit is exclusive to core profile, and none of the
other functions added by the extension are advertised in other profiles.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Dave Airlie airl...@redhat.com
Cc: Ilia Mirkin
99 matches
Mail list logo