[Mesa-dev] [PATCH v2 1/4] gallium/winsys/kms: Fix double refcount when importing from prime FD (v2)

Currently the code creates a display target struct with refcount field initialized to 1 and then the caller again increments it, leading to a leaked reference. Let's remove the unnecessary increment. v2: Split from one big patch into four fixing one thing at a time. Signed-off-by: Tomasz Figa

[Mesa-dev] [PATCH v2 0/4] gallium/winsys/kms: Fix issues with prime import

Current prime import code in kms-dri Gallium winsys backend has several issues: - It leaks imported buffers, because of duplicate reference count increment at import time. - It does not check whether an already imported (or local) buffer is not being imported, which is problematic because

[Mesa-dev] [PATCH v2 4/4] gallium/winsys/kms: Look up the GEM handle after importing a prime FD (v2)

drmPrimeHandleToFD() will return the same GEM handle every time the same buffer is imported, even from a different prime FD. Since GEM handles are not reference counted, we need to make sure that each GEM handle is referenced only by one display target struct, by looking it up in kms_sw->bo_list

[Mesa-dev] [PATCH v2 2/4] gallium/winsys/kms: Fully initialize kms_sw_dt at prime import time (v2)

Currently kms_sw_displaytarget_add_from_prime() allocates the struct and fills in only some of the fields, resulting in a half-baked struct that needs to be further completed by the caller. To make this a bit more consistent, pass width, height and stride to this function and fill in everything

[Mesa-dev] [PATCH v2 3/4] gallium/winsys/kms: Move display target handle lookup to separate function (v2)

As a preparation to use the lookup in more than once place, move the code that looks up given KMS/GEM handle to a separate function. This change should not introduce any functional changes. v2: Split into separate patch. Move lookup code into separate function. Signed-off-by: Tomasz Figa

Re: [Mesa-dev] Mesa (master): egl/dri2: Add reference count for dri2_egl_display

On 29/07/16 09:20, Michel Dänzer wrote: On 28.07.2016 22:11, Emil Velikov wrote: Module: Mesa Branch: master Commit: 9ee683f877b283020c6f24776236f1145cb7a4ea URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=9ee683f877b283020c6f24776236f1145cb7a4ea Author: Nicolas Boichat

[Mesa-dev] [PATCH 4/8] gallium/radeon: count contexts

From: Marek Olšák We don't wanna use unflushed fences when we have multiple contexts. --- src/gallium/drivers/radeon/r600_pipe_common.c | 3 +++ src/gallium/drivers/radeon/r600_pipe_common.h | 1 + 2 files changed, 4 insertions(+) diff --git

[Mesa-dev] [PATCH 2/8] r600g: use last_gfx_fence like radeonsi

From: Marek Olšák --- src/gallium/drivers/r600/r600_hw_context.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/r600/r600_hw_context.c b/src/gallium/drivers/r600/r600_hw_context.c index 1bc3bf4..113991f 100644 ---

[Mesa-dev] [PATCH 3/8] gallium/radeon: count gfx IB flushes

From: Marek Olšák This will be used as a counter for whether fence_finish needs to flush the IB. --- src/gallium/drivers/r600/r600_hw_context.c| 1 + src/gallium/drivers/radeon/r600_pipe_common.h | 1 + src/gallium/drivers/radeonsi/si_hw_context.c | 2 +- 3 files

[Mesa-dev] [PATCH 6/8] winsys/amdgpu: implement cs_get_next_fence

