Re: [Mesa-dev] [PATCH 2/2] Android: EGL: fix missing nativewindow.h include on O

2017-08-24 22:02 GMT+08:00 Rob Herring : > On Thu, Aug 24, 2017 at 4:08 AM, Chih-Wei Huang > wrote: >> 2017-08-24 1:25 GMT+08:00 Rob Herring : >> >> I'm also trying to fix it. >> Seems it only requires the headers instead >> of the

Re: [Mesa-dev] [PATCH 00/10] gallium: normalize CONST file accesses to 2D

Am 23.08.2017 18:41, schrieb Nicolai Hähnle: Hi all, Following the discussion on Timothy's std430 packing series, here's a quick proposal to just always use 2D accesses to the CONST file in TGSI. The first patch should be sufficient for all drivers to accept those 2D accesses. It seems that

Re: [Mesa-dev] [PATCH 1/2] anv: implementation of VK_EXT_debug_report extension

On 08/25/2017 08:08 AM, Jason Ekstrand wrote: On Thu, Aug 24, 2017 at 9:52 PM, Tapani Pälli > wrote: + + vk_foreach_struct(info, pCreateInfo) { Usually, we handle the primary structure directly and

Re: [Mesa-dev] [PATCH 1/2] anv: implementation of VK_EXT_debug_report extension

On Thu, Aug 24, 2017 at 9:52 PM, Tapani Pälli wrote: > Hi; > > On 08/24/2017 08:36 PM, Jason Ekstrand wrote: > >> On Wed, Aug 23, 2017 at 11:23 PM, Tapani Pälli > > wrote: >> >> Patch adds required functionality

Re: [Mesa-dev] [PATCH] i965: add 2xMSAA and 16xMSAA to DRI configs for Gen9.

Hi; On 08/25/2017 12:30 AM, Kenneth Graunke wrote: On Thursday, August 24, 2017 4:16:39 AM PDT kevin.rogo...@intel.com wrote: From: Kevin Rogovin Special thanks to Eero Tamminen for reporting rasterizer numbers being twice what it should be for 2xMSAA under a

Re: [Mesa-dev] [PATCH 1/2] anv: implementation of VK_EXT_debug_report extension

Hi; On 08/24/2017 08:36 PM, Jason Ekstrand wrote: On Wed, Aug 23, 2017 at 11:23 PM, Tapani Pälli > wrote: Patch adds required functionality for extension to manage a list of application provided callbacks and handle debug

[Mesa-dev] [PATCH] a2xx: add support for a few 16-bit color rendering formats

The rest should be possible too, just needs some additional investigation. Passes fbo-*-formats piglit tests. Signed-off-by: Ilia Mirkin --- src/gallium/drivers/freedreno/a2xx/fd2_gmem.c | 5 + src/gallium/drivers/freedreno/a2xx/fd2_screen.c | 7 ++- 2 files

[Mesa-dev] [PATCH] disk_cache: assert if a cache entries keys don't match mesa

In ef42423e7be9 I enable the check for release builds however we still want to assert in debug builds to alert to collisions or just general bugs with the key building/compare code. Otherwise it will just fail silently effectively disabling the cache. --- src/util/disk_cache.c | 4 +++- 1 file

Re: [Mesa-dev] [PATCH V3 1/5] util/disk_cache: rename mesa cache dir and introduce cache versioning

For the series: Tested-by: Dieter Nützel on RX580 with Nine Dieter Am 24.08.2017 03:11, schrieb Timothy Arceri: Steam is already analysing cache items, unfortunatly we did not introduce a versioning mechanism for identifying structural changes to cache entries earlier

Re: [Mesa-dev] [PATCH v2 08/10] anv: Use DRM sync objects to back fences whenever possible

A v3 of this will be coming tomorrow. Dave asked me to rework some kernel apis. On August 24, 2017 4:49:53 PM Jason Ekstrand wrote: In order to implement VK_KHR_external_fence, we need to back our fences with something that's shareable. Since the kernel wait

Re: [Mesa-dev] [PATCH 0/5] intel/isl: Set MOCS based on view usage

Ken, Did you ever look at this? --Jason On August 23, 2017 12:50:25 AM "Pohjolainen, Topi" wrote: On Tue, Aug 01, 2017 at 03:48:29PM -0700, Jason Ekstrand wrote: This little series changes things around so that, instead of passing MOCS values into ISL, ISL

[Mesa-dev] [Bug 102394] RBDOOM3BFG digital vomit

https://bugs.freedesktop.org/show_bug.cgi?id=102394 --- Comment #2 from dlol...@gmail.com --- To be honest I'm not sure what version it worked. On my laptop which runs a Radeon HD 6000 series GPU, also ran fine a few weeks ago, and then this started to happen after an update. Removing the Padoka

Re: [Mesa-dev] [PATCH v9 0/7] mesa/st: glsl_to_tgsi: refined register merge algorithm

On 25/08/17 02:38 AM, Gert Wollny wrote: > > The patch doesn't introduce piglit regression (I tested the shader subset). I'd recommend testing at least the gpu profile, ideally running on X. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast

[Mesa-dev] [Bug 102394] RBDOOM3BFG digital vomit

https://bugs.freedesktop.org/show_bug.cgi?id=102394 --- Comment #1 from Timothy Arceri --- Please find out what version of Mesa was working vs the Padoka PPA version and add that to this bug report. To get that you can use: glxinfo | grep OpenGL >From there it would be

[Mesa-dev] [PATCH 2/2] radeonsi: rewrite late alloc VS limit computation

From: Marek Olšák This is still very simple, but it's better than before. Loosely ported from Vulkan. --- src/gallium/drivers/radeonsi/si_state.c | 38 ++--- 1 file changed, 26 insertions(+), 12 deletions(-) diff --git

[Mesa-dev] [PATCH 1/2] gallium/radeon: set EVENT_WRITE_EOP.INT_SEL = wait for write confirmation

From: Marek Olšák Ported from Vulkan. Not sure what this is good for.. maybe write confirmation from L2 flushes? --- src/amd/common/r600d_common.h | 3 +++ src/gallium/drivers/radeon/r600_pipe_common.c | 12 +--- 2 files changed, 12 insertions(+), 3

Re: [Mesa-dev] [PATCH 00/14] KHR_no_error support to various TFB functions

On 24/08/17 23:21, Samuel Pitoiset wrote: Hi, Some more simple patches for KHR_no_error support. Please review, thanks! With the change suggested in patch 3 series is: Reviewed-by: Timothy Arceri ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH 03/14] mesa: add begin_transform_feedback() helper

