On Thu, 2016-03-03 at 09:38 +1100, Timothy Arceri wrote:
> On Mon, 2015-09-14 at 13:49 +0200, Iago Toral Quiroga wrote:
> > When we find indirect indexing into an array, the current
> > implementation
> > of the array spliiting optimization pass does not look further into
> > the
> > expression tre
On Wed, Mar 2, 2016 at 11:48 PM, Marek Olšák wrote:
> On Wed, Mar 2, 2016 at 8:22 PM, Oded Gabbay wrote:
>> There is an old if statement (dated to 2011) that prevented doing
>> endian swap for colorformat, in case the buffer is marked
>> as PIPE_USAGE_STAGING.
>>
>> This is now wrong because st_R
On Wednesday, March 2, 2016 3:45:57 PM PST Matt Turner wrote:
> Two shaders that appear in Unigine benchmarks (Heaven and Valley) unpack
> three bytes from an integer and convert each into a float:
>
>float((val >> 16u) & 0xffu)
>float((val >> 8u) & 0xffu)
>float((val >> 0u) & 0xffu)
This resolves some order dependencies between the already existing
callback the newly created one.
---
src/gallium/state_trackers/glx/xlib/glx_api.c | 1 +
src/gallium/state_trackers/glx/xlib/xm_api.c | 58 +++
src/gallium/state_trackers/glx/xlib/xm_api.h | 3 ++
3 file
There is a limit of 10 display connections, which was a
problem for apps/tests that were continuously opening/closing display
connections.
This fix uses XAddExtension() and XESetCloseDisplay() to keep track
of the status of the display connections from the X server, freeing
mesa-related data as X
I am re-sending an updated patch that fixes this stomping, however, there is an
additional issue
that is exposed in programs that use multiple windows (for example manywin).
The issue is that
resources are shared between contexts, so when we destroy context 2, it tries
to free resources
that al
No shader-db changes, but does recognize some extract_u16 which enables
the next patch to optimize some code.
---
src/compiler/nir/nir_opt_algebraic.py | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/compiler/nir/nir_opt_algebraic.py
b/src/compiler/nir/nir_opt_algebraic.py
index dd26
Two shaders that appear in Unigine benchmarks (Heaven and Valley) unpack
three bytes from an integer and convert each into a float:
float((val >> 16u) & 0xffu)
float((val >> 8u) & 0xffu)
float((val >> 0u) & 0xffu)
Instead of shifting, masking, and type converting like this:
shr(8)
instructions in affected programs: 31535 -> 29966 (-4.98%)
helped: 23
cycles in affected programs: 272648 -> 266022 (-2.43%)
helped: 14
HURT: 1
The patch decreases the number of instructions in the two Unigine
programs by:
#1721: 4374 -> 4155 instructions (-5.01%)
#1706: 3582 ->
On Mon, 2015-09-14 at 13:49 +0200, Iago Toral Quiroga wrote:
> When we find indirect indexing into an array, the current
> implementation
> of the array spliiting optimization pass does not look further into
> the
> expression tree. However, if the variable expression involves
> variable
> indexing
Reviewed-by: Marek Olšák
Marek
On Wed, Mar 2, 2016 at 8:22 PM, Oded Gabbay wrote:
> There is an old if statement (dated to 2011) that prevented doing
> endian swap for colorformat, in case the buffer is marked as
> PIPE_USAGE_STAGING.
>
> This is now wrong because st_ReadPixels() reads into a d
On Wed, Mar 2, 2016 at 8:22 PM, Oded Gabbay wrote:
> There is an old if statement (dated to 2011) that prevented doing
> endian swap for colorformat, in case the buffer is marked
> as PIPE_USAGE_STAGING.
>
> This is now wrong because st_ReadPixels() reads into a destination
> texture that is marke
On Wed, Mar 2, 2016 at 1:46 PM, Jason Ekstrand wrote:
> On Wed, Mar 2, 2016 at 9:03 AM, Sedat Dilek wrote:
>
>> Any news on that?
>>
>
> Out-of-tree builds including make check should be working now.
> --Jason
>
Confirmed. I just did a pull/rebuild out of tree, and make check works
again.
--A
On 2 March 2016 at 22:06, Eduardo Lima Mitev wrote:
> From: Antia Puentes
Looks good to me,
Reviewed-by: Dave Airlie
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Wed, Mar 2, 2016 at 9:03 AM, Sedat Dilek wrote:
> Any news on that?
>
Out-of-tree builds including make check should be working now.
--Jason
>
> - Sedat -
>
> On 2/24/16, Aaron Watry wrote:
> > To clarify, I've also been doing out-of-tree builds with make check and a
> > vulkan build (on a
There is an old if statement (dated to 2011) that prevented doing
endian swap for colorformat, in case the buffer is marked
as PIPE_USAGE_STAGING.
This is now wrong because st_ReadPixels() reads into a destination
texture that is marked with PIPE_USAGE_STAGING. Therefore, even if
the texture is re
There is an old if statement (dated to 2011) that prevented doing
endian swap for colorformat, in case the buffer is marked as
PIPE_USAGE_STAGING.
This is now wrong because st_ReadPixels() reads into a destination
texture that is marked with PIPE_USAGE_STAGING. Therefore, even if
the texture is re
On Wed, Mar 2, 2016 at 2:09 PM, Emil Velikov wrote:
> On 2 March 2016 at 17:19, Alex Deucher wrote:
>> On Wed, Mar 2, 2016 at 11:48 AM, Emil Velikov
>> wrote:
>>> Just like everywhere else in the radeon codebase.
>>>
>>> Cc: Marek Olšák
>>> Signed-off-by: Emil Velikov
>>
>> NACK, amdgpu has m
Reviewed-by: Ilia Mirkin
On Wed, Mar 2, 2016 at 12:11 PM, Samuel Pitoiset
wrote:
> Spotted by Coverity.
>
> Signed-off-by: Samuel Pitoiset
> Reported-by: Matt Turner
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
On 2 March 2016 at 17:19, Alex Deucher wrote:
> On Wed, Mar 2, 2016 at 11:48 AM, Emil Velikov
> wrote:
>> Just like everywhere else in the radeon codebase.
>>
>> Cc: Marek Olšák
>> Signed-off-by: Emil Velikov
>
> NACK, amdgpu has major version 3.
>
Silly me confused use_vui with use_vm.
I tak
Any news on that?
- Sedat -
On 2/24/16, Aaron Watry wrote:
> To clarify, I've also been doing out-of-tree builds with make check and a
> vulkan build (on a haswell system, but I don't think that matters at build
> time).
>
> On Tue, Feb 23, 2016 at 8:46 PM, Aaron Watry wrote:
>
>> To expand on
We are seeking nominations for candidates for election to the X.Org
Foundation Board of Directors. All X.Org Foundation members are
eligible for election to the board.
Nominations for the 2016 election are now open and will remain open
until 23:59 UTC on 15 March 2016.
The Board consists of dire
On 03/02/2016 10:33 AM, Kyriazis, George wrote:
Brian,
I looked for a trivial/tri test in the mesa repo and I only found the
gallium/trivial/tri test. This one I found is loading the device pipes
directly and is not using glx so it's not affected by my change. It shows some
valgrind memory
Brian,
I looked for a trivial/tri test in the mesa repo and I only found the
gallium/trivial/tri test. This one I found is loading the device pipes
directly and is not using glx so it's not affected by my change. It shows some
valgrind memory leaks upon exit regardless of my change.
Or maybe
On 03/01/2016 05:12 PM, Rodrigo Vivi wrote:
These IDs were already part of the kernel since:
kernel commit 985dd4360fdf2533fe48a33a4a2094f2e4718dc0
Author: Imre Deak
Date: Thu Jan 28 16:04:12 2016 +0200
drm/i915/bxt: update list of PCIIDs
Signed-off-by: Rodrigo Vivi
Cc: Venkateswarlu
On Wed, Mar 2, 2016 at 11:48 AM, Emil Velikov wrote:
> Just like everywhere else in the radeon codebase.
>
> Cc: Marek Olšák
> Signed-off-by: Emil Velikov
NACK, amdgpu has major version 3.
> ---
>
> Spotted while grepping for drm_minor - i.e. which version will be used
> for the next round.
>
Spotted by Coverity.
Signed-off-by: Samuel Pitoiset
Reported-by: Matt Turner
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
b/src/gallium/drivers/nouve
Just like everywhere else in the radeon codebase.
Cc: Marek Olšák
Signed-off-by: Emil Velikov
---
Spotted while grepping for drm_minor - i.e. which version will be used
for the next round.
-Emil
src/gallium/drivers/radeon/radeon_vce.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
d
From: Nicolai Hähnle
This will allow it to be re-used for shader image descriptors.
---
src/gallium/drivers/radeonsi/si_state.c | 73 ++---
1 file changed, 48 insertions(+), 25 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/r
From: Marek Olšák
and rename .._buffers -> .._buffer
Based loosely on Nicolai's patch. This will make it easier to cherry-pick
Nicolai's patches from his image support branch.
---
src/gallium/drivers/radeonsi/si_descriptors.c | 23 ---
1 file changed, 12 insertions(+), 11 de
From: Marek Olšák
---
src/gallium/drivers/r600/r600_pipe.c | 2 +-
src/gallium/drivers/radeon/r600_pipe_common.h | 6 +++
src/gallium/drivers/radeon/r600_texture.c | 4 ++
src/gallium/drivers/radeonsi/si_pipe.c| 4 +-
src/gallium/drivers/radeonsi/si_state.c | 63 ++
From: Nicolai Hähnle
view->resource is redundant with view->base.texture, so get rid of it.
---
src/gallium/drivers/radeonsi/si_descriptors.c | 2 +-
src/gallium/drivers/radeonsi/si_pipe.h| 1 -
src/gallium/drivers/radeonsi/si_state.c | 3 +--
3 files changed, 2 insertions(+), 4 de
From: Marek Olšák
This will be queried by the OpenCL stack using an interop call.
I have tested that the values match lspci.
---
src/gallium/drivers/radeon/radeon_winsys.h| 6 ++
src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 17 +++--
2 files changed, 21 insertions(+), 2
From: Marek Olšák
This should be okay except that sampler views and images are not re-set.
---
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +++
src/gallium/drivers/radeon/r600_texture.c | 33 +++
src/gallium/drivers/radeonsi/si_blit.c| 12 ++
3
From: Nicolai Hähnle
This will allow this code to be re-used for shader images.
---
src/gallium/drivers/radeonsi/si_state.c | 350 +---
1 file changed, 186 insertions(+), 164 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/rad
From: Marek Olšák
To allow sharing textures with DCC enabled.
---
src/gallium/drivers/radeon/r600_pipe_common.h | 2 +-
src/gallium/drivers/radeon/r600_texture.c | 51 ---
src/gallium/drivers/radeonsi/cik_sdma.c | 2 +-
src/gallium/drivers/radeonsi/si_blit.c
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 33f4042..377dd3d 100644
--- a/src/gallium/drivers/radeon
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 8fc236e..21bb3f9 100644
--- a/src/gallium/drivers/radeon/r600_texture.c
+++ b/sr
From: Marek Olšák
---
src/gallium/drivers/radeon/radeon_winsys.h | 10 ++
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 5 +
2 files changed, 15 insertions(+)
diff --git a/src/gallium/drivers/radeon/radeon_winsys.h
b/src/gallium/drivers/radeon/radeon_winsys.h
index 566322c..5ca2414
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 21bb3f9..229fa5e 100644
--- a/src/gallium/drivers/radeon/r600_texture.c
+++ b/s
From: Bas Nieuwenhuizen
This is currently not needed but will be necessary when we have
features that do not work with DCC enabled, such as image stores
and sharing non-scanout surfaces.
v2: Marek: rebase, remove decompression from si_flush_resource (not needed)
---
src/gallium/drivers/radeonsi
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_buffer_common.c | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c
b/src/gallium/drivers/radeon/r600_buffer_common.c
index fb3a80e..70c8eb2 100644
--- a/src/gall
From: Marek Olšák
The disabling of CMASK is simple, but notifying all contexts about it is not:
- The screen must have a list of all contexts.
- Each context must have a monotonic counter that is incremented only when
the screen wants to re-emit framebuffer states.
- Each context must check in
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 16
src/gallium/drivers/radeonsi/si_blit.c| 2 +-
2 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 22
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_buffer_common.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c
b/src/gallium/drivers/radeon/r600_buffer_common.c
index 70c8eb2..33ba0fb 100644
--- a/src/gallium/drivers/radeon/r600_buf
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 5f49dd7..8fc236e 100644
--- a/src/gallium/drivers/radeon/r6
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.h | 1 +
src/gallium/drivers/radeon/r600_texture.c | 3 +++
2 files changed, 4 insertions(+)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h
b/src/gallium/drivers/radeon/r600_pipe_common.h
index 4cef5ef..3bbbfbb 1
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_buffer_common.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c
b/src/gallium/drivers/radeon/r600_buffer_common.c
index 439a3cb..fb3a80e 100644
--- a/src/gallium/drivers/radeon/r600_buffer_
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 38 +++
1 file changed, 23 insertions(+), 15 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 4f18788..33f4042 100644
--- a/src/gal
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_buffer_common.c | 1 +
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +++
src/gallium/drivers/radeon/r600_texture.c | 7 +--
3 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_buffer
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 5e77080..558dc73 100644
--- a/src/gallium/drivers/radeon/r600_te
From: Marek Olšák
---
src/gallium/drivers/r300/r300_texture.c | 4 ++--
src/gallium/drivers/radeon/r600_texture.c | 4 ++--
src/gallium/drivers/radeon/radeon_winsys.h| 8
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 12 ++--
src/gallium/winsys/radeon/drm/rade
From: Marek Olšák
and call it radeon_bo_metadata
---
src/gallium/drivers/r300/r300_texture.c | 22 -
src/gallium/drivers/radeon/r600_texture.c | 52 ++--
src/gallium/drivers/radeon/radeon_winsys.h| 50 +---
src/gallium/winsys/amdgpu/drm/amdgp
Hi,
This patch series contains necessary radeonsi changes in order to support
OpenGL-OpenCL interop. This only covers buffer and texture sharing.
The changes can be summarized to:
- write an image descriptor to amdgpu buffer metadata for OpenCL to use it
- buffer map/unmap optimizations need to
From: Marek Olšák
This was needed for DRM < 2.12.0 where the kernel was rewriting tiling flags
in IBs.
---
src/gallium/drivers/r300/r300_texture.c | 2 +-
src/gallium/drivers/radeon/r600_texture.c | 2 +-
src/gallium/drivers/radeon/radeon_winsys.h| 2 --
src/gallium/winsys/amdgpu/d
On 03/02/2016 04:35 PM, Matt Turner wrote:
On Wed, Feb 3, 2016 at 7:44 AM, Eduardo Lima Mitev wrote:
Use QueryInternalFormat instead of QuerySamplesForFormat to obtain the
highest supported sample. QuerySamplesForFormat is to be removed.
---
src/mesa/main/multisample.c | 10 ++
1 fil
On 03/02/2016 08:46 AM, Marek Olšák wrote:
From: Marek Olšák
---
These will be queried by our OpenCL stack to identify the device.
The radeon code depends on a patch I haven't sent out yet.
src/gallium/docs/source/screen.rst | 4
src/gallium/drivers/freedreno/freedreno_sc
From: Marek Olšák
---
These will be queried by our OpenCL stack to identify the device.
The radeon code depends on a patch I haven't sent out yet.
src/gallium/docs/source/screen.rst | 4
src/gallium/drivers/freedreno/freedreno_screen.c | 4
src/gallium/drivers/i915/i915_
On 03/02/2016 04:32 PM, Matt Turner wrote:
On Sun, Feb 28, 2016 at 11:44 AM, Samuel Pitoiset
wrote:
Changes from v2:
- add missing NOT modifier for GK110/GM107
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir.h | 4 +++
.../drivers/nouveau/codegen/nv
On Wed, Feb 3, 2016 at 7:44 AM, Eduardo Lima Mitev wrote:
> Use QueryInternalFormat instead of QuerySamplesForFormat to obtain the
> highest supported sample. QuerySamplesForFormat is to be removed.
> ---
> src/mesa/main/multisample.c | 10 ++
> 1 file changed, 6 insertions(+), 4 deletion
On Sun, Feb 28, 2016 at 11:44 AM, Samuel Pitoiset
wrote:
> Changes from v2:
> - add missing NOT modifier for GK110/GM107
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir.h | 4 +++
> .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 23
On Wed, 2016-03-02 at 13:21 +0100, Juan A. Suarez Romero wrote:
> opt_vector_float() transforms several scalar MOV operations to a
> single
> vectorial MOV.
>
I had sent this patch a couple of months ago, but seems I lost its
track. So I've rebased it, check it still gets some improvement, and
up
opt_vector_float() transforms several scalar MOV operations to a single
vectorial MOV.
This is done when those MOV covers all the components of the destination
register. So something like:
mov vgrf3.0.xy:D, 0D
mov vgrf3.0.w:D, 1065353216D
mov vgrf3.0.z:D, 0D
is transformed in:
mov vgrf3.0:F, [0
On 03/02/2016 02:14 AM, Dave Airlie wrote:
On 1 March 2016 at 19:04, Eduardo Lima Mitev wrote:
Hi,
I have sent updates for some of the patches, fixing a few dEQP regressions
we found, as well as rebase conflicts.
At this point, we don't have a any piglit or dEQP regression (GLES3 or
GLES31).
On 03/02/2016 01:51 AM, Brian Paul wrote:
On 03/01/2016 05:30 PM, Dave Airlie wrote:
On 4 February 2016 at 01:44, Eduardo Lima Mitev wrote:
Use QueryInternalFormat instead of QuerySamplesForFormat to obtain the
highest supported sample. QuerySamplesForFormat is to be removed.
---
src/mesa/ma
On 03/02/2016 01:27 AM, Dave Airlie wrote:
On 4 February 2016 at 01:44, Eduardo Lima Mitev wrote:
From: Antia Puentes
---
src/mesa/main/extensions_table.h | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions_table.h b/src/m
From: Antia Puentes
Specifically MIPMAP, MANUAL_GENERATE_MIPMAP and AUTO_GENERATE_MIPMAP
queries.
From the ARB_internalformat_query2 specification:
"- MIPMAP: If the resource supports mipmaps, TRUE is returned in .
If the resource is not supported, or if mipmaps are not supported for
When a user defines a point size array and enables it, the point
size value set via glPointSize should be ignored. To achieve this,
we can simply set ctx->VertexProgram.PointSizeEnabled whenever
GL_POINT_SIZE_ARRAY_OES is set so that the right point size is
used.
Bugzilla: https://bugs.freedesktop
https://bugs.freedesktop.org/show_bug.cgi?id=94295
--- Comment #7 from Plamena Manolova ---
(In reply to Vinson Lee from comment #6)
> (In reply to Plamena Manolova from comment #5)
> > Could you please try out the new patch?
>
> #0 find_empty_block (uniform=, prog=) at
> glsl/link_uniforms.cpp
Hi Kenneth,
You're absolutely right, just setting ctx->VertexProgram.PointSizeEnabled
would be much simpler and cleaner, I'll go ahead and update the patch.
Thank you for reviewing!
On Tue, Mar 1, 2016 at 10:07 PM, Kenneth Graunke
wrote:
> On Tuesday, March 1, 2016 6:39:49 PM PST Plamena Manolov
On 01/03/16 16:33, Olivier Galibert wrote:
I can confirm tri/cube work with latest git. Talos Principle refuses
to start because of missing vkCmdBeginQuery, time to jump into the
docs to see how much of gen8 is copy-able there.
OG.
Indeed, I got vkcube to work on HSW. I recompiled everythi
On 02/03/16 02:04, Dave Airlie wrote:
> On 2 March 2016 at 10:49, Dave Airlie wrote:
>> On 1 March 2016 at 19:04, Eduardo Lima Mitev wrote:
>>> Hi,
>>>
>>> I have sent updates for some of the patches, fixing a few dEQP regressions
>>> we found, as well as rebase conflicts.
>>>
>>> At this point
72 matches
Mail list logo