Re: [Mesa-dev] [PATCH mesa] meson: add missing radv_extensions.c generation for libvulkan_radeon

On Tue, 2017-10-17 at 12:00 +0100, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom I would add a line like: fixes: 17201a2eb0b (radv: port to using updated anv entrypoint/extension generator.) > --- > src/amd/vulkan/meson.build | 10 +- > 1 file

Re: [Mesa-dev] Build fail since configure.ac: rework llvm libs handling for 3.9+

Emil Velikov wrote: On 17 October 2017 at 13:40, Emil Velikov wrote: With both: -DLLVM_BUILD_LLVM_DYLIB=ON and -DLLVM_LINK_LLVM_DYLIB=ON Something's a bit strange: ... setting -DLLVM_LINK_LLVM_DYLIB=ON should also set -DLLVM_BUILD_LLVM_DYLIB=ON [1]. If that's

Re: [Mesa-dev] [PATCH v2] st/mesa: Initialize textures array in st_framebuffer_validate

On 10/17/2017 04:39 PM, Michel Dänzer wrote: From: Michel Dänzer And just reference pipe_resources to it in the validate callbacks. Avoids pipe_resource leaks when st_framebuffer_validate ends up calling the validate callback multiple times, e.g. when a window is

Re: [Mesa-dev] [PATCH 2/2] loader/dri3: Make sure we invalidate a drawable on size change

On 16/10/17 07:16 PM, Thomas Hellstrom wrote: > On 10/16/2017 04:39 PM, Michel Dänzer wrote: >> On 16/10/17 02:21 PM, Thomas Hellstrom wrote: >>> On 10/16/2017 12:53 PM, Thomas Hellstrom wrote: Hi, Michel, On 10/16/2017 12:35 PM, Michel Dänzer wrote: > Hi Thomas, > > On

Re: [Mesa-dev] Build fail since configure.ac: rework llvm libs handling for 3.9+

Emil Velikov wrote: On a "bad" build (-DLLVM_BUILD_LLVM_DYLIB=ON) I get andy [~]$ llvm-config --link-shared --libs bitwriter llvm-config: error: missing: /usr/lib/libLLVMDemangle.so llvm-config: error: missing: /usr/lib/libLLVMSupport.so llvm-config: error: missing:

[Mesa-dev] [PATCH v2] st/mesa: Initialize textures array in st_framebuffer_validate

From: Michel Dänzer And just reference pipe_resources to it in the validate callbacks. Avoids pipe_resource leaks when st_framebuffer_validate ends up calling the validate callback multiple times, e.g. when a window is resized. v2: * Use generic stable tag instead of

Re: [Mesa-dev] [PATCH v2 1/4] nir: set default lod to texture opcodes that needed it but don't provide it