On 24/08/17 23:21, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- src/mesa/main/transformfeedback.c | 49 --- 1 file changed, 30 insertions(+), 19 deletions(-) diff --git a/src/mesa/main/transformfeedback.c

Re: [Mesa-dev] [PATCH 3/3] Revert "st/va: add enviromental variable to disable interlace"

On 25/08/17 12:11 AM, Leo Liu wrote: > This reverts commit 10dec2de2d9f568675d66d736b48701fa26f7b50. > > The environment variable is no longer needed with the previous change Thanks for clarifying the commit log. Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer

Re: [Mesa-dev] [PATCH] mesa: port the LastLookedUpVAO optimisation to _mesa_lookup_vao()

Thanks for fixing this I'd been meaning to look into it. Reviewed-by: Timothy Arceri On 24/08/17 23:53, Samuel Pitoiset wrote: It was only used in the errors path. Signed-off-by: Samuel Pitoiset --- src/mesa/main/arrayobj.c | 20

Re: [Mesa-dev] [PATCH] mesa: don't error check the default buffer object in glBindBufferOffsetEXT()

Reviewed-by: Timothy Arceri On 24/08/17 21:44, Samuel Pitoiset wrote: An allocation check is already done when the buffer is created at context creation. Signed-off-by: Samuel Pitoiset --- src/mesa/main/transformfeedback.c | 11 +--

[Mesa-dev] [PATCH] radv/gfx9: fix buffer size on gfx9.

From: Dave Airlie The VI sizing only applies to VI. This fixes: dEQP-VK.image.image_size.buffer.* Signed-off-by: Dave Airlie --- src/amd/common/ac_nir_to_llvm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH 2/4] mesa: check allocation failures in new_transform_feedback()

On 24/08/17 21:43, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- src/mesa/main/transformfeedback.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/main/transformfeedback.c b/src/mesa/main/transformfeedback.c index

Re: [Mesa-dev] [PATCH] radv: fix predication on gfx9

Both are Reviewed-by: Bas Nieuwenhuizen On Fri, Aug 25, 2017, at 01:42, Dave Airlie wrote: > From: Dave Airlie > > When I added gfx9 I did it wrong, this fixes it. > > Fixes: 5247b311e9 "radv/gfx9: fix set predication packet." > Signed-off-by:

[Mesa-dev] [PATCH v2 10/10] anv: Add support for the SYNC_FD handle type for fences

