Am 29.07.2015 06:35, schrieb Romain Failliot:
Hi!
First, I wanted to thank you for the incredible work you've done to
achieve OpenGL 4.1. The number of visitors exploded on mesamatrix,
while the average was around 100 visits per day, I had more than 2500
visits just for the day of the news! This
On Tue, 2015-07-28 at 18:17 +0300, Francisco Jerez wrote:
> Iago Toral Quiroga writes:
>
> > Link to v1:
> > http://lists.freedesktop.org/archives/mesa-dev/2015-July/089766.html
> >
> > Changes after review (Curro)
> > - Drop the patch that asserted that the reg size should always be 1
> > -
Yes, thanks.
Il 28/lug/2015 11:27 PM, Ilia Mirkin ha scritto:
>
> Fine by me. Do you need someone to commit this?
>
> On Tue, Jul 28, 2015 at 2:53 PM, Fabio Pedretti wrote:
> > ---
> > docs/GL3.txt | 36 ++--
> > 1 file changed, 18 insertions(+), 18 deletions
On Tuesday, July 28, 2015 08:00:54 PM Ben Widawsky wrote:
> This is easily accomplished by moving simd16 3src to GEN9_FEATURES.
>
> Signed-off-by: Ben Widawsky
> ---
> src/mesa/drivers/dri/i965/brw_device_info.c | 6 ++
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/src/
Reviewed-by: Tapani Pälli
On 07/29/2015 04:48 AM, Kenneth Graunke wrote:
Other stages can be miserably slow too!
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/br
Reviewed-by: Tapani Pälli
On 07/29/2015 02:45 AM, Anuj Phogat wrote:
brw_cs.cpp:386:27: warning: comparison between signed and unsigned
integer expressions [-Wsign-compare]
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletio
Reviewed-by: Tapani Pälli
On 07/29/2015 02:48 AM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/common/meta_blit.c | 6 +++---
src/mesa/drivers/dri/i915/i915_texstate.c | 2 +-
src/mesa/drivers/dri/i965/brw_fs_vector_splitting.cpp | 2 +
Hi!
First, I wanted to thank you for the incredible work you've done to
achieve OpenGL 4.1. The number of visitors exploded on mesamatrix,
while the average was around 100 visits per day, I had more than 2500
visits just for the day of the news! This simple fact shows how much
this news was expect
This is easily accomplished by moving simd16 3src to GEN9_FEATURES.
Signed-off-by: Ben Widawsky
---
src/mesa/drivers/dri/i965/brw_device_info.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_device_info.c
b/src/mesa/drivers/dri/i965/brw_d
On 07/28/2015 02:38 PM, Chad Versace wrote:
> On Tue 28 Jul 2015, Emil Velikov wrote:
>> Hello Jim,
>>
>> On 28 July 2015 at 02:57, Bish, Jim wrote:
>>> From: Jim Bish
>>>
>> Would you mind splitting this into separate patches ? Adding a few
>> words in the commit log(s) would be highly prefera
Other stages can be miserably slow too!
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965/brw_shader.cpp
index 7808212..154c9a2
On 07/28/2015 10:05 AM, Emil Velikov wrote:
> On 28/07/15 17:54, Bish, Jim wrote:
>>
>>
>> On 07/28/2015 06:11 AM, Emil Velikov wrote:
>>> Hello Jim,
>>>
>>> On 28 July 2015 at 02:57, Bish, Jim wrote:
From: Jim Bish
>>> Would you mind splitting this into separate patches ? Adding a fe
From: Dave Airlie
One of the oglconform tests was crashing here, and it was
due to not cloning the actual parameters before creating the
new call. This makes a call clone function that does the right
things to make sure we clone all the needed info, and points
the callee at it. (It differs from -
On 28 July 2015 at 20:05, Marek Olšák wrote:
> From: Marek Olšák
>
Reviewed-by: Dave Airlie
> ---
> src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 10 +++---
> 1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c
> b/sr
On 28 July 2015 at 20:05, Marek Olšák wrote:
> From: Marek Olšák
Reviewed-by: Dave Airlie
>
> ---
> src/gallium/drivers/r600/r600_llvm.c| 1 -
> src/gallium/drivers/r600/r600_shader.c | 1 -
> src/gallium/drivers/radeon/radeon_llvm.h| 19 -
On 28 July 2015 at 20:05, Marek Olšák wrote:
> From: Marek Olšák
For 2 & 3,
Reviewed-by: Dave Airlie
>
> duplicated now
> ---
> src/gallium/drivers/r600/r600_llvm.c | 34 +--
> src/gallium/drivers/radeon/radeon_llvm.h | 8 ---
> .../drivers/radeon/radeon_setup
On 28 July 2015 at 20:05, Marek Olšák wrote:
> From: Marek Olšák
Looks good,
Reviewed-by: Dave Airlie
>
> ---
> src/gallium/drivers/radeon/r600_pipe_common.c | 1 +
> src/gallium/drivers/radeon/r600_pipe_common.h | 1 +
> src/gallium/drivers/radeonsi/si_shader.c | 2 +-
> 3 files changed
On 28 July 2015 at 20:05, Marek Olšák wrote:
> From: Marek Olšák
Reviewed-by: Dave Airlie
>
> ---
> src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 66
> ++
> 1 file changed, 66 insertions(+)
>
> diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> b/src/mesa/st
On Tue, 2015-07-28 at 16:34 -0700, Kenneth Graunke wrote:
> A simple shader such as
>
>vec4 color;
>color.xy.x = 1.0;
>
> would cause ir_assignment::set_lhs() to generate bogus IR:
>
>(swiz xy (swiz x (constant float (1.0
>
> We were setting the number of components of each new
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/common/meta_blit.c | 6 +++---
src/mesa/drivers/dri/i915/i915_texstate.c | 2 +-
src/mesa/drivers/dri/i965/brw_fs_vector_splitting.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_reg.h | 2 +-
src/mesa/d
brw_cs.cpp:386:27: warning: comparison between signed and unsigned
integer expressions [-Wsign-compare]
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_cs.cpp
b/src/mesa/drivers
https://bugs.freedesktop.org/show_bug.cgi?id=91496
Bug ID: 91496
Summary: [clover] Multiple device probe fails since
a27ec5dc460b91dc44675f48cddbbb2631ee824f
Product: Mesa
Version: git
Hardware: Other
OS: Al
A simple shader such as
vec4 color;
color.xy.x = 1.0;
would cause ir_assignment::set_lhs() to generate bogus IR:
(swiz xy (swiz x (constant float (1.0
We were setting the number of components of each new RHS swizzle based
on the highest channel used in the LHS swizzle. So, .xy.y w
On 28/07/15 23:49, Jason Ekstrand wrote:
> On Mon, Jul 27, 2015 at 3:37 PM, Alejandro Piñeiro
> wrote:
>> Uses the nir structure to get all the info needed (sources,
>> dest reg, etc), and then it uses the common
>> vec4_visitor::emit_texture to emit the final code.
>> ---
>>
>> v3: added glsl_
From: Dave Airlie
This enables GL4.1 for radeonsi, and updates the
docs in the correct places.
v2: enable only for llvm 3.7 which has fixes in place.
Reviewed-by: Marek Olšák
Signed-off-by: Dave Airlie
---
docs/GL3.txt | 16
docs/relnotes/10.7.0.htm
From: Dave Airlie
This is the final piece for ARB_gpu_shader5,
The code is based on the r600 code from Glenn Kennard,
and myself.
While developing this, I'm not 100% sure of all the calculations
made in the GS registers, this is why the max_stream is worked
out there and used to limit the chang
From: Dave Airlie
This will be used here later.
v2: update atom sizes
add check for old vs new enabled mask
Signed-off-by: Dave Airlie
---
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +++
src/gallium/drivers/radeon/r600_streamout.c | 21 -
2 files changed, 15 inse
From: Dave Airlie
This adds to the common radeon streamout code, support
for multiple streams.
It updates radeonsi/r600 to set the enabled mask up.
v2: update for changes in previous patch.
Reviewed-by: Marek Olšák
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/r600_shader.c
The last 4 had some review fixes that I rebased badly, and failed to test.
This set actually work.
Dave.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Dave Airlie
This enables GL4.1 for radeonsi, and updates the
docs in the correct places.
v2: enable only for llvm 3.7 which has fixes in place.
Reviewed-by: Marek Olšák
Signed-off-by: Dave Airlie
---
docs/GL3.txt | 16
docs/relnotes/10.7.0.htm
From: Dave Airlie
This adds to the common radeon streamout code, support
for multiple streams.
It updates radeonsi/r600 to set the enabled mask up.
Reviewed-by: Marek Olšák
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/r600_shader.c | 7 +
src/gallium/drivers/r600/r60
From: Dave Airlie
This is the final piece for ARB_gpu_shader5,
The code is based on the r600 code from Glenn Kennard,
and myself.
While developing this, I'm not 100% sure of all the calculations
made in the GS registers, this is why the max_stream is worked
out there and used to limit the chang
From: Dave Airlie
This will be used here later.
v2: update atom sizes
add check for old vs new enabled mask
Signed-off-by: Dave Airlie
---
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +++
src/gallium/drivers/radeon/r600_streamout.c | 21 -
2 files changed, 15 inse
From: Nanley Chery
The following patches add missing functionality to my prior ASTC patch series
needed to advertise support for the extension. The missing first patch was an
oversight, while the second patch is due to the release of a newer spec
revision.
The series will be injected into my pri
From: Nanley Chery
The ASTC spec was revised as follows:
Revision 2, April 28, 2015 - added CompressedTex{Sub,}Image3D to
commands accepting ASTC format tokens in the New Tokens section [...].
Support only exists in the HDR submode:
Add a second new column "3D Tex." which is empty for
From: Nanley Chery
According to the spec, Tex*Storage* commands are supported if the GL
implementation has support for immutable textures.
Signed-off-by: Nanley Chery
---
src/mesa/main/texformat.c | 56 +++
1 file changed, 56 insertions(+)
diff --gi
On Tue, 2015-07-28 at 13:37 +0300, Tapani Pälli wrote:
> Change function to get all gl_constants for inspection, this is used
> by follow-up patch.
>
> v2: rebase, update function documentation
>
> Signed-off-by: Tapani Pälli
> ---
> src/glsl/linker.cpp | 23 +++
> 1 file ch
Reviewed-by: Jason Ekstrand
On Mon, Jul 27, 2015 at 3:26 PM, Alejandro Piñeiro wrote:
> Splitted in two. The emission is moved to a new vec4_visitor
> method, vec4_visitor::emit_texture, ir order to be reused
> on the nir path.
> ---
>
> v3: removed shadow_compare and has_nonconstant_offset bool
On Mon, Jul 27, 2015 at 3:37 PM, Alejandro Piñeiro wrote:
> Uses the nir structure to get all the info needed (sources,
> dest reg, etc), and then it uses the common
> vec4_visitor::emit_texture to emit the final code.
> ---
>
> v3: added glsl_type_for_nir_alu_type helper as suggested (it could be
On Tue 28 Jul 2015, Emil Velikov wrote:
> Hello Jim,
>
> On 28 July 2015 at 02:57, Bish, Jim wrote:
> > From: Jim Bish
> >
> Would you mind splitting this into separate patches ? Adding a few
> words in the commit log(s) would be highly preferable.
Yes, please submit separate patches. "One patc
Fine by me. Do you need someone to commit this?
On Tue, Jul 28, 2015 at 2:53 PM, Fabio Pedretti wrote:
> ---
> docs/GL3.txt | 36 ++--
> 1 file changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/docs/GL3.txt b/docs/GL3.txt
> index 15bb57f..2e2a7e3 100644
All patches in this series except for 8 and 14 are
Reviewed-by: Jason Ekstrand
8 looks fishy and I think the code we have there today is probably not
what we want as well. Shouldn't take too much time to fix.
Once 8 gets sorted, I'm fine with 14 but we obviously can't push it
until we have 8.
Everybody is using git send-email. Sending an email from git works the
same as sending it from, say, Thunderbird. Just set the address, port,
TLS, etc.
The patch looks good. I'll push it momentarily.
Marek
On Tue, Jul 28, 2015 at 4:07 PM, Edward O'Callaghan
wrote:
> Hi Marek,
>
> Yea I don't le
On Tue, Jul 28, 2015 at 1:23 AM, Francisco Jerez wrote:
> Instead of relying on the default one. This shouldn't lead to any
> functional changes because DEP_RESOLVE_MOV overrides the execution
> controls of the instruction anyway.
Actually, DEP_RESOLVE_MOV calls half() on the builder which doesn
On Jul 28, 2015 2:43 AM, "Francisco Jerez" wrote:
>
> The register coalesce pass wasn't rewriting the destination and
> sources of instructions that accessed the second half of a coalesced
> register previously copied with a 16-wide MOV instruction. E.g.:
>
> | ADD (16) vgrf0:f, vgrf0:f, 1.0:f
>
R-B me
On Jul 28, 2015 2:43 AM, "Francisco Jerez" wrote:
> For some reason the loop that rewrites all occurrences of the
> coalesced register was iterating over all possible offsets until it
> would find one that compares equal to the offset of a source or
> destination of any instruction in the
I thought someone else R-b'd this.
LGTM. Reviewed-by: Brian Paul
On 07/28/2015 01:45 PM, Marek Olšák wrote:
Ping.
Marek
On Fri, Jul 24, 2015 at 7:16 PM, Marek Olšák wrote:
From: Marek Olšák
Fixes piglit:
spec@glsl-1.30@execution@fs-texture-sampler2dshadow-10
spec@glsl-1.30@ex
Ping.
Marek
On Fri, Jul 24, 2015 at 7:16 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Fixes piglit:
> spec@glsl-1.30@execution@fs-texture-sampler2dshadow-10
> spec@glsl-1.30@execution@fs-texture-sampler2dshadow-11
>
> v2: use st_shader_stage_to_ptarget
> ---
> src/mesa/state_tracker/
On Tue, Jul 28, 2015 at 6:35 PM, Kai Wasserbäch
wrote:
> Marek Olšák wrote on 28.07.2015 12:05:
>> From: Marek Olšák
>>
>> ---
>> src/gallium/auxiliary/util/u_debug.c | 8
>> src/gallium/auxiliary/util/u_debug.h | 6 +++---
>> 2 files changed, 7 insertions(+), 7 deletions(-)
>>
>> diff
On Tue, Jul 28, 2015 at 12:06 PM, Jason Ekstrand wrote:
> The meta CopyImageSubData path uses BlitFramebuffers to do the actual copy.
> The only thing that can affect BlitFramebuffers other than the currently
> bound framebuffers is the scissor so we need to save that off and reset it.
> If we don
The meta CopyImageSubData path uses BlitFramebuffers to do the actual copy.
The only thing that can affect BlitFramebuffers other than the currently
bound framebuffers is the scissor so we need to save that off and reset it.
If we don't do this, applications that use a scissor together with
CopyIma
---
docs/GL3.txt | 36 ++--
1 file changed, 18 insertions(+), 18 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 15bb57f..2e2a7e3 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -92,11 +92,11 @@ GL 3.3, GLSL 3.30 --- all DONE: i965, nv50, nvc0, r600,
r
On 07/28/2015 12:44 PM, Marek Olšák wrote:
From: Marek Olšák
---
src/mesa/state_tracker/st_atom_shader.c | 29 ++---
src/mesa/state_tracker/st_context.h | 1 -
2 files changed, 2 insertions(+), 28 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_shader.
From: Marek Olšák
---
src/mesa/state_tracker/st_atom_shader.c | 29 ++---
src/mesa/state_tracker/st_context.h | 1 -
2 files changed, 2 insertions(+), 28 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_shader.c
b/src/mesa/state_tracker/st_atom_shader.c
ind
If you add the check for LLVM 3.7, this is:
Reviewed-by: Marek Olšák
Marek
On Mon, Jul 27, 2015 at 3:50 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This enables GL4.1 for radeonsi, and updates the
> docs in the correct places.
>
> Signed-off-by: Dave Airlie
> ---
> docs/GL3.txt
On Mon, Jul 27, 2015 at 3:50 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This adds to the common radeon streamout code, support
> for multiple streams.
>
> It updates radeonsi/r600 to set the enabled mask up.
>
> Signed-off-by: Dave Airlie
> ---
> src/gallium/drivers/r600/r600_shader.c
On Mon, Jul 27, 2015 at 3:50 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This will be used here later.
>
> Signed-off-by: Dave Airlie
> ---
> src/gallium/drivers/radeon/r600_streamout.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/gallium/drivers/ra
Reviewed-by: Marek Olšák
Marek
On Mon, Jul 27, 2015 at 3:49 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This adds support for queries against the non-0 vertex streams.
>
> Signed-off-by: Dave Airlie
> ---
> src/gallium/drivers/radeon/r600_query.c | 18 --
> src/gallium/d
Just thought I'd mention that this is a bit RFC.
I was just throwing it out there before dinner so you guys could
have a look at it if you feel like it. I was hoping to put the loop analysis
on here to, but I'll fix it up to use a DFS approach before sending
it out on the list. The way I've solved
Reviewed-by: Marek Olšák
Marek
On Mon, Jul 27, 2015 at 3:49 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This is part of ARB_gpu_shader5, and this passes
> all the piglit tests currently available.
>
> v2: use macros from the fine derivs commit.
> add comments.
> Signed-off-by: Dave Airlie
I was hoping this should fix the problem with the pass
getting rerun forever, but apparently not. We still need
a solution like this though, so sending it out for some
feedback of whether or not this is the way to do it.
Signed-off-by: Thomas Helland
---
src/glsl/nir/nir.h | 1 +
We were already doing this internally for iterating over a function
implementation, so just expose it directly.
Signed-off-by: Connor Abbott
---
src/glsl/nir/nir.c | 7 +++
src/glsl/nir/nir.h | 3 +++
2 files changed, 10 insertions(+)
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c
ind
V2: Do a "depth first search" to convert to LCSSA
I'm still a bit unsure if the way I'm inserting
the phi nodes is correct, but the flow of the pass
seems to be correct. Unfortunately it keeps getting
rerun time and time again. Most likely due to some optimizations
removing the phi-nodes or other
Brian Paul wrote on 28.07.2015 18:40:
> On 07/28/2015 10:35 AM, Kai Wasserbäch wrote:
>> Marek Olšák wrote on 28.07.2015 12:05:
>>> From: Marek Olšák
>>>
>>> ---
>>> src/gallium/auxiliary/util/u_debug.c | 8
>>> src/gallium/auxiliary/util/u_debug.h | 6 +++---
>>> 2 files changed, 7 i
On 28/07/15 17:54, Bish, Jim wrote:
>
>
> On 07/28/2015 06:11 AM, Emil Velikov wrote:
>> Hello Jim,
>>
>> On 28 July 2015 at 02:57, Bish, Jim wrote:
>>> From: Jim Bish
>>>
>> Would you mind splitting this into separate patches ? Adding a few
>> words in the commit log(s) would be highly prefera
On Tue, Jul 28, 2015 at 3:37 AM, Tapani Pälli wrote:
> Same check is made for glBindFragDataLocationIndexed but it was missing
> when using layout qualifiers.
>
> Fixes following Piglit test:
> arb_blend_func_extended-output-location
>
> Signed-off-by: Tapani Pälli
> ---
> src/glsl/linke
On 07/28/2015 06:11 AM, Emil Velikov wrote:
> Hello Jim,
>
> On 28 July 2015 at 02:57, Bish, Jim wrote:
>> From: Jim Bish
>>
> Would you mind splitting this into separate patches ? Adding a few
> words in the commit log(s) would be highly preferable.
sure. thought this was small enough but ok
Marek Olšák wrote on 28.07.2015 12:05:
> From: Marek Olšák
>
> ---
> src/gallium/auxiliary/util/u_debug.c | 8
> src/gallium/auxiliary/util/u_debug.h | 6 +++---
> 2 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/src/gallium/auxiliary/util/u_debug.c
> b/src/gallium/au
On Tue, Jul 28, 2015 at 12:05:40PM +0200, Marek Olšák wrote:
> From: Marek Olšák
>
Reviewed-by: Tom Stellard
> ---
> src/gallium/drivers/r600/r600_llvm.c | 152
> -
> .../drivers/radeon/radeon_setup_tgsi_llvm.c| 150
> 2 files cha
On Tue, Jul 28, 2015 at 12:05:42PM +0200, Marek Olšák wrote:
> From: Marek Olšák
>
> This reverts commit 4db985a5fa9ea985616a726b1770727309502d81.
>
> The grass no longer disappears, which was the reason the commit was reverted.
> This might affect tessellation. We'll see.
>
Hi Marek,
This pa
This also helps making my egl/dri3 code easier, so
Acked-by: Boyan Ding
2015-07-11 1:49 GMT+08:00 Emil Velikov :
> Cc: Axel Davy
> Signed-off-by: Emil Velikov
> ---
> configure.ac| 2 +-
> src/egl/drivers/dri2/platform_wayland.c | 20 ++--
> 2 file
https://bugs.freedesktop.org/show_bug.cgi?id=91468
--- Comment #7 from Emil Velikov ---
(In reply to Krzysztof A. Sobiecki from comment #6)
> (In reply to Emil Velikov from comment #3)
> > Please send the patch to mesa-dev (ideally using git send-email). It'll get
> > wider exposure there.
>
> I
Iago Toral Quiroga writes:
> Link to v1:
> http://lists.freedesktop.org/archives/mesa-dev/2015-July/089766.html
>
> Changes after review (Curro)
> - Drop the patch that asserted that the reg size should always be 1
> - Expand this so that we do not unspill a register if we have just
> uns
https://bugs.freedesktop.org/show_bug.cgi?id=91468
--- Comment #6 from Krzysztof A. Sobiecki ---
(In reply to Emil Velikov from comment #3)
> Please send the patch to mesa-dev (ideally using git send-email). It'll get
> wider exposure there.
I don't see how It would be appropriate to even try to
https://bugs.freedesktop.org/show_bug.cgi?id=91468
--- Comment #5 from Krzysztof A. Sobiecki ---
(In reply to Serge Martin from comment #4)
> Please note that autotools buils have the "svn" part in the libname.
> I don't knows if it will be deprecated in 3.8 and remove later or if it will
> be re
https://bugs.freedesktop.org/show_bug.cgi?id=91130
--- Comment #4 from Dennis Schridde ---
Just to make sure I understand. You say that:
1) Mesa should provide a shared library that actually implements all functions
advertised in the header (which will be fixed in Mesa 10.7/11.0)
2) applications
Jason Ekstrand writes:
> On Jul 28, 2015 2:43 AM, "Francisco Jerez" wrote:
>>
>> register_coalesce() was considering the exec_size of the MOV
>> instruction alone to decide whether the register at offset+1 of the
>> source VGRF was being copied to inst->dst.reg_offset+1 of the
>> destination VGR
Hi Marek,
Yea I don't let git send emails via any mailer.. It would be better if I
could just do pull requests rather than mailing lists if at all
possible? I really seriously hate mailing lists.
In any case, attached is a revised patch with all the feedback included.
Kind Regards,
Edward.
--
On 07/28/2015 04:05 AM, Marek Olšák wrote:
From: Marek Olšák
It only checks fragment textures and ignores other shaders, which makes it
incomplete, and texture are already finalized in update_single_texture.
There are no piglit regressions.
---
src/mesa/state_tracker/st_atom.c | 1 -
On 07/28/2015 07:33 AM, Emil Velikov wrote:
On 20 July 2015 at 17:12, Brian Paul wrote:
On 07/17/2015 02:14 PM, Emil Velikov wrote:
On 17 July 2015 at 20:22, Brian Paul wrote:
Can you elaborate in the commit message why this is being done or why the
HAVE_SYS_TYPES_H definition is not neede
On 20 July 2015 at 17:12, Brian Paul wrote:
> On 07/17/2015 02:14 PM, Emil Velikov wrote:
>>
>> On 17 July 2015 at 20:22, Brian Paul wrote:
>>>
>>> Can you elaborate in the commit message why this is being done or why the
>>> HAVE_SYS_TYPES_H definition is not needed? That saves me time having t
On 17/07/15 21:57, Emil Velikov wrote:
> Would anyone be interested is skimming over this trivial series ?
>
Thomas kindly looked over patches 1,2 and 5. Would anyone care to throw
a quick ack/nack for the rest ?
I'm inclined to push the series unless there are any objections in the
next few days
https://bugs.freedesktop.org/show_bug.cgi?id=91468
--- Comment #4 from Serge Martin ---
Please note that autotools buils have the "svn" part in the libname.
I don't knows if it will be deprecated in 3.8 and remove later or if it will be
removed in 3.8.
--
You are receiving this mail because:
Yo
On Jul 28, 2015 2:43 AM, "Francisco Jerez" wrote:
>
> register_coalesce() was considering the exec_size of the MOV
> instruction alone to decide whether the register at offset+1 of the
> source VGRF was being copied to inst->dst.reg_offset+1 of the
> destination VGRF, which is only a valid assumpt
https://bugs.freedesktop.org/show_bug.cgi?id=91468
--- Comment #3 from Emil Velikov ---
Please send the patch to mesa-dev (ideally using git send-email). It'll get
wider exposure there.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=91130
--- Comment #3 from Serge Martin ---
Sorry I mean "OpenCL 1.1 MESA 10.7.0-devel". 1.2 is only clain on my local tree
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
_
Hello Jim,
On 28 July 2015 at 02:57, Bish, Jim wrote:
> From: Jim Bish
>
Would you mind splitting this into separate patches ? Adding a few
words in the commit log(s) would be highly preferable.
> ---
> Android.common.mk | 10 ++
> Android.mk
https://bugs.freedesktop.org/show_bug.cgi?id=91130
--- Comment #2 from Serge Martin ---
Hello
this should be mitigated with the "clover: allow the creation of Piglit tests
for the missing CL 1.2 fonction" serie available on mesa-dev list.
However, even if the program will compile, those fonctio
Hello Boyan,
On 21 July 2015 at 16:43, Boyan Ding wrote:
> This is the second version of my effort to bring dri3 to EGL on X. This
> time I took Emil's advice and moved common logic of GLX and EGL into
> loader/loader_dri3_helper.[ch]. With this, the dri3 code in libEGL is
> only ~650 lines, with
On 21 July 2015 at 16:44, Boyan Ding wrote:
> Signed-off-by: Boyan Ding
> ---
> configure.ac | 6 +
> src/egl/drivers/dri2/Makefile.am | 5 +
> src/egl/drivers/dri2/egl_dri2.c | 66 +++-
> src/egl/drivers/dri2/egl_dri2.h | 14 +-
> src/
On 21 July 2015 at 16:43, Boyan Ding wrote:
> Signed-off-by: Boyan Ding
> ---
> configure.ac|8 +-
> src/loader/Makefile.am |9 +
> src/loader/loader_dri3_helper.c | 1397
> +++
> src/loader/loader_dri3_helper.h | 226 +++
https://bugs.freedesktop.org/show_bug.cgi?id=91474
--- Comment #2 from Emil Velikov ---
Using the device extensions one will be able to enumerate and select the device
they want without the need of X or the DRI_PRIME variable.
Jonny Lamb posted a series that implements the "base" - device_enumer
https://bugs.freedesktop.org/show_bug.cgi?id=91474
--- Comment #1 from Axel Davy ---
Mesa egl does indeed support DRI_PRIME for the wayland backend,
however unless DRI_PRIME is set, it uses the same card that the server.
I guess either the user has set DRI_PRIME or the equivalent drirc option,
o
Hi Edward,
please use git send-email next time. Now I have to paste your patch here.
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 15bb57f..6d9d424 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -189,7 +189,7 @@ GL 4.5, GLSL 4.50:
GL_ARB_ES3_1_compatibility not st
On Tue, 2015-07-28 at 13:10 +1000, Edward O'Callaghan wrote:
> Hi,
>
> Attached is a patch that should support the said extension. I am using
> the following piglit patch to test with:
>
> https://github.com/victoredwardocallaghan/piglit
> -work/commit/783d5aa28b2e5ea4f67e439fe9e74c0391f5d7d0
>
Same check is made for glBindFragDataLocationIndexed but it was missing
when using layout qualifiers.
Fixes following Piglit test:
arb_blend_func_extended-output-location
Signed-off-by: Tapani Pälli
---
src/glsl/linker.cpp | 19 +++
1 file changed, 19 insertions(+)
diff
Change function to get all gl_constants for inspection, this is used
by follow-up patch.
v2: rebase, update function documentation
Signed-off-by: Tapani Pälli
---
src/glsl/linker.cpp | 23 +++
1 file changed, 15 insertions(+), 8 deletions(-)
diff --git a/src/glsl/linker.cpp
Similar to reads inmmediately after spilling, there is no gain for register
allocation in unspilling the same vgrf in consecutive instructions, since
the gain only comes when there are gaps in the program that don't use the
vgrf. In these scenarios we can simply unspill once and reuse the result
fo
When we have code such as this:
mov vgrf1.0.x:F, vgrf2.:F
mov vgrf3.0.x:F, vgrf1.:F
...
mov vgrf3.0.x:F, vgrf1.:F
And vgrf1 is chosen for spilling, we can emit this:
mov vgrf1.0.x:F, vgrf2.:F
gen4_scratch_write hw_reg0:F, vgrf1.:D, 22D
mov vgrf3.0.x:F, vgrf1.:F
...
gen4_s
---
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4.cpp| 2 +-
src/mesa/drivers/dri/i965/intel_debug.c | 3 ++-
src/mesa/drivers/dri/i965/intel_debug.h | 5 +++--
4 files changed, 7 insertions(+), 5 deletions(-)
diff --g
1 - 100 of 146 matches
Mail list logo
