On 07/24/2014 07:16 PM, Matt Turner wrote:
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/c
This patch fixes regressions caused by commit 48deb4d. Regressions
happened because 'run_default' var did not initialized when default
case was the last one.
Now all the switch tests in es3conform suite are passing.
Signed-off-by: Tapani Pälli
---
src/glsl/ast_to_hir.cpp | 12 ++--
1 fi
On Fri, 25 Jul 2014 09:14:40 +0300
Pekka Paalanen wrote:
> On Thu, 24 Jul 2014 17:16:39 +0100
> Emil Velikov wrote:
>
> > 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:
> > >>> O
On Thu, 24 Jul 2014 17:16:39 +0100
Emil Velikov wrote:
> 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:
> >>>
> Fro
https://bugs.freedesktop.org/show_bug.cgi?id=80615
--- Comment #6 from Alexandre Demers ---
Christian, could you give us your input on this issue and proposed fix?
(In reply to comment #5)
> AFAICS the whole thing is quite inconsistent.
>
> (In reply to comment #3)
> > That's something I hadn't
On Friday, July 25, 2014 04:02:48 AM Emil Velikov wrote:
> On 25/07/14 03:14, Matt Turner wrote:
> > 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
On 07/25/2014 06:51 AM, Dave Airlie wrote:
> Hi guys,
>
> Just fell over these in passing, run piglit
> glsl-1.50-transform-feedback-type-and-size test
>
> Dave.
>
>
8<
> ==8650== Invalid read of size 4
> ==8650==at 0x65F7FD6: brw::vec4_visitor::opt_cse_local(bblock_t*)
> (brw_vec4_cse.cpp:22
Hi guys,
Just fell over these in passing, run piglit
glsl-1.50-transform-feedback-type-and-size test
Dave.
==8650== Invalid read of size 8
==8650==at 0x6440C9D: ast_interface_block::hir(exec_list*,
_mesa_glsl_parse_state*) (ast_to_hir.cpp:5647)
==8650==by 0x64334CA: _mesa_ast_to_hir(exec
On Thu, Jul 24, 2014 at 8:02 PM, Emil Velikov wrote:
> On 25/07/14 03:14, Matt Turner wrote:
>> 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 w
On 25/07/14 03:14, Matt Turner wrote:
> 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 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
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
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.
-
---
src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
b/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
index d64cd98..f609138 100644
--- a/src/mesa/drivers/dri/
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 59 +++
src/mesa/drivers/dri/i965/brw_cfg.h | 2 ++
2 files changed, 61 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp
b/src/mesa/drivers/dri/i965/brw_cfg.cpp
index 3895469..a51d0d2 100644
--- a/src/
---
src/mesa/drivers/dri/i965/brw_cfg.h | 1 +
src/mesa/drivers/dri/i965/brw_shader.cpp | 32
src/mesa/drivers/dri/i965/brw_shader.h | 4
3 files changed, 37 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h
b/src/mesa/drivers/dri/i965/
... 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
---
.../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
---
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
---
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_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.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
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
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
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
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
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
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
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 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
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
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
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.
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
> >>
> >>
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
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
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
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
+++
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
@@
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
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:
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
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
>
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 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
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: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
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 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
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
---
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
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 ++
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
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 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
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 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,
>>
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 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 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 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 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 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 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 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
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 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
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
>
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
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
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
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,
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
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
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 [-
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
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
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, 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 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 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 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 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
1 - 100 of 113 matches
Mail list logo