From: Marek Olšák --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 38 +++ src/gallium/winsys/amdgpu/drm/amdgpu_cs.h | 1 + 2 files changed, 35 insertions(+), 4 deletions(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c

[Mesa-dev] [PATCH 0/8] RadeonSI: Unflushed fences for better Bioshock performance

Hi, Most interested people already know what this is, so I'm just gonna share the performance numbers here. This series makes Bioshock Infinite 23% faster. Along with the CPU overhead optimizations that are in the master branch already, the combined improvement is more than 40%. The first 2

[Mesa-dev] [PATCH 7/8] winsys/radeon: implement cs_get_next_fence

From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 30 +-- src/gallium/winsys/radeon/drm/radeon_drm_cs.h | 1 + 2 files changed, 29 insertions(+), 2 deletions(-) diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c

[Mesa-dev] [PATCH 1/8] gallium/radeon: move last_gfx_fence from radeonsi to common code

From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 1 + src/gallium/drivers/radeon/r600_pipe_common.h | 1 + src/gallium/drivers/radeonsi/si_hw_context.c | 10 +- src/gallium/drivers/radeonsi/si_pipe.c| 1 -

[Mesa-dev] [PATCH v2 4/7] egl/android: Refactor image creation to separate flink and prime paths (v2)

This patch splits current dri2_create_image_android_native_buffer() into main entry point and two additional functions, one for creating an image from flink name and one for handling prime FDs using the generic DMA-buf path. This makes the code cleaner and also prepares for disabling flink path

Re: [Mesa-dev] [PATCH 12/12] gallium/radeon/winsyses: decrease max_alloc_size to 1/3 of largest heap

On Tue, Aug 2, 2016 at 4:55 AM, Marek Olšák wrote: > On Tue, Aug 2, 2016 at 3:13 AM, Michel Dänzer wrote: >> On 01.08.2016 16:35, Michel Dänzer wrote: >>> On 30.07.2016 06:42, Marek Olšák wrote: From: Marek Olšák This is

[Mesa-dev] [PATCH] radeonsi: skip unnecessary si_update_shaders calls

From: Marek Olšák Small decrease in draw call overhead. --- src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_state.c | 6 ++ src/gallium/drivers/radeonsi/si_state_draw.c| 20 +---

Re: [Mesa-dev] [PATCH 0/8] RadeonSI: Unflushed fences for better Bioshock performance

On Tue, Aug 2, 2016 at 12:27 PM, Marek Olšák wrote: > Hi, > > Most interested people already know what this is, so I'm just gonna share the > performance numbers here. > > This series makes Bioshock Infinite 23% faster. Along with the CPU overhead > optimizations that are in

[Mesa-dev] [PATCH 5/8] gallium/radeon: add cs_get_next_fence winsys callback

From: Marek Olšák --- src/gallium/drivers/radeon/radeon_winsys.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/gallium/drivers/radeon/radeon_winsys.h b/src/gallium/drivers/radeon/radeon_winsys.h index 3be4738..e7787d3 100644 ---

[Mesa-dev] [PATCH 8/8] gallium/radeon: use unflushed fences for deferred flushes

From: Marek Olšák +23% Bioshock Infinite performance. --- src/gallium/drivers/radeon/r600_pipe_common.c | 45 ++- 1 file changed, 44 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c

Re: [Mesa-dev] [PATCH] ast: Updated AST_NUM_OPERATORS for coherence with ast_operators

On Mon, 2016-08-01 at 14:02 +0100, Eric Engestrom wrote: > On Sun, Jul 31, 2016 at 07:07:34PM +0300, Andres Gomez wrote: [snip] > > diff --git a/src/compiler/glsl/ast.h b/src/compiler/glsl/ast.h > > index 157895d..89f531c 100644 > > --- a/src/compiler/glsl/ast.h > > +++ b/src/compiler/glsl/ast.h

[Mesa-dev] [PATCH v2 1/7] egl/android: Remove unused variables in droid_get_buffers_with_format()

Fix compilation warnings due to unused variables left after some earlier code changes. Signed-off-by: Tomasz Figa --- src/egl/drivers/dri2/platform_android.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/egl/drivers/dri2/platform_android.c

[Mesa-dev] [PATCH v2 3/7] egl/android: Fix support for pbuffers (v2)

From: Nicolas Boichat Existing image loader code supports creating images only for window surfaces. Moreover droid_create_surface() passes wrong surface type to dri2_get_dri_config(), resulting in incorrect configs being returned for pbuffers. This patch fixes these

[Mesa-dev] [PATCH v2 5/7] egl/android: Make get_fourcc() accept HAL formats

There are DRI_IMAGE_FOURCC macros, for which there are no corresponding DRI_IMAGE_FORMAT macros. To support such formats we need to make the lookup function take the native format directly. As a side effect, it simplifies all existing calls to this function, because they all called get_format()

[Mesa-dev] [PATCH v2 6/7] egl/android: Add support for YV12 pixel format (v2)

This patch adds support for YV12 pixel format to the Android platform backend. Only creating EGL images is supported, it is not added to the list of available visuals. v2: Use const array defined just for YV12 instead of trying to be overly generic. Signed-off-by: Tomasz Figa

[Mesa-dev] [PATCH v2 7/7] egl/android: Make drm_gralloc headers optional (v2)

Make the code at least compile when being built without drm_gralloc headers. v2: Replaced #ifdefs with stubs for gralloc_drm_get_gem_handle() and GRALLOC_MODULE_PERFORM_GET_DRM_FD. Removed explicit render node probing code. Signed-off-by: Tomasz Figa ---

[Mesa-dev] [PATCH v2 2/7] egl/android: Respect buffer mask in droid_image_get_buffers (v2)

Drivers can request different set of buffers depending on the buffer mask they pass to the get_buffers callback. This patch makes droid_image_get_buffers() respect this mask. v2: Return error only in case of real error condition and ignore requests of unavailable buffers. Signed-off-by:

[Mesa-dev] [PATCH v2 0/7] egl/android: Improve the Android EGL backend

Hi, This series is a collection of various fixes and extensions we came up with during our attempt to use Mesa for Android. Fixes included in this series: - fixed multiple issues with handling pbuffers in the backend, - made the implementation of DRI image loader .getBuffers callback

Re: [Mesa-dev] [PATCH] nir: fix validation message

On Aug 2, 2016 2:46 AM, "Timothy Arceri" wrote: > > Looks like a copy and paste error from f752effa087 Agreed. Rb > Cc: Jason Ekstrand > --- > src/compiler/nir/nir_validate.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >

Re: [Mesa-dev] [PATCH 2/2] radeonsi: print the command line to VM fault reports (v2)

Obviously, the R-b applies to this rebased series as well. Nicolai On 01.08.2016 23:55, Marek Olšák wrote: From: Marek Olšák v2: rebase on top of Brian's commit --- src/gallium/drivers/radeonsi/si_debug.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

Re: [Mesa-dev] [PATCH] vbo: fix debug message format string in case of 32-bit machines

On Tue, Aug 2, 2016 at 4:13 PM, Brian Paul wrote: > In bufferobj.c, it looks like we're just using %d or %s and casting the > arguments to int. But in other places, we're using %ld and casting to long. > That's not going to accurately report 64-bit values, but I guess that

Re: [Mesa-dev] [PATCH 00/12] RadeonSI: Prevent IB submissions with illegal memory usage

On 29.07.2016 23:42, Marek Olšák wrote: This series ensures that memory usage of gfx IBs is below the kernel- exposed memory limits in most cases. It's not possible to prevent CS ioctl failures completely because it depends on whether continuous free space for all buffers is available, i.e. it

Re: [Mesa-dev] [PATCH] dri: use 'bool' instead 'int' for boolean variables

On 01.08.2016 15:21, Jan Ziak wrote: Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> --- src/egl/drivers/dri2/egl_dri2.h | 19 ++- src/gallium/auxiliary/vl/vl_winsys_dri3.c | 3 ++- src/glx/dri3_priv.h | 5 +++--

Re: [Mesa-dev] [PATCH 3/3] radeonsi: print the command line to VM fault reports

Patch 2 & 3: Reviewed-by: Nicolai Hähnle On 30.07.2016 17:22, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/radeonsi/si_debug.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_debug.c

Re: [Mesa-dev] [PATCH 8/8] gallium/radeon: use unflushed fences for deferred flushes

On 02.08.2016 12:27, Marek Olšák wrote: From: Marek Olšák +23% Bioshock Infinite performance. --- src/gallium/drivers/radeon/r600_pipe_common.c | 45 ++- 1 file changed, 44 insertions(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH 0/3] Destroy Present event contexts

On 08/01/2016 10:20 PM, Michel Dänzer wrote: On 28.07.2016 18:34, Michel Dänzer wrote: Running XTS (the X test suite) on Xephyr using glamor running on Xorg, I stumbled upon an issue: * XTS causes Xephyr to keep resetting and starting new server generations * glamor creates a new GLX context

Re: [Mesa-dev] [PATCH] vbo: fix debug message format string in case of 32-bit machines

On 08/01/2016 04:35 PM, Jan Ziak wrote: Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> --- src/mesa/vbo/vbo_exec_array.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c index

Re: [Mesa-dev] [PATCH 0/3] RadeonSI GLSL "lit" tests

On 31.07.2016 22:29, Marek Olšák wrote: Inspired by LLVM, I created a small test suite that looks like "lit" tests, except that the input is GLSL and it verifies asm. The motivation was to verify that we generate optimal code when starting from GLSL. [PATCH 1/3] radeonsi: add environment

Re: [Mesa-dev] [PATCH 7/8] winsys/radeon: implement cs_get_next_fence

Patches 1-7: Reviewed-by: Nicolai Hähnle On 02.08.2016 12:27, Marek Olšák wrote: From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 30 +-- src/gallium/winsys/radeon/drm/radeon_drm_cs.h | 1 + 2 files

Re: [Mesa-dev] [PATCH 12/12] gallium/radeon/winsyses: decrease max_alloc_size to 1/3 of largest heap

On 02.08.2016 10:55, Marek Olšák wrote: On Tue, Aug 2, 2016 at 3:13 AM, Michel Dänzer wrote: On 01.08.2016 16:35, Michel Dänzer wrote: On 30.07.2016 06:42, Marek Olšák wrote: From: Marek Olšák This is controversial, but I don't see a better way out

Re: [Mesa-dev] [PATCH 4/4] radeonsi: don't set the last parameter component of llvm.AMDGPU.cube

The series is Reviewed-by: Nicolai Hähnle On 31.07.2016 22:34, Marek Olšák wrote: From: Marek Olšák LLVM doesn't use it. --- src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-)

Re: [Mesa-dev] [PATCH] radeonsi: skip unnecessary si_update_shaders calls

Reviewed-by: Nicolai Hähnle On 02.08.2016 12:34, Marek Olšák wrote: From: Marek Olšák Small decrease in draw call overhead. --- src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_state.c | 6 ++

[Mesa-dev] [PATCH] vbo: fix format string compiler warning for 32-bit machines

Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> --- src/mesa/vbo/vbo_exec_array.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c index f371890..ae6a3da 100644 ---

Re: [Mesa-dev] [PATCH] ddebug: don't use fmemopen on non-Linux OS

Reviewed-by: Nicolai Hähnle On 31.07.2016 22:57, Marek Olšák wrote: From: Marek Olšák Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97140 --- src/gallium/drivers/ddebug/dd_draw.c | 5 + 1 file changed, 5 insertions(+) diff --git

Re: [Mesa-dev] [PATCH 08/12] gallium/radeon/winsyses: print CS submission error number

On 29.07.2016 23:42, Marek Olšák wrote: From: Marek Olšák --- src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 2 +- src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 16/95] i965/vec4: We only support 32-bit integer ALU operations for now