I sent them out yesterday (sorry you weren't on the CC) and lionel reviewed them. They were pushed as: commit 759ab66db036dd911cb589429eb4dbb3eb4fdc4c Author: Jason Ekstrand Date: Mon Oct 16 08:50:44 2017 -0700 anv/apply_pipeline_layout: Use

Re: [Mesa-dev] [PATCH v2 4/8] egl: add dri2_egl_surface_update_buffer_age() helper (v2)

On 6 October 2017 at 22:38, Gwan-gyeong Mun wrote: > To share common update buffer age code. > This updates old buffer's age and sets current back buffer's age to 1. > > In preparation to adding of new platform which uses this helper. > > v2: > - Fixes from Eric's review: >

[Mesa-dev] [PATCH] vulkan/wsi: Avoid waiting indefinitely for present completion in x11_manage_fifo_queues().

In particular, if the window was destroyed before the present request completed, xcb_wait_for_special_event() may never return. Note that the usage of xcb_poll_for_special_event() requires a version of libxcb that includes commit fad81b63422105f9345215ab2716c4b804ec7986 to work properly.

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

On Tue, Oct 17, 2017 at 03:46:24PM +0200, Michel Dänzer wrote: > On 17/10/17 02:22 PM, Daniel Vetter wrote: > > On Tue, Oct 17, 2017 at 12:28:17PM +0200, Michel Dänzer wrote: > >> On 17/10/17 11:34 AM, Nicolai Hähnle wrote: > > > >>> Common sense suggests that there need to be two side to

Re: [Mesa-dev] [PATCH v2 8/8] egl/wayland: add dri2_wl_free_buffers() helper

On Friday, 2017-10-06 21:38:35 +, Gwan-gyeong Mun wrote: > This deduplicates free routines of color_buffers array. > > Signed-off-by: Mun Gwan-gyeong > --- > src/egl/drivers/dri2/platform_wayland.c | 60 > + > 1 file changed, 31

Re: [Mesa-dev] [PATCH v2 3/8] egl: add dri2_egl_surface_record_buffers_and_update_back_buffer() helper (v2)

Hi Gwan-gyeong, There's a small nit but otherwise looks good. On 6 October 2017 at 22:38, Gwan-gyeong Mun wrote: > To share common record buffers and update back buffer code. > This records all the buffers created by each platform's native window and > update back buffer for

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

Am 17.10.2017 um 15:46 schrieb Michel Dänzer: On 17/10/17 02:22 PM, Daniel Vetter wrote: [SNIP] Finally I'm not sure we want to insist on a target time for freesync. At least as far as I understand things you just want "as soon as possible". This might change with some of the VK/EGL/GLX

Re: [Mesa-dev] [PATCH v2 2/8] egl: refactor color_buffers structure for deduplicating

On 6 October 2017 at 22:38, Gwan-gyeong Mun wrote: > This is added for preventing adding of new color buffers structure and back* > when new platform backend is added. > This refactoring separates out the common and platform specific bits. > This makes odd casting in the

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

On 17/10/17 02:22 PM, Daniel Vetter wrote: > On Tue, Oct 17, 2017 at 12:28:17PM +0200, Michel Dänzer wrote: >> On 17/10/17 11:34 AM, Nicolai Hähnle wrote: > >>> Common sense suggests that there need to be two side to FreeSync / VESA >>> Adaptive Sync support: >>> >>> 1. Query the display

Re: [Mesa-dev] seems meson vulkan build is currently broken on travis

On Tuesday, 2017-10-17 13:03:28 +, Gert Wollny wrote: > By testing my own patches I saw that the meson/vulcan specific build > failed on travis: > > https://travis-ci.org/gerddie/mesa/builds/288995180 > > To check that it is not related to my changes I also did that specific > build with

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

On 17/10/17 01:04 PM, Nicolai Hähnle wrote: > On 17.10.2017 12:28, Michel Dänzer wrote: >> On 17/10/17 11:34 AM, Nicolai Hähnle wrote: >>> >>> Common sense suggests that there need to be two side to FreeSync / VESA >>> Adaptive Sync support: >>> >>> 1. Query the display capabilities. This means

Re: [Mesa-dev] [PATCH v2 1/8] egl: add dri2_egl_surface_free_outdated_buffers_and_update_size() helper (v2)

Hi Gwan-gyeong, On 6 October 2017 at 22:38, Gwan-gyeong Mun wrote: > To share common free outdated buffers and update size code. > This compares width and height arguments with current egl surface dimension, > if the compared surface dimension is differ, then it free local

Re: [Mesa-dev] [PATCH] [AMD] dri3: Add adaptive_sync_enable driconf option

On 17/10/17 01:18 PM, Nicolai Hähnle wrote: > On 17.10.2017 12:41, Michel Dänzer wrote: >> On 17/10/17 12:29 PM, Nicolai Hähnle wrote: >>> On 17.10.2017 12:07, Michel Dänzer wrote: On 17/10/17 11:33 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > >

Re: [Mesa-dev] [PATCH v3 25/43] compiler: Mark when input/ouput attribute at VS uses 16-bit

On 15/10/17 12:00, Pohjolainen, Topi wrote: > On Thu, Oct 12, 2017 at 08:38:14PM +0200, Jose Maria Casanova Crespo wrote: >> New shader attribute to mark when a location has 16-bit >> value. This patch includes support on mesa glsl and nir. >> --- >> src/compiler/glsl_types.h | 24

[Mesa-dev] seems meson vulkan build is currently broken on travis

By testing my own patches I saw that the meson/vulcan specific build failed on travis: https://travis-ci.org/gerddie/mesa/builds/288995180 To check that it is not related to my changes I also did that specific build with the latest git master (35c66f3e4017) that failed.

[Mesa-dev] [PATCH v2 07/10] mesa/st/tests: Add tests for improved tracking of temporaries

Additional tests are added that check the tracking of access to temporaries in if-else branches. Signed-off-by: Gert Wollny --- .../tests/test_glsl_to_tgsi_lifetime.cpp | 493 - 1 file changed, 486 insertions(+), 7 deletions(-) diff --git

[Mesa-dev] [PATCH v2 06/10] mesa/st/glsl_to_tgsi: Add tracking of ifelse writes in register merging

Improve the life-time evaluation of temporary registers by also tracking writes in both if and else branches and in up to 32 nested scopes. As a result the estimated required register life-times can be further reduced enabling more registers to be merged. Signed-off-by: Gert Wollny

[Mesa-dev] [PATCH v2 09/10] mesa/st/tests: Add tests for lifetime tracking with indirect addressing

Add a code line type that accepts one layer of indirect addressing and add tests to check that temporary register access used for indirect addressing is accounted for in the lifetime estimation. Signed-off-by: Gert Wollny --- .../tests/test_glsl_to_tgsi_lifetime.cpp

[Mesa-dev] [PATCH v2 03/10] mesa/st/tests: base check of number of registers on opcode info

* Test number of operands by using num_inst_src_regs/num_inst_dst_regs and fix tests accordingly. Signed-off-by: Gert Wollny --- src/mesa/state_tracker/tests/test_glsl_to_tgsi_lifetime.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH v2 10/10] mesa/st/glsl_to_tgsi: remove now unneeded assert.