--- src/intel/vulkan/anv_gem.c | 28 + src/intel/vulkan/anv_gem_stubs.c | 13 ++ src/intel/vulkan/anv_private.h | 4 +++ src/intel/vulkan/anv_queue.c | 53 +++- 4 files changed, 87 insertions(+), 11 deletions(-) diff

[Mesa-dev] [PATCH v2 04/10] anv: Rename anv_fence_state to anv_bo_fence_state

It only applies to legacy BO fences. --- src/intel/vulkan/anv_batch_chain.c | 2 +- src/intel/vulkan/anv_private.h | 10 +- src/intel/vulkan/anv_queue.c | 24 3 files changed, 18 insertions(+), 18 deletions(-) diff --git

[Mesa-dev] [PATCH v2 08/10] anv: Use DRM sync objects to back fences whenever possible

In order to implement VK_KHR_external_fence, we need to back our fences with something that's shareable. Since the kernel wait interface for sync objects already supports waiting for multiple fences in one go, it makes anv_WaitForFences much simpler if we only have one type of fence. ---

Re: [Mesa-dev] [PATCH] glsl: fix glsl_struct_field size calculations for shader cache

Whoops. Reviewed-by: Timothy Arceri Thanks! On 24/08/17 23:42, Nicolai Hähnle wrote: From: Nicolai Hähnle Found by address sanitizer: ==22621==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x6140cbd8 at pc 0x7f561610a4ff bp

[Mesa-dev] [PATCH v2 09/10] anv: Implement VK_KHR_external_fence

--- src/intel/vulkan/anv_batch_chain.c | 19 - src/intel/vulkan/anv_extensions.py | 5 ++ src/intel/vulkan/anv_queue.c | 142 - 3 files changed, 161 insertions(+), 5 deletions(-) diff --git a/src/intel/vulkan/anv_batch_chain.c

[Mesa-dev] [PATCH v2 07/10] anv/gem: Add support for syncobj wait and reset

--- src/intel/vulkan/anv_gem.c | 71 src/intel/vulkan/anv_gem_stubs.c | 26 +++ src/intel/vulkan/anv_private.h | 6 3 files changed, 103 insertions(+) diff --git a/src/intel/vulkan/anv_gem.c b/src/intel/vulkan/anv_gem.c index

[Mesa-dev] [PATCH v2 06/10] drm-uapi/drm: Add DRM_IOCTL_SYNCOBJ_WAIT and RESET

