Re: [Mesa-dev] [PATCH] st/nine: Remove code for no USER_INDEX_BUFFERS as these are always on

2017-02-25 Thread Mike Lothian
Ah that's what I did first, but figured it was probably in accurate naming after that On Sat, 25 Feb 2017 at 08:21 Axel Davy wrote: > Hi Mike, > > We really want not to use user index buffers when csmt is active (thus > the !This->csmt_active). > This should be a one line

[Mesa-dev] [PATCH v2] st/nine: Remove check for USER_INDEX_BUFFERS use csmt_active instead

2017-02-25 Thread Mike Lothian
This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the pipe cap was removed Now USER_INDEX_BUFFERS are always enabled remove the check and only check for cmst_active v2: Axel pointed out the code was still needed when cmst was inactive, Rebase on master too Fixes: 4a883966c1f7

Re: [Mesa-dev] [PATCH] st/nine: Remove code for no USER_INDEX_BUFFERS as these are always on

2017-02-25 Thread Axel Davy
Hi Mike, We really want not to use user index buffers when csmt is active (thus the !This->csmt_active). This should be a one line patch to just remove the part GET_PCAP(USER_INDEX_BUFFERS) Yours, Axel Davy On 25/02/2017 06:23, Mike Lothian wrote: This fixes

Re: [Mesa-dev] [PATCH v2] st/nine: Remove check for USER_INDEX_BUFFERS use csmt_active instead

2017-02-25 Thread Mike Lothian
There's too many negatives, not sure if this should really be "if (This->cmst_active) {" or not On Sat, 25 Feb 2017 at 10:42 Mike Lothian wrote: > This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the pipe cap > was removed > > Now USER_INDEX_BUFFERS are always

[Mesa-dev] [Bug 99319] godot engine poor performance

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99319 --- Comment #7 from Bas Nieuwenhuizen --- I think this is actually fixed in Mesa git. I get 60 fps for the platformer demo now. -- You are receiving this mail because: You are the assignee for the bug. You are the QA

[Mesa-dev] [PATCH] gallium/nine: Fix build regression

2017-02-25 Thread Edward O'Callaghan
commit 4a88396 dropped 'PIPE_CAP_USER_INDEX_BUFFERS' however this case was missed. Signed-off-by: Edward O'Callaghan --- src/gallium/state_trackers/nine/device9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH] glsl: remove unecessary flags.q.subroutine_def

2017-02-25 Thread Samuel Pitoiset
This bit is definitely not necessary because subroutine_list can be used instead. This frees one more bit in the flags.q struct which is nice because arb_bindless_texture will need 4 bits for the new layout qualifiers. No piglit regressions found (including compiler tests) with "-t subroutine".

Re: [Mesa-dev] [PATCH] gallium/nine: Fix build regression

2017-02-25 Thread Mike Lothian
Are we better off just checking for cmst_active now? Since that's all we're checking On Sat, 25 Feb 2017 at 11:12 Edward O'Callaghan wrote: > commit 4a88396 dropped 'PIPE_CAP_USER_INDEX_BUFFERS' however > this case was missed. > > Signed-off-by: Edward O'Callaghan

[Mesa-dev] [PATCH] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Edward O'Callaghan
From: Mike Lothian This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the PIPE_CAP was removed. Now USER_INDEX_BUFFERS are always enabled remove the check and only check for cmst_active directly. v2: Axel pointed out the code was still needed when cmst was inactive,

Re: [Mesa-dev] [PATCH] gallium/nine: Fix build regression

2017-02-25 Thread Edward O'Callaghan
yes sorry disregard this one, I got report this independently. sorry for the noise. On 02/25/2017 10:18 PM, Mike Lothian wrote: > Are we better off just checking for cmst_active now? Since that's all > we're checking > > On Sat, 25 Feb 2017 at 11:12 Edward O'Callaghan >

Re: [Mesa-dev] [PATCH] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Mike Lothian
I think my question regarding the double negative still stands. Is it "!This->cmst_active" ot "This->cmst_active" On Sat, 25 Feb 2017 at 11:23 Edward O'Callaghan wrote: > From: Mike Lothian > > This fixes