With the implementation of the tracking of the registers used in reladdr asserting that a driver calling merge_register() uses the address register is no longer needed. Signed-off-by: Gert Wollny --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 1 - 1 file changed, 1

[Mesa-dev] [PATCH v2 08/10] mesa/st/glsl_to_tgsi: Add tracking of indirect addressing registers

So far indirect addressing was not tracked to estimate the temporary life time, and it was not needed, because code to load the address registers was always emitted eliminating the reladdr* handles in the past glsl-to.tgsi stages. Now, with Mareks patch 9a88580a4b3d allowing any 1D register to be

[Mesa-dev] [PATCH v2 04/10] mesa/st/tests: cleanup whitespace usage and correct some comments

Signed-off-by: Gert Wollny --- .../tests/test_glsl_to_tgsi_lifetime.cpp | 127 ++--- 1 file changed, 63 insertions(+), 64 deletions(-) diff --git a/src/mesa/state_tracker/tests/test_glsl_to_tgsi_lifetime.cpp

[Mesa-dev] [PATCH v2 05/10] mesa/st/glsl_to_tgsi: Correct debug output for indirect access

For arrays print the array ID, and with indirect access also print the reladdr* registers. The reladdr* registers are always used in the printout, even though the actual code may use an address register. Specifically, a sequence involving src.reladdr = TEMP[2] and src.index=10 that emits the

[Mesa-dev] [PATCH v2 00/10] glsl_to_tgsi: Further improvement of lifetime tracking for register merge

Dear all, this is the updated patch set that adds enhanced tracking of IF/ELSE branches and tracking of reladdr* registers for the register_merge step. So far patches 1 & 5 (now 8) are Reviewed-by: Nicolai Hähnle Changes w.r.t. v1: * patches 2-4(new): As

[Mesa-dev] [Bug 103312] meson/macOS: Dependency libdrm_intel found: NO

