The lowered code reads from the destination, which isn't possible from
message registers.
Fixes the dEQP functional.shaders.precision.int.highp_mul_fragment test
on SNB.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/mesa/d
On 09/01/2015 04:40 PM, Brian Paul wrote:
On 09/01/2015 06:04 AM, Tapani Pälli wrote:
On 08/31/2015 05:56 PM, Brian Paul wrote:
Looks good. Just some minor nitpicks below.
Reviewed-by: Brian Paul
Thanks! I have one comment below ..
On 08/31/2015 01:23 AM, Tapani Pälli wrote:
Patch mod
On Tue, 2015-09-01 at 14:46 +0100, Emil Velikov wrote:
> On 29 July 2015 at 13:46, Timothy Arceri wrote:
> > On Wed, 2015-07-29 at 09:57 +0200, Iago Toral wrote:
> > > On Sun, 2015-07-26 at 18:35 +1000, Timothy Arceri wrote:
> > > > Since commit c0cd5b var->data.binding was being used as a replace
This will allow us to access the uniform later on without resorting to
building a name string and looking it up in UniformHash.
V2: store slot number for all non-UBO uniforms to make code more
consitent, renamed explicit_binding to explicit_location and added
comment about what it does. Store the
As a bonus we get indirect support for arrays of arrays for free.
V3: Use nir_instr_rewrite_src() with empty src rather then clearing
the use_link list directly for the old indirects as suggested by Jason
V2: Fixed validation error in debug build
---
src/glsl/nir/nir_lower_samplers.cpp | 169 +++
---
src/glsl/link_uniforms.cpp | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
index f68ea9d..c28ac52 100644
--- a/src/glsl/link_uniforms.cpp
+++ b/src/glsl/link_uniforms.cpp
@@ -805,6 +805,17 @@ p
Removes another dependency on the UniformHash hash table.
---
src/mesa/program/sampler.cpp | 123 +++
1 file changed, 53 insertions(+), 70 deletions(-)
diff --git a/src/mesa/program/sampler.cpp b/src/mesa/program/sampler.cpp
index ea3024d..01150fa 100644
--
This is required so that the next patch can safely assign the slot id
to the var.
The ids are now assigned in the order we want before allocating storage
so there is no need to sort the storage array and move things around.
---
src/glsl/link_uniforms.cpp | 90 +
This allows the correct offset to be easily calculated for indirect
indexing when a struct array contains multiple samplers, or any crazy
nesting.
The indices for the folling struct will now look like this:
Sampler index: 0 Name: s[0].tex
Sampler index: 1 Name: s[1].tex
Sampler index: 2 Name: s[0]
The following patch series fixes the indirect sampler indexing piglit tests
/tests/spec/arb_gpu_shader5/execution/sampler_array_indexing/* for nir.
It also takes steps toward removing the use of the UniformHash table.
There are no piglit regressions and I've also sent a couple more meaner
indirec
From: Ian Romanick
builtin_variables.cpp:1062:53: warning: unused parameter 'name_as_gs_input'
[-Wunused-parameter]
const char *name_as_gs_input)
^
builtin_functions.cpp:4774:47: warning: unused paramet
I was actually going to suggest loading from the sysconfig dir instead
of /etc a while back, but this works for me too. I haven't looked over
the details, but the idea of this series is
Acked-by: Ilia Mirkin
On Tue, Sep 1, 2015 at 8:26 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> People are
From: Marek Olšák
A user can be using Mesa 11.0, but /etc/drirc can be from Mesa 10.5.
We don't want the old drirc to affect Mesa 11.0.
There are 2 options:
- use a different file name (e.g. /etc/drirc_global) for people wanting
a global drirc file, but they must supply it by themselves
- just
From: Marek Olšák
People are having issues with apps because drirc wasn't installed
into /etc. I've lost patience.
---
src/mesa/drivers/dri/common/Makefile.am | 4 +-
src/mesa/drivers/dri/common/Makefile.sources | 3 +-
src/mesa/drivers/dri/common/drirc| 84
On 2 September 2015 at 09:58, Brian Paul wrote:
> On 09/01/2015 05:39 PM, Dave Airlie wrote:
>>
>> From: Dave Airlie
>>
>> GL 3.2 introduced new language in this area, and CTS enforces it,
>> this patches checks all the vertex buffers and the index buffer
>> are mapped in the core profile case. I
On 09/01/2015 05:39 PM, Dave Airlie wrote:
From: Dave Airlie
GL 3.2 introduced new language in this area, and CTS enforces it,
this patches checks all the vertex buffers and the index buffer
are mapped in the core profile case. I'm not sure what GLES
s/mapped/unmapped/ ?
expects here.
Thi
On Tue, Sep 1, 2015 at 7:50 PM, Dave Airlie wrote:
> On 2 September 2015 at 09:42, Ilia Mirkin wrote:
>> On Tue, Sep 1, 2015 at 7:39 PM, Dave Airlie wrote:
>>> From: Dave Airlie
>>>
>>> GL 3.2 introduced new language in this area, and CTS enforces it,
>>> this patches checks all the vertex buff
On 2 September 2015 at 09:42, Ilia Mirkin wrote:
> On Tue, Sep 1, 2015 at 7:39 PM, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> GL 3.2 introduced new language in this area, and CTS enforces it,
>> this patches checks all the vertex buffers and the index buffer
>> are mapped in the core profile
On Tue, Sep 1, 2015 at 7:39 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> GL 3.2 introduced new language in this area, and CTS enforces it,
> this patches checks all the vertex buffers and the index buffer
> are mapped in the core profile case. I'm not sure what GLES
> expects here.
>
> This fix
From: Dave Airlie
GL 3.2 introduced new language in this area, and CTS enforces it,
this patches checks all the vertex buffers and the index buffer
are mapped in the core profile case. I'm not sure what GLES
expects here.
This fixes
GL33-CTS.draw_elements_base_vertex_tests.invalid_mapped_bos
Si
On 2 September 2015 at 08:57, Matt Turner wrote:
> This reverts commit ad208d975a6d3aebe14f7c2c16039ee200d8b30c.
>
> This code attempted to ensure that we didn't use more than the maximum
> number of attribute slots, but it did not consider attribute aliasing so
> it would miscount slots used by a
On 2 September 2015 at 08:57, Matt Turner wrote:
> ---
> I checked the uses of count_attribute_slots() and it looks like they're
> expecting this already, but these two patches definitely need testing on
> a driver that supports fp64.
>
Don't think so,
As Ilia said they can count as storage for 2
On Tue, Sep 1, 2015 at 6:53 PM, Matt Turner wrote:
> On Tue, Sep 1, 2015 at 3:57 PM, Matt Turner wrote:
>> ---
>> I checked the uses of count_attribute_slots() and it looks like they're
>> expecting this already, but these two patches definitely need testing on
>> a driver that supports fp64.
>
>
I'm all for TGIS simplification, but I just checked, and we rely on TGSI
predicates for some of our internal state trackers. So I'll need more
time to evaluate how much effort it would be to not rely on it, and when
can it be done.
Jose
On 01/09/15 22:19, Marek Olšák wrote:
From: Marek Olšá
On Tue, Sep 1, 2015 at 3:57 PM, Matt Turner wrote:
> ---
> I checked the uses of count_attribute_slots() and it looks like they're
> expecting this already, but these two patches definitely need testing on
> a driver that supports fp64.
Whoops, assuming things go well this should also be Cc'd to
---
I checked the uses of count_attribute_slots() and it looks like they're
expecting this already, but these two patches definitely need testing on
a driver that supports fp64.
src/glsl/glsl_types.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/glsl/glsl_types.cpp
This reverts commit ad208d975a6d3aebe14f7c2c16039ee200d8b30c.
This code attempted to ensure that we didn't use more than the maximum
number of attribute slots, but it did not consider attribute aliasing so
it would miscount slots used by aliased attributes and incorrectly
reject valid shaders.
Wo
https://bugs.freedesktop.org/show_bug.cgi?id=91840
--- Comment #2 from Timothy Arceri ---
There are patches on the list not sure if they still apply:
http://lists.freedesktop.org/archives/mesa-dev/2014-August/066941.html
The piglit tests have already been updated to support ES, this was done at
Hi Ian,
On 1 September 2015 at 19:30, Ian Romanick wrote:
> On 07/17/2015 10:29 AM, Emil Velikov wrote:
>> Signed-off-by: Emil Velikov
>> ---
>> src/gallium/auxiliary/Makefile.am | 29 +
>> 1 file changed, 17 insertions(+), 12 deletions(-)
>>
>> diff --git a/src/gall
On 09/01/2015 11:17 AM, Chris Wilson wrote:
> On Tue, Sep 01, 2015 at 06:40:39PM +0100, Neil Roberts wrote:
>> It's legal to call glTexSubImage with zero values for the width,
>> height or depth. Previously this was breaking the PBO access
>> validation because it tries to work out the last pixel a
https://bugs.freedesktop.org/show_bug.cgi?id=91840
--- Comment #1 from Ian Romanick ---
I think the main effort would be in creating piglit tests. Someone would also
need to run the Khronos conformance tests. Either I or someone at Intel should
be able to do that once patches are available to e
Hi,
There is predication in direct3D9 HLSL,
however we don't implement them and never had any complaint about that.
Reason is that HLSL Microsoft compiler doesn't generate any predication.
I read somewhere they did this choice because for drivers it is possible
to lower efficiently if conditions
From: Marek Olšák
Neved used.
---
Take 2. Let's see how people feel about TGSI now.
src/gallium/auxiliary/gallivm/lp_bld_limits.h | 4 -
src/gallium/auxiliary/gallivm/lp_bld_tgsi.h| 2 -
src/gallium/auxiliary/gallivm/lp_bld_tgsi_aos.c| 46 ---
src/gallium/auxiliary/gal
From: Marek Olšák
Just a cleanup I had made a long time ago and forgot about.
---
src/gallium/drivers/radeonsi/si_shader.c | 468 +--
1 file changed, 191 insertions(+), 277 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/rade
https://bugs.freedesktop.org/show_bug.cgi?id=91747
--- Comment #3 from Brian Paul ---
Thanks to Sinclair Yeh's detective work, it appears that the unity shell is
disabling transparency when it finds "LLVM" in the GL_RENDERER string.
We've seen the same issue here with our latest VMware driver wh
Please stop using a non-existent email address. alterapraxis.com doesn't exist.
Marek
On Tue, Sep 1, 2015 at 10:38 AM, Edward O'Callaghan
wrote:
> From: Edward O'Callaghan
>
> Signed-off-by: Edward O'Callaghan
> ---
> src/gallium/drivers/r600/r600_shader.c | 8 ++--
> 1 file changed, 2 in
Pushed, thanks.
Marek
On Tue, Sep 1, 2015 at 10:38 AM, Edward O'Callaghan
wrote:
> From: Edward O'Callaghan
>
> Signed-off-by: Edward O'Callaghan
> ---
> src/gallium/drivers/r600/r600_shader.c | 8 ++--
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/src/gallium/drivers
On Tue, Sep 1, 2015 at 3:45 PM, Eirik Byrkjeflot Anonsen
wrote:
> Ilia Mirkin writes:
>
>> On Tue, Sep 1, 2015 at 12:15 PM, Ian Romanick wrote:
>>> For a bunch of the small changes, I don't care too much what the
>>> difference is. I just want to know whether after is better than before.
>>
>>
Ilia Mirkin writes:
> On Tue, Sep 1, 2015 at 12:15 PM, Ian Romanick wrote:
>> For a bunch of the small changes, I don't care too much what the
>> difference is. I just want to know whether after is better than before.
>
> And that gets back to my comment that you can't *measure* the impact
> of
From: Marek Olšák
Cc: 11.0
---
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
index 50c42e3..fe55dc3 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
++
From: Marek Olšák
This must be done before exporting a buffer as dmabuf fds, because
we lose track of who is using it and can't trust the reference counter.
Cc: 11.0
---
src/gallium/auxiliary/pipebuffer/pb_bufmgr.h | 5 +++
src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c | 42 +++
From: Marek Olšák
Cc: 11.0
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
b/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
index 600ced9..2878c8f 100644
--- a/src/gallium/winsys/radeon/drm/ra
Matt Turner writes:
> On Tue, Sep 1, 2015 at 10:10 AM, Albert Freeman
> wrote:
>> Clang tryed to declare the non type member of struct module (enum type type)
>> (in clover/core/module.hpp) instead of a variable of type enum (enum type).
>>
>> Signed-off-by: Albert Freeman
>> ---
>
> Thanks fo
https://bugs.freedesktop.org/show_bug.cgi?id=91840
Jose Fonseca changed:
What|Removed |Added
CC||jfons...@vmware.com
--
You are receiving
On 07/17/2015 10:29 AM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/gallium/auxiliary/Makefile.am | 29 +
> 1 file changed, 17 insertions(+), 12 deletions(-)
>
> diff --git a/src/gallium/auxiliary/Makefile.am
> b/src/gallium/auxiliary/Makefile.am
>
On Tue, Sep 1, 2015 at 10:10 AM, Albert Freeman
wrote:
> Clang tryed to declare the non type member of struct module (enum type type)
> (in clover/core/module.hpp) instead of a variable of type enum (enum type).
>
> Signed-off-by: Albert Freeman
> ---
Thanks for your first patch :)
The subject
On Tue, Sep 01, 2015 at 06:40:39PM +0100, Neil Roberts wrote:
> It's legal to call glTexSubImage with zero values for the width,
> height or depth. Previously this was breaking the PBO access
> validation because it tries to work out the last pixel accessed by
> getting the pixel at height-1 and de
On Tue, Sep 1, 2015 at 4:58 AM, Julien Isorce wrote:
> Hi,
>
> Thx for the comments. I forgot to mention that there is nothing really new
> in these patches.
> All the low level code relative to nouveau was already there.
> The patches are almost "just" moving code in order to call nouveau_vp3_bsp
On Tue, Sep 1, 2015 at 8:02 AM, Alejandro Piñeiro wrote:
> The src_reg constructor that received the glsl_type was using it
> only to build the swizzle, but not to fill this->type as dst_reg
> is doing.
>
> This caused some type mismatch between movs and alu operations
> on the NIR path, so copy p
On Tue, Sep 1, 2015 at 1:40 PM, Neil Roberts wrote:
> It's legal to call glTexSubImage with zero values for the width,
> height or depth. Previously this was breaking the PBO access
> validation because it tries to work out the last pixel accessed by
> getting the pixel at height-1 and depth-1 whi
On Tue, Sep 1, 2015 at 6:41 AM, Emil Velikov wrote:
> If you'd like some objective justification why these patches make
> things better, please give me some merits that I can check against.
As I've stated... quite a few times now, I don't like that the series
removes the AM_V_LEX/AM_V_YACC when t
It's legal to call glTexSubImage with zero values for the width,
height or depth. Previously this was breaking the PBO access
validation because it tries to work out the last pixel accessed by
getting the pixel at height-1 and depth-1 which would end up with
bogus values.
This was causing GL error
For 5&6, I didn't check every tiny detail, but looks good to me.
Reviewed-by: Brian Paul
On 08/25/2015 07:14 PM, Dave Airlie wrote:
From: Dave Airlie
Add enough 8-bit channel formats to handle all the
different things CTS throws at us.
Signed-off-by: Dave Airlie
---
src/mesa/main/formats
2015-09-01 18:55 GMT+02:00 Matt Turner :
> On Tue, Sep 1, 2015 at 9:42 AM, Andreas Boll
> wrote:
>> Hi Matt,
>>
>> 2015-08-29 1:09 GMT+02:00 Matt Turner :
>>> The remnants of the extension were removed from Mesa in commit 7a58262e.
>>>
>>> Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=555186
I looked at the change that added this (97185bf2), and there was no
explanation why it was done this way... and the rest of the
MAPI_MODE_BRIDGE stuff seems fairly nuts. This change, however, seems
reasonable.
Reviewed-by: Ian Romanick
On 08/28/2015 11:47 AM, Matt Turner wrote:
> ---
> Here's a
On Tue, Sep 1, 2015 at 9:42 AM, Andreas Boll wrote:
> Hi Matt,
>
> 2015-08-29 1:09 GMT+02:00 Matt Turner :
>> The remnants of the extension were removed from Mesa in commit 7a58262e.
>>
>> Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=555186
>> Bugzilla: https://bugs.freedesktop.org/show_bug.c
On Tue, Sep 01, 2015 at 05:44:30PM +0100, Neil Roberts wrote:
> Looks good to me.
>
> Chris Wilson writes:
>
> > @@ -324,10 +340,9 @@ _mesa_meta_pbo_GetTexSubImage(struct gl_context *ctx,
> > GLuint dims,
> > * property.
> > */
> > image_height = packing->ImageHeight == 0 ? height
On Mon, Aug 10, 2015 at 5:44 AM, Michel Dänzer wrote:
> From: Michel Dänzer
>
> Fixes Gallium based DRI drivers failing to load on big endian hosts
> because they can't find any matching fbconfigs.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71789
> Signed-off-by: Michel Dänzer
>
Looks good to me.
Chris Wilson writes:
> @@ -324,10 +340,9 @@ _mesa_meta_pbo_GetTexSubImage(struct gl_context *ctx,
> GLuint dims,
> * property.
> */
> image_height = packing->ImageHeight == 0 ? height : packing->ImageHeight;
> - full_height = image_height * (depth - 1) + height
Hi Matt,
2015-08-29 1:09 GMT+02:00 Matt Turner :
> The remnants of the extension were removed from Mesa in commit 7a58262e.
>
> Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=555186
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91020
> Bugzilla: https://bugs.freedesktop.org/show_bug.
On Tue, Sep 1, 2015 at 12:15 PM, Ian Romanick wrote:
> For a bunch of the small changes, I don't care too much what the
> difference is. I just want to know whether after is better than before.
And that gets back to my comment that you can't *measure* the impact
of a change. Not with something w
On 08/31/2015 11:25 PM, Ilia Mirkin wrote:
> On Tue, Sep 1, 2015 at 1:48 AM, Eirik Byrkjeflot Anonsen
> wrote:
>> Ian Romanick writes:
>>
>>> ping. :)
>>>
>>> On 08/10/2015 11:48 AM, Matt Turner wrote:
On Mon, Aug 10, 2015 at 10:12 AM, Ian Romanick
wrote:
> From: Ian Romanick
>>>
If the user is specifying a subregion of a buffer using SKIP_ROWS and
SKIP_PIXELS, we must compute the buffer size carefully as the end of the
last row may be much shorter than stride*image_height*depth. The current
code tries to memcpy from beyond the end of the user data, for example
causing:
==
https://bugs.freedesktop.org/show_bug.cgi?id=91840
Bug ID: 91840
Summary: Expose GL_KHR_debug to ES contexts
Product: Mesa
Version: git
Hardware: All
URL: https://www.opengl.org/registry/specs/KHR/debug.txt
OS: A
https://bugs.freedesktop.org/show_bug.cgi?id=37397
Link Mauve changed:
What|Removed |Added
Depends on||91840
--
You are receiving this mail becau
The src_reg constructor that received the glsl_type was using it
only to build the swizzle, but not to fill this->type as dst_reg
is doing.
This caused some type mismatch between movs and alu operations
on the NIR path, so copy propagation optimization was not applied
to remove unneeded movs if ne
On Tue, Sep 1, 2015 at 5:30 AM, Chris Wilson wrote:
> On Tue, Sep 01, 2015 at 01:09:33PM +0100, Neil Roberts wrote:
>> Good catch and it seems like a nice way to fix it.
>>
>> Reviewed-by: Neil Roberts
>>
>> I wonder if it might be worth avoiding copying the padding and pack the
>> rows more tigh
On Tue, 2015-09-01 at 16:41 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> The fastpath currently checks the stride != width, but
Maybe replace stride with RowLength in the line above to make things
more clear.
> if you have a RowLength of 7, and Alignment of 4, then
> that shuoldn't match.
On 08/31/2015 10:50 PM, Dave Airlie wrote:
This case just wasn't handled, so add support for it.
Signed-off-by: Dave Airlie
---
src/mesa/main/texgetimage.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c
index 0c23687..52ed
Yeah, this looks better. Just minor nit-picks below.
Otherwise, Reviewed-by: Brian Paul
On 08/31/2015 10:50 PM, Dave Airlie wrote:
From: Dave Airlie
In a number of places the SwapBytes handling
didn't handle cases with GL_(UN)PACK_ALIGNMENT
set and 7 byte width cases aligned to 8 bytes.
T
On Tue, Sep 01, 2015 at 02:48:20PM +0100, Emil Velikov wrote:
> On 1 August 2015 at 17:57, Emil Velikov wrote:
> > Hello all,
> >
> > On 20 July 2015 at 18:08, Anuj Phogat wrote:
> >> On Sat, Jul 18, 2015 at 1:24 AM, Chris Wilson
> >> wrote:
> >>> On Fri, Jul 17, 2015 at 05:12:54PM -0700, Anuj
On 08/31/2015 03:57 PM, Jason Ekstrand wrote:
It is a shader enum after all...
---
src/glsl/shader_enums.h | 108
src/mesa/main/mtypes.h | 107 ---
2 files changed, 108 insertions(+), 107 deletions(-
On 26 August 2015 at 12:52, Boyan Ding wrote:
> This avoids generation of undefined packing in qir and qpu instructions,
> fixing a lot of rendering errors.
>
> Fixes 8b36d107fdd (vc4: Pack the unorm-packing bits into a src MUL
> instruction when possible.)
>
> Cc: mesa-sta...@lists.freedesktop.or
Francisco Jerez writes:
> The hardware documentation relating to the UAV HW-assisted coherency
> mechanism and UAV access enable bits is scarce and sometimes
> contradictory, and there's quite some guesswork behind this commit, so
> let me summarize the background first: HSW and later hardware ha
Reviewed-by: Iago Toral Quiroga
On Tue, 2015-09-01 at 16:41 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> The CTS packed_pixels test checks that readpixels doesn't write
> into the space between rows, however we fail that here unless
> we check the format and stride match.
> This fixes all
Hi all
On 21 August 2015 at 23:04, Anuj Phogat wrote:
> We have a similar check in meta pbo path.
>
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/intel_pixel_read.c | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_pixel_read.c
On 09/01/2015 03:49 PM, Emil Velikov wrote:
> Hi all,
>
> On 1 August 2015 at 20:40, Thomas Hellstrom wrote:
>> Hi!
>>
>> On 08/01/2015 07:16 PM, Emil Velikov wrote:
>>> On 22 July 2015 at 00:00, Rob Clark wrote:
From: Rob Clark
Like xa_surface_from_handle(), but takes a handle ty
On 8 August 2015 at 12:11, Francisco Jerez wrote:
> Tom Stellard writes:
>
>> Calls to LLVMIntialize* fail when we are linking against individual
>> component libraries rather than one large shared object, because
>> we only include component libraries that are required by the drivers.
>>
>> We n
Reviewed-by: Iago Toral Quiroga
On Tue, 2015-09-01 at 16:41 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> We don't need to use the 3d image address here as that will
> include SKIP_IMAGES, and we are only blitting a single
> 2D anyways, so just use the 2D path.
>
> This fixes some memory o
Hi all,
On 1 August 2015 at 20:40, Thomas Hellstrom wrote:
> Hi!
>
> On 08/01/2015 07:16 PM, Emil Velikov wrote:
>> On 22 July 2015 at 00:00, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> Like xa_surface_from_handle(), but takes a handle type, rather than
>>> hard-coding 'shared' handle. This i
On 1 August 2015 at 17:57, Emil Velikov wrote:
> Hello all,
>
> On 20 July 2015 at 18:08, Anuj Phogat wrote:
>> On Sat, Jul 18, 2015 at 1:24 AM, Chris Wilson
>> wrote:
>>> On Fri, Jul 17, 2015 at 05:12:54PM -0700, Anuj Phogat wrote:
On Mon, Jul 6, 2015 at 3:33 AM, Chris Wilson
wrote
On 29 July 2015 at 13:46, Timothy Arceri wrote:
> On Wed, 2015-07-29 at 09:57 +0200, Iago Toral wrote:
>> On Sun, 2015-07-26 at 18:35 +1000, Timothy Arceri wrote:
>> > Since commit c0cd5b var->data.binding was being used as a replacement
>> > for atomic buffer index, but they don't have to be the
Hi guys
On 11 July 2015 at 11:47, Francisco Jerez wrote:
> Tom Stellard writes:
>
>> Events can be added to an OpenCL command queue concurrently from multiple
>> threads, but pipe_context bjects are not threadsafe. The threadsafe
>> wrappers protect all pipe_context function calls with a mutex,
On 21 August 2015 at 19:09, Emil Velikov wrote:
> On 03/08/15 19:09, Emil Velikov wrote:
>> On 3 August 2015 at 17:17, Matt Turner wrote:
>>> On Fri, Jul 17, 2015 at 11:53 AM, Emil Velikov
>>> wrote:
On 17 July 2015 at 19:09, Matt Turner wrote:
> On Fri, Jul 17, 2015 at 10:29 AM, Emil
On 09/01/2015 06:04 AM, Tapani Pälli wrote:
On 08/31/2015 05:56 PM, Brian Paul wrote:
Looks good. Just some minor nitpicks below.
Reviewed-by: Brian Paul
Thanks! I have one comment below ..
On 08/31/2015 01:23 AM, Tapani Pälli wrote:
Patch modifies existing shader source and replace f
On 31 August 2015 at 13:32, Emil Velikov wrote:
> On 30 August 2015 at 00:21, Matt Turner wrote:
>> Hi Emil,
>>
>> src/gallium/targets/vdpau has a block that installs per-driver
>> hardlinks, but src/gallium/targets/va does not (presumably because it
>> was added later), which leads to:
>>
>> htt
On Tue, Sep 01, 2015 at 01:09:33PM +0100, Neil Roberts wrote:
> Good catch and it seems like a nice way to fix it.
>
> Reviewed-by: Neil Roberts
>
> I wonder if it might be worth avoiding copying the padding and pack the
> rows more tightly in the temporary buffer. Ie, we would allocate a
> buff
Good catch and it seems like a nice way to fix it.
Reviewed-by: Neil Roberts
I wonder if it might be worth avoiding copying the padding and pack the
rows more tightly in the temporary buffer. Ie, we would allocate a
buffer of align(width*cpp)*height*depth and copy the rows in one at a
time inste
On 08/31/2015 05:56 PM, Brian Paul wrote:
Looks good. Just some minor nitpicks below.
Reviewed-by: Brian Paul
Thanks! I have one comment below ..
On 08/31/2015 01:23 AM, Tapani Pälli wrote:
Patch modifies existing shader source and replace functionality to work
with environment variabl
Specified in OpenGL ES 3.1 spec, Table 23.32: Program Object State.
Signed-off-by: Tapani Pälli
---
src/mesa/main/get_hash_params.py | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_params.py
index dc5ba6f..e2e3d0
From OpenGL 4.5 Core spec (7.13):
"If pipeline is a name that has been generated (without subsequent
deletion) by GenProgramPipelines, but refers to a program pipeline
object that has not been previously bound, the GL first creates a
new state vector in the same manner as when Bind
I haven't found clear spec evidence of this behaviour but this is
expected by a conformance test that changes the value with
glProgramParameteri but does not link the program. Test says:
"The query for PROGRAM_SEPARABLE must query latched state. In other
words, the state of the binary after it was
On 1 September 2015 at 20:34, Iago Toral wrote:
> Hey,
>
> On Tue, 2015-08-18 at 12:52 +1000, Dave Airlie wrote:
>> Hey,
>>
>> while running CTS under valgrind I got to see a lot of
>>
>> ==32256== Invalid read of size 2
>> ==32256==at 0x5B53F07: convert_ushort (format_utils.c:1155)
>> ==32256
On Tue, 2015-09-01 at 12:34 +0200, Iago Toral wrote:
> Hey,
>
> On Tue, 2015-08-18 at 12:52 +1000, Dave Airlie wrote:
> > Hey,
> >
> > while running CTS under valgrind I got to see a lot of
> >
> > ==32256== Invalid read of size 2
> > ==32256==at 0x5B53F07: convert_ushort (format_utils.c:115
Hey,
On Tue, 2015-08-18 at 12:52 +1000, Dave Airlie wrote:
> Hey,
>
> while running CTS under valgrind I got to see a lot of
>
> ==32256== Invalid read of size 2
> ==32256==at 0x5B53F07: convert_ushort (format_utils.c:1155)
> ==32256==by 0x5B8523A: _mesa_swizzle_and_convert (format_utils
https://bugs.freedesktop.org/show_bug.cgi?id=91826
Albert Freeman changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=91826
--- Comment #6 from Albert Freeman ---
Yeah, I don't know too much about C++11 but it MIGHT be a bug in clang.
On 1 September 2015 at 09:30, wrote:
> Comment # 5 on bug 91826 from Koop Mast
>
> The patch from #3 fixes the build with clang, tha
Commit 2126c68e5cba killed the array elements parameter on load/store
intrinsics that was stored in const_index[1]. It looks like that
patch missed to remove this assignment in the UBO path.
---
src/glsl/nir/glsl_to_nir.cpp | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/glsl/nir/glsl_to_ni
https://bugs.freedesktop.org/show_bug.cgi?id=91826
--- Comment #5 from Koop Mast ---
The patch from #3 fixes the build with clang, thanks.
Report what to clang upstream, that they error on this issue versus gcc which
only gives a warning?
--
You are receiving this mail because:
You are the QA
Hi,
Thx for the comments. I forgot to mention that there is nothing really new
in these patches.
All the low level code relative to nouveau was already there.
The patches are almost "just" moving code in order to call nouveau_vp3_bsp
and nvc0_decoder_bsp
multiple times between each begin/end frame
1 - 100 of 108 matches
Mail list logo