Re: [Mesa-dev] [PATCH] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Edward O'Callaghan
oh yes I missed that, you want to have drop the negation in your patch Mike. On 02/25/2017 10:28 PM, Mike Lothian wrote: > I think my question regarding the double negative still stands. Is it > "!This->cmst_active" ot "This->cmst_active" > > On Sat, 25 Feb 2017 at 11:23 Edward O'Callaghan >

Re: [Mesa-dev] [PATCH] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Mike Lothian
Will do. On Sat, 25 Feb 2017 at 11:31 Edward O'Callaghan wrote: > oh yes I missed that, you want to have drop the negation in your patch > Mike. > > On 02/25/2017 10:28 PM, Mike Lothian wrote: > > I think my question regarding the double negative still stands. Is it

[Mesa-dev] [PATCH v4] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Mike Lothian
This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the PIPE_CAP was removed. Now USER_INDEX_BUFFERS are always enabled remove the check and only check for cmst_active directly. v2: Axel pointed out the code was still needed when cmst was inactive, Rebase on master too v3: Drop struct

Re: [Mesa-dev] [PATCH v4] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Edward O'Callaghan
On 02/25/2017 10:39 PM, Mike Lothian wrote: > This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the > PIPE_CAP was removed. > > Now USER_INDEX_BUFFERS are always enabled remove the check and only > check for cmst_active directly. > > v2: Axel pointed out the code was still needed when

[Mesa-dev] [PATCH v5] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Mike Lothian
This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the PIPE_CAP was removed. Now USER_INDEX_BUFFERS are always enabled remove the check and only check for cmst_active directly. v2: Axel pointed out the code was still needed when cmst was inactive, Rebase on master too v3: Drop struct

Re: [Mesa-dev] [PATCH v5] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Edward O'Callaghan
Very noisy but you got there in the end, Reviewed-by: Edward O'Callaghan On 02/25/2017 10:50 PM, Mike Lothian wrote: > This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the > PIPE_CAP was removed. > > Now USER_INDEX_BUFFERS are always enabled remove the

Re: [Mesa-dev] [PATCH v5] st/nine: Drop USER_INDEX_BUFFERS check

2017-02-25 Thread Mike Lothian
Thanks for pushing and being patient with me On Sat, 25 Feb 2017 at 12:13 Edward O'Callaghan wrote: > Very noisy but you got there in the end, > > Reviewed-by: Edward O'Callaghan > > On 02/25/2017 10:50 PM, Mike Lothian wrote: > > This

[Mesa-dev] [Bug 99953] device9.c:122:49: error: ‘PIPE_CAP_USER_INDEX_BUFFERS’ undeclared (first use in this function)

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99953 Mike Lothian changed: What|Removed |Added Resolution|--- |FIXED

[Mesa-dev] PSA: kmscube moved to fd.o

2017-02-25 Thread Rob Clark
Anyone with mesa/libdrm push access should in theory be able to push directly. Not really any point to wait for me to have time to merge pullreqs on github ;-) I've spiffed it out slightly, so it supports both legacy pageflip mode and atomic/fence mode, etc.

Re: [Mesa-dev] [PATCH 1/2] radeonsi: add support for an on-disk shader cache

2017-02-25 Thread Dieter Nützel
Am 25.02.2017 05:56, schrieb Timothy Arceri: On 24/02/17 21:02, Marek Olšák wrote: On Fri, Feb 24, 2017 at 3:18 AM, Timothy Arceri wrote: On 24/02/17 08:49, Timothy Arceri wrote: On 24/02/17 05:12, Marek Olšák wrote: On Thu, Feb 23, 2017 at 3:09 AM, Timothy

Re: [Mesa-dev] [RESEND 08/13] anv: generate anv_entrypoints.{h, c} in one command

2017-02-25 Thread Mauro Rossi
> > Besides this issue, also 8e03250fcf4fc5de31e92ca4919959d932888a69 > "vulkan: Combine wsi and util makefiles" requires changes/an Android.mk > makefile > > Regarding the -o argument, > it is indeed essential for Android Build System that generated > sources/headers > go in

[Mesa-dev] [PATCH] android: vulkan: add support for libmesa_vulkan_{util, wsi}