Iago Toral Quiroga writes: > Add asserts so we remember to address this when we enable 64-bit > integer support, as suggested by Connor and Jason. Reviewed-by: Francisco Jerez > --- > src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 70 >

Re: [Mesa-dev] [PATCH 15/95] i965/disasm: align16 DF source regions have a width of 2

Reviewed-by: Francisco Jerez Iago Toral Quiroga writes: > --- > src/mesa/drivers/dri/i965/brw_disasm.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c >

Re: [Mesa-dev] [PATCH 13/95] i965: add brw_vecn_grf()

Iago Toral Quiroga writes: > From: Connor Abbott > Reviewed-by: Francisco Jerez > --- > src/mesa/drivers/dri/i965/brw_reg.h | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_reg.h

Re: [Mesa-dev] [PATCH 17/95] i965/vec4: add dst_null_df()

Reviewed-by: Francisco Jerez Iago Toral Quiroga writes: > --- > src/mesa/drivers/dri/i965/brw_vec4.h | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h > b/src/mesa/drivers/dri/i965/brw_vec4.h > index

Re: [Mesa-dev] [PATCH v2 3/7] egl/android: Fix support for pbuffers (v2)

Hi Rob, On Wed, Aug 3, 2016 at 2:32 AM, Rob Herring wrote: > On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: >> From: Nicolas Boichat >> >> Existing image loader code supports creating images only for window >> surfaces. Moreover

