Re: [Mesa-dev] About tests for extensions that are later integrated in the core specs

2016-03-08 Thread Andres Gomez
On Fri, 2016-03-04 at 11:49 -0800, Matt Turner wrote: ... > So to answer your main question, I would generate tests for both > glsl-4.00 and arb_gpu_shader_fp64, but only manually write tests for > arb_gpu_shader_fp64. Thanks for the early reply, Matt! We will do as you suggest. -- Br, Andres

[Mesa-dev] [PATCH] glsl: dont allow undefined array sizes in ES

2016-03-08 Thread Timothy Arceri
This applies the rule to empty declarations. Fixes: dEQP-GLES3.functional.shaders.arrays.invalid.empty_declaration_without_var_name_vertex dEQP-GLES3.functional.shaders.arrays.invalid.empty_declaration_without_var_name_fragment --- src/compiler/glsl/ast_to_hir.cpp | 11 +++ 1 file changed

[Mesa-dev] Mesa 11.2.0 release candidate 3

2016-03-08 Thread Emil Velikov
The third release candidate for Mesa 11.2.0 is now available. Axel Davy (5): st/nine: Introduce helper for Position shader input st/nine: Use Position input helper for ps3 declared inputs st/nine: Handle READONLY for buffer MANAGED pool st/nine: Fix Multithreading issue wi

Re: [Mesa-dev] [Mesa-stable] [PATCH v2] egl: support EGL_LARGEST_PBUFFER in eglCreatePbufferSurface(...)

2016-03-08 Thread Emil Velikov
On 7 March 2016 at 05:56, Tapani Pälli wrote: > ping Emil > I thought I've already replied. Sorry about that Reviewed-by: Emil Velikov -Emil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-de

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

2016-03-08 Thread Emil Velikov
Just like everywhere else in the radeon codebase. v2: Don't forget about drm_major == 3 (Alex) Cc: Alex Deucher Cc: Marek Olšák Signed-off-by: Emil Velikov --- Untested, as previous version. src/gallium/drivers/radeon/radeon_vce.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) dif

[Mesa-dev] [PATCH] scons: build osmesa swrast on windows again

2016-03-08 Thread Andreas Fänger
We are still using classic osmesa (swrast) in a lot of projects on windows and linux. As discussed with Brian on the user list, here is a patch that makes building osmesa swrast on windows and linux again using scons. Andreas Fänger (1): scons: build osmesa swrast and gallium src/mesa/drivers/

[Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Andreas Fänger
This patch makes it possible to build classic osmesa/swrast on windows again. It was removed in commit 69db422218b0264b5b8eef45bd003a2544e9cbd6. Although there is a gallium version of osmesa now, the swrast version still has more features lacking in llvmpipe, e.g. anisotropic filtering. --- src/me

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

2016-03-08 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Mar 8, 2016 at 11:55 AM, Emil Velikov wrote: > Just like everywhere else in the radeon codebase. > > v2: Don't forget about drm_major == 3 (Alex) > > Cc: Alex Deucher > Cc: Marek Olšák > Signed-off-by: Emil Velikov > --- > > Untested, as previous versio

Re: [Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Emil Velikov
On 8 March 2016 at 11:04, Andreas Fänger wrote: > This patch makes it possible to build classic osmesa/swrast on windows > again. It was removed in commit 69db422218b0264b5b8eef45bd003a2544e9cbd6. > Although there is a gallium version of osmesa now, the swrast version > still has more features lac

Re: [Mesa-dev] [PATCH RFC 0/2] GBM API extension to support fusing KMS and render devices

2016-03-08 Thread Daniel Stone
Hi, On 7 March 2016 at 20:45, Emil Velikov wrote: > On 7 March 2016 at 10:35, Daniel Stone wrote: >> On 7 March 2016 at 10:19, Thierry Reding wrote: >>> On Mon, Mar 07, 2016 at 10:46:52AM +0100, Lucas Stach wrote: The wrapped driver takes away the ability of the application to decide

[Mesa-dev] Intent to work on support of ARB_vertex_attrib_64bit

2016-03-08 Thread Alejandro Piñeiro
Hello everybody, this is an announcement that some of the developers at Igalia are planning to work on adding i965 support for ARB_vertex_attrib_64bit. I just filed a bug [1] to track progress. Feel free to add your comments there too. I was also about to send a patch to update docs/GL3.txt and

Re: [Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Andreas Fänger
> -Ursprüngliche Nachricht- > Von: Emil Velikov [mailto:emil.l.veli...@gmail.com] > Gesendet: Dienstag, 8. März 2016 12:33 > > On 8 March 2016 at 11:04, Andreas Fänger wrote: > > This patch makes it possible to build classic osmesa/swrast on windows > > again. It was removed in commit > 6

[Mesa-dev] [PATCH 02/10] gallium/winsys/drm: add layer to struct winsys_handle

2016-03-08 Thread Christian König
From: Christian König For exporting a specific layer of an array texture. Signed-off-by: Christian König --- src/gallium/include/state_tracker/drm_driver.h | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/include/state_tracker/drm_driver.h b/src/gallium/include/state_track

[Mesa-dev] New DMA-Buf based interop for VDPAU/OpenGL.

2016-03-08 Thread Christian König
Hi everyone, the following series of patches define and implement an new DMA-Buf based interface for interop between VDPAU and OpenGL in Mesa. The basic idea is that we want to be independent of the source code versions used in both and so avoid using Mesa internal structures for the interop. I

[Mesa-dev] [PATCH 04/10] radeon/winsys: add layer support for BO export

2016-03-08 Thread Christian König
From: Christian König Add layer support to export individual array layers. Signed-off-by: Christian König --- src/gallium/drivers/radeon/r600_texture.c | 1 + src/gallium/drivers/radeon/radeon_winsys.h| 1 + src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 2 ++ src/gallium/winsys/radeo

[Mesa-dev] [PATCH 06/10] st/vdpau: use linear layout for output surfaces

2016-03-08 Thread Christian König
From: Christian König Works around a bug in radeonsi and tiling is actually not very beneficial in this use case. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/output.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/vdpau/

[Mesa-dev] [PATCH 05/10] radeonsi: ignore PIPE_BIND_LINEAR in si_is_format_supported

2016-03-08 Thread Christian König
From: Christian König Linear layout should work for all formats as well. Signed-off-by: Christian König --- src/gallium/drivers/radeonsi/si_state.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index 157

[Mesa-dev] [PATCH 08/10] st/vdpau: move FormatRGBAToPipe into the interop

2016-03-08 Thread Christian König
From: Christian König We are going to need that in the Mesa state tracker as well. Signed-off-by: Christian König --- src/gallium/include/state_tracker/vdpau_funcs.h | 65 src/gallium/state_trackers/vdpau/bitmap.c| 2 +- src/gallium/state_trackers/vdpau/outpu

[Mesa-dev] [PATCH 01/10] gallium/winsys/drm: add offset to struct winsys_handle

2016-03-08 Thread Christian König
From: Christian König We are going to need this for EGL_EXT_image_dma_buf_import. Signed-off-by: Christian König --- src/gallium/include/state_tracker/drm_driver.h| 5 + src/gallium/state_trackers/dri/dri2.c | 2 ++ src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 1 + s

[Mesa-dev] [PATCH 10/10] st/mesa: implement new DMA-buf based VDPAU interop

2016-03-08 Thread Christian König
From: Christian König Avoid using internal structures from another API. Signed-off-by: Christian König --- src/mesa/state_tracker/st_vdpau.c | 176 -- 1 file changed, 129 insertions(+), 47 deletions(-) diff --git a/src/mesa/state_tracker/st_vdpau.c b/src/m

[Mesa-dev] [PATCH 07/10] st/vdpau: add new interop interface

2016-03-08 Thread Christian König
From: Christian König Use DMA-buf for the VDPAU interop interface instead of using internal structures. Signed-off-by: Christian König --- src/gallium/include/state_tracker/vdpau_dmabuf.h | 94 +++ src/gallium/include/state_tracker/vdpau_interop.h | 7 +- 2 files changed,

[Mesa-dev] [PATCH 09/10] st/vdpau: implement the new DMA-buf based interop

2016-03-08 Thread Christian König
From: Christian König That should allow us to get away from passing internal structures around. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/ftab.c | 6 ++- src/gallium/state_trackers/vdpau/output.c| 40 +- src/gallium/state_trackers/vdpau/s

[Mesa-dev] [PATCH 03/10] radeon/winsys: add offset support for BO import/export

2016-03-08 Thread Christian König
From: Christian König Add offset support to handle NV12 offsets as well. Signed-off-by: Christian König --- src/gallium/drivers/r300/r300_texture.c | 6 +++--- src/gallium/drivers/radeon/r600_texture.c | 26 ++ src/gallium/drivers/radeon/radeon_winsys.h|

[Mesa-dev] i965: Spilling non-contiguous registers

2016-03-08 Thread Iago Toral
Hi, I am trying to improve register spilling for fp64 programs. Specifically for the varying-packing-simple piglit test with double types. Because this test uses all available varying slots, register pressure is significant and spilling is necessary for it to pass, even for non-fp64 types. The ma

Re: [Mesa-dev] [PATCH RFC 0/2] GBM API extension to support fusing KMS and render devices

2016-03-08 Thread Christian Gmeiner
2016-03-08 12:36 GMT+01:00 Daniel Stone : > Hi, > > On 7 March 2016 at 20:45, Emil Velikov wrote: >> On 7 March 2016 at 10:35, Daniel Stone wrote: >>> On 7 March 2016 at 10:19, Thierry Reding wrote: On Mon, Mar 07, 2016 at 10:46:52AM +0100, Lucas Stach wrote: > The wrapped driver takes

[Mesa-dev] [Bug 94394] Compile Mesa , specific compilation , /usr/bin/ld: cannot find -ldrm

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94394 --- Comment #5 from Emil Velikov --- *** Bug 94443 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug.

Re: [Mesa-dev] [PATCH 0/9] Skip automatic execsize for instructions with a width of 4

2016-03-08 Thread Samuel Iglesias Gonsálvez
On Mon, 2016-03-07 at 15:12 +0100, Samuel Iglesias Gonsálvez wrote: > On Mon, 2016-03-07 at 16:03 +0200, Pohjolainen, Topi wrote: > > On Mon, Mar 07, 2016 at 10:48:49AM +0100, Samuel Iglesias Gons?lvez > > wrote: > > > Hello, > > > > > > There is only one patch from this series that has been revie

[Mesa-dev] [PATCH 1/3] glsl: don't validate ifc blocks using validation meant for variables

2016-03-08 Thread Timothy Arceri
--- src/compiler/glsl/link_varyings.cpp | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/link_varyings.cpp b/src/compiler/glsl/link_varyings.cpp index c40ed58..34eb848 100644 --- a/src/compiler/glsl/link_varyings.cpp +++ b/src/compiler/glsl/link_var

[Mesa-dev] [PATCH 3/3] glsl: don't always reject shaders with mismatching ifc blocks

2016-03-08 Thread Timothy Arceri
Since we store some member qualifiers in the interface type we need to be more careful about rejecting shaders just because the pointer doesn't match. Its perfectly valid for some qualifiers such as precision to not match across shader interfaces. --- src/compiler/glsl/link_interface_blocks.cpp |

[Mesa-dev] [PATCH 2/3] glsl: make interstage_match() static

2016-03-08 Thread Timothy Arceri
--- src/compiler/glsl/link_interface_blocks.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/glsl/link_interface_blocks.cpp b/src/compiler/glsl/link_interface_blocks.cpp index 64c30fe..9d36836 100644 --- a/src/compiler/glsl/link_interface_blocks.cpp +++ b/src/c

[Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Andreas Fänger
This patch makes it possible to build classic osmesa/swrast on windows again. It was removed in commit 69db422218b0264b5b8eef45bd003a2544e9cbd6. Although there is a gallium version of osmesa now, the swrast version still has more features lacking in llvmpipe, e.g. anisotropic filtering. --- src/me

Re: [Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Andreas Fänger
Here is the updated patch. It reverts changes to Makefile.am, too. Andreas Fänger (1): scons: build osmesa swrast and gallium src/mesa/drivers/SConscript | 2 ++ src/mesa/drivers/osmesa/Makefile.am | 2 ++ src/mesa/drivers/osmesa/SConscript | 40 +

Re: [Mesa-dev] [PATCH] i965/fs/nir: "surface_access::" prefix not needed

2016-03-08 Thread Iago Toral
Reviewed-by: Iago Toral Quiroga On Fri, 2016-03-04 at 20:38 +0100, Alejandro Piñeiro wrote: > "using namespace brw::surface_access" is already present at the > top of the source file. > --- > src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 24 ++-- > 1 file changed, 10 insertions(

Re: [Mesa-dev] [PATCH] glsl: avoid stack smashing when there are too many attributes

2016-03-08 Thread Iago Toral
On Sun, 2016-03-06 at 12:21 -0500, Ilia Mirkin wrote: > This fixes a crash in > > dEQP-GLES3.functional.transform_feedback.array_element.separate.points.lowp_mat3x2 > > and likely others. The vertex shader has > 16 input variables (without > explicit locations), which causes us to index outside o

Re: [Mesa-dev] [PATCH] glsl: avoid stack smashing when there are too many attributes

2016-03-08 Thread Ilia Mirkin
On Tue, Mar 8, 2016 at 9:54 AM, Iago Toral wrote: > On Sun, 2016-03-06 at 12:21 -0500, Ilia Mirkin wrote: >> This fixes a crash in >> >> dEQP-GLES3.functional.transform_feedback.array_element.separate.points.lowp_mat3x2 >> >> and likely others. The vertex shader has > 16 input variables (without >

[Mesa-dev] [PATCH] winsys/amdgpu: enlarge buffer_indices_hashlist

2016-03-08 Thread Bas Nieuwenhuizen
Enlarge the buffer hashlist to prevent large numbers of misses due to adding more buffers than can be cached in the hashlist. The game I tested had CS's with up to 1500 buffers and the overhead of amdgpu_lookup_buffer for various sizes was: 4096 1.97% (new value) 2048 4.37% 1024 6.92% 512 9.47%

Re: [Mesa-dev] [PATCH] glsl: avoid stack smashing when there are too many attributes

2016-03-08 Thread Iago Toral
On Tue, 2016-03-08 at 10:01 -0500, Ilia Mirkin wrote: > On Tue, Mar 8, 2016 at 9:54 AM, Iago Toral wrote: > > On Sun, 2016-03-06 at 12:21 -0500, Ilia Mirkin wrote: > >> This fixes a crash in > >> > >> dEQP-GLES3.functional.transform_feedback.array_element.separate.points.lowp_mat3x2 > >> > >> and

Re: [Mesa-dev] [PATCH 3/3] glsl: don't always reject shaders with mismatching ifc blocks

2016-03-08 Thread Samuel Iglesias Gonsálvez
On Wed, 2016-03-09 at 00:47 +1100, Timothy Arceri wrote: > Since we store some member qualifiers in the interface type > we need to be more careful about rejecting shaders just because > the pointer doesn't match. Its perfectly valid for some qualifiers > such as precision to not match across shade

Re: [Mesa-dev] [PATCH] i965/fs: Recognize constants can be loaded by subtracting from 1.0.

2016-03-08 Thread Iago Toral
On Mon, 2016-02-29 at 22:06 -0800, Matt Turner wrote: > Some shaders from Synmark contain this loop: > >for (float i = 0.02; i < 0.9; i += 0.11) > > and in its body it uses both i and (1.0 - i). All 16 immediates are > promoted to registers (they're used by 3-src MAD instructions). By > recog

Re: [Mesa-dev] [PATCH 3/3] glsl: don't always reject shaders with mismatching ifc blocks

2016-03-08 Thread Ilia Mirkin
On Tue, Mar 8, 2016 at 8:47 AM, Timothy Arceri wrote: > Since we store some member qualifiers in the interface type > we need to be more careful about rejecting shaders just because > the pointer doesn't match. Its perfectly valid for some qualifiers > such as precision to not match across shader

Re: [Mesa-dev] [PATCH 1/5] include/GL: add mesa_glinterop.h for OpenGL-OpenCL interop (v3)

2016-03-08 Thread Marek Olšák
On Sat, Mar 5, 2016 at 9:53 AM, Michel Dänzer wrote: > On 04.03.2016 04:46, Marek Olšák wrote: >> >> +/** >> + * Device information returned by Mesa. >> + */ >> +typedef struct _mesa_glinterop_device_info { >> + uint32_t size; /* size of this structure */ > > Callees determine how much data they

Re: [Mesa-dev] [PATCH 1/5] include/GL: add mesa_glinterop.h for OpenGL-OpenCL interop (v3)

2016-03-08 Thread Marek Olšák
On Thu, Mar 3, 2016 at 11:56 PM, Emil Velikov wrote: > Hi Marek, > > A small question, and a few trivial suggestions. Hopefully I'm not too > late for the party. > > On 3 March 2016 at 19:46, Marek Olšák wrote: > >> +typedef struct _mesa_glinterop_device_info { >> + uint32_t size; /* size of th

[Mesa-dev] [Bug 94383] build error on i386 when enabling swr

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94383 Fabio Pedretti changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

Re: [Mesa-dev] [PATCH RFC 0/2] GBM API extension to support fusing KMS and render devices

2016-03-08 Thread Emil Velikov
Hi Dan, Summarising and stating some unsaid assumptions. Assumptions: - The proposed solution is a replacement of the wrapped drivers approach. No, it's meant to introduce an API mostly gears towards DRI_PRIME setups. - Wrapped drivers will/could/should be done outside the SoC world (i.e. with

Re: [Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Roland Scheidegger
Not that I really care what you can or can't build (and I won't comment on build changes), what are those features lacking in llvmpipe, beside from anisotropic filtering (which I always considered essentially useless for a software renderer, albeit interesting if you're curious about the math invol

Re: [Mesa-dev] [PATCH] scons: build osmesa swrast and gallium

2016-03-08 Thread Emil Velikov
On 8 March 2016 at 12:06, Andreas Fänger wrote: >> -Ursprüngliche Nachricht- >> Von: Emil Velikov [mailto:emil.l.veli...@gmail.com] >> Gesendet: Dienstag, 8. März 2016 12:33 >> >> On 8 March 2016 at 11:04, Andreas Fänger wrote: >> > This patch makes it possible to build classic osmesa/swr

[Mesa-dev] [PATCH] gallium/swr: remove use of BYTE from swr driver

2016-03-08 Thread Tim Rowley
Remove use of a win32-style type leaked from the swr rasterizer. --- src/gallium/drivers/swr/swr_memory.h| 8 src/gallium/drivers/swr/swr_scratch.cpp | 8 src/gallium/drivers/swr/swr_screen.cpp | 4 ++-- src/gallium/drivers/swr/swr_state.cpp | 8 4 files changed,

[Mesa-dev] [PATCH] nv50, nvc0: make sure to destroy the mutex used for blits

2016-03-08 Thread Samuel Pitoiset
This mutex is initialized when the blitter is created, but it is never destroyed. This doesn't hurt anything but it makes sense to destroy it at blitter deletion. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nv50/nv50_surface.c | 1 + src/gallium/drivers/nouveau/nvc0/nvc0_surfa

Re: [Mesa-dev] [PATCH] nv50, nvc0: make sure to destroy the mutex used for blits

2016-03-08 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Tue, Mar 8, 2016 at 1:01 PM, Samuel Pitoiset wrote: > This mutex is initialized when the blitter is created, but it is never > destroyed. This doesn't hurt anything but it makes sense to destroy it > at blitter deletion. > > Signed-off-by: Samuel Pitoiset > --- > sr

[Mesa-dev] [Bug 92877] Add support for libglvnd

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92877 --- Comment #1 from Kyle Brenneman --- The current libglvnd interface only allows a dispatch table for each vendor library. Using libglvnd's dispatching in place of Mesa's would either require changing that interface or exposing Mesa as multiple

[Mesa-dev] [Bug 92877] Add support for libglvnd

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92877 Kyle Brenneman changed: What|Removed |Added CC||kyle.brenne...@gmail.com -- You are re

Re: [Mesa-dev] [PATCH 15/21] nir: Add a function for comparing cursors

2016-03-08 Thread Jason Ekstrand
On Wed, Feb 24, 2016 at 10:45 PM, Connor Abbott wrote: > On Sat, Feb 13, 2016 at 9:14 PM, Jason Ekstrand > wrote: > > --- > > src/compiler/nir/nir.c | 63 > ++ > > src/compiler/nir/nir.h | 2 ++ > > 2 files changed, 65 insertions(+) > > > > diff

Re: [Mesa-dev] [PATCH] nir: Add a pass for lower indirect variable dereferences

2016-03-08 Thread Jason Ekstrand
On Wed, Feb 24, 2016 at 10:25 PM, Connor Abbott wrote: > On Sat, Feb 13, 2016 at 8:48 PM, Jason Ekstrand > wrote: > > This new pass lowers load/store_var intrinsics that act on indirect > derefs > > to if-ladder of direct load/store_var intrinsics. The if-ladders > perform a > > simple binary s

Re: [Mesa-dev] [PATCH] anv/meta: Use ISL to get the image tile height

2016-03-08 Thread Anuj Phogat
On Fri, Mar 4, 2016 at 2:03 PM, Nanley Chery wrote: > From: Nanley Chery > > In addition to making the height addition more understandable, this > future-proofs the code for new tiling modes and keeps the image > height as small as possible. > > Signed-off-by: Nanley Chery > --- > src/intel/vul

Re: [Mesa-dev] [PATCH 15/21] nir: Add a function for comparing cursors

2016-03-08 Thread Connor Abbott
On Tue, Mar 8, 2016 at 1:34 PM, Jason Ekstrand wrote: > > > On Wed, Feb 24, 2016 at 10:45 PM, Connor Abbott wrote: >> >> On Sat, Feb 13, 2016 at 9:14 PM, Jason Ekstrand >> wrote: >> > --- >> > src/compiler/nir/nir.c | 63 >> > ++ >> > src/compiler

Re: [Mesa-dev] [PATCH 1/2] dri: add backbuffer use flag

2016-03-08 Thread Marek Olšák
On Wed, Mar 2, 2016 at 12:05 AM, Marek Olšák wrote: > On Wed, Mar 2, 2016 at 12:01 AM, Ian Romanick wrote: >> On 03/01/2016 01:41 PM, Marek Olšák wrote: >>> From: Axel Davy >>> >>> This will be used by the next commit. >>> --- >>> include/GL/internal/dri_interface.h | 1 + >>> src/egl/drive

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

2016-03-08 Thread Marek Olšák
On Tue, Mar 8, 2016 at 4:40 AM, Michel Dänzer wrote: > On 03.03.2016 01:36, Marek Olšák wrote: >> 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/g

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

2016-03-08 Thread Marek Olšák
On Tue, Mar 8, 2016 at 4:41 AM, Michel Dänzer wrote: > On 03.03.2016 01:36, Marek Olšák wrote: >> 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

[Mesa-dev] [PATCH v2 1/2] mesa: docs: i965: Use correct doxygen groupings syntax

2016-03-08 Thread Sarah Sharp
v2: - use \name doxygen format instead of @defgroup, which creates a separate module - addresses comments by Ian back in December 8<->8 When reading the source code, it's useful to indicate that a group of fields in a struct are

[Mesa-dev] [PATCH v2 2/2] mesa: docs: Intel i965 hardware limits.

2016-03-08 Thread Sarah Sharp
v2: - use \name doxygen format instead of @defgroup, which creates a separate module - addresses comments by Ian back in December 8<->8 This should help the next person working on hardware enabling figure out where in the Intel P

[Mesa-dev] [PATCH 3/3] st/va: add HEVC main 10 profile

2016-03-08 Thread Boyuan Zhang
Signed-off-by: Boyuan Zhang Reviewed-by: Christian König Reviewed-by: Alex Deucher --- src/gallium/state_trackers/va/va_private.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/va/va_private.h b/src/gallium/state_trackers/va/va_private.h inde

[Mesa-dev] [PATCH 2/2] nvc0: add a new validation path for compute

2016-03-08 Thread Samuel Pitoiset
This makes use of the new state validation interface to be consistent with 3d. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_compute.c | 47 +++-- src/gallium/drivers/nouveau/nvc0/nve4_compute.c | 42 +++--- 2 files changed, 41 inser

[Mesa-dev] [PATCH 1/2] nvc0: rework the validation path for 3D

2016-03-08 Thread Samuel Pitoiset
This exposes an interface for state validation that will be also used to rework the compute validation path. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/Makefile.sources | 1 + src/gallium/drivers/nouveau/nvc0/nvc0_context.h| 3 -- .../drivers/nouveau/nvc0/nvc0_sta

Re: [Mesa-dev] [PATCH 1/2] nvc0: rework the validation path for 3D

2016-03-08 Thread Ilia Mirkin
On Tue, Mar 8, 2016 at 3:08 PM, Samuel Pitoiset wrote: > This exposes an interface for state validation that will be also used > to rework the compute validation path. > > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/Makefile.sources | 1 + > src/gallium/drivers/nouv

Re: [Mesa-dev] [PATCH 1/2] nvc0: rework the validation path for 3D

2016-03-08 Thread Samuel Pitoiset
On 03/08/2016 09:11 PM, Ilia Mirkin wrote: On Tue, Mar 8, 2016 at 3:08 PM, Samuel Pitoiset wrote: This exposes an interface for state validation that will be also used to rework the compute validation path. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/Makefile.sources

[Mesa-dev] [PATCH 2/3] radeon/video: enable HEVC main 10 decode

2016-03-08 Thread Boyuan Zhang
Signed-off-by: Boyuan Zhang Reviewed-by: Christian König Reviewed-by: Alex Deucher --- src/gallium/drivers/radeon/radeon_video.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_video.c b/src/gallium/drivers/radeon/radeon_video.c ind

[Mesa-dev] [PATCH 1/3] radeon/uvd: handle HEVC main 10 decode

2016-03-08 Thread Boyuan Zhang
Signed-off-by: Boyuan Zhang Reviewed-by: Alex Deucher --- src/gallium/drivers/radeon/radeon_uvd.c | 69 +++-- 1 file changed, 58 insertions(+), 11 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_uvd.c b/src/gallium/drivers/radeon/radeon_uvd.c index 6ea07

[Mesa-dev] [PATCH v2 1/2] nvc0: rework the validation path for 3D

2016-03-08 Thread Samuel Pitoiset
This exposes an interface for state validation that will be also used to rework the compute validation path. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_context.h| 10 +- .../drivers/nouveau/nvc0/nvc0_state_validate.c | 36 ++ src/gall

[Mesa-dev] [PATCH v2 2/2] nvc0: add a new validation path for compute

2016-03-08 Thread Samuel Pitoiset
This makes use of the new state validation interface to be consistent with 3d. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_compute.c | 46 +++-- src/gallium/drivers/nouveau/nvc0/nve4_compute.c | 41 ++ 2 files changed, 39 inser

Re: [Mesa-dev] [PATCH v2 1/2] nvc0: rework the validation path for 3D

2016-03-08 Thread Ilia Mirkin
Thanks. Series is Reviewed-by: Ilia Mirkin On Tue, Mar 8, 2016 at 3:36 PM, Samuel Pitoiset wrote: > This exposes an interface for state validation that will be also used > to rework the compute validation path. > > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/nvc0/nvc0_co

[Mesa-dev] [PATCH 3/6] anv/meta: Make meta_emit_blit() public

2016-03-08 Thread Nanley Chery
From: Nanley Chery This can be reverted if the only other consumer, anv_meta_blit2d(), uses a different method. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_meta.h | 11 +++ src/intel/vulkan/anv_meta_blit.c | 2 +- 2 files changed, 12 insertions(+), 1 deletion(-) diff --

[Mesa-dev] [PATCH 1/6] anv/meta: Minimize height of images used for copies

2016-03-08 Thread Nanley Chery
From: Nanley Chery In addition to demystifying the value being added to the height, this future-proofs the code for new tiling modes and keeps the image height as small as possible. v2: Actually use the smallest height possible. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_meta_blit.c

[Mesa-dev] [PATCH 4/6] anv/meta: Split anv_meta_blit.c into three files

2016-03-08 Thread Nanley Chery
From: Nanley Chery The new organization is as follows: * anv_meta_blit.c: Blit and state setup/teardown commands * anv_meta_copy.c: Copy and update commands * anv_meta_blit2d.c: 2D Blitter API commands Also, change the formatting to contain most lines within 80 columns. Signed-off-by: Nanley Ch

[Mesa-dev] [PATCH 5/6] anv/meta: Prefix anv_ to meta_emit_blit()

2016-03-08 Thread Nanley Chery
From: Nanley Chery Follow the convention for non-static functions. Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_meta.h| 2 +- src/intel/vulkan/anv_meta_blit.c | 4 ++-- src/intel/vulkan/anv_meta_blit2d.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/s

[Mesa-dev] [PATCH 6/6] anv/blit2d: Use the tiling enum for simplicity

2016-03-08 Thread Nanley Chery
From: Nanley Chery Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_meta_blit2d.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/intel/vulkan/anv_meta_blit2d.c b/src/intel/vulkan/anv_meta_blit2d.c index d49b470..6f07342 100644 --- a/src/intel/vulkan/anv_meta_

[Mesa-dev] [PATCH 2/6] anv/meta: Store src and dst usage flags in a variable

2016-03-08 Thread Nanley Chery
From: Nanley Chery Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_meta_blit.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/intel/vulkan/anv_meta_blit.c b/src/intel/vulkan/anv_meta_blit.c index ecd4d2d..82b79b8 100644 --- a/src/intel/vulkan/anv_meta_bl

Re: [Mesa-dev] [PATCH] anv/meta: Use ISL to get the image tile height

2016-03-08 Thread Nanley Chery
On Tue, Mar 08, 2016 at 10:41:24AM -0800, Anuj Phogat wrote: > On Fri, Mar 4, 2016 at 2:03 PM, Nanley Chery wrote: > > From: Nanley Chery > > > > In addition to making the height addition more understandable, this > > future-proofs the code for new tiling modes and keeps the image > > height as s

Re: [Mesa-dev] [PATCH 3/3] glsl: don't always reject shaders with mismatching ifc blocks

2016-03-08 Thread Timothy Arceri
On Tue, 2016-03-08 at 16:24 +0100, Samuel Iglesias Gonsálvez wrote: > On Wed, 2016-03-09 at 00:47 +1100, Timothy Arceri wrote: > > Since we store some member qualifiers in the interface type > > we need to be more careful about rejecting shaders just because > > the pointer doesn't match. Its perfe

Re: [Mesa-dev] [PATCH 3/3] glsl: don't always reject shaders with mismatching ifc blocks

2016-03-08 Thread Timothy Arceri
On Tue, 2016-03-08 at 10:37 -0500, Ilia Mirkin wrote: > On Tue, Mar 8, 2016 at 8:47 AM, Timothy Arceri > wrote: > > Since we store some member qualifiers in the interface type > > we need to be more careful about rejecting shaders just because > > the pointer doesn't match. Its perfectly valid for

Re: [Mesa-dev] [PATCH 1/5] include/GL: add mesa_glinterop.h for OpenGL-OpenCL interop (v3)

2016-03-08 Thread Marek Olšák
On Tue, Mar 8, 2016 at 4:39 PM, Marek Olšák wrote: > On Sat, Mar 5, 2016 at 9:53 AM, Michel Dänzer wrote: >> On 04.03.2016 04:46, Marek Olšák wrote: >>> >>> +/** >>> + * Device information returned by Mesa. >>> + */ >>> +typedef struct _mesa_glinterop_device_info { >>> + uint32_t size; /* size

[Mesa-dev] [PATCH 3/5] egl: implement EGL part of interop interface (v2)

2016-03-08 Thread Marek Olšák
From: Marek Olšák v2: - use const --- src/egl/drivers/dri2/egl_dri2.c | 32 ++ src/egl/drivers/dri2/egl_dri2.h | 1 + src/egl/main/eglapi.c | 72 + src/egl/main/eglapi.h | 9 ++ 4 files changed, 114 insertions(+)

[Mesa-dev] [PATCH 1/5] include/GL: add mesa_glinterop.h for OpenGL-OpenCL interop (v4)

2016-03-08 Thread Marek Olšák
From: Marek Olšák v2: - use "enum" to define stuff v3: - more comments, define MESA_GLINTEROP_UNSUPPORTED v4: - add mesa_glinterop_device_info::interop_version - more comments - remove #define MESA_GLINTEROP_VERSION - use const for "in" --- include/GL/mesa_glinterop.h | 272 +

[Mesa-dev] [PATCH 4/5] glx: implement GLX part of interop interface (v2)

2016-03-08 Thread Marek Olšák
From: Marek Olšák v2: - use const --- src/glx/Makefile.am | 1 + src/glx/dri2_glx.c | 11 +++--- src/glx/dri2_priv.h | 16 src/glx/dri3_glx.c | 5 +++ src/glx/dri3_priv.h | 10 + src/glx/dri_common_interop.c | 92 +

[Mesa-dev] [PATCH 2/5] dri_interface: add interface for GL interop with other APIs (v2)

2016-03-08 Thread Marek Olšák
From: Marek Olšák v2: - use const --- include/GL/internal/dri_interface.h | 26 ++ 1 file changed, 26 insertions(+) diff --git a/include/GL/internal/dri_interface.h b/include/GL/internal/dri_interface.h index 2b49a29..84731a0 100644 --- a/include/GL/internal/dri_interfa

[Mesa-dev] [PATCH 5/5] st/dri: implement the GL interop DRI extension (v2)

2016-03-08 Thread Marek Olšák
From: Marek Olšák v2: - set interop_version - simplify the offset_after macro --- src/gallium/state_trackers/dri/dri2.c | 255 ++ 1 file changed, 255 insertions(+) diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium/state_trackers/dri/dri2.c ind

Re: [Mesa-dev] [PATCH 09/10] i965: Add and use is_scheduling_barrier() function.

2016-03-08 Thread Matt Turner
On Fri, Mar 4, 2016 at 8:49 PM, Francisco Jerez wrote: > Matt Turner writes: > >> Though there is a lot of overlap with has_side_effects(), these do mean >> different things. > > Can we do it the other way around and implement is_scheduling_barrier() > in terms of has_side_effects()? has_side_ef

Re: [Mesa-dev] [PATCH 1/6] anv/meta: Minimize height of images used for copies

2016-03-08 Thread Anuj Phogat
On Tue, Mar 8, 2016 at 1:00 PM, Nanley Chery wrote: > From: Nanley Chery > > In addition to demystifying the value being added to the height, > this future-proofs the code for new tiling modes and keeps the > image height as small as possible. > > v2: Actually use the smallest height possible. >

Re: [Mesa-dev] [PATCH] nv50/ir: Check for valid insn instead of defs size

2016-03-08 Thread Samuel Pitoiset
This seems like correct. Reviewed-by: Samuel Pitoiset On 02/25/2016 02:03 AM, Pierre Moreau wrote: On Tesla cards, the first register $r0 contains the thread id; later generations use a specialised register for it. In order to prevent the register from being given to anyone, and thus lose the

Re: [Mesa-dev] [PATCH] nv50/ir: Check for valid insn instead of defs size

2016-03-08 Thread Ilia Mirkin
Patch is fine, description is wrong (or at least inaccurate). The real issue is that function arguments have defs, but no defining instruction. As a result, there's nothing to do when allocating registers. This has nothing to do with $r0, but it does have something to do with the fact that nv50 co

[Mesa-dev] [Bug 94447] glsl/glcpp/tests/glcpp-test-cr-lf regression

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94447 Bug ID: 94447 Summary: glsl/glcpp/tests/glcpp-test-cr-lf regression Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: All Status: NEW Keywords: bise

Re: [Mesa-dev] [PATCH] winsys/amdgpu: enlarge buffer_indices_hashlist

2016-03-08 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Mar 8, 2016 at 4:01 PM, Bas Nieuwenhuizen wrote: > Enlarge the buffer hashlist to prevent large numbers of misses > due to adding more buffers than can be cached in the hashlist. > > The game I tested had CS's with up to 1500 buffers and the overhead > of

Re: [Mesa-dev] [PATCH] winsys/amdgpu: enlarge buffer_indices_hashlist

2016-03-08 Thread Marek Olšák
Pushed. Do you even have push access? Marek On Wed, Mar 9, 2016 at 12:50 AM, Marek Olšák wrote: > Reviewed-by: Marek Olšák > > Marek > > On Tue, Mar 8, 2016 at 4:01 PM, Bas Nieuwenhuizen > wrote: >> Enlarge the buffer hashlist to prevent large numbers of misses >> due to adding more buffers t

[Mesa-dev] [PATCH] mesa: Raise GL_INVALID_ENUM for bad textarget in FramebufferTexture*D

2016-03-08 Thread Kenneth Graunke
dEQP-GLES3.functional.negative_api.buffer.framebuffer_texture2d expects glFramebufferTexture[123]D to raise GL_INVALID_ENUM when supplied a completely bogus textarget parameter (i.e. 0x). This patch changes our behavior from GL_INVALID_OPERATION to GL_INVALID_ENUM in this case. The spec i

Re: [Mesa-dev] [PATCH] mesa: Raise GL_INVALID_ENUM for bad textarget in FramebufferTexture*D

2016-03-08 Thread Ilia Mirkin
Actually I filed a dEQP bug for that one: https://code.google.com/p/android/issues/detail?id=200911 The text says "An INVALID_OPERATION error is generated if texture is not zero and textarget is not one of TEXTURE_2D, TEXTURE_2D_MULTISAMPLE, or one of the cube map face targets from table 8.21". A

[Mesa-dev] [Bug 94447] glsl/glcpp/tests/glcpp-test-cr-lf regression

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94447 --- Comment #1 from Ian Romanick --- That is odd. Since Ken added some tests, I would think he would have seen such a failure. Vinson, what were you building this on? -- You are receiving this mail because: You are the QA Contact for the bug.

[Mesa-dev] [Bug 94383] build error on i386 when enabling swr

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94383 --- Comment #4 from Tim Rowley --- Hmm, interesting problem. In drivers/swr/avx[2], we set the compiler code generation in AM_CXXFLAGS with -march=. This works fine for normal development building of mesa. In the distribution build environment

[Mesa-dev] [Bug 94447] glsl/glcpp/tests/glcpp-test-cr-lf regression

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94447 --- Comment #2 from Vinson Lee --- (In reply to Ian Romanick from comment #1) > That is odd. Since Ken added some tests, I would think he would have seen > such a failure. > > Vinson, what were you building this on? I can reproduce the make ch

Re: [Mesa-dev] [PATCH 09/10] i965: Add and use is_scheduling_barrier() function.

2016-03-08 Thread Francisco Jerez
Matt Turner writes: > On Fri, Mar 4, 2016 at 8:49 PM, Francisco Jerez wrote: >> Matt Turner writes: >> >>> Though there is a lot of overlap with has_side_effects(), these do mean >>> different things. >> >> Can we do it the other way around and implement is_scheduling_barrier() >> in terms of h

[Mesa-dev] [Bug 94452] dEQP-GLES3.functional.negative_api.texture.generatemipmap fails

2016-03-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94452 Bug ID: 94452 Summary: dEQP-GLES3.functional.negative_api.texture.generatemip map fails Product: Mesa Version: git Hardware: Other OS: All Statu

[Mesa-dev] [PATCH] egl: adds EGL_KHR_reusable_sync to egl_dri

2016-03-08 Thread Dongwon Kim
This patch enables an EGL extension, EGL_KHR_reusable_sync. This new extension basically provides a way for multiple APIs or threads to be excuted synchronously via a "reusable sync" primitive shared by those threads/API calls. This was implemented based on the specification at https://www.khrono

  1   2   >