On Thu, 24 Jul 2014 01:43:35 +0100
Emil Velikov emil.l.veli...@gmail.com wrote:
From: Giovanni Campagna gcampa...@src.gnome.org
Turn GBM into a swrast loader (providing putimage/getimage backed
by a dumb KMS buffer). This allows to run KMS+DRM GL applications
(such as weston or
https://bugs.freedesktop.org/show_bug.cgi?id=62202
Tapani Pälli lem...@gmail.com changed:
What|Removed |Added
Version|9.1 |git
--- Comment #1 from
On Thu, Jul 03, 2014 at 11:23:13AM -0700, Ben Widawsky wrote:
Viewport extents are a 3rd rectangle that defines which pixels get
discarded as part of the rasterization process. This can potentially
improve performance by reducing cache usage, and freeing up PS cycles.
This will get hit if
https://bugs.freedesktop.org/show_bug.cgi?id=62202
--- Comment #2 from Tapani Pälli lem...@gmail.com ---
warning: Source file is more recent than executable.
Sorry about this, it came from unrelated changes .. however I just tested that
the function names match fine with a clean tree. Had also
On Wed, 2014-07-23 at 10:33 -0400, Tom Stellard wrote:
On Fri, Jul 18, 2014 at 01:09:03PM +0200, Bruno Jimenez wrote:
On Thu, 2014-07-17 at 22:56 -0400, Tom Stellard wrote:
On Wed, Jul 16, 2014 at 11:12:42PM +0200, Bruno Jiménez wrote:
Hi,
This series finally adds support for
On Wed, 2014-07-23 at 10:46 -0400, Tom Stellard wrote:
On Sat, Jul 19, 2014 at 07:35:49PM +0200, Bruno Jiménez wrote:
---
src/gallium/drivers/r600/compute_memory_pool.c | 43
++
src/gallium/drivers/r600/compute_memory_pool.h | 1 +
2 files changed, 25
v2: Remove unnecesary variables
---
src/gallium/drivers/r600/compute_memory_pool.c | 31 +-
src/gallium/drivers/r600/compute_memory_pool.h | 1 +
2 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/r600/compute_memory_pool.c
On Wed, Jun 18, 2014 at 03:11:52PM -0700, Anuj Phogat wrote:
On Wed, Jul 24, 2013 at 5:06 PM, Chad Versace
chad.vers...@linux.intel.com wrote:
This patch is
Reviewed-by: Chad Versace chad.vers...@linux.intel.com
with the understanding that it depends on previous patches.
I'll begin
On Wed, 2014-07-23 at 15:23 -0700, Ian Romanick wrote:
On 07/21/2014 08:03 PM, Timothy Arceri wrote:
On Mon, 2014-07-21 at 14:04 -0700, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
There are a bunch of places, especially in the UBO code, where we check
whether
Yes, the commit message and your git username needs to be fixed.
Marek
On Thu, Jul 24, 2014 at 12:49 AM, Aditya Avinash
adityaavina...@gmail.com wrote:
Hi,
I am sorry. This is my first patch. I'll correct it for the next time. Do
you want me to resend it?
On Wednesday, July 23, 2014, Ian
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Thu, Jul 24, 2014 at 2:37 AM, Tom Stellard thomas.stell...@amd.com wrote:
---
src/gallium/docs/source/screen.rst| 2 ++
src/gallium/drivers/radeon/r600_pipe_common.c | 7 +++
src/gallium/include/pipe/p_defines.h
Isn't this redundant with get_shader_param(PIPE_SHADER_COMPUTE,
PIPE_SHADER_CAP_MAX_CONSTS) * 16?
Marek
On Thu, Jul 24, 2014 at 3:05 AM, Tom Stellard thomas.stell...@amd.com wrote:
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/include/pipe/p_defines.h | 3 ++-
2 files
From: Alon Levy al...@redhat.com
Fixed build error at glsl_parser.yy for Visual Studio 2013
Signed-off-by: Alon Levy al...@redhat.com
---
Thanks for the review. I verified and VS2013 (vs12) doesn't include strcasecmp,
I corrected the file you mentioned, here is a better patch. (this supercedes
On 07/22/2014 12:09 PM, Neil Roberts wrote:
+ * The spec for GL_ARB_texture_compression_bptc doesn't mention whether it
+ * should be included in GL_COMPRESSED_TEXTURE_FORMATS.
Ian Romanick i...@freedesktop.org writes:
What do NVIDIA and AMD do? We should mimic that.
I don't have an NVidia
ping
On Sat, Jul 19, 2014 at 11:45 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
Brian reviewed the first bits of this, but it turns out I had to change
src_register as well. Also I realized that instead of doing the UADD I could
just
On 24/07/14 07:23, Pekka Paalanen wrote:
On Thu, 24 Jul 2014 01:43:35 +0100
Emil Velikov emil.l.veli...@gmail.com wrote:
From: Giovanni Campagna gcampa...@src.gnome.org
Turn GBM into a swrast loader (providing putimage/getimage backed
by a dumb KMS buffer). This allows to run KMS+DRM GL
On 06/27/2014 10:59 AM, Knut Andre Tidemann wrote:
v2: fix style and wrong major version comparison.
v3: fix version check in context creation.
---
src/gallium/state_trackers/egl/common/egl_g3d.c | 1 +
src/gallium/state_trackers/egl/common/egl_g3d_api.c | 12
2 files
On 23/07/14 22:16, Ian Romanick wrote:
On 07/22/2014 02:07 PM, Alon Levy wrote:
Signed-off-by: Alon Levy al...@redhat.com
---
src/glsl/glsl_parser.yy | 4
1 file changed, 4 insertions(+)
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
index faaf438..25370cd 100644
---
On Thu, 24 Jul 2014 13:34:42 +0100
Emil Velikov emil.l.veli...@gmail.com wrote:
On 24/07/14 07:23, Pekka Paalanen wrote:
On Thu, 24 Jul 2014 01:43:35 +0100
Emil Velikov emil.l.veli...@gmail.com wrote:
From: Giovanni Campagna gcampa...@src.gnome.org
Turn GBM into a swrast loader
On Thu, Jul 24, 2014 at 01:09:49PM +0200, Marek Olšák wrote:
Isn't this redundant with get_shader_param(PIPE_SHADER_COMPUTE,
PIPE_SHADER_CAP_MAX_CONSTS) * 16?
This is what clover was using, but I was confused about what the value
was supposed to represent. Now, I think I understand (number
On Wed, Jul 23, 2014 at 5:57 PM, Ian Romanick i...@freedesktop.org wrote:
On 07/22/2014 12:09 PM, Neil Roberts wrote:
diff --git a/src/mesa/main/texcompress.c b/src/mesa/main/texcompress.c
index 9dbfe9f..b708b49 100644
--- a/src/mesa/main/texcompress.c
+++ b/src/mesa/main/texcompress.c
@@
On Thu, 2014-07-24 at 06:35 -0700, Tom Stellard wrote:
On Thu, Jul 24, 2014 at 01:09:49PM +0200, Marek Olšák wrote:
Isn't this redundant with get_shader_param(PIPE_SHADER_COMPUTE,
PIPE_SHADER_CAP_MAX_CONSTS) * 16?
This is what clover was using, but I was confused about what the value
A couple of small fixes to teach platform checks about Cygwin's characteristics
Ideally, these would be autoconf checks, and we wouldn't have to do anything
here, but it seems that ship has long sailed.
Yaakov Selkowitz (2):
xmlconfig: Use program_invocation_short_name when building for
From: Yaakov Selkowitz yselkow...@users.sourceforge.net
mesa/mesa/src/mesa/drivers/dri/common/xmlconfig.c:104:10: warning: #warning
Per application configuration won't work with your OS version. [-Wcpp]
#warning Per application configuration won't work with your OS
version.
From: Yaakov Selkowitz yselkow...@users.sourceforge.net
mesa/mesa/src/gallium/auxiliary/os/os_process.c:40:2: warning: #warning
unexpected platform in os_process.c [-Wcpp]
#warning unexpected platform in os_process.c
mesa/mesa/src/gallium/auxiliary/os/os_process.c:77:2: warning: #warning
Ian Romanick i...@freedesktop.org writes:
Is that NVIDIA's off-line compression tool, or is that the compressor
in the driver?
I was talking about the offline compressor. I don't know what NVidia's
online compressor is like. Yes, perhaps if we can get a quick compressor
with reasonable results
On 07/24/2014 02:12 PM, Alon Levy wrote:
From: Alon Levy al...@redhat.com
Forgot to add Ian as cc.
Fixed build error at glsl_parser.yy for Visual Studio 2013
Signed-off-by: Alon Levy al...@redhat.com
---
Thanks for the review. I verified and VS2013 (vs12) doesn't include
strcasecmp,
On Thu, Jul 24, 2014 at 10:37 AM, Neil Roberts n...@linux.intel.com wrote:
Ian Romanick i...@freedesktop.org writes:
Is that NVIDIA's off-line compression tool, or is that the compressor
in the driver?
I was talking about the offline compressor. I don't know what NVidia's
online compressor
From: José Fonseca jfons...@vmware.com
While running https://github.com/nvMcJohn/apitest with apitrace I noticed that
Mesa was producing bogus results:
wglChoosePixelFormatARB(hdc, piAttribIList = {...}, pfAttribFList = 0,
nMaxFormats = 1, piFormats = {19, 65576, 37, 198656, 131075, 0,
OpenGL 3.1 requires 4096 vec4s (65536 bytes) per constant buffer and
the hardware supports 16 constant buffers. I'm not sure what the
constant registers are, but they cannot have anything to do with the
constant buffer limit, because otherwise we wouldn't be able to meet
the requirement for
Matt Turner matts...@gmail.com writes:
I was thinking one of the cool things we might be able to do a ETC2 -
BC7 transcode on platforms without ETC2 hardware decompression. We
won't be able to do that without a good compressor.
That sounds like a fun project. It would be a shame to have to
Sorry, GL 3.1 actually only requires 1024 vec4s. The UBO extension
spec contains a mistake.
Marek
On Thu, Jul 24, 2014 at 4:55 PM, Marek Olšák mar...@gmail.com wrote:
OpenGL 3.1 requires 4096 vec4s (65536 bytes) per constant buffer and
the hardware supports 16 constant buffers. I'm not sure
On 07/24/2014 07:43 AM, Ilia Mirkin wrote:
On Thu, Jul 24, 2014 at 10:37 AM, Neil Roberts n...@linux.intel.com wrote:
Ian Romanick i...@freedesktop.org writes:
Is that NVIDIA's off-line compression tool, or is that the compressor
in the driver?
I was talking about the offline compressor. I
On 07/23/2014 07:41 PM, Matt Turner wrote:
On Wed, Jul 23, 2014 at 3:16 PM, Ian Romanick i...@freedesktop.org wrote:
On 07/22/2014 12:09 PM, Neil Roberts wrote:
Here's a first attempt at a patch series to implement BPTC texture
compression in the i965 driver on Gen=7.
Getting it to work on
Ilia Mirkin imir...@alum.mit.edu writes:
Just a thought -- if online compression is highly unexpected, perhaps
it'd be reasonably to make a *horrid* compressor that doesn't rely on
any external libraries? I don't know how complex the BPTC format is,
but I suspect it may be possible to do a
On 24 July 2014 16:55, Marek Olšák mar...@gmail.com wrote:
the hardware supports 16 constant buffers. I'm not sure what the
constant registers are, but they cannot have anything to do with the
Probably the old CFILE constants, of which there actually only were
256, and which IIRC were removed
On Thu, 2014-07-24 at 17:07 +0200, Marek Olšák wrote:
Sorry, GL 3.1 actually only requires 1024 vec4s. The UBO extension
spec contains a mistake.
Marek
On Thu, Jul 24, 2014 at 4:55 PM, Marek Olšák mar...@gmail.com wrote:
OpenGL 3.1 requires 4096 vec4s (65536 bytes) per constant buffer
On Thu, Jul 24, 2014 at 5:44 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
On 23/07/14 22:16, Ian Romanick wrote:
On 07/22/2014 02:07 PM, Alon Levy wrote:
Signed-off-by: Alon Levy al...@redhat.com
---
src/glsl/glsl_parser.yy | 4
1 file changed, 4 insertions(+)
diff --git
On 24/07/14 14:30, Pekka Paalanen wrote:
On Thu, 24 Jul 2014 13:34:42 +0100
Emil Velikov emil.l.veli...@gmail.com wrote:
On 24/07/14 07:23, Pekka Paalanen wrote:
On Thu, 24 Jul 2014 01:43:35 +0100
Emil Velikov emil.l.veli...@gmail.com wrote:
From: Giovanni Campagna gcampa...@src.gnome.org
On 24/07/14 17:07, Matt Turner wrote:
On Thu, Jul 24, 2014 at 5:44 AM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 23/07/14 22:16, Ian Romanick wrote:
On 07/22/2014 02:07 PM, Alon Levy wrote:
Signed-off-by: Alon Levy al...@redhat.com
---
src/glsl/glsl_parser.yy | 4
1 file
On Thu, Jul 24, 2014 at 5:37 AM, Knut Andre Tidemann
knut.tidem...@gmail.com wrote:
On 06/27/2014 10:59 AM, Knut Andre Tidemann wrote:
v2: fix style and wrong major version comparison.
v3: fix version check in context creation.
---
src/gallium/state_trackers/egl/common/egl_g3d.c | 1 +
On Fri, Jun 27, 2014 at 1:59 AM, Knut Andre Tidemann
knut.tidem...@gmail.com wrote:
v2: fix style and wrong major version comparison.
v3: fix version check in context creation.
---
src/gallium/state_trackers/egl/common/egl_g3d.c | 1 +
src/gallium/state_trackers/egl/common/egl_g3d_api.c
On Thu, Jul 24, 2014 at 10:29:11AM +0300, Pohjolainen, Topi wrote:
On Thu, Jul 03, 2014 at 11:23:13AM -0700, Ben Widawsky wrote:
Viewport extents are a 3rd rectangle that defines which pixels get
discarded as part of the rasterization process. This can potentially
improve performance by
On Thu, Jul 24, 2014 at 8:55 AM, Neil Roberts n...@linux.intel.com wrote:
Ilia Mirkin imir...@alum.mit.edu writes:
Just a thought -- if online compression is highly unexpected, perhaps
it'd be reasonably to make a *horrid* compressor that doesn't rely on
any external libraries? I don't know
From: Marek Olšák marek.ol...@amd.com
it also matches GL 4.2
further discussion:
http://lists.freedesktop.org/archives/mesa-dev/2013-August/042680.html
---
src/gallium/drivers/r600/evergreen_state.c | 2 --
src/gallium/drivers/r600/r600_asm.c| 1 -
src/gallium/drivers/r600/r600_shader.c
On Thu, Jul 24, 2014 at 10:22:25AM -0700, Ben Widawsky wrote:
On Thu, Jul 24, 2014 at 10:29:11AM +0300, Pohjolainen, Topi wrote:
On Thu, Jul 03, 2014 at 11:23:13AM -0700, Ben Widawsky wrote:
Viewport extents are a 3rd rectangle that defines which pixels get
discarded as part of the
On Thu, Jul 24, 2014 at 12:05 PM, Jan Vesely jan.ves...@rutgers.edu wrote:
On Thu, 2014-07-24 at 17:07 +0200, Marek Olšák wrote:
Sorry, GL 3.1 actually only requires 1024 vec4s. The UBO extension
spec contains a mistake.
Marek
On Thu, Jul 24, 2014 at 4:55 PM, Marek Olšák mar...@gmail.com
On 07/24/2014 07:50 PM, Emil Velikov wrote:
On 24/07/14 17:07, Matt Turner wrote:
On Thu, Jul 24, 2014 at 5:44 AM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 23/07/14 22:16, Ian Romanick wrote:
On 07/22/2014 02:07 PM, Alon Levy wrote:
Signed-off-by: Alon Levy al...@redhat.com
---
From: Marek Olšák marek.ol...@amd.com
This new name isn't so confusing.
I also changed the gallivm limit, because it looked wrong.
---
src/gallium/auxiliary/gallivm/lp_bld_limits.h | 4 ++--
src/gallium/auxiliary/tgsi/tgsi_exec.h| 8
src/gallium/auxiliary/util/u_caps.c
---
src/mesa/main/bufferobj.c | 31 +++
1 file changed, 31 insertions(+)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 7b1bba0..1dfcda3 100644
--- a/src/mesa/main/bufferobj.c
+++ b/src/mesa/main/bufferobj.c
@@ -832,6 +832,9 @@
The commit message should be something like
mesa: Add missing atomic buffer bindings and unbindings.
On Thu, Jul 24, 2014 at 12:18 PM, Aditya Atluri
adityaavina...@gmail.com wrote:
---
src/mesa/main/bufferobj.c | 31 +++
1 file changed, 31 insertions(+)
diff
On 22/07/14 19:43, Dylan Baker wrote:
GBM_DRIVERS_PATH is not documented, and only used to set the location of
gbm drivers, while LIBGL_DRIVERS_PATH is used for everything else, and
is documented.
Generally this split leads to confusion as to why gbm doesn't work.
This patch makes
Hi,
Do you want me to resend it?
On Thursday, July 24, 2014, Matt Turner matts...@gmail.com wrote:
The commit message should be something like
mesa: Add missing atomic buffer bindings and unbindings.
On Thu, Jul 24, 2014 at 12:18 PM, Aditya Atluri
adityaavina...@gmail.com javascript:;
On Thu, Jul 24, 2014 at 1:32 PM, Emil Velikov emil.l.veli...@gmail.com wrote:
On 22/07/14 19:43, Dylan Baker wrote:
GBM_DRIVERS_PATH is not documented, and only used to set the location of
gbm drivers, while LIBGL_DRIVERS_PATH is used for everything else, and
is documented.
Generally this
On Thu, Jul 24, 2014 at 1:40 PM, Aditya Avinash
adityaavina...@gmail.com wrote:
Do you want me to resend it?
Not just for those comments. If there are other comments that
necessitate resending, please fix the two things I mentioned.
Otherwise someone will just fix them when they commit your
On Thu, Jul 24, 2014 at 1:43 PM, Kristian Høgsberg k...@bitplanet.net wrote:
On Thu, Jul 24, 2014 at 1:32 PM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 22/07/14 19:43, Dylan Baker wrote:
GBM_DRIVERS_PATH is not documented, and only used to set the location of
gbm drivers, while
From: Ian Romanick ian.d.roman...@intel.com
Previously we had to keep unreachable global symbols in the symbol table
because the symbol table is used during linking. Having the symbol
table retain pointers to freed memory... what could possibly go wrong?
At the same time, this meant that we kept
Historically, we've implemented the rules for overriding built-in
functions by creating multiple ir_functions and relying on the symbol
table to hide the one containing built-in functions. That works, but
has a few drawbacks, so the next patch will change it.
Instead, we'll have a single
Piglit's spec/glsl-1.10/linker/override-builtin-{const,uniform}-05 tests
do the following:
1. Call abs(float) - a built-in function.
2. Create a user-defined replacement for abs(float).
3. Call abs(float) again - now the user function.
At step 1, we created an ir_function which included the
On Tuesday, July 15, 2014 08:05:31 PM Kenneth Graunke wrote:
On Monday, July 14, 2014 03:48:34 PM Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
Previously we had to keep unreachable global symbols in the symbol table
because the symbol table is used during linking.
On Thursday, July 24, 2014 09:32:38 PM Emil Velikov wrote:
On 22/07/14 19:43, Dylan Baker wrote:
GBM_DRIVERS_PATH is not documented, and only used to set the location of
gbm drivers, while LIBGL_DRIVERS_PATH is used for everything else, and
is documented.
Generally this split leads to
From: Jerome Glisse jgli...@redhat.com
The gpu packet prefetcher hates the ugly big nop packet those leads
to prefetching some invalid memory in some case. Apparently hawaii
is particularly sensible to this.
Note this only partialy fix hawaii issues and some zbuffer tiling
issues are still
From: Jérôme Glisse jgli...@redhat.com
Trace buffer allow to dump a command buffer which is fully repliable
as a standalone c program. This make debuging lockup immensively
simpler. This patch only plug the core minimal stuff and is still
missing the more fancy aspect that are in r600g. It
A while back we've mandated that gbm requires enable_dri,
thus this check is no longer required.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac | 4
1 file changed, 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index 4482161..46ed001 100644
---
The former is the only user of the latter. As such building gbm
without egl makes little to no sense.
Cc: 10.2 mesa-sta...@lists.freedesktop.org
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac | 4
1 file changed, 4 insertions(+)
diff --git a/configure.ac
On Thu, Jul 24, 2014 at 05:42:21PM -0400, j.gli...@gmail.com wrote:
From: Jerome Glisse jgli...@redhat.com
The gpu packet prefetcher hates the ugly big nop packet those leads
to prefetching some invalid memory in some case. Apparently hawaii
is particularly sensible to this.
Note this
From: Jerome Glisse jgli...@redhat.com
The ucode we got for hawaii does not support 0x1000 special nop
packet type 3 and this leads to gpu reading invalid memory. As packet
type 2 still exist just use packet type 2.
Note this only partialy fix hawaii issues and some zbuffer tiling
issues are
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Jul 25, 2014 at 12:07 AM, j.gli...@gmail.com wrote:
From: Jérôme Glisse jgli...@redhat.com
Trace buffer allow to dump a command buffer which is fully repliable
as a standalone c program. This make debuging lockup immensively
On Thu, 2014-07-24 at 13:52 -0400, Alex Deucher wrote:
On Thu, Jul 24, 2014 at 12:05 PM, Jan Vesely jan.ves...@rutgers.edu wrote:
On Thu, 2014-07-24 at 17:07 +0200, Marek Olšák wrote:
Sorry, GL 3.1 actually only requires 1024 vec4s. The UBO extension
spec contains a mistake.
Marek
This fixes piglit (with GL_ARB_compute_shader force enabled):
built-in-constants tests/spec/arb_compute_shader/minimum-maximums.txt
(Also requires piglit patch arb_compute_shader glsl min/max: Update
gl_MaxComputeUniformComponents)
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
This fixes piglit's arb_compute_shader-minmax test.
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
Re-send (originally sent June 9)
src/mesa/main/config.h | 11 +++
src/mesa/main/get_hash_params.py | 7 +++
2 files changed, 18 insertions(+)
diff --git
On Thu, Jul 24, 2014 at 6:44 PM, Jordan Justen
jordan.l.jus...@intel.com wrote:
This fixes piglit's arb_compute_shader-minmax test.
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
Re-send (originally sent June 9)
Is there any point in making these Const.Foo so that individual
On Thu, Jul 24, 2014 at 3:56 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Thu, Jul 24, 2014 at 6:44 PM, Jordan Justen
jordan.l.jus...@intel.com wrote:
This fixes piglit's arb_compute_shader-minmax test.
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
Re-send (originally sent
These are
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Fri, Jul 25, 2014 at 11:04 AM, Jordan Justen jljus...@gmail.com wrote:
On Thu, Jul 24, 2014 at 3:56 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Thu, Jul 24, 2014 at 6:44 PM, Jordan Justen
jordan.l.jus...@intel.com wrote:
This fixes
On Thu, Jul 24, 2014 at 6:28 PM, j.gli...@gmail.com wrote:
From: Jerome Glisse jgli...@redhat.com
The ucode we got for hawaii does not support 0x1000 special nop
packet type 3 and this leads to gpu reading invalid memory. As packet
type 2 still exist just use packet type 2.
Note this
From: Kenneth Graunke kenn...@whitecape.org
This hash table is used in core Mesa, the GLSL compiler, and the i965
driver, which makes it a good candidate for the new src/util module.
It's much faster than program/hash_table.[ch] (see commit 6991c2922f5
for data), and José's u_hash_table.c has a
This gathers macros that have been included across components into util so
that the include chain can be more vertical. In particular, this makes
util stand on its own without any dependence whatsoever on the rest of
mesa.
Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com
---
Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com
---
src/mesa/Makefile.sources | 3 +
src/mesa/SConscript | 341 ++
2 files changed, 11 insertions(+), 333 deletions(-)
diff --git a/src/mesa/Makefile.sources b/src/mesa/Makefile.sources
Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com
---
src/mesa/main/format_pack.c | 102 ++-
src/mesa/main/format_unpack.c| 69 --
src/mesa/main/format_unpack.h| 3 --
src/mesa/main/texcompress_etc.c | 19
From: Kenneth Graunke kenn...@whitecape.org
For a long time, we've wanted a place to put utility code which isn't
directly tied to Mesa or Gallium internals. This patch creates a new
src/util directory for exactly that purpose, and builds the contents as
libmesautil.la.
ralloc seemed like a
Signed-off-by: Jason Ekstrand jason.ekstr...@intel.com
---
src/gallium/Automake.inc | 2 +
src/gallium/auxiliary/Makefile.am | 4 -
src/gallium/auxiliary/Makefile.sources| 1 -
src/gallium/auxiliary/SConscript | 8 +-
There is a lot of code duplicated or awkwardly shared right now between
mesa/main, glsl, and gallium. It would be good to gather these useful
things into a central place that does not depend on any of the above.
People have talked about doing this from time to time; this series aims to
get the
On 24/07/14 22:08, Dylan Baker wrote:
On Thursday, July 24, 2014 09:32:38 PM Emil Velikov wrote:
On 22/07/14 19:43, Dylan Baker wrote:
GBM_DRIVERS_PATH is not documented, and only used to set the location of
gbm drivers, while LIBGL_DRIVERS_PATH is used for everything else, and
is documented.
On Thu, Jul 24, 2014 at 5:17 PM, Emil Velikov emil.l.veli...@gmail.com wrote:
On 24/07/14 22:08, Dylan Baker wrote:
On Thursday, July 24, 2014 09:32:38 PM Emil Velikov wrote:
On 22/07/14 19:43, Dylan Baker wrote:
GBM_DRIVERS_PATH is not documented, and only used to set the location of
gbm
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 58 +--
src/mesa/drivers/dri/i965/brw_cfg.h | 4 ++-
2 files changed, 58 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp
b/src/mesa/drivers/dri/i965/brw_cfg.cpp
index 9cd8b9f..c39edad
---
src/mesa/drivers/dri/i965/brw_cfg.h | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h
b/src/mesa/drivers/dri/i965/brw_cfg.h
index a5d2df5..913a1ed 100644
--- a/src/mesa/drivers/dri/i965/brw_cfg.h
+++ b/src/mesa/drivers/dri/i965/brw_cfg.h
@@
These 19 patches reduce the number of times we calculate the CFG by
61.74% (after the previous series, which already reduced it by 55%).
To do so, I've added some infrastructure to the CFG code to combine
blocks, and to insert and remove instructions while preserving the
information stored in the
---
src/mesa/drivers/dri/i965/brw_cfg.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h
b/src/mesa/drivers/dri/i965/brw_cfg.h
index f7203e2..a5d2df5 100644
--- a/src/mesa/drivers/dri/i965/brw_cfg.h
+++ b/src/mesa/drivers/dri/i965/brw_cfg.h
@@ -112,6
The next patch adds a foreach_block (block, cfg) macro, which works
better if it provides a direct bblock_t pointer, rather than a
bblock_link pointer that you have to use to find the actual block.
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 10 +-
src/mesa/drivers/dri/i965/brw_cfg.h |
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
src/mesa/drivers/dri/i965/brw_shader.cpp | 4 +++-
src/mesa/drivers/dri/i965/brw_shader.h | 1 +
src/mesa/drivers/dri/i965/brw_vec4.cpp | 1 +
4 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
Use this as an opportunity to rename 'block_num' to 'num'. block-num is
clear, and block-block_num has always been redundant.
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 17 ++---
src/mesa/drivers/dri/i965/brw_cfg.h| 5 +-
---
src/mesa/drivers/dri/i965/brw_blorp_blit_eu.cpp | 4 +++-
src/mesa/drivers/dri/i965/brw_fs.cpp | 10 ++
src/mesa/drivers/dri/i965/brw_fs.h| 12 ++--
src/mesa/drivers/dri/i965/brw_fs_generator.cpp| 22 +-
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 80
src/mesa/drivers/dri/i965/brw_shader.h | 5 ++
2 files changed, 85 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965/brw_shader.cpp
index 47535a9..ba93cbc 100644
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 24
src/mesa/drivers/dri/i965/brw_cfg.h | 2 ++
2 files changed, 26 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp
b/src/mesa/drivers/dri/i965/brw_cfg.cpp
index d806b83..9cd8b9f 100644
---
---
src/mesa/drivers/dri/i965/brw_cfg.h | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h
b/src/mesa/drivers/dri/i965/brw_cfg.h
index 913a1ed..29e31e7 100644
--- a/src/mesa/drivers/dri/i965/brw_cfg.h
+++ b/src/mesa/drivers/dri/i965/brw_cfg.h
@@ -109,6
... rather than pointing directly to the associated instruction. This
will let us set the block containing the IF statement's else-pointer to
NULL, when we delete a useless ELSE instruction, as in the case
(+f0) if(8)
...
else(8)
endif(8)
Also, remove the pointer to the ENDIF, since
---
src/mesa/drivers/dri/i965/brw_fs.h| 4 ++--
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 20
2 files changed, 14 insertions(+), 10 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.h
b/src/mesa/drivers/dri/i965/brw_fs.h
index
This pass deletes an IF/ELSE/ENDIF or IF/ENDIF sequence, or the ELSE in
an ELSE/ENDIF sequence.
In the typical case (where IF and ENDIF) aren't the only instructions in
their basic blocks, we can simply remove the instructions (implicitly
deleting the block containing only the ELSE), and attempt
Will let us avoid invalidating the CFG if the optimization pass has
removed instructions using the new basic block methods.
---
src/mesa/drivers/dri/i965/brw_fs.h| 2 +-
src/mesa/drivers/dri/i965/brw_fs_live_variables.cpp | 5 +++--
src/mesa/drivers/dri/i965/brw_shader.h
To avoid invalidating and recreating the control flow graph. Also stop
invalidating the CFG in places we didn't add or remove an instruction.
cfg calculations: 202951 - 80307 (-60.43%)
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 45 +-
1 - 100 of 109 matches
Mail list logo