--- include/drm-uapi/drm.h | 25 + 1 file changed, 25 insertions(+) diff --git a/include/drm-uapi/drm.h b/include/drm-uapi/drm.h index bf3674a..f93f80a 100644 --- a/include/drm-uapi/drm.h +++ b/include/drm-uapi/drm.h @@ -712,6 +712,28 @@ struct drm_syncobj_handle {

[Mesa-dev] [PATCH v2 05/10] vulkan/util: Add a vk_zalloc helper

--- src/vulkan/util/vk_alloc.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/vulkan/util/vk_alloc.h b/src/vulkan/util/vk_alloc.h index 2915021..f58a806 100644 --- a/src/vulkan/util/vk_alloc.h +++ b/src/vulkan/util/vk_alloc.h @@ -37,6 +37,20 @@ vk_alloc(const

[Mesa-dev] [PATCH v2 01/10] anv: Rework fences to work more like BO semaphores

This commit changes fences to work a bit more like BO semaphores. Instead of the fence being a batch, it's simply a BO that gets added to the validation list for the last execbuf call in the QueueSubmit operation. It's a bit annoying finding the last submit in the execbuf but this allows us to

[Mesa-dev] [PATCH v2 03/10] anv: Pull the guts of anv_fence into anv_fence_impl

This is just a refactor, similar to what we did for semaphores, in preparation for handling VK_KHR_external_fence. --- src/intel/vulkan/anv_batch_chain.c | 22 -- src/intel/vulkan/anv_private.h | 42 ++- src/intel/vulkan/anv_queue.c | 144

[Mesa-dev] [PATCH v2 02/10] anv/wsi: Use QueueSubmit to trigger the fence in AcquireNextImage

--- src/intel/vulkan/anv_wsi.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/intel/vulkan/anv_wsi.c b/src/intel/vulkan/anv_wsi.c index 9369f26..00edb22 100644 --- a/src/intel/vulkan/anv_wsi.c +++ b/src/intel/vulkan/anv_wsi.c @@ -364,22 +364,25 @@ VkResult

[Mesa-dev] [PATCH] radv/gfx9: gfx9 has buffer sizing rules like pre-VI.

From: Dave Airlie This fixes: dEQP-VK.robustness.buffer_access.* on GFX9. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_image.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/amd/vulkan/radv_image.c

[Mesa-dev] [PATCH] radv: fix predication on gfx9

From: Dave Airlie When I added gfx9 I did it wrong, this fixes it. Fixes: 5247b311e9 "radv/gfx9: fix set predication packet." Signed-off-by: Dave Airlie --- src/amd/vulkan/si_cmd_buffer.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

Re: [Mesa-dev] [PATCH 1/3] st/omx: move YUV deinterlace function to common

Christian König wrote: Am 24.08.2017 um 17:11 schrieb Leo Liu: Signed-off-by: Leo Liu Reviewed-by: Christian König for the series. Andy do you want to test this? Should make VA-API transcoding simpler to use. Oh, nice it will be great to loose

Re: [Mesa-dev] [PATCH 2/2] radv: Don't set a new subpass on compute resolve.

On 25 August 2017 at 09:17, Bas Nieuwenhuizen wrote: > We don't use the render path so totally unneeded. > > Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" I think there is a more recent commit to fix this one :-) either way, Reviewed-by:

[Mesa-dev] [PATCH 2/2] radv: Don't set a new subpass on compute resolve.

We don't use the render path so totally unneeded. Fixes: f4e499ec791 "radv: add initial non-conformant radv vulkan driver" --- src/amd/vulkan/radv_meta_resolve_cs.c | 8 1 file changed, 8 deletions(-) diff --git a/src/amd/vulkan/radv_meta_resolve_cs.c

[Mesa-dev] [PATCH 1/2] radv: Remove some intel comments from the resolve code.

These are clearly not applicable to radv. --- src/amd/vulkan/radv_meta_resolve.c| 7 --- src/amd/vulkan/radv_meta_resolve_cs.c | 7 --- src/amd/vulkan/radv_meta_resolve_fs.c | 7 --- 3 files changed, 21 deletions(-) diff --git a/src/amd/vulkan/radv_meta_resolve.c

Re: [Mesa-dev] glmark2 terrain errors on imx6q

When running on Mesa 17.2, I believe this has already been fixed with the GC3000 but is still not working with the GC2000. Here's the fix I'm thinking addresses the issue with the GC3000:

[Mesa-dev] [Bug 101851] [regression] libEGL_common.a undefined reference to '__gxx_personality_v0'

https://bugs.freedesktop.org/show_bug.cgi?id=101851 Alec Moskvin changed: What|Removed |Added CC||al...@gmx.com -- You

[Mesa-dev] glmark2 terrain errors on imx6q

Hi, Getting the following errors when running glmark2 terrain test on imx6q: # glmark2-es2-drm -b terrain ** Failed to set swap interval. Results may be bounded above by refresh rate. === glmark2 2014.03

Re: [Mesa-dev] [PATCH] ac/debug: use util_strchrnul() to fix android build error

On Thu, Aug 24, 2017 at 4:52 PM, Mauro Rossi wrote: > Similar to e09d04cd56 "radeonsi: use util_strchrnul() to fix android build > error" > > Android Bionic does not support strchrnul() string function, > gallium auxiliary util/u_string.h provides util_strchrnul() > > This

Re: [Mesa-dev] [PATCH] ac/debug: use util_strchrnul() to fix android build error

Reviewed-by: Marek Olšák Marek On Thu, Aug 24, 2017 at 11:52 PM, Mauro Rossi wrote: > Similar to e09d04cd56 "radeonsi: use util_strchrnul() to fix android build > error" > > Android Bionic does not support strchrnul() string function, > gallium

[Mesa-dev] [PATCH] ac/debug: use util_strchrnul() to fix android build error

Similar to e09d04cd56 "radeonsi: use util_strchrnul() to fix android build error" Android Bionic does not support strchrnul() string function, gallium auxiliary util/u_string.h provides util_strchrnul() This change avoids the following warning and error:

Re: [Mesa-dev] [PATCH] radv/meta: missing initialisations in create_pass().

On Thu, Aug 24, 2017, at 11:10, Xavier Bouchoux wrote: > Otherwise radv_cmd_state_setup_attachments() will complain it has no > clearvalues, > when called via radv_process_depth_image_inplace(). > > Signed-off-by: Xavier Bouchoux > --- > src/amd/vulkan/radv_meta_decompress.c

Re: [Mesa-dev] [PATCH] mesa: Implement GL_ARB_polygon_offset_clamp

My personal inclination would have been the inverse (i.e. alias glPolygonOffsetClampEXT to glPolygonOffsetClamp), but ... you've been through enough silly revisions, and I don't sufficiently care. Reviewed-by: Ilia Mirkin On Thu, Aug 24, 2017 at 5:00 PM, Adam Jackson

Re: [Mesa-dev] [PATCH 4/6] ac/debug: annotate IB dumps with the raw values

On Thu, Aug 24, 2017 at 4:24 PM, Rob Herring wrote: > On Tue, Aug 22, 2017 at 10:45 AM, Nicolai Hähnle wrote: >> From: Nicolai Hähnle >> >> --- >> src/amd/common/ac_debug.c | 84 >> +-- >>

Re: [Mesa-dev] [PATCH] i965: add 2xMSAA and 16xMSAA to DRI configs for Gen9.

On Thursday, August 24, 2017 4:16:39 AM PDT kevin.rogo...@intel.com wrote: > From: Kevin Rogovin > > Special thanks to Eero Tamminen for reporting rasterizer > numbers being twice what it should be for 2xMSAA under > a benchmark. > > Signed-off-by: Kevin Rogovin

Re: [Mesa-dev] [PATCH 4/6] ac/debug: annotate IB dumps with the raw values

On Tue, Aug 22, 2017 at 10:45 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > --- > src/amd/common/ac_debug.c | 84 > +-- > 1 file changed, 66 insertions(+), 18 deletions(-) > > diff --git

[Mesa-dev] [PATCH] mesa: Implement GL_ARB_polygon_offset_clamp

Semantically identical to the EXT version (whose string is still valid for GLES), so rename the bit but expose both extension strings. (Suggested by Ilia Mirkin and Ian Romanick.) v3: Fix the entrypoint alias in GL4x.xml (Ilia) Signed-off-by: Adam Jackson --- docs/features.txt

[Mesa-dev] [Bug 102377] PIPE_*_4BYTE_ALIGNED_ONLY caps crashing

https://bugs.freedesktop.org/show_bug.cgi?id=102377 --- Comment #3 from Bruce Cherniak --- Tim headed out on vacation today. He'll be out nearly 2 weeks. I'm going to try to get to testing this in the next day or so. -- You are receiving this mail because: You are

[Mesa-dev] [Bug 102377] PIPE_*_4BYTE_ALIGNED_ONLY caps crashing

https://bugs.freedesktop.org/show_bug.cgi?id=102377 --- Comment #2 from Brian Paul --- Tim, I'll hold off on pushing this patch until you can test it. -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the

Re: [Mesa-dev] [PATCH 41/47] i965/fs: Add reuse_16bit_conversions_register optimization

Hi Alejandro, This seems really suspicious. If the live ranges are really independent, then the register allocator should be able to assign the two virtual registers to the same physical register if it needs to. This change forces the two to be the same, which constrains the register allocator

[Mesa-dev] [PATCH] egl/drm: Don't "fall back" to /dev/dri/card0 if the first open fails

The snprintf stuff here already constructs the right name for the device node, and if it doesn't, you configured Mesa wrong, don't do that. Signed-off-by: Adam Jackson --- src/egl/drivers/dri2/platform_drm.c | 2 -- 1 file changed, 2 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 2/2] mesa: Implement GL_ARB_polygon_offset_clamp

On Thu, Aug 24, 2017 at 2:40 PM, Adam Jackson wrote: > Semantically identical to the EXT version (whose string is still valid > for GLES), so rename the bit but expose both extension strings. > (Suggested by Ilia Mirkin and Ian Romanick.) > > Signed-off-by: Adam Jackson

[Mesa-dev] [PATCH] st/va: move YUV content to deinterlaced buffer when reallocated for encoder

v2: use deinterlace common function v3: make sure deinterlace only Signed-off-by: Leo Liu --- src/gallium/state_trackers/va/picture.c | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/src/gallium/state_trackers/va/picture.c

Re: [Mesa-dev] [PATCH 1/3] st/omx: move YUV deinterlace function to common

On 08/24/2017 11:34 AM, Christian König wrote: Am 24.08.2017 um 17:11 schrieb Leo Liu: Signed-off-by: Leo Liu Reviewed-by: Christian König for the series. Andy do you want to test this? Should make VA-API transcoding simpler to use. Just got

Re: [Mesa-dev] [PATCH mesa] egl/android: add missing include

On Thu, Aug 24, 2017 at 10:08 AM, Eric Engestrom wrote: > On Thursday, 2017-08-24 10:02:29 -0500, Rob Herring wrote: >> On Thu, Aug 24, 2017 at 9:26 AM, Rob Herring wrote: >> > On Thu, Aug 24, 2017 at 9:22 AM, Eric Engestrom >> >

[Mesa-dev] [PATCH 2/2] mesa: Implement GL_ARB_polygon_offset_clamp

Semantically identical to the EXT version (whose string is still valid for GLES), so rename the bit but expose both extension strings. (Suggested by Ilia Mirkin and Ian Romanick.) Signed-off-by: Adam Jackson --- docs/features.txt| 2 +-

[Mesa-dev] [PATCH 1/2] mesa: Implement GL_ARB_texture_filter_anisotropic

The only difference from the EXT version is bumping the minmax to 16, so just hit all the drivers at once. v2: Fix driver names, add to 17.3 release notes (Ilia Mirkin) Reviewed-by: Ilia Mirkin Signed-off-by: Adam Jackson --- docs/features.txt

Re: [Mesa-dev] [PATCH 08/10] anv: Use DRM sync objects to back fences whenever possible

On Thu, Aug 24, 2017 at 10:39 AM, Jason Ekstrand wrote: > On Thu, Aug 24, 2017 at 10:20 AM, Lionel Landwerlin < > lionel.g.landwer...@intel.com> wrote: > >> On 08/08/17 23:45, Jason Ekstrand wrote: >> >>> In order to implement VK_KHR_external_fence, we need to back our

Re: [Mesa-dev] [PATCH 08/10] anv: Use DRM sync objects to back fences whenever possible

On Thu, Aug 24, 2017 at 10:20 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > On 08/08/17 23:45, Jason Ekstrand wrote: > >> In order to implement VK_KHR_external_fence, we need to back our fences >> with something that's shareable. Since the kernel wait interface for >> sync

[Mesa-dev] [PATCH v9 4/7] mesa/st: glsl_to_tgsi: add tests for the new temporary lifetime tracker

This patch adds a set of unit tests for the new lifetime tracker. --- configure.ac |1 + src/mesa/Makefile.am |2 +- .../state_tracker/st_glsl_to_tgsi_temprename.cpp | 12 +-

[Mesa-dev] [PATCH v9 7/7] mesa/st: glsl_to_tgsi: tie in new temporary register merge approach

This patch replaces the old register lifetime estiamtion and rename mapping evaluation with the new one. Performance to compare between the current and the new implementation were measured by running the shader-db in one thread. ---

[Mesa-dev] [PATCH v9 5/7] mesa/st: glsl_to_tgsi: add register rename mapping evaluator

The remapping evaluator first sorts the temporary registers ascending based on their first life time instruction, and then uses a binary search to find merge canidates. For the initial sorting it uses std::sort because qsort is quite slow in comparison. By removing the define USE_STL_SORT in

[Mesa-dev] [PATCH v9 6/7] mesa/st: glsl_to_tgsi: Add test set for evaluation of rename mapping

The patch adds tests for the register rename mapping evaluation and combined life time estimation and renaming. --- .../tests/test_glsl_to_tgsi_lifetime.cpp | 192 + 1 file changed, 192 insertions(+) diff --git

[Mesa-dev] [PATCH v9 3/7] mesa/st: glsl_to_tgsi: implement new temporary register lifetime tracker

This patch adds a class for tracking the life times of temporary registers in the glsl to tgsi translation. The algorithm runs in three steps: First, in order to minimize the number of needed memory allocations the program is scanned to evaluate the number of scopes. Then, the program is scanned

[Mesa-dev] [PATCH v9 2/7] mesa/st: glsl_to_tgsi move some helper classes to extra files

To prepare the implementation of a temp register lifetime tracker some of the classes are moved into seperate header/implementation files to make them accessible from other files. Specifically these are: class st_src_reg; class st_dst_reg; class glsl_to_tgsi_instruction; struct

[Mesa-dev] [PATCH v9 0/7] mesa/st: glsl_to_tgsi: refined register merge algorithm

Dear all, I thought I might send out this patch another time with its full history and freshly rebased. All the changes that I applied were a result of reviews by Nicolai (mostly) and Emil (thanks again to both of you). The set is mirroed at

[Mesa-dev] [PATCH v9 1/7] st_glsl_to_tgsi: rewrite rename registers to use array fully.

From: Dave Airlie Instead of having to search the whole array, just use the whole thing and store a valid bit in there with the rename. Removes this from the profile on some of the fp64 tests Reviewed-by: Timothy Arceri Signed-off-by: Dave Airlie

Re: [Mesa-dev] [PATCH 1/2] anv: implementation of VK_EXT_debug_report extension

On Wed, Aug 23, 2017 at 11:23 PM, Tapani Pälli wrote: > Patch adds required functionality for extension to manage a list of > application provided callbacks and handle debug reporting from driver > and application side. > > Signed-off-by: Tapani Pälli

Re: [Mesa-dev] [PATCH 08/10] anv: Use DRM sync objects to back fences whenever possible

On 08/08/17 23:45, Jason Ekstrand wrote: In order to implement VK_KHR_external_fence, we need to back our fences with something that's shareable. Since the kernel wait interface for sync objects already supports waiting for multiple fences in one go, it makes anv_WaitForFences much simpler if

Re: [Mesa-dev] [PATCH] i965: add 2xMSAA and 16xMSAA to DRI configs for Gen9.

On 17-08-24 14:16:39, kevin.rogo...@intel.com wrote: From: Kevin Rogovin Special thanks to Eero Tamminen for reporting rasterizer numbers being twice what it should be for 2xMSAA under a benchmark. Signed-off-by: Kevin Rogovin ---

Re: [Mesa-dev] [PATCH mesa] i965: add missing `const` in function signature

On 17-08-24 11:01:34, Matt Turner wrote: Reviewed-by: Matt Turner I'm blaming this one on someone else's rebase ;-) Reviewed-by: Ben Widawsky ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] gallium/vbuf: fix buffer reference bugs

Reviewed-by: Marek Olšák Marek On Thu, Aug 24, 2017 at 6:48 PM, Brian Paul wrote: > In two places we called pipe_resource_reference() to remove a reference > to a vertex buffer resource. But we neglected to check if the buffer was > a user buffer and

[Mesa-dev] [PATCH] gallium/vbuf: fix buffer reference bugs

In two places we called pipe_resource_reference() to remove a reference to a vertex buffer resource. But we neglected to check if the buffer was a user buffer and not a pipe_resource. This caused us to pass an invalid pipe_resource pointer to pipe_resource_reference(). Instead of calling

[Mesa-dev] [Bug 102377] PIPE_*_4BYTE_ALIGNED_ONLY caps crashing

https://bugs.freedesktop.org/show_bug.cgi?id=102377 --- Comment #1 from Brian Paul --- See proposed patch on mesa-dev. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.___

Re: [Mesa-dev] TGSI 16-bit support

On Thu, Aug 24, 2017 at 3:39 PM, Nicolai Hähnle wrote: > On 24.08.2017 14:19, Erik Faye-Lund wrote: >> >> On Wed, Aug 23, 2017 at 6:19 PM, Marek Olšák wrote: >>> >>> On Wed, Aug 23, 2017 at 3:08 PM, Nicolai Hähnle >>> wrote:

Re: [Mesa-dev] [PATCH 02/47] mesa/st: Handle 16-bit types at st_glsl_storage_type_size()

Reviewed-by: Marek Olšák Marek On Thu, Aug 24, 2017 at 3:54 PM, Alejandro Piñeiro wrote: > From: Eduardo Lima Mitev > > This is basically to avoid "not handle in switch" warnings. > > v2: Let the new types hit the assertion instead.

Re: [Mesa-dev] [PATCH] gallium/docs: add reference links for resource_create method

Reviewed-by: Marek Olšák Marek On Thu, Aug 24, 2017 at 4:17 PM, Gwan-gyeong Mun wrote: > It adds reference links for arguments usage and bind of resource_create(). > > Signed-off-by: Mun Gwan-gyeong > --- >

Re: [Mesa-dev] [PATCH] gallium/docs: fix a reference link for get_paramf

Reviewed-by: Marek Olšák Marek On Thu, Aug 24, 2017 at 3:53 PM, Gwan-gyeong Mun wrote: > Previous get_paramf links same as get_param. It changes the reference link to > PIPE_CAPF_* > > Signed-off-by: Mun Gwan-gyeong > --- >

Re: [Mesa-dev] [PATCH] radeonsi: set IF_THRESHOLD to 4

Acked-by: Marek Olšák Marek On Thu, Aug 24, 2017 at 2:46 PM, Timothy Arceri wrote: > In 74e39de9324d it was set to 3 and it was reported that 4 caused > tesseract to start spilling VGPRs. This no longer seems to be the > case. > > Totals: > SGPRS:

Re: [Mesa-dev] [PATCH 1/4] radeonsi: correct maximum wave count per SIMD

On Thu, Aug 24, 2017 at 3:24 PM, Nicolai Hähnle wrote: > On 24.08.2017 13:32, Marek Olšák wrote: >> >> >> >> On Aug 24, 2017 1:21 PM, "Nicolai Hähnle" > > wrote: >> >> On 24.08.2017 12 :34, Marek Olšák

Re: [Mesa-dev] [PATCH 00/10] gallium: normalize CONST file accesses to 2D

We'll definitely have to adjust some code too, but looks alright to me. For the series: Acked-by: Roland Scheidegger Am 23.08.2017 um 18:41 schrieb Nicolai Hähnle: > Hi all, > > Following the discussion on Timothy's std430 packing series, here's > a quick proposal to just

Re: [Mesa-dev] [PATCH 4/4] radeonsi: get the raster config from AMDGPU on SI

On Thu, Aug 24, 2017 at 3:44 PM, Alex Deucher wrote: > On Thu, Aug 24, 2017 at 4:20 AM, Nicolai Hähnle wrote: >> Patches 2-4: >> >> Reviewed-by: Nicolai Hähnle >> >> >> On 23.08.2017 22:44, Marek Olšák wrote: >>> >>> From:

[Mesa-dev] [Bug 102394] RBDOOM3BFG digital vomit

https://bugs.freedesktop.org/show_bug.cgi?id=102394 Bug ID: 102394 Summary: RBDOOM3BFG digital vomit Product: Mesa Version: unspecified Hardware: Other OS: All Status: NEW Severity: normal

[Mesa-dev] [Bug 100038] [BDW] gpu hangs ecode 8:0:0x84d77c1c

https://bugs.freedesktop.org/show_bug.cgi?id=100038 Elizabeth changed: What|Removed |Added Product|DRI |Mesa

Re: [Mesa-dev] [PATCH 1/3] st/omx: move YUV deinterlace function to common

Am 24.08.2017 um 17:11 schrieb Leo Liu: Signed-off-by: Leo Liu Reviewed-by: Christian König for the series. Andy do you want to test this? Should make VA-API transcoding simpler to use. Regards, Christian. ---

[Mesa-dev] [Bug 101982] Weston crashes when running an OpenGL program on i965

https://bugs.freedesktop.org/show_bug.cgi?id=101982 Link Mauve changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [PATCH 1/2] anv: implementation of VK_EXT_debug_report extension

I've noticed one missing thing, for VK_ERROR_OUT_OF_HOST_MEMORY situations we would need function to call debug report without actual object (since we failed to allocate it!) by only passing object enum, maybe have a separate vk_memory_error macro for these cases? On 08/24/2017 09:23 AM,

Re: [Mesa-dev] [PATCH mesa] egl/android: add missing include

On Thu, Aug 24, 2017 at 10:09 AM, Tapani Pälli wrote: > > > On 08/24/2017 06:02 PM, Rob Herring wrote: >> >> On Thu, Aug 24, 2017 at 9:26 AM, Rob Herring wrote: >>> >>> On Thu, Aug 24, 2017 at 9:22 AM, Eric Engestrom >>> wrote:

Re: [Mesa-dev] [PATCH] i965: Simplify MOCS mashing in genX_state_upload.c.

On 24/08/17 16:15, Kenneth Graunke wrote: On Thursday, August 24, 2017 4:04:26 AM PDT Lionel Landwerlin wrote: Looks good, but it looks like you could replace an additional one in upload_push_constant_packets(). That one is a bit weird - it uses 0 on Gen8+. I've wondered about that, actually

[Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

When a conditional branch has the same labels in the "if" part and in the "else" part, then we have the same cfg block, and it must be handled once. Fixes: dEQP-VK.spirv_assembly.instruction.compute.conditional_branch.same_labels*

Re: [Mesa-dev] [PATCH] i965: Simplify MOCS mashing in genX_state_upload.c.

On Thursday, August 24, 2017 4:04:26 AM PDT Lionel Landwerlin wrote: > Looks good, but it looks like you could replace an additional one in > upload_push_constant_packets(). That one is a bit weird - it uses 0 on Gen8+. I've wondered about that, actually - the docs claim that you must use 0 -

[Mesa-dev] [Bug 102390] centroid interpolation causes broken attribute values

https://bugs.freedesktop.org/show_bug.cgi?id=102390 --- Comment #1 from timon --- (In reply to timon from comment #0) > If I add the the centroid interpolation decoration to any attribute: > - the values will be mostly broken To clarify the values for that attribute only,

[Mesa-dev] [Bug 101941] Getting different output depending on attribute declaration order

https://bugs.freedesktop.org/show_bug.cgi?id=101941 Brian Paul changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [PATCH 2/3] st/va: move YUV content to deinterlaced buffer when reallocated for encoder

v2: use deinterlace common function Signed-off-by: Leo Liu --- src/gallium/state_trackers/va/picture.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c index

  1   2   3   >