[Mesa-dev] [PATCH] i965: use mt->offset in intel_miptree_map_movntdqa().

We need to include mt->offset in the calculation of src pointer because its value may be non-zero, for example in a cubemap texture. Signed-off-by: Haixia Shi Cc: Jason Ekstrand Cc: kenneth.w.grau...@intel.com Cc: Chad Versace

Re: [Mesa-dev] [PATCH 0/3] RadeonSI GLSL "lit" tests

On Tue, Aug 2, 2016 at 9:23 PM, Nicolai Hähnle wrote: > > > On 02.08.2016 21:06, Marek Olšák wrote: >> >> On Tue, Aug 2, 2016 at 3:40 PM, Nicolai Hähnle wrote: >>> >>> On 31.07.2016 22:29, Marek Olšák wrote: Inspired by LLVM, I created a

Re: [Mesa-dev] [PATCH] nir: Always print non-identity swizzles.

Reviewed-by: Timothy Arceri ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 0/3] Destroy Present event contexts

On 02.08.2016 23:01, Leo Liu wrote: > On 08/01/2016 10:20 PM, Michel Dänzer wrote: >> On 28.07.2016 18:34, Michel Dänzer wrote: >>> Running XTS (the X test suite) on Xephyr using glamor running on Xorg, >>> I stumbled upon an issue: >>> >>> * XTS causes Xephyr to keep resetting and starting new