2017-02-25 Thread Mauro Rossi
The following commits require android porting: e9dcb17 "vulkan/util: Add generator for enum_to_str functions" 8e03250 "vulkan: Combine wsi and util makefiles" The following changes are implemented in this patch: Add src/vulkan/Android.mk to build libmesa_vulkan_util and libmesa_vulkan_wsi

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #4 from Aidan Thornton --- I've been having the same issue on Gentoo for a while with Mesa and libdrm from git master, and it appears someone's been having the same problem with the Debian-packaged version of

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #5 from Henrique Dante de Almeida --- I'll try with an upstream environment and generate strace output soon -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #8 from Henrique Dante de Almeida --- Should I have verde-r600-* ? I'm not using amdgpu. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #9 from Henrique Dante de Almeida --- I'm currently unable to use amdgpu with kernel 4.9. I'm getting this when exiting X and unable to shutdown cleanly: fev 25 18:00:44 dragonmount kernel: amdgpu :01:00.0:

Re: [Mesa-dev] [PATCH 1/2] radeonsi: add support for an on-disk shader cache

2017-02-25 Thread Marek Olšák
On Sat, Feb 25, 2017 at 5:06 PM, Dieter Nützel wrote: > Am 25.02.2017 05:56, schrieb Timothy Arceri: >> >> On 24/02/17 21:02, Marek Olšák wrote: >>> >>> On Fri, Feb 24, 2017 at 3:18 AM, Timothy Arceri >>> wrote: On 24/02/17 08:49,

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #6 from Jan Vesely --- (In reply to Henrique Dante de Almeida from comment #2) > Yes: > > Nome : libclc > Versão : 0.2.0+334+520743b-1 > Descrição: Library

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #7 from Henrique Dante de Almeida --- libclc /usr/ libclc /usr/include/ libclc /usr/include/clc/ libclc /usr/include/clc/as_type.h libclc /usr/include/clc/async/ libclc /usr/include/clc/async/async_work_group_copy.h

[Mesa-dev] [PATCH v4 6/8] nir: Add a simple int64 lowering pass

2017-02-25 Thread Jason Ekstrand
The algorithms used by this pass, especially for division, are heavily based on the work Ian Romanick did for the similar int64 lowering pass in the GLSL compiler. v2: Properly handle vectors v3: Get rid of log2_denom stuff. Since we're using bcsel, we do all the calculations anyway and

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #10 from Jan Vesely --- (In reply to Henrique Dante de Almeida from comment #8) > Should I have verde-r600-* ? I'm not using amdgpu. no. verde-amdgcn- is correct. the Linux kernel driver does not

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #11 from Henrique Dante de Almeida --- Created attachment 129915 --> https://bugs.freedesktop.org/attachment.cgi?id=129915=edit OpenCL hello strace -- You are receiving this mail because: You are the QA Contact

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #12 from Jan Vesely --- (In reply to Henrique Dante de Almeida from comment #11) > Created attachment 129915 [details] > OpenCL hello strace thank you. libclc is located and loaded correctly. can you

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #13 from Henrique Dante de Almeida --- Created attachment 129916 --> https://bugs.freedesktop.org/attachment.cgi?id=129916=edit Arch Linux libclc tahiti-amdgcn--.bc -- You are receiving this mail because: You

[Mesa-dev] [PATCH 07/24] radeonsi: move image intrinsic building to amd/common

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 68 ++ src/amd/common/ac_llvm_build.h | 29 ++ src/gallium/drivers/radeonsi/si_shader.c | 154 +-- 3 files changed, 159 insertions(+), 92 deletions(-)

[Mesa-dev] [PATCH 16/24] radeonsi: merge and simplify tbuffer_store functions

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 62 + src/amd/common/ac_llvm_build.h | 34 -- src/amd/common/ac_nir_to_llvm.c | 16 +++ src/gallium/drivers/radeonsi/si_shader.c | 79

[Mesa-dev] [PATCH 20/24] ac: replace llvm.SI.tbuffer.store with llvm.amdgcn.buffer.store if ADD_TID=0

2017-02-25 Thread Marek Olšák
From: Marek Olšák TODO: ADD_TID doesn't work. Needs more investigation. --- src/amd/common/ac_llvm_build.c| 58 ++- src/amd/common/ac_llvm_build.h| 4 +- src/amd/common/ac_nir_to_llvm.c | 4 +-

[Mesa-dev] [PATCH 22/24] radeonsi: move SI.vs.load.input building into amd/common

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 17 + src/amd/common/ac_llvm_build.h | 6 ++ src/gallium/drivers/radeonsi/si_shader.c | 25 ++--- 3 files changed, 33 insertions(+), 15 deletions(-) diff

[Mesa-dev] [PATCH 02/24] gallivm, ac: add writeonly and inaccessiblememonly attributes

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_util.c | 2 ++ src/amd/common/ac_llvm_util.h | 2 ++ src/gallium/auxiliary/gallivm/lp_bld_intr.c | 2 ++ src/gallium/auxiliary/gallivm/lp_bld_intr.h | 2 ++ 4 files changed, 8 insertions(+) diff --git

[Mesa-dev] [PATCH 05/24] radeonsi: move llvm.SI.export building to amd/common

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 15 ++ src/amd/common/ac_llvm_build.h | 11 ++ src/gallium/drivers/radeonsi/si_shader.c | 306 +++ 3 files changed, 170 insertions(+), 162 deletions(-) diff --git

[Mesa-dev] [PATCH 12/24] radeonsi: set readnone on reads from read-only memory

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 11 +-- src/amd/common/ac_llvm_build.h | 3 ++- src/gallium/drivers/radeonsi/si_shader.c | 20 ++-- 3 files changed, 21 insertions(+), 13 deletions(-) diff --git

[Mesa-dev] [PATCH 14/24] radeonsi: replace AMDGPU.bfe.* with amdgcn.*bfe

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c| 26 +++ src/amd/common/ac_llvm_build.h| 3 +++ src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 11 -- 3 files changed, 33 insertions(+), 7

[Mesa-dev] [PATCH 08/24] radeonsi: remove last use of llvm.SI.resinfo

2017-02-25 Thread Marek Olšák
From: Marek Olšák and move one function up to reuse the code. --- src/gallium/drivers/radeonsi/si_shader.c | 97 1 file changed, 49 insertions(+), 48 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c

[Mesa-dev] [PATCH 11/24] radeonsi: replace SI.buffer.load.dword with amdgcn.buffer.load

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 64 ++-- 1 file changed, 19 insertions(+), 45 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index af031c7..887e6a4

[Mesa-dev] [PATCH 06/24] ac: replace SI.export with amdgcn.exp.*

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 31 +++ src/gallium/drivers/radeonsi/si_shader.c | 8 +--- 2 files changed, 36 insertions(+), 3 deletions(-) diff --git a/src/amd/common/ac_llvm_build.c

[Mesa-dev] [PATCH 04/24] ac: unify build_type_name_for_intr functions

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 37 src/amd/common/ac_llvm_build.h | 2 ++ src/amd/common/ac_nir_to_llvm.c | 41 ++ src/gallium/drivers/radeonsi/si_shader.c | 49

[Mesa-dev] [PATCH 10/24] radeonsi: replace SI.packf16 with amdgcn.cvt.pkrtz

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 18 ++ src/amd/common/ac_llvm_build.h | 2 ++ src/gallium/drivers/radeonsi/si_shader.c | 6 +- 3 files changed, 21 insertions(+), 5 deletions(-) diff --git

[Mesa-dev] [PATCH 09/24] ac: replace old image intrinsics with new ones

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 80 ++ 1 file changed, 80 insertions(+) diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c index 3a1ef93..187c2cb 100644 ---

[Mesa-dev] [PATCH 13/24] radeonsi: move kill intrinsic building into amd/common

2017-02-25 Thread Marek Olšák
From: Marek Olšák just a cleanup --- src/amd/common/ac_llvm_build.c| 16 src/amd/common/ac_llvm_build.h| 1 + src/gallium/drivers/radeonsi/si_shader.c | 12

[Mesa-dev] [PATCH 00/24] RadeonSI moving away from legacy intrinsics & more

2017-02-25 Thread Marek Olšák
Hi, This series depends on the patch adding attributes at call sites: https://patchwork.freedesktop.org/series/20100/ The only legacy intrinsics still in use are: - llvm.SI.load.const - llvm.SI.tbuffer.store (when ADD_TID = 1) - llvm.AMDGPU.kill - llvm.AMDGPU.kilp A lot of the code that builds

[Mesa-dev] [PATCH 19/24] radeonsi: use the writeonly LLVM attribute

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index dd5bdf6..16579af 100644 ---