https://bugs.freedesktop.org/show_bug.cgi?id=103312 --- Comment #2 from Emil Velikov --- autotools detects possible drivers per platform and per cpu arch. The latter may be an overkill but the former is a must IMHO. -- You are receiving this mail because: You are the

[Mesa-dev] [PATCH v2 01/10] mesa/st/tests: Fix zero-byte allocation leaks

Don't allocate a zero-sized array, when no texture offsets are given. Reviewed-by: Nicolai Hähnle Signed-off-by: Gert Wollny --- .../tests/test_glsl_to_tgsi_lifetime.cpp | 23 +++--- 1 file changed, 16 insertions(+), 7

[Mesa-dev] [PATCH v2 02/10] mesa/st/tests: unify MockCodeLine* classes

* Merge the classes MockCodeLine and MockCodelineWithSwizzle into one and refactor tests accordingly. * Change memory allocations to use ralloc* interface. Signed-off-by: Gert Wollny --- .../tests/test_glsl_to_tgsi_lifetime.cpp | 481 ++--- 1

Re: [Mesa-dev] Build fail since configure.ac: rework llvm libs handling for 3.9+

On 17 October 2017 at 13:40, Emil Velikov wrote: >> With both: >> -DLLVM_BUILD_LLVM_DYLIB=ON and -DLLVM_LINK_LLVM_DYLIB=ON >> > Something's a bit strange: > ... setting -DLLVM_LINK_LLVM_DYLIB=ON should also set -DLLVM_BUILD_LLVM_DYLIB=ON [1]. If that's not the case

Re: [Mesa-dev] Build fail since configure.ac: rework llvm libs handling for 3.9+

On 16 October 2017 at 23:32, Dieter Nützel wrote: > Am 16.10.2017 20:13, schrieb Andy Furniss: >> >> Emil Velikov wrote: >>> >>> On 16 October 2017 at 03:22, Jan Vesely wrote: On Sun, 2017-10-15 at 00:00 +0100, Andy Furniss wrote: >

[Mesa-dev] [Bug 103312] meson/macOS: Dependency libdrm_intel found: NO

https://bugs.freedesktop.org/show_bug.cgi?id=103312 --- Comment #1 from Eric Engestrom --- libdrm_intel is needed to build i915; if you want to disable it you can use `-Ddri-drivers=i965` (or any other value that doesn't contain `i915`, although right now those are the

Re: [Mesa-dev] [PATCH 16/16] radeonsi: if there's just const buffer 0, set it in place of CONST/SSBO pointer

On 13.10.2017 14:04, Marek Olšák wrote: From: Marek Olšák SI_SGPR_CONST_AND_SHADER_BUFFERS now contains the pointer to const buffer 0 if there is no other buffer there. Benefits: - there is no constbuf descriptor upload and shader load It's assumed that all constant

Re: [Mesa-dev] [PATCH 15/16] ac: clean up ac_build_indexed_load function interfaces

This patch and patches 1 - 13: Reviewed-by: Nicolai Hähnle On 13.10.2017 14:04, Marek Olšák wrote: From: Marek Olšák --- src/amd/common/ac_llvm_build.c| 42 ++- src/amd/common/ac_llvm_build.h

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

On Tue, Oct 17, 2017 at 12:28:17PM +0200, Michel Dänzer wrote: > On 17/10/17 11:34 AM, Nicolai Hähnle wrote: > > Hi all, > > > > I just sent out a patch that enables FreeSync in Mesa for the somewhat > > hacked implementation of FreeSync that exists in our hybrid (amdgpu-pro) > > stack's DDX and

Re: [Mesa-dev] [PATCH 08/10] i965: implement DRIImage::createImageFromRenderbuffer2

On 17 October 2017 at 11:29, Eric Engestrom wrote: > On Monday, 2017-10-16 16:04:10 +, Emil Velikov wrote: >> From: Emil Velikov >> >> The new entry point has a way to feedback the error. Thus we no longer >> need to call _mesa_error()

Re: [Mesa-dev] [PATCH 05/10] gbm: handle queryImage() failure through rest of gbm_dri_bo_import()