Re: [Mesa-dev] [PATCH 09/95] i965/vec4: add support for printing DF immediates

Iago Toral Quiroga writes: > From: Connor Abbott > Reviewed-by: Francisco Jerez > --- > src/mesa/drivers/dri/i965/brw_vec4.cpp | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git

Re: [Mesa-dev] [PATCH] i965: use mt->offset in intel_miptree_map_movntdqa().

On 08/02/2016 04:35 PM, Chad Versace wrote: On 08/02/2016 04:17 PM, Haixia Shi wrote: We need to include mt->offset in the calculation of src pointer because its value may be non-zero, for example in a cubemap texture. Signed-off-by: Haixia Shi Cc: Jason Ekstrand

[Mesa-dev] [PATCH v2] i965: use mt->offset in intel_miptree_map_movntdqa().

We need to include mt->offset in the calculation of src pointer because its value may be non-zero, for example in a cubemap texture. Signed-off-by: Haixia Shi Cc: Jason Ekstrand Cc: kenneth.w.grau...@intel.com Cc: Chad Versace

Re: [Mesa-dev] [PATCH v2] i965: use mt->offset in intel_miptree_map_movntdqa().

On 08/02/2016 05:03 PM, Haixia Shi wrote: We need to include mt->offset in the calculation of src pointer because its value may be non-zero, for example in a cubemap texture. Signed-off-by: Haixia Shi Cc: Jason Ekstrand Cc:

Re: [Mesa-dev] [PATCH] nir: add missing x swizzle in nir_print()

On Tue, Aug 2, 2016 at 5:28 PM, Timothy Arceri wrote: > Currently we only print the swizzle if the components are not > used in order. This means x is not printed when it is the only > component used. By checking if the last component is used we > can print the

Re: [Mesa-dev] [PATCH] i965: use mt->offset in intel_miptree_map_movntdqa().

On 08/02/2016 04:17 PM, Haixia Shi wrote: We need to include mt->offset in the calculation of src pointer because its value may be non-zero, for example in a cubemap texture. Signed-off-by: Haixia Shi Cc: Jason Ekstrand Cc:

[Mesa-dev] [PATCH] nir: add missing x swizzle in nir_print()

Currently we only print the swizzle if the components are not used in order. This means x is not printed when it is the only component used. By checking if the last component is used we can print the swizzle for this case. --- src/compiler/nir/nir_print.c | 10 +- 1 file changed, 9

Re: [Mesa-dev] [PATCH 33/95] i965/vec4: implement d2b

Iago Toral Quiroga writes: > --- > src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp > b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp > index 1525a3d..4014020

Re: [Mesa-dev] [PATCH] nir: Always print non-identity swizzles.

Seems reasonable to be too. Rb if you'd like On Aug 2, 2016 8:25 PM, "Timothy Arceri" wrote: > Reviewed-by: Timothy Arceri > ___ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org >

Re: [Mesa-dev] [PATCH 25/95] i965/vec4: fix base offset for nir_registers with doubles

Iago Toral Quiroga writes: > --- > src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp > b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp > index cf35f2e..fde7b60

[Mesa-dev] non-shared glapi still needed ?

Hi folks, is there still a real need for having non-shared glapi ? Otherwise, should we remove it ? --mtx ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] dri: use 'bool' instead of 'int' for boolean variables

Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> --- src/egl/drivers/dri2/egl_dri2.h | 19 ++- src/egl/drivers/dri2/platform_drm.c | 6 +++--- src/egl/drivers/dri2/platform_wayland.c | 20 ++--

[Mesa-dev] [PATCH] dir-locals.el: show-trailing-whitespace and whitespace support

Signed-off-by: Andres Gomez --- .dir-locals.el | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/.dir-locals.el b/.dir-locals.el index 4b53931..9c89a63 100644 --- a/.dir-locals.el +++ b/.dir-locals.el @@ -1,4 +1,5 @@ -((prog-mode +((nil .

Re: [Mesa-dev] [PATCH] dri: use 'bool' instead 'int' for boolean variables

On Mon, Aug 01, 2016 at 03:43:17PM +0200, ⚛ wrote: > On Mon, Aug 1, 2016 at 2:46 PM, Eric Engestrom > wrote: > > On Sun, Jul 31, 2016 at 05:49:02PM +0200, Jan Ziak wrote: > >> Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> > > > > This is a

Re: [Mesa-dev] [PATCH] dri: use 'bool' instead of 'int' for boolean variables

On Tue, Aug 02, 2016 at 05:32:16PM +0200, Jan Ziak wrote: > Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> Much better with the s/0/false/;s/1/true/ :) Reviewed-by: Eric Engestrom > --- > src/egl/drivers/dri2/egl_dri2.h | 19

[Mesa-dev] [PATCH mesa] glx/dri3: remove never-used field

This field was added in 2d94601582e4f0fcaf8c ("Add DRI3+Present loader"), which was a v2. Guessing it was used in v1 and Keith forgot to remove it in v2. Signed-off-by: Eric Engestrom --- Fair warning: This patch will conflict with Jan Ziak's "[PATCH] dri: use 'bool'

Re: [Mesa-dev] [PATCH] ast: Updated AST_NUM_OPERATORS for coherence with ast_operators

On Tue, Aug 02, 2016 at 12:20:54PM +0300, Andres Gomez wrote: > On Mon, 2016-08-01 at 14:02 +0100, Eric Engestrom wrote: > > On Sun, Jul 31, 2016 at 07:07:34PM +0300, Andres Gomez wrote: > > [snip] > > > > diff --git a/src/compiler/glsl/ast.h b/src/compiler/glsl/ast.h > > > index

Re: [Mesa-dev] [PATCH] vbo: fix debug message format string in case of 32-bit machines

On 08/02/2016 08:45 AM, ⚛ wrote: On Tue, Aug 2, 2016 at 4:13 PM, Brian Paul wrote: In bufferobj.c, it looks like we're just using %d or %s and casting the arguments to int. But in other places, we're using %ld and casting to long. That's not going to accurately report

Re: [Mesa-dev] [PATCH] vbo: fix format string compiler warning for 32-bit machines

On 08/02/2016 08:40 AM, Jan Ziak wrote: Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com> --- src/mesa/vbo/vbo_exec_array.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c

Re: [Mesa-dev] [PATCH v2 7/7] egl/android: Make drm_gralloc headers optional (v2)

On Wed, Aug 3, 2016 at 5:12 AM, Rob Herring wrote: > On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: >> Make the code at least compile when being built without drm_gralloc >> headers. >> >> v2: Replaced #ifdefs with stubs for gralloc_drm_get_gem_handle()

Re: [Mesa-dev] [PATCH v2] i965: use mt->offset in intel_miptree_map_movntdqa().

On Tuesday, August 2, 2016 5:03:32 PM PDT Haixia Shi wrote: > We need to include mt->offset in the calculation of src pointer because its > value may be non-zero, for example in a cubemap texture. > > Signed-off-by: Haixia Shi > Cc: Jason Ekstrand >

Re: [Mesa-dev] [PATCH 28/95] i965/vec4: fix register allocation for 64-bit undef sources

Iago Toral Quiroga writes: > --- > src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp > b/src/mesa/drivers/dri/i965/brw_vec4_nir.cpp > index 7b8e30d..65fa057 100644

Re: [Mesa-dev] [PATCH] ast: Updated AST_NUM_OPERATORS for coherence with ast_operators

On Tue, 2016-08-02 at 17:04 +0100, Eric Engestrom wrote: > On Tue, Aug 02, 2016 at 12:20:54PM +0300, Andres Gomez wrote: [snip] > > I'm not sure I'm understanding what you mean. > > > > If you mean to remove the #define and add the value as and additional > > element to the enum, the existence

[Mesa-dev] [PATCH 1/2] anv/blit2d: Add a format parameter to bind_dst and create_iview

Signed-off-by: Jasosn Ekstrand Cc: "12.0" --- src/intel/vulkan/anv_meta_blit2d.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/src/intel/vulkan/anv_meta_blit2d.c b/src/intel/vulkan/anv_meta_blit2d.c

[Mesa-dev] [PATCH 2/2] anv/blit2d: Add support for RGB destinations

This fixes 104 of the new image_clearing and copy_and_blit Vulkan CTS tests. Signed-off-by: Jason Ekstrand Cc: "12.0" --- src/intel/vulkan/anv_meta_blit2d.c | 210 - 1 file changed, 208 insertions(+),

Re: [Mesa-dev] [PATCH v2 3/7] egl/android: Fix support for pbuffers (v2)

On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: > From: Nicolas Boichat > > Existing image loader code supports creating images only for window > surfaces. Moreover droid_create_surface() passes wrong surface type to > dri2_get_dri_config(),

Re: [Mesa-dev] [PATCH 12/12] gallium/radeon/winsyses: decrease max_alloc_size to 1/3 of largest heap

Am 02.08.2016 um 14:57 schrieb Alex Deucher: On Tue, Aug 2, 2016 at 4:55 AM, Marek Olšák wrote: On Tue, Aug 2, 2016 at 3:13 AM, Michel Dänzer wrote: On 01.08.2016 16:35, Michel Dänzer wrote: On 30.07.2016 06:42, Marek Olšák wrote: From: Marek Olšák

Re: [Mesa-dev] [PATCH] i965: Rework the unlit centroid workaround.

On Mon, Aug 1, 2016 at 8:59 PM, Kenneth Graunke wrote: > Previously, for every input, we moved the dispatch mask to the flag > register, then emitted two predicated PLN instructions, one with > centroid barycentric coordinates (for normal pixels), and one with > pixel

Re: [Mesa-dev] [PATCH 09/21] i965/fs: Rework representation of fragment output locations in NIR.

On Sunday, July 31, 2016 12:22:40 PM PDT Francisco Jerez wrote: > Kenneth Graunke writes: > > > On Friday, July 22, 2016 8:59:03 PM PDT Francisco Jerez wrote: > >> The problem with the current approach is that driver output locations > >> are represented as a linear offset

Re: [Mesa-dev] [PATCH] configure.ac/swr: build swr with -fno-strict-aliasing

> On Aug 2, 2016, at 1:00 PM, Matt Turner wrote: > > On Tue, Aug 2, 2016 at 10:53 AM, Tim Rowley > wrote: >> swr rasterizer contains numerous data transfers between vectors >> and ordinary C types. Fixing for strict aliasing will take time. >

[Mesa-dev] [PATCH v2] swr: build swr with -fno-strict-aliasing

swr rasterizer contains numerous data transfers between vectors and ordinary C types. Fixing for strict aliasing will take time. --- src/gallium/drivers/swr/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/swr/Makefile.am b/src/gallium/drivers/swr/Makefile.am

Re: [Mesa-dev] [PATCH] i965: Rework the unlit centroid workaround.

On Aug 2, 2016 12:36 PM, "Matt Turner" wrote: > > On Mon, Aug 1, 2016 at 8:59 PM, Kenneth Graunke wrote: > > Previously, for every input, we moved the dispatch mask to the flag > > register, then emitted two predicated PLN instructions, one with > >

[Mesa-dev] [PATCH] configure.ac/swr: build swr with -fno-strict-aliasing

swr rasterizer contains numerous data transfers between vectors and ordinary C types. Fixing for strict aliasing will take time. --- configure.ac| 7 +++ src/gallium/drivers/swr/Makefile.am | 1 + 2 files changed, 8 insertions(+) diff --git a/configure.ac

[Mesa-dev] [PATCH] configure.ac: add llvm inteljitevents component if enabled

Needed to successfully link llvmpipe or swr when using shared llvm libs. --- configure.ac | 5 + 1 file changed, 5 insertions(+) diff --git a/configure.ac b/configure.ac index fb4a12a..edbc95b 100644 --- a/configure.ac +++ b/configure.ac @@ -2497,6 +2497,11 @@ dnl in LLVM_LIBS. if test

Re: [Mesa-dev] [PATCH] configure.ac/swr: build swr with -fno-strict-aliasing

On Tue, Aug 2, 2016 at 10:53 AM, Tim Rowley wrote: > swr rasterizer contains numerous data transfers between vectors > and ordinary C types. Fixing for strict aliasing will take time. Oh, sorry! I forgot about swr. > --- > configure.ac| 7

Re: [Mesa-dev] [PATCH 00/12] RadeonSI: Prevent IB submissions with illegal memory usage

On Tue, Aug 2, 2016 at 5:15 PM, Nicolai Hähnle wrote: > On 29.07.2016 23:42, Marek Olšák wrote: >> >> This series ensures that memory usage of gfx IBs is below the kernel- >> exposed memory limits in most cases. >> >> It's not possible to prevent CS ioctl failures completely

Re: [Mesa-dev] [PATCH v2] swr: build swr with -fno-strict-aliasing

Thanks! Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 1/7] winsys/amdgpu: query ME/PFP/CE firmware versions

From: Nicolai Hähnle The radeon kernel module doesn't have the firmware query interface, so the corresponding values will remain 0. --- src/gallium/drivers/radeon/r600_pipe_common.c | 3 +++ src/gallium/drivers/radeon/radeon_winsys.h| 3 +++

[Mesa-dev] [PATCH 4/7] radeonsi: transpose indirect/index draw dispatch

From: Nicolai Hähnle This allows better code sharing for indirect draw calls. --- src/gallium/drivers/radeonsi/si_state_draw.c | 76 1 file changed, 31 insertions(+), 45 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c

[Mesa-dev] [PATCH 3/7] radeonsi: move index buffer calculations in si_emit_draw_packets up

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_state_draw.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index

[Mesa-dev] [PATCH 6/7] radeonsi: program multi draw and indirect draw count parameters

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_state_draw.c | 23 ++- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index

Re: [Mesa-dev] [PATCH v2 2/7] egl/android: Respect buffer mask in droid_image_get_buffers (v2)

On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: > Drivers can request different set of buffers depending on the buffer > mask they pass to the get_buffers callback. This patch makes > droid_image_get_buffers() respect this mask. > > v2: Return error only in case of real

Re: [Mesa-dev] [PATCH v2 1/7] egl/android: Remove unused variables in droid_get_buffers_with_format()

On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: > Fix compilation warnings due to unused variables left after some earlier > code changes. > > Signed-off-by: Tomasz Figa Tested-by: Rob Herring

Re: [Mesa-dev] [PATCH v2 5/7] egl/android: Make get_fourcc() accept HAL formats

On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: > There are DRI_IMAGE_FOURCC macros, for which there are no corresponding > DRI_IMAGE_FORMAT macros. To support such formats we need to make the > lookup function take the native format directly. As a side effect, it >

Re: [Mesa-dev] [PATCH 0/3] RadeonSI GLSL "lit" tests

On Tue, Aug 2, 2016 at 3:40 PM, Nicolai Hähnle wrote: > On 31.07.2016 22:29, Marek Olšák wrote: >> >> Inspired by LLVM, I created a small test suite that looks like "lit" >> tests, except that the input is GLSL and it verifies asm. >> >> The motivation was to verify that we

Re: [Mesa-dev] [PATCH 0/3] RadeonSI GLSL "lit" tests

On 02.08.2016 21:06, Marek Olšák wrote: On Tue, Aug 2, 2016 at 3:40 PM, Nicolai Hähnle wrote: On 31.07.2016 22:29, Marek Olšák wrote: Inspired by LLVM, I created a small test suite that looks like "lit" tests, except that the input is GLSL and it verifies asm. The

Re: [Mesa-dev] [PATCH v2 6/7] egl/android: Add support for YV12 pixel format (v2)

On Tue, Aug 2, 2016 at 6:07 AM, Tomasz Figa wrote: > This patch adds support for YV12 pixel format to the Android platform > backend. Only creating EGL images is supported, it is not added to the > list of available visuals. > > v2: Use const array defined just for YV12

  1   2   >