[Mesa-dev] [PATCH 01/24] tgsi/scan: record load/store/atomic image usage

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/tgsi/tgsi_scan.c | 15 ++- src/gallium/auxiliary/tgsi/tgsi_scan.h | 8 +++- src/gallium/drivers/radeonsi/si_shader.c | 4 +++- 3 files changed, 16 insertions(+), 11 deletions(-) diff --git

[Mesa-dev] [PATCH 15/24] radeonsi: set noalias on input shader pointers

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index caff95b..699fefd 100644 ---

Re: [Mesa-dev] [PATCH 02/24] gallivm, ac: add writeonly and inaccessiblememonly attributes

2017-02-25 Thread Jan Vesely
On Sun, 2017-02-26 at 00:58 +0100, Marek Olšák wrote: > From: Marek Olšák > > --- > src/amd/common/ac_llvm_util.c | 2 ++ > src/amd/common/ac_llvm_util.h | 2 ++ > src/gallium/auxiliary/gallivm/lp_bld_intr.c | 2 ++ >

[Mesa-dev] [Bug 73777] xf86drm.h:40:17: error: drm.h: No such file or directory

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73777 Joaquín Montero Salinas changed: What|Removed |Added CC|

Re: [Mesa-dev] [Mesa-stable] [PATCH v3 1/3] i965/fs: mark last DF uniform array element as 64 bit live one

2017-02-25 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > This bug can make that we don't detect the end of a contiguous area > correctly and push larger areas than the real ones. > > Signed-off-by: Samuel Iglesias Gonsálvez > Cc: "17.0"

[Mesa-dev] [Bug 99856] OpenCL Hello world returns "unsupported call to function get_local_size"

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99856 --- Comment #14 from Jan Vesely --- I think the problem is that the libclc implementation contains uninlined calls (function calls are not supported an AMD GPUs): ; Function Attrs: alwaysinline nounwind define

[Mesa-dev] [PATCH 17/24] radeonsi: enable TC L2 for tessellation offchip stores

2017-02-25 Thread Marek Olšák
From: Marek Olšák Vulkan does the same thing. --- src/gallium/drivers/radeonsi/si_shader.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index

[Mesa-dev] [PATCH 23/24] ac: replace SI.vs.load.input with amdgcn.buffer.load.format

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 20 1 file changed, 20 insertions(+) diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c index 8fac89c..6364657 100644 --- a/src/amd/common/ac_llvm_build.c +++

[Mesa-dev] [PATCH 24/24] ac: normalize build helper names

2017-02-25 Thread Marek Olšák
From: Marek Olšák s/emit/build/ --- src/amd/common/ac_llvm_build.c | 299 ++--- src/amd/common/ac_llvm_build.h | 58 ++-- src/amd/common/ac_nir_to_llvm.c| 208 +++---

[Mesa-dev] [PATCH 18/24] ac: remove offen parameter from ac_build_buffer_store_dword

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_llvm_build.c | 9 - src/amd/common/ac_llvm_build.h | 3 +-- src/amd/common/ac_nir_to_llvm.c | 6 +++--- src/gallium/drivers/radeonsi/si_shader.c | 25 - 4 files changed,

[Mesa-dev] [PATCH 21/24] radeonsi: detect and mark loads/stores from read-only/write-only memory

2017-02-25 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 114 --- 1 file changed, 104 insertions(+), 10 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index 6ccb407..467d0bd

Re: [Mesa-dev] [PATCH] vulkan/wsi: Improve the DRI3 error message

2017-02-25 Thread Jacob Lifshay
Just to double check, is there anything else I need to do to have this patch committed? Jacob Lifshay On Feb 19, 2017 02:08, "Kai Wasserbäch" wrote: > Jason Ekstrand wrote on 19.02.2017 06:01: > > On Feb 18, 2017 12:37 PM, "Kai Wasserbäch"

[Mesa-dev] [PATCH] swr: enable clear_texture with util_clear_texture

2017-02-25 Thread Bruce Cherniak
Passes corresponding piglit tests. --- src/gallium/drivers/swr/swr_context.cpp | 1 + src/gallium/drivers/swr/swr_screen.cpp | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/swr/swr_context.cpp b/src/gallium/drivers/swr/swr_context.cpp index

[Mesa-dev] [PATCH 03/24] radeonsi: set unorm=1 for TGSI_TEXTURE_SHADOWRECT as well

2017-02-25 Thread Marek Olšák
From: Marek Olšák It was harmless, because we also set unorm in the sampler state. --- src/gallium/drivers/radeonsi/si_shader.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c

[Mesa-dev] [PATCH] radv/ac: Add integer->integer casts.

2017-02-25 Thread Bas Nieuwenhuizen
Signed-off-by: Bas Nieuwenhuizen --- src/amd/common/ac_nir_to_llvm.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c index 9778581a1ef..4f3d689db7e 100644 ---

[Mesa-dev] [PATCH 2/2] gallium/u_queue: set num_threads correctly if not all threads start

2017-02-25 Thread Grazvydas Ignotas
If i-th thread could not be created it means we have i threads, not i+1, because we start from 0. Fixes: 404d0d5 "gallium/u_queue: add an option to have multiple worker threads" Signed-off-by: Grazvydas Ignotas --- no commit access src/gallium/auxiliary/util/u_queue.c | 2 +-

[Mesa-dev] [PATCH 1/2] gallium/u_queue: fix a crash with atexit handlers

2017-02-25 Thread Grazvydas Ignotas
Commit 4aea8fe ("gallium/u_queue: fix random crashes when the app calls exit()") added a atexit handler which calls util_queue_killall_and_wait() for each queue to stop the threads. However the app is also free to use atexit handlers to clean up things, leading to util_queue_destroy() call which

Re: [Mesa-dev] [PATCH] swr: enable clear_texture with util_clear_texture

2017-02-25 Thread Edward O'Callaghan
Reviewed-by: Edward O'Callaghan On 02/26/2017 02:09 PM, Bruce Cherniak wrote: > Passes corresponding piglit tests. > --- > src/gallium/drivers/swr/swr_context.cpp | 1 + > src/gallium/drivers/swr/swr_screen.cpp | 2 +- > 2 files changed, 2 insertions(+), 1

Re: [Mesa-dev] [PATCH] radv/ac: Add integer->integer casts.

2017-02-25 Thread Edward O'Callaghan
Acked-by: Edward O'Callaghan On 02/26/2017 12:39 PM, Bas Nieuwenhuizen wrote: > Signed-off-by: Bas Nieuwenhuizen > --- > src/amd/common/ac_nir_to_llvm.c | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git

[Mesa-dev] [Bug 73777] xf86drm.h:40:17: error: drm.h: No such file or directory

2017-02-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73777 Joaquín Montero Salinas changed: What|Removed |Added Status|RESOLVED|REOPENED

Re: [Mesa-dev] [PATCH 10/24] radeonsi: replace SI.packf16 with amdgcn.cvt.pkrtz

2017-02-25 Thread Matt Arsenault
> On Feb 25, 2017, at 15:58, Marek Olšák wrote: > > } > + > +LLVMValueRef ac_emit_cvt_pkrtz_f16(struct ac_llvm_context *ctx, > +LLVMValueRef args[2]) > +{ > + if (HAVE_LLVM >= 0x0500) { > + LLVMTypeRef v2f16 = > +

[Mesa-dev] [PATCH] docs: update features.txt for GL_ARB_clear_texture with swr

2017-02-25 Thread Bruce Cherniak
--- docs/features.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/features.txt b/docs/features.txt index d9528e9..c42581a 100644 --- a/docs/features.txt +++ b/docs/features.txt @@ -192,7 +192,7 @@ GL 4.4, GLSL 4.40 -- all DONE: i965/gen8+, nvc0, radeonsi

Re: [Mesa-dev] [PATCH v3 3/3] i965/fs: fix indirect load DF uniforms on BSW/BXT

2017-02-25 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > The lowered BSW/BXT indirect move instructions had incorrect > source types, which luckily wasn't causing incorrect assembly to be > generated due to the bug fixed in the next patch, but would have > confused the remaining back-end IR

Re: [Mesa-dev] [PATCH v2 1/1] clover: Dump linked binary to a different file

2017-02-25 Thread Francisco Jerez
Jan Vesely writes: > this allows to pass the generated files directly to llc or bugpoint > > v2: add atomic counter ID > > Signed-off-by: Jan Vesely > --- > src/gallium/state_trackers/clover/llvm/invocation.cpp | 8 ++-- > 1 file changed, 6