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 o
https://bugs.freedesktop.org/show_bug.cgi?id=62202
--- Comment #2 from Tapani Pälli ---
> 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 a bit more look
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 ad
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, 2
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
b/src/gallium/
On Wed, Jun 18, 2014 at 03:11:52PM -0700, Anuj Phogat wrote:
> On Wed, Jul 24, 2013 at 5:06 PM, Chad Versace
> wrote:
> > This patch is
> > Reviewed-by: Chad Versace
> > with the understanding that it depends on previous patches.
> >
> > I'll begin reviewing your Piglit tests now.
> >
> >
> > On
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
> >>
> >> There are a bunch of places, especially in the UBO code, where we check
> >> whether something is a ma
Yes, the commit message and your git username needs to be fixed.
Marek
On Thu, Jul 24, 2014 at 12:49 AM, Aditya Avinash
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 Romanick wrote:
>>
Reviewed-by: Marek Olšák
Marek
On Thu, Jul 24, 2014 at 2:37 AM, Tom Stellard wrote:
> ---
> src/gallium/docs/source/screen.rst| 2 ++
> src/gallium/drivers/radeon/r600_pipe_common.c | 7 +++
> src/gallium/include/pipe/p_defines.h | 3 ++-
> 3 files changed, 11 insertio
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 wrote:
> ---
> src/gallium/docs/source/screen.rst | 2 ++
> src/gallium/include/pipe/p_defines.h | 3 ++-
> 2 files changed, 4 insertions(+), 1
From: Alon Levy
Fixed build error at glsl_parser.yy for Visual Studio 2013
Signed-off-by: Alon Levy
---
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
the previous patch)
src/mesa/mai
> 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 writes:
> What do NVIDIA and AMD do? We should mimic that.
I don't have an NVidia or AMD card ha
ping
On Sat, Jul 19, 2014 at 11:45 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
>
> 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 index2D to 1 in order to generate somethi
On 24/07/14 07:23, Pekka Paalanen wrote:
> On Thu, 24 Jul 2014 01:43:35 +0100
> Emil Velikov wrote:
>
>> From: Giovanni Campagna
>>
>> 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 mutt
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 change
On 23/07/14 22:16, Ian Romanick wrote:
> On 07/22/2014 02:07 PM, Alon Levy wrote:
>> Signed-off-by: Alon Levy
>> ---
>> 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 wrote:
> On 24/07/14 07:23, Pekka Paalanen wrote:
> > On Thu, 24 Jul 2014 01:43:35 +0100
> > Emil Velikov wrote:
> >
> >> From: Giovanni Campagna
> >>
> >> Turn GBM into a swrast loader (providing putimage/getimage backed
> >> by a dumb KMS buffe
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 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
>> @@ -235,6 +235,
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
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 cygwin
From: Yaakov Selkowitz
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."
Signed-off-by: Yaakov Selkowitz
Review
From: Yaakov Selkowitz
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
unexpected platform in os_process.c [-
Ian Romanick 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 then it starts to
On 07/24/2014 02:12 PM, Alon Levy wrote:
> From: Alon Levy
Forgot to add Ian as cc.
>
> Fixed build error at glsl_parser.yy for Visual Studio 2013
>
> Signed-off-by: Alon Levy
> ---
> Thanks for the review. I verified and VS2013 (vs12) doesn't include
> strcasecmp,
> I corrected the file you
On Thu, Jul 24, 2014 at 10:37 AM, Neil Roberts wrote:
> Ian Romanick 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
From: José Fonseca
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, 402653184, 0,
0, 0, 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 unifo
Matt Turner 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
completely decomp
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 wrote:
> OpenGL 3.1 requires 4096 vec4s (65536 bytes) per constant buffer and
> the hardware supports 16 constant buffers. I'm not sure what the
> "const
On 07/24/2014 07:43 AM, Ilia Mirkin wrote:
> On Thu, Jul 24, 2014 at 10:37 AM, Neil Roberts wrote:
>> Ian Romanick 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
>
On 07/23/2014 07:41 PM, Matt Turner wrote:
> On Wed, Jul 23, 2014 at 3:16 PM, Ian Romanick 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 the ha
Ilia Mirkin 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 simple conversion t
On 24 July 2014 16:55, Marek Olšák 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 since Evergree
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 wrote:
> > OpenGL 3.1 requires 4096 vec4s (65536 bytes) per constant buffer and
> > th
On Thu, Jul 24, 2014 at 5:44 AM, Emil Velikov 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
>>> ---
>>> src/glsl/glsl_parser.yy | 4
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/src/glsl/glsl_parser.yy b/
On 24/07/14 14:30, Pekka Paalanen wrote:
> On Thu, 24 Jul 2014 13:34:42 +0100
> Emil Velikov wrote:
>
>> On 24/07/14 07:23, Pekka Paalanen wrote:
>>> On Thu, 24 Jul 2014 01:43:35 +0100
>>> Emil Velikov wrote:
>>>
From: Giovanni Campagna
Turn GBM into a swrast loader (providing pu
On 24/07/14 17:07, Matt Turner wrote:
> On Thu, Jul 24, 2014 at 5:44 AM, Emil Velikov
> 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
---
src/glsl/glsl_parser.yy | 4
1 file changed, 4 insertions(+
On Thu, Jul 24, 2014 at 5:37 AM, Knut Andre Tidemann
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 +
>> src/gal
On Fri, Jun 27, 2014 at 1: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
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
On Thu, Jul 24, 2014 at 8:55 AM, Neil Roberts wrote:
> Ilia Mirkin 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,
>>
From: Marek Olšák
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 | 2 --
src/gal
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 th
On Thu, Jul 24, 2014 at 12:05 PM, Jan Vesely 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 wrote:
>> > OpenGL 3.1 req
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
>> 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
> ---
> src/glsl/glsl_pa
From: Marek Olšák
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 | 4 ++
---
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 @@ _mesa_init_bu
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
wrote:
> ---
> src/mesa/main/bufferobj.c | 31 +++
> 1 file changed, 31 insertions(+)
>
> diff --git a/src/mesa/ma
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 L
Hi,
Do you want me to resend it?
On Thursday, July 24, 2014, Matt Turner 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
> > wrote:
> > ---
> > src/mesa/main/bufferobj.c | 3
On Thu, Jul 24, 2014 at 1:32 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 c
On Thu, Jul 24, 2014 at 1:40 PM, Aditya Avinash
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 patch.
_
On Thu, Jul 24, 2014 at 1:43 PM, Kristian Høgsberg wrote:
> On Thu, Jul 24, 2014 at 1:32 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 everythin
From: Ian Romanick
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 live references to tons
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 ir_funct
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 built
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
> >
> > Previously we had to keep unreachable global symbols in the symbol table
> > because the symbol table is used during linking. Having the symbol
>
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
From: Jerome Glisse
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 present.
Signed-off-by:
From: Jérôme Glisse
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 however already
proved us
A while back we've mandated that gbm requires enable_dri,
thus this check is no longer required.
Signed-off-by: Emil Velikov
---
configure.ac | 4
1 file changed, 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index 4482161..46ed001 100644
--- a/configure.ac
+++ b/configure.ac
@@
The former is the only user of the latter. As such building gbm
without egl makes little to no sense.
Cc: "10.2"
Signed-off-by: Emil Velikov
---
configure.ac | 4
1 file changed, 4 insertions(+)
diff --git a/configure.ac b/configure.ac
index 744e55c..4482161 100644
--- a/configure.ac
+++
On Thu, Jul 24, 2014 at 05:42:21PM -0400, j.gli...@gmail.com wrote:
> From: Jerome Glisse
>
> 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 f
From: Jerome Glisse
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 still present.
Ch
Reviewed-by: Marek Olšák
Marek
On Fri, Jul 25, 2014 at 12:07 AM, wrote:
> From: Jérôme Glisse
>
> 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
On Thu, 2014-07-24 at 13:52 -0400, Alex Deucher wrote:
> On Thu, Jul 24, 2014 at 12:05 PM, Jan Vesely 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
---
src/glsl/builtin_variables.
This fixes piglit's arb_compute_shader-minmax test.
Signed-off-by: Jordan Justen
---
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 a/src/mesa/main/config.h b/src/m
On Thu, Jul 24, 2014 at 6:44 PM, Jordan Justen
wrote:
> This fixes piglit's arb_compute_shader-minmax test.
>
> Signed-off-by: Jordan Justen
> ---
> Re-send (originally sent June 9)
Is there any point in making these Const.Foo so that individual
drivers can specify different values? Or would al
On Thu, Jul 24, 2014 at 3:56 PM, Ilia Mirkin wrote:
> On Thu, Jul 24, 2014 at 6:44 PM, Jordan Justen
> wrote:
>> This fixes piglit's arb_compute_shader-minmax test.
>>
>> Signed-off-by: Jordan Justen
>> ---
>> Re-send (originally sent June 9)
>
> Is there any point in making these Const.Foo so
These are
Reviewed-by: Chris Forbes
On Fri, Jul 25, 2014 at 11:04 AM, Jordan Justen wrote:
> On Thu, Jul 24, 2014 at 3:56 PM, Ilia Mirkin wrote:
>> On Thu, Jul 24, 2014 at 6:44 PM, Jordan Justen
>> wrote:
>>> This fixes piglit's arb_compute_shader-minmax test.
>>>
>>> Signed-off-by: Jordan Ju
On Thu, Jul 24, 2014 at 6:28 PM, wrote:
> From: Jerome Glisse
>
> 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
From: Kenneth Graunke
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 comment saying Gallium
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"
---
src/gallium/Automake.inc
Signed-off-by: Jason Ekstrand
---
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
index f4904fb..fdfb2d2 10064
Signed-off-by: Jason Ekstrand
---
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
src/mesa/main/texcompres
From: Kenneth Graunke
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 good first candidate. Th
Signed-off-by: Jason Ekstrand
---
src/gallium/Automake.inc | 2 +
src/gallium/auxiliary/Makefile.am | 4 -
src/gallium/auxiliary/Makefile.sources| 1 -
src/gallium/auxiliary/SConscript | 8 +-
src/gallium/auxiliary/util/u_format_pack.p
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 bal
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
On Thu, Jul 24, 2014 at 5:17 PM, Emil Velikov 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 drivers, while
---
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
@@ -120,6
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 +112
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 +-
.../drivers/dri/i965/brw_dead_control_flow.
---
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/d
---
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
--- a/src/mesa/driver
---
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 +10
... 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 i
---
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 6fe23d2..c01
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 t
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 +-
.../drivers/dri/i965/br
---
.../drivers/dri/i965/brw_fs_peephole_predicated_break.cpp | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_peephole_predicated_break.cpp
b/src/mesa/drivers/dri/i965/brw_fs_peephole_predicated_break.cpp
index fe3812d..445d10e
Operating on this code,
B0: ...
cmp.ne.f0(8)
(+f0) if(8)
B1: break(8)
B2: endif(8)
We can delete B2 without attempting to merge any blocks, since the
break/continue instruction necessarily ends the previous block.
After deleting the if instruction, we attempt to merge blocks B0 and B1.
-
1 - 100 of 113 matches
Mail list logo