On 17 October 2017 at 11:12, Eric Engestrom wrote: > On Monday, 2017-10-16 16:04:07 +, Emil Velikov wrote: >> From: Emil Velikov >> >> Fixes: 6a7dea93fa7 ("dri: Rework planar image interface") >> Cc: Jakob Bornecrantz

[Mesa-dev] [Bug 103312] meson/macOS: Dependency libdrm_intel found: NO

https://bugs.freedesktop.org/show_bug.cgi?id=103312 Yurii Kolesnykov changed: What|Removed |Added URL|

[Mesa-dev] [Bug 103312] meson/macOS: Dependency libdrm_intel found: NO

https://bugs.freedesktop.org/show_bug.cgi?id=103312 Bug ID: 103312 Summary: meson/macOS: Dependency libdrm_intel found: NO Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Mac OS X (All) Status: NEW

Re: [Mesa-dev] [PATCH 04/10] gbm: handle queryImage() failure for GBM_BO_IMPORT_EGL_IMAGE

On 17 October 2017 at 11:09, Eric Engestrom wrote: > On Monday, 2017-10-16 16:04:06 +, Emil Velikov wrote: >> From: Emil Velikov >> >> The function can fail. Check and teardown accordingly. >> >> Fixes: a43d286ef7f ("gbm: Add import from

Re: [Mesa-dev] [PATCH 2/2] anv/apply_pipeline_layout: Use nir_tex_instr_remove_src

Reviewed-by: Lionel Landwerlin On 17/10/17 02:09, Jason Ekstrand wrote: --- src/intel/vulkan/anv_nir_apply_pipeline_layout.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/src/intel/vulkan/anv_nir_apply_pipeline_layout.c

Re: [Mesa-dev] [PATCH v3 23/43] i965/fs: Add byte scattered read message and fs support

On 15/10/17 11:47, Pohjolainen, Topi wrote: > On Thu, Oct 12, 2017 at 08:38:12PM +0200, Jose Maria Casanova Crespo wrote: >> --- >> src/intel/compiler/brw_eu.h| 7 + >> src/intel/compiler/brw_eu_defines.h| 2 ++ >> src/intel/compiler/brw_eu_emit.c

Re: [Mesa-dev] [PATCH] [AMD] dri3: Add adaptive_sync_enable driconf option

On 17.10.2017 12:41, Michel Dänzer wrote: On 17/10/17 12:29 PM, Nicolai Hähnle wrote: On 17.10.2017 12:07, Michel Dänzer wrote: On 17/10/17 11:33 AM, Nicolai Hähnle wrote: From: Nicolai Hähnle When enabled, this will request FreeSync via the hybrid amdgpu DDX's

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

On 17.10.2017 12:28, Michel Dänzer wrote: On 17/10/17 11:34 AM, Nicolai Hähnle wrote: Hi all, I just sent out a patch that enables FreeSync in Mesa for the somewhat hacked implementation of FreeSync that exists in our hybrid (amdgpu-pro) stack's DDX and kernel module. [0] While this patch

[Mesa-dev] [PATCH mesa] meson: add missing radv_extensions.c generation for libvulkan_radeon

Signed-off-by: Eric Engestrom --- src/amd/vulkan/meson.build | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/amd/vulkan/meson.build b/src/amd/vulkan/meson.build index a5a4f81352807beac92d..6a416d988674504281c6 100644 ---

Re: [Mesa-dev] [PATCH V3] automake: intel: move expat handling where it's used

On 17 October 2017 at 02:10, Hongxu Jia wrote: > Linking libvulkan_intel.so can fail, due to unresolved references to > libexpat.so. > > EXPAT_CFLAGS should be moved as well. > The EXPAT_CFLAGS changes seems to be missing. Did you forget git add? -Emil

Re: [Mesa-dev] [PATCH] [AMD] dri3: Add adaptive_sync_enable driconf option

On 17/10/17 12:29 PM, Nicolai Hähnle wrote: > On 17.10.2017 12:07, Michel Dänzer wrote: >> On 17/10/17 11:33 AM, Nicolai Hähnle wrote: >>> From: Nicolai Hähnle >>> >>> When enabled, this will request FreeSync via the hybrid amdgpu DDX's >>> AMDGPU X11 protocol extension.

Re: [Mesa-dev] [PATCH 10/10] radeon: implement DRIImage::createImageFromRenderbuffer2

On Monday, 2017-10-16 16:04:12 +, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > src/mesa/drivers/dri/radeon/radeon_screen.c | 37 > +++-- > 1 file changed, 25 insertions(+),

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] auxiliary: use vl_drm_screen_create method for surfaceless

