Re: [Mesa-dev] [PATCH 1/2] glsl/opt_array_splitting: Fix crash when doing array indexing into other arrays

2016-03-02 Thread Iago Toral
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

Re: [Mesa-dev] [PATCH 1/2] r600g: Do colorformat endian swap for PIPE_USAGE_STAGING

2016-03-02 Thread Oded Gabbay
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

Re: [Mesa-dev] [PATCH 1/3] nir: Recognize open-coded extract_u8.

2016-03-02 Thread Kenneth Graunke
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)

[Mesa-dev] [PATCH v3 2/2] Hang off screen destructor off main XCloseDisplay() callback.

2016-03-02 Thread George Kyriazis
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

[Mesa-dev] [PATCH v3 1/2] Support unlimited number of display connections

2016-03-02 Thread George Kyriazis
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

Re: [Mesa-dev] [PATCH v2] Support unlimited number of display connections

2016-03-02 Thread Kyriazis, George
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

[Mesa-dev] [PATCH 2/3] nir: Recognize open-coded extract_u16.

2016-03-02 Thread Matt Turner
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

[Mesa-dev] [PATCH 1/3] nir: Recognize open-coded extract_u8.

2016-03-02 Thread Matt Turner
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)

[Mesa-dev] [PATCH 3/3] i965/fs: Optimize float conversions of byte/word extract.

2016-03-02 Thread Matt Turner
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 ->

Re: [Mesa-dev] [PATCH 1/2] glsl/opt_array_splitting: Fix crash when doing array indexing into other arrays

2016-03-02 Thread Timothy Arceri
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

Re: [Mesa-dev] [PATCH 2/2] radeonsi: Do colorformat endian swap for PIPE_USAGE_STAGING

2016-03-02 Thread Marek Olšák
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

Re: [Mesa-dev] [PATCH 1/2] r600g: Do colorformat endian swap for PIPE_USAGE_STAGING

2016-03-02 Thread Marek Olšák
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

Re: [Mesa-dev] [vulkan] Fix AM_CPPFLAGS

2016-03-02 Thread Aaron Watry
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

Re: [Mesa-dev] [PATCH v2 39/65] mesa/formatquery: Added mipmap related queries

2016-03-02 Thread Dave Airlie
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

Re: [Mesa-dev] [vulkan] Fix AM_CPPFLAGS

2016-03-02 Thread Jason Ekstrand
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

[Mesa-dev] [PATCH 1/2] r600g: Do colorformat endian swap for PIPE_USAGE_STAGING

2016-03-02 Thread Oded Gabbay
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

[Mesa-dev] [PATCH 2/2] radeonsi: Do colorformat endian swap for PIPE_USAGE_STAGING

2016-03-02 Thread Oded Gabbay
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

Re: [Mesa-dev] [PATCH] gallium/radeon: use explicit drm_major, drm_minor check

2016-03-02 Thread Alex Deucher
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

Re: [Mesa-dev] [PATCH] gk110/ir: fix wrong emission of NOT modifier for VOTE

2016-03-02 Thread Ilia Mirkin
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(-) > >

Re: [Mesa-dev] [PATCH] gallium/radeon: use explicit drm_major, drm_minor check

2016-03-02 Thread Emil Velikov
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

Re: [Mesa-dev] [vulkan] Fix AM_CPPFLAGS

2016-03-02 Thread Sedat Dilek
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

[Mesa-dev] Reminder: 2016 X.Org Board of Directors Elections Nomination period is NOW

2016-03-02 Thread Peter Hutterer
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

Re: [Mesa-dev] [PATCH v2] Support unlimited number of display connections

2016-03-02 Thread Brian Paul
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

Re: [Mesa-dev] [PATCH v2] Support unlimited number of display connections

2016-03-02 Thread Kyriazis, George
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

Re: [Mesa-dev] [Intel-gfx] [PATCH] intel: Adding missing Broxton PCI IDs.

2016-03-02 Thread Clint Taylor
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

Re: [Mesa-dev] [PATCH] gallium/radeon: use explicit drm_major, drm_minor check

2016-03-02 Thread Alex Deucher
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. >

[Mesa-dev] [PATCH] gk110/ir: fix wrong emission of NOT modifier for VOTE

2016-03-02 Thread Samuel Pitoiset
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

[Mesa-dev] [PATCH] gallium/radeon: use explicit drm_major, drm_minor check

2016-03-02 Thread Emil Velikov
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

[Mesa-dev] [PATCH 23/26] radeonsi: extract the buffer descriptor computation into its own function

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 21/26] radeonsi: accept pipe_resource in si_sampler_view_add_buffer

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 25/26] radeonsi: set amdgpu metadata before exporting a texture

2016-03-02 Thread Marek Olšák
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 ++

[Mesa-dev] [PATCH 22/26] radeonsi: remove resource field from si_sampler_view

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 26/26] winsys/amdgpu: get PCI info

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 20/26] radeonsi: disable DCC on handle export if expecting write access

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 24/26] radeonsi: extract the texture descriptor computation into its own function

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 18/26] radeonsi: allocate DCC in the same backing buffer as the texture

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 06/26] gallium/radeon: clean up r600_texture_get_handle

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 14/26] gallium/radeon: disallow handle export for MSAA & depth textures

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 04/26] winsys/amdgpu: allow drivers to set/get opaque metadata

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 15/26] gallium/radeon: don't use fast color clear if sharing doesn't allow it

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 19/26] radeonsi: add DCC decompression (v2)

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 10/26] gallium/radeon: if we can't discard a whole resource, discard the range instead

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 17/26] gallium/radeon: disable CMASK on handle export if sharing doesn't allow it

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 16/26] gallium/radeon: eliminate fast color clear before sharing

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 11/26] galilum/radeon: disallow reallocation of shared buffers

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 13/26] gallium/radeon: remember that texture_from_handle was called and its flags

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 12/26] gallium/radeon: check that handle usage doesn't change for a resource

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 09/26] gallium/radeon: buffer valid range tracking only works with unshared buffers

2016-03-02 Thread Marek Olšák
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_

[Mesa-dev] [PATCH 05/26] gallium/radeon: move code initializing texture metadata to its own function

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 07/26] gallium/radeon: set texture metadata only once

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 08/26] gallium/radeon: don't set texture metadata for buffers

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 03/26] gallium/radeon: rename winsys buffer_get/set_tiling to buffer_get/set_metadata

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 01/26] gallium/radeon: use a structure for passing tiling flags from/to winsys

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 00/26] RadeonSI getting ready for interop with HSA/OpenCL

2016-03-02 Thread Marek Olšák
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

[Mesa-dev] [PATCH 02/26] gallium/radeon: remove rcs parameter from radeon_winsys::buffer_set_tiling

2016-03-02 Thread Marek Olšák
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

Re: [Mesa-dev] [PATCH 07/65] mesa/multisample: Check sample count using the new driver hook

2016-03-02 Thread Eduardo Lima Mitev
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

Re: [Mesa-dev] [PATCH] gallium: add CAPs returning PCI device location

2016-03-02 Thread Brian Paul
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

[Mesa-dev] [PATCH] gallium: add CAPs returning PCI device location

2016-03-02 Thread Marek Olšák
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_

Re: [Mesa-dev] [PATCH v3] nv50/ir: emit VOTE instruction

2016-03-02 Thread Samuel Pitoiset
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

Re: [Mesa-dev] [PATCH 07/65] mesa/multisample: Check sample count using the new driver hook

2016-03-02 Thread Matt Turner
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

Re: [Mesa-dev] [PATCH v3] nv50/ir: emit VOTE instruction

2016-03-02 Thread Matt Turner
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

Re: [Mesa-dev] [PATCH v5] i965: add opportunistic behaviour to opt_vector_float()

2016-03-02 Thread Juan A. Suarez Romero
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

[Mesa-dev] [PATCH v5] i965: add opportunistic behaviour to opt_vector_float()

2016-03-02 Thread Juan A. Suarez Romero
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

Re: [Mesa-dev] [PATCH 00/65] ARB_internalformat_query2 support for Mesa and i965

2016-03-02 Thread Eduardo Lima Mitev
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).

Re: [Mesa-dev] [PATCH 07/65] mesa/multisample: Check sample count using the new driver hook

2016-03-02 Thread Eduardo Lima Mitev
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

Re: [Mesa-dev] [PATCH 11/65] mesa/main: Add extension tracking bit for ARB_internalformat_query2

2016-03-02 Thread Eduardo Lima Mitev
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

[Mesa-dev] [PATCH v2 39/65] mesa/formatquery: Added mipmap related queries

2016-03-02 Thread Eduardo Lima Mitev
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

[Mesa-dev] [PATCH] i965: Ignore glPointSize when GL_POINT_SIZE_ARRAY_OES is enabled

2016-03-02 Thread Plamena Manolova
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

[Mesa-dev] [Bug 94295] [swrast] piglit shader_runner fast_color_clear/all-colors regression

2016-03-02 Thread bugzilla-daemon
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

Re: [Mesa-dev] [PATCH] i965: Ignore glPointSize when GL_POINT_SIZE_ARRAY_OES is enabled

2016-03-02 Thread Manolova, Plamena
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

Re: [Mesa-dev] ANNOUNCE: An open-source Vulkan driver for Intel hardware

2016-03-02 Thread Martin Peres
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

Re: [Mesa-dev] [PATCH 00/65] ARB_internalformat_query2 support for Mesa and i965

2016-03-02 Thread Alejandro Piñeiro
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