HI Suresh, Please try to avoid HTML emails. If that's not possible, make sure your reply is more readable. At the moment it has 3 different fonts and 3 different colours, which makes is distracting and hard to read. Please tweak the quoting format - older replies should be a level deeper. See

Re: [Mesa-dev] [PATCH 09/10] i915: implement DRIImage::createImageFromRenderbuffer2

On Monday, 2017-10-16 16:04:11 +, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov (same code, same comments as the previous/i965 patch apply here) > --- > src/mesa/drivers/dri/i915/intel_screen.c | 33 >

Re: [Mesa-dev] [RFC] radv: copy indirect lowering settings from radeonsi

On 17/10/17 17:49, Bas Nieuwenhuizen wrote: On Tue, Oct 17, 2017 at 7:41 AM, Timothy Arceri wrote: It looks the original indirect mask was probably copied from ANV. Here we drop lowering locals altogether and allow indirects on inputs where supported. Sascha Willems

Re: [Mesa-dev] [PATCH V3] automake: intel: move expat handling where it's used

Yeah, it also applies to i965. I'm guessing we haven't seen that problem because expat gets pulled in by other bits of the mesa. Looks good to me, thanks : Reviewed-by: Lionel Landwerlin On 17/10/17 02:10, Hongxu Jia wrote: Linking libvulkan_intel.so can

Re: [Mesa-dev] [PATCH 08/10] i965: implement DRIImage::createImageFromRenderbuffer2

On Monday, 2017-10-16 16:04:10 +, Emil Velikov wrote: > From: Emil Velikov > > The new entry point has a way to feedback the error. Thus we no longer > need to call _mesa_error() but instead we can pass the correct value. > > Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH] [AMD] dri3: Add adaptive_sync_enable driconf option

On 17.10.2017 12:07, Michel Dänzer wrote: On 17/10/17 11:33 AM, Nicolai Hähnle wrote: From: Nicolai Hähnle When enabled, this will request FreeSync via the hybrid amdgpu DDX's AMDGPU X11 protocol extension. Due to limitations in the DDX this will only work for

Re: [Mesa-dev] Upstream support for FreeSync / Adaptive Sync

On 17/10/17 11:34 AM, Nicolai Hähnle wrote: > Hi all, > > I just sent out a patch that enables FreeSync in Mesa for the somewhat > hacked implementation of FreeSync that exists in our hybrid (amdgpu-pro) > stack's DDX and kernel module. [0] > > While this patch isn't meant for upstream, that's

Re: [Mesa-dev] [PATCH 1/2] nir: Add a helper for adding texture instruction sources

Thanks! Reviewed-by: Lionel Landwerlin On 17/10/17 02:09, Jason Ekstrand wrote: --- src/compiler/nir/nir.c | 22 +++ src/compiler/nir/nir.h | 4 src/compiler/nir/nir_lower_samplers.c

Re: [Mesa-dev] [PATCH 05/10] gbm: handle queryImage() failure through rest of gbm_dri_bo_import()

On Monday, 2017-10-16 16:04:07 +, Emil Velikov wrote: > From: Emil Velikov > > Fixes: 6a7dea93fa7 ("dri: Rework planar image interface") > Cc: Jakob Bornecrantz > Signed-off-by: Emil Velikov > --- >

Re: [Mesa-dev] [PATCH 04/10] gbm: handle queryImage() failure for GBM_BO_IMPORT_EGL_IMAGE

On Monday, 2017-10-16 16:04:06 +, Emil Velikov wrote: > From: Emil Velikov > > The function can fail. Check and teardown accordingly. > > Fixes: a43d286ef7f ("gbm: Add import from fd") > Cc: Kristian Høgsberg > Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH] [AMD] dri3: Add adaptive_sync_enable driconf option

On 17/10/17 11:33 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > When enabled, this will request FreeSync via the hybrid amdgpu DDX's > AMDGPU X11 protocol extension. > > Due to limitations in the DDX this will only work for applications > that cover the entire X

[Mesa-dev] [PATCH] radv: use the dispatch initiator for indirect dispatches

Missed that when I allowed waves to be launched out-of-order. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c

[Mesa-dev] Upstream support for FreeSync / Adaptive Sync

Hi all, I just sent out a patch that enables FreeSync in Mesa for the somewhat hacked implementation of FreeSync that exists in our hybrid (amdgpu-pro) stack's DDX and kernel module. [0] While this patch isn't meant for upstream, that's as good a time as any to raise the issue of how a

[Mesa-dev] [PATCH] [AMD] dri3: Add adaptive_sync_enable driconf option

From: Nicolai Hähnle When enabled, this will request FreeSync via the hybrid amdgpu DDX's AMDGPU X11 protocol extension. Due to limitations in the DDX this will only work for applications that cover the entire X screen (which is important to keep in mind when you have a

Re: [Mesa-dev] [PATCH 1/9] radv: refactor simple and indexed draws with radv_draw_info

Just noticed that I missed the predicate stuff. On 10/17/2017 11:03 AM, Samuel Pitoiset wrote: Similar to the dispatch compute logic but for draw calls. For convenience, indirect draws will be converted in a separate patch. Signed-off-by: Samuel Pitoiset ---

[Mesa-dev] [PATCH 4/9] radv: add radv_emit_draw_registers()

Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 46 +--- 1 file changed, 34 insertions(+), 12 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index

[Mesa-dev] [PATCH 6/9] radv: rename radv_cmd_buffer_flush_state() to radv_draw()

Similar to the dispatch codepath. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 85 ++-- 1 file changed, 39 insertions(+), 46 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c

[Mesa-dev] [PATCH 7/9] radv: add radv_emit_shader_prefetch()

Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 47 +++- 1 file changed, 22 insertions(+), 25 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index

[Mesa-dev] [PATCH 0/9] radv: draw codepath refactoring + optimal packet order

Hi, This series first refactors the draw codepath to follow the dispatch codepath (ie. using a new structure called radv_draw_info). Then it adds few helpers, and finally it tries to use a better packet order in order to reduce the time where shaders are idle. This is loosely based on RadeonSI

[Mesa-dev] [PATCH 9/9] radv: use optimal packet order for draws

Ported from RadeonSI. The time where shaders are idle should be shorter now. This can give a little boost, like +6% with the dynamicubo Vulkan demo. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 94 1

[Mesa-dev] [PATCH 8/9] radv: add radv_emit_shaders_prefetch()

Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 38 ++ 1 file changed, 26 insertions(+), 12 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index

[Mesa-dev] [PATCH 1/9] radv: refactor simple and indexed draws with radv_draw_info

Similar to the dispatch compute logic but for draw calls. For convenience, indirect draws will be converted in a separate patch. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 202 --- 1 file changed, 127

[Mesa-dev] [PATCH] radv: remove XtoY_temps structs

Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_meta_bufimage.c | 62 - 1 file changed, 26 insertions(+), 36 deletions(-) diff --git a/src/amd/vulkan/radv_meta_bufimage.c b/src/amd/vulkan/radv_meta_bufimage.c index

[Mesa-dev] [PATCH 3/9] radv: refactor indirect draws (+count buffer) with radv_draw_info

Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 169 +++ 1 file changed, 48 insertions(+), 121 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index

[Mesa-dev] [PATCH 5/9] radv: emit primitive restart from radv_emit_draw_registers()

Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 59 1 file changed, 30 insertions(+), 29 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index

[Mesa-dev] [PATCH 2/9] radv: refactor indirect draws with radv_draw_info

Indirect draws with a count buffer will be refactored in a separate patch. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 217 ++- 1 file changed, 143 insertions(+), 74 deletions(-) diff --git

[Mesa-dev] [PATCH v2] glsl/linker: produce error when invalid explicit locations are used

We only need to add a check to validate output locations here. For inputs with invalid locations we will fail to link when we can't find a matching output in the same (invalid) location. v2: compute location slots properly depending on shader stage and variable type / direction Fixes:

Re: [Mesa-dev] [PATCH 1/6] i965: Only put external handles into the handle ht

Quoting Kenneth Graunke (2017-10-16 20:07:05) > I'd like to try out the > AMD_pinned_memory support with Left 4 Dead 2, as I know a bunch of > the Source 1 games use AMD_pinned_memory for better performance... Finally got around to building a 32b mesa for steam, Left4Dead2 reports: SDL failed to

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] auxiliary: use vl_drm_screen_create method for surfaceless

Hi Suresh, in general please don't use HTML mail (e.g. no color or bold formating etc.. in mails). Mailing list usually reject that, but Mesa isn't so strict with that so it hopefully works out. For more comments see below. Am 17.10.2017 um 08:36 schrieb Guttula, Suresh: On 11 October

[Mesa-dev] [PATCH 1/2] drirc: Enable glthread for more games (Saints Row 4 & Gat out of Hell).

“Saints Row: Gat out of Hell” benefits from this on slower CPUs in that usage spikes on individual cores are avoided, which in turn makes it harder to hit a bug which causes broken audio and the game to hang on exit. “Saints Row IV” appears to be fine either way, but also exhibits the audio

[Mesa-dev] [PATCH V3] automake: intel: move expat handling where it's used

Linking libvulkan_intel.so can fail, due to unresolved references to libexpat.so. EXPAT_CFLAGS should be moved as well. Signed-off-by: Hongxu Jia --- src/intel/Makefile.common.am | 1 + src/intel/Makefile.tools.am | 2 -- 2 files changed, 1 insertion(+), 2

[Mesa-dev] [PATCH 2/2] drirc: Group a few games in the glthread whitelist together.

--- src/util/drirc | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/src/util/drirc b/src/util/drirc index 3cf3d8dc69..39ac3c858c 100644 --- a/src/util/drirc +++ b/src/util/drirc @@ -166,27 +166,37 @@ TODO: document the other workarounds.

[Mesa-dev] [PATCH] radv/image: bump all the offset to uint64_t.

From: Dave Airlie So one of the CTS tests tries to allocate a 16384x1 2048 array texture. This overflows a bunch of calculations when we want it tiled as the heights goes to 128. addrlib returns us the correct size (16GB or so), but we mangle it in the htile calcs due to the

Re: [Mesa-dev] [RFC] radv: copy indirect lowering settings from radeonsi

On Tue, Oct 17, 2017 at 7:41 AM, Timothy Arceri wrote: > It looks the original indirect mask was probably copied from > ANV. > > Here we drop lowering locals altogether and allow indirects > on inputs where supported. > > Sascha Willems demo results: > > tessellation ~4000

Re: [Mesa-dev] [PATCH] radv: port to using updated anv entrypoint/extension generator.

On Tue, Oct 17, 2017 at 7:36 AM, Dave Airlie wrote: > From: Dave Airlie > > This ports radv to using the anv entrypoint/extension generator > code. > > No differences on enabled extensions list in vulkaninfo. > > Signed-off-by: Dave Airlie

[Mesa-dev] [Bug 103062] Error starting game FS17

https://bugs.freedesktop.org/show_bug.cgi?id=103062 --- Comment #11 from alexander --- It will be possible to download an older version of FS17. -- You are receiving this mail because: You are the assignee for the bug.___

<    1   2