On Tue, Apr 27, 2021 at 7:50 AM Luke A. Guest wrote:
>
>
> On 27/04/2021 08:00, Pierre Moreau wrote:
> > Hello Luke,
> >
> > If you set `PKG_CONFIG_PATH=$PATH_TO_LIBCLC_INSTALL/share/pkgconfig` when
> > running meson, it should pick that version instead of the system one.
> >
> > I run it as
On Thu, 2020-11-12 at 06:37 +1000, Dave Airlie wrote:
> On Thu, 12 Nov 2020 at 06:23, Francisco Jerez wrote:
> > I don't remember the specifics of why we ended up interfacing with Clang
> > this way. What is technically wrong with it, specifically? I don't
> > have any objection to switching to
On Wed, 2020-04-15 at 06:31 +0200, Dieter Nützel wrote:
> Am 15.04.2020 05:50, schrieb Jan Vesely:
> > On Wed, 2020-04-15 at 05:40 +0200, Dieter Nützel wrote:
> > > Addendum
> > >
> > > I'm building LLVM git (llvm + SPIRV-LLVM-Translator) for X86;AMDGPU
>
On Wed, 2020-04-15 at 05:40 +0200, Dieter Nützel wrote:
> Addendum
>
> I'm building LLVM git (llvm + SPIRV-LLVM-Translator) for X86;AMDGPU like
> this:
there's nothing wrong with your build. rocm libs are required by clang
when it targets amdgcn--amdhsa target. Unless you know that you want
CodeGenFileType enum was moved to Support/CodeGen.h
Signed-off-by: Jan Vesely
---
Tested by building and running CLOVER_DEBUG=llvm,native clinfo using
both llvm-git (10) and llvm-9
.../state_trackers/clover/llvm/codegen/native.cpp| 10 --
src/gallium/state_trackers/clover/llvm
shared pool to accommodate global buffers).
cl global buffers are created in cl context so I think there shouldn't
be too much trouble moving it to the driver equivalent of that.
Jan
>
> Marek
>
> On Tue, Aug 6, 2019 at 10:50 PM Dieter Nützel wrote:
>
> > Am 20.06.201
g 6, 2019 at 12:59 PM Jan Vesely wrote:
> > v2: Drop special case of llvm-9
> > Signed-off-by: Jan Vesely
> > ---
> > src/gallium/state_trackers/clover/llvm/compat.hpp | 10 --
> > 1 file changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --gi
v2: Drop special case of llvm-9
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/llvm/compat.hpp | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/compat.hpp
b/src/gallium/state_trackers/clover/llvm/compat.hpp
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/llvm/compat.hpp | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/compat.hpp
b/src/gallium/state_trackers/clover/llvm/compat.hpp
index 0ecf622a9af..b273abf75af
On Sat, 2019-06-15 at 07:38 +0200, Dieter Nützel wrote:
> Am 14.06.2019 08:13, schrieb Jan Vesely:
> > On Thu, 2019-06-13 at 21:20 +0200, Dieter Nützel wrote:
> > > Am 13.06.2019 07:10, schrieb Marek Olšák:
> > > > FYI, I just pushed the new linker.
> > >
finished in 2.08s.
>
> blender: ../src/gallium/drivers/radeonsi/si_compute.c:319:
> si_set_global_binding: Assertion `first + n <= MAX_GLOBAL_BUFFERS'
> failed.
>
> [1]Abbruch blender (core dumped)
The number of max global buffers was bumped in 06bf567
On Thu, 2019-06-13 at 01:10 -0400, Marek Olšák wrote:
> FYI, I just pushed the new linker.
thanks. I've checked the handling and the current approach works for
sections as well (even if not handled explicitly).
Jan
>
> Marek
>
> On Mon, Jun 3, 2019 at 10:39 PM Jan Vesely wrot
nes to patch that in. is there an ETA for
the linker series?
Jan
>
> Marek
>
> On Mon, Jun 3, 2019 at 10:39 PM Jan Vesely wrote:
>
> > Cc: mesa-sta...@lists.freedesktop.org
> > Signed-off-by: Jan Vesely
> > ---
> > src/amd/common/ac_binary.c | 2 ++
> >
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Jan Vesely
---
src/gallium/drivers/radeonsi/si_compute.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_compute.c
b/src/gallium/drivers/radeonsi/si_compute.c
index 51da06fe550
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Jan Vesely
---
src/amd/common/ac_binary.c | 2 ++
src/amd/common/ac_binary.h | 2 ++
2 files changed, 4 insertions(+)
diff --git a/src/amd/common/ac_binary.c b/src/amd/common/ac_binary.c
index 8f4755ebe16..18dc72c61f0 100644
--- a/src/amd
Fixes piglits:
call.cl
calls-larget-struct.cl
calls-struct.cl
calls-workitem-id.cl
realign-stack.cl
tail-calls.cl
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Jan Vesely
---
The piglit test now pass using llvm-7,8,git.
ImageMagick works
On Mon, 2019-06-03 at 11:12 +1000, Dave Airlie wrote:
> On Mon, 3 Jun 2019 at 10:58, Jan Vesely wrote:
> > On Sun, 2019-06-02 at 20:09 -0400, James Harvey wrote:
> > > I've started a thread on the llvm mailing list. See
> > > https://nam02.safelinks.protection.
taking the LLVM fix.
I've started looking, but the elf structure lists relocations against
a section (STT_SECTION), so I still don't know where to get target
address.
Jan
>
> On Sat, Jun 1, 2019 at 10:56 PM Jan Vesely wrote:
> > Hi,
> >
> > On Sat, 2019-06-01 at 18:21
On Mon, 2019-06-03 at 09:14 +1000, Dave Airlie wrote:
> On Mon, 3 Jun 2019 at 09:13, Dave Airlie wrote:
> > On Sun, 2 Jun 2019 at 23:13, Jan Vesely wrote:
> > > On Sun, 2019-06-02 at 07:17 -0400, James Harvey wrote:
> > > > On Sun, Jun 2, 2019 at 7:01 AM Ba
s.
something like 'OCD_ICD_VENDORS=/var/empty' (any directory other than
/etc/OpenCL/vendors will do) effectively disables OpenCL by reporting
0 available platforms.
Jan
--
Jan Vesely
signature.asc
Description: This is a digitally signed message part
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
contributions and needs
are inferior to corporate stakeholders (not even worth CI cycles).
thus, clover on amd gpus is in damage mitigation mode; errors are
better than crashes, crashes are better than hangs.
Chances are that clover over NIR will work before the current issues
are fixed.
regards,
assert(16 * ctx->screen->info.num_render_backends <=
> scratch->b.b.width0);
> radeon_emit(cs, PKT3(PKT3_EVENT_WRITE, 2, 0));
> radeon_emit(cs, EVENT_TYPE(EVENT_TYPE_ZPASS_DONE
On Mon, May 13, 2019 at 6:40 PM Marek Olšák wrote:
>
> From: Marek Olšák
>
> ---
> src/gallium/drivers/radeonsi/si_pipe.c | 2 +-
> src/gallium/drivers/radeonsi/si_query.c | 7 ---
> 2 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
>
nly process compute flags. */
> + flags &= SI_CONTEXT_INV_ICACHE |
> + SI_CONTEXT_INV_SMEM_L1 |
> + SI_CONTEXT_INV_VMEM_L1 |
> + SI_CONTEXT_INV_GLOBAL_L2 |
> + SI_CONTEXT_WRITEBACK_GLO
On Wed, Apr 10, 2019 at 9:33 PM Marek Olšák wrote:
>
> Reviewed-by: Marek Olšák
thank you. pushed.
Jan
>
> Marek
>
> On Wed, Apr 3, 2019 at 2:02 AM Jan Vesely wrote:
>>
>> Skip over non-existent files.
>> Signed-off-by: Jan Vesely
>> ---
>>
On Tue, Apr 9, 2019 at 5:36 PM Vinson Lee wrote:
>
> On Tue, Apr 9, 2019 at 1:29 PM Jan Vesely wrote:
> >
> > On Sat, Apr 6, 2019 at 10:11 PM Dieter Nützel wrote:
> > >
> > > Tested-by: Dieter Nützel
> > > Acked-by: Dieter Nützel
> >
&
On Wed, Apr 3, 2019 at 2:02 AM Jan Vesely wrote:
>
> Skip over non-existent files.
> Signed-off-by: Jan Vesely
> ---
> This should help detect instances of messed up/missing symbols in the driver.
> windows build seems OK:
> https://ci.appveyor.com/project/jvesely/mesa/
rs.
>
> Dieter
>
> Am 06.04.2019 06:43, schrieb Jan Vesely:
> > This partially reverts commit 356ec7a21960d77db282f67af577dcdb46966b5a.
> > There are missing symbols needed by libglsl, so we might as well skip
> > the entire library (which should be present in the me
>>>> On Mon, Apr 1, 2019 at 1:28 PM Jan Vesely
> >>> wrote:
> >>>>> On Mon, 2019-04-01 at 12:30 -0400, Marek Olšák wrote:
> >>>>>> Does the attached patch fix the copy-buffer test?
> >>>>>
> >>>>&
Hi,
is there a release with that change planned any time soon?
thanks,
Jan
On Mon, Apr 8, 2019 at 4:23 PM Dylan Baker wrote:
>
> As per the discussion last time around, the last feature meson needed to grow
> to
> be able to remove autotools was support for remembering PKG_CONFIG_PATH, since
This partially reverts commit 356ec7a21960d77db282f67af577dcdb46966b5a.
There are missing symbols needed by libglsl, so we might as well skip the
entire library (which should be present in the mesa stat tracker).
Signed-off-by: Jan Vesely
---
Hi Timur, Vinson,
this patch is enough to get
Skip over non-existent files.
Signed-off-by: Jan Vesely
---
This should help detect instances of messed up/missing symbols in the driver.
windows build seems OK:
https://ci.appveyor.com/project/jvesely/mesa/builds/23550498#L1292
.../auxiliary/pipe-loader/pipe_loader.c | 5 +-
src/gallium
On Mon, 2019-04-01 at 18:37 -0400, Marek Olšák wrote:
> From: Marek Olšák
Tested-by: Jan Vesely
Can you add a note along the lines; "compute rings don't have PFP" or
anything more descriptive on the commit message?
thanks,
Jan
>
> Fixes: a1378639ab1 "radeonsi:
On Tue, 2019-04-02 at 02:37 +0200, Dieter Nützel wrote:
> Am 01.04.2019 07:43, schrieb Jan Vesely:
> > Hi,
> >
> > On Mon, 2019-04-01 at 06:24 +0200, Dieter Nützel wrote:
> > > Hello,
> > >
> > > commit #356ec7a2196 'gallium: fix autotools
nks,
Jan
>
> Thanks,
> Marek
>
> On Sat, Mar 2, 2019 at 10:58 PM Jan Vesely wrote:
>
> > On Thu, 2019-02-28 at 17:48 -0500, Marek Olšák wrote:
> > > On Thu, Feb 28, 2019 at 4:44 AM Jan Vesely
> > wrote:
> > > > On Tue, 2019-02-26 at 18:34 -
gallium-xvmc=false
> -Dgallium-omx=disabled -Dgallium-xa=false
>
> Only -Dgallium-opencl=disabled works.
>
> Thanks,
> Dieter
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedes
On Thu, 2019-02-28 at 17:48 -0500, Marek Olšák wrote:
> On Thu, Feb 28, 2019 at 4:44 AM Jan Vesely wrote:
>
> > On Tue, 2019-02-26 at 18:34 -0500, Marek Olšák wrote:
> > > I ran a simple test verifying that compute is working properly on the
> > >
When clover is using compute rings, it doesn't stall/block graphics
> operations.
I'd be nice to include this information in the commit message.
Jan
>
> Marek
>
> On Tue, Feb 26, 2019 at 4:10 PM Jan Vesely wrote:
>
> > Can you add a bit of background why clover should/should
Can you add a bit of background why clover should/should not use other
rings?
I planned to test this, but my raven system can't run clover since kernel
4.20 release (BZ 109649), so I need to bisect that first.
Can this patch help address the soft lockup issue on CIK (BZ 108879)?
presumably, it
d
> in platform
>
> ICD loader properties
> ICD loader Name OpenCL ICD Loader
> ICD loader Vendor OCL Icd free software
> ICD loader Version 2.2.12
> ICD loader Profile
>
> The stable 18.3.1 it works.
>
> Don't know how to analise this kind of problem.
>
> Sincerely
>
> AndyBe
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mai
On Wed, 2018-12-26 at 15:26 +, Eric Engestrom wrote:
> On Tuesday, 2018-12-25 23:09:53 +0100, Jan Vesely wrote:
> > Guess my meson-fu is still pretty weak.
> > Now I see the build failure again:
> > In file included from ../mesa/src/intel/vulkan/anv_private.h:77:0,
>
2018 at 10:51 AM Jan Vesely wrote:
> On Sun, 2018-12-23 at 15:35 +, Eric Engestrom wrote:
> > On Sunday, 2018-12-23 12:31:20 +0100, Jan Vesely wrote:
> > > From: Jan Vesely
> > >
> > > intel_vulkan.h uses '#include "vulkan.h"' so the file needs to be
On Sun, 2018-12-23 at 15:35 +, Eric Engestrom wrote:
> On Sunday, 2018-12-23 12:31:20 +0100, Jan Vesely wrote:
> > From: Jan Vesely
> >
> > intel_vulkan.h uses '#include "vulkan.h"' so the file needs to be in
> > include path.
> > Fixes mes
From: Jan Vesely
intel_vulkan.h uses '#include "vulkan.h"' so the file needs to be in
include path.
Fixes meson build of anv
Signed-off-by: Jan Vesely
---
src/intel/vulkan/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/meson.build b
On Sun, 2018-12-16 at 10:36 +0100, Kai Wasserbäch wrote:
> Jan Vesely wrote on 13.12.18 22:17:
> > CodeGenOptions were moved to Basic.
> >
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109039
> > Signed-off-by: Jan Vesely
>
> Since this
CodeGenOptions were moved to Basic.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109039
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/llvm/compat.hpp | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/llvm
On Mon, 2018-12-03 at 16:17 -0800, Dylan Baker wrote:
> Quoting Bas Nieuwenhuizen (2018-12-03 14:14:19)
> > On Mon, Dec 3, 2018 at 7:30 PM Jan Vesely wrote:
> > > On Mon, 2018-12-03 at 10:21 +, Eric Engestrom wrote:
> > > > Cc: Emil Velikov
> > &g
On Mon, 2018-12-03 at 10:21 +, Eric Engestrom wrote:
> Cc: Emil Velikov
> Cc: Andres Gomez
> Cc: Juan A. Suarez Romero
> Cc: Dylan Baker
> Signed-off-by: Eric Engestrom
> ---
> This patch depends on the releasing procedure in docs/releasing.html to
> be updated to not rely on autotools
On Wed, 2018-11-07 at 13:05 -0800, Matt Turner wrote:
> On Wed, Nov 7, 2018 at 12:53 PM Jan Vesely wrote:
> > On Wed, 2018-11-07 at 18:51 +, Emil Velikov wrote:
> > > On Wed, 7 Nov 2018 at 18:39, Jan Vesely wrote:
> > > > On Tue, 2018-11-06 at 1
On Wed, 2018-11-07 at 11:34 -0800, Dylan Baker wrote:
> Quoting Jan Vesely (2018-11-07 10:39:48)
> > On Tue, 2018-11-06 at 12:09 +, Emil Velikov wrote:
> > > On Thu, 1 Nov 2018 at 16:13, Michel Dänzer wrote:
> > > > On 2018-11-01 5:03 p.m., Jan Vesely wrote:
>
On Wed, 2018-11-07 at 18:51 +, Emil Velikov wrote:
> On Wed, 7 Nov 2018 at 18:39, Jan Vesely wrote:
> > On Tue, 2018-11-06 at 12:09 +, Emil Velikov wrote:
> > > On Thu, 1 Nov 2018 at 16:13, Michel Dänzer wrote:
> > > > On 2018-11-01 5:03 p.m., Jan Vesely wr
On Tue, 2018-11-06 at 12:09 +, Emil Velikov wrote:
> On Thu, 1 Nov 2018 at 16:13, Michel Dänzer wrote:
> > On 2018-11-01 5:03 p.m., Jan Vesely wrote:
> > > On Wed, 2018-10-31 at 18:40 +, Emil Velikov wrote:
> > > > On Wed, 31 Oct 2018 at 17:41, Jan Vesely wr
The option was removed in LLVM r345763
Signed-off-by: Jan Vesely
---
src/amd/common/ac_llvm_util.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_llvm_util.c b/src/amd/common/ac_llvm_util.c
index 69d9f7b9f3..dc9b684e9d 100644
--- a/src/amd/common
On Wed, 2018-10-31 at 18:40 +, Emil Velikov wrote:
> On Wed, 31 Oct 2018 at 17:41, Jan Vesely wrote:
> > On Wed, 2018-10-31 at 17:22 +, Emil Velikov wrote:
> > > On Wed, 31 Oct 2018 at 16:24, Michel Dänzer wrote:
> > > > On 2018-10-31 5:19 p.m., Jan Vesely
On Wed, 2018-10-31 at 16:14 -0700, Dylan Baker wrote:
> Quoting Jan Vesely (2018-10-31 11:37:19)
> > On Wed, 2018-10-31 at 11:22 -0700, Francisco Jerez wrote:
> > > I don't object removing the pre-LLVM 5.0 paths, nor keeping them around
> > > in order to make r600
o the new style of metadata passing and
updating libclc builtins.
There are things higher on my priority list wrt clover/libclc (fixing
rounding routines, replacing the build system, adding support for
buffer sharing between devices). it will take some time unless someone
else volunteers.
Jan
>
On Wed, 2018-10-31 at 17:02 +, Emil Velikov wrote:
> On Wed, 31 Oct 2018 at 15:57, Jan Vesely wrote:
> > On Wed, 2018-10-31 at 13:30 +, Emil Velikov wrote:
> > > From: Emil Velikov
> > >
> > > LLVM versions earlier than 5.0.1 are no longer supported.
On Wed, 2018-10-31 at 17:22 +, Emil Velikov wrote:
> On Wed, 31 Oct 2018 at 16:24, Michel Dänzer wrote:
> > On 2018-10-31 5:19 p.m., Jan Vesely wrote:
> > > On Wed, 2018-10-31 at 13:30 +, Emil Velikov wrote:
> > > > From: Emil Velikov
> > > >
&
On Wed, 2018-10-31 at 13:30 +, Emil Velikov wrote:
> From: Emil Velikov
>
> The option has been long superseded with LLVM_CONFIG.
> Distributions have been using it for a couple of years now.
I've been using it in my configure setup. This might be a stupid
question; is the LLVM_CONFIG env
On Wed, 2018-10-31 at 13:30 +, Emil Velikov wrote:
> From: Emil Velikov
>
> LLVM versions earlier than 5.0.1 are no longer supported.
I'd prefer to keep llvm-3.9, since it's the last version that supports
images for r600, but I'd leave the final decision to Francisco.
Jan
>
Hi,
I think this patch breaks the build:
https://travis-ci.org/jvesely/mesa/jobs/443356781
Jan
On Tue, 2018-10-16 at 08:18 -0500, Jason Ekstrand wrote:
> FYI, patch 1 is required for this patch to build. It also means this patch
> found a nice little bug. I'll respond to patch 1 in more
Fixes assertion failure/crash when running luxmark/luxball on clover.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108272
CC: mesa-sta...@lists.freedesktop.org
Signed-off-by: Jan Vesely
---
This is just a stop gap measure. OpenCL does not limit the number of
global buffers, but we can
On Thu, 2018-10-18 at 09:32 +0200, Michel Dänzer wrote:
> On 2018-10-17 6:43 p.m., Marek Olšák wrote:
> > Can you test the attached patch?
>
> Doesn't help, unfortunately. Backtraces with the patch attached.
>
> FWIW, this is on Bonaire.
Hi,
fwiw, I don't see this hang on my carrizo/iceland
On Tue, 2018-09-25 at 18:11 +0100, Emil Velikov wrote:
> On 12 September 2018 at 23:20, Jan Vesely wrote:
> > SWR takes 17+ minutes to build. Clover builds take ~6-7 minutes.
> >
>
> Clover takes the same time as all the other state-trackers combined.
> Ideally we'll t
On Wed, 2018-09-19 at 11:43 -0400, Jan Vesely wrote:
> On Wed, 2018-09-12 at 18:20 -0400, Jan Vesely wrote:
> > SWR takes 17+ minutes to build. Clover builds take ~6-7 minutes.
> >
> > Signed-off-by: Jan Vesely
> > ---
> > .travis.yml | 4
> > 1 fil
The warning is correct. In the first case, memset tries to zero "Target"
object which has a non-trivial constructor and non-trivial
copy-constructor. The original code is broken in the way it mixes C and C++
initialization and the patch only papers over the issue.
The correct fix would be to
On Wed, 2018-09-12 at 18:20 -0400, Jan Vesely wrote:
> SWR takes 17+ minutes to build. Clover builds take ~6-7 minutes.
>
> Signed-off-by: Jan Vesely
> ---
> .travis.yml | 4
> 1 file changed, 4 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 40
Hi,
On Mon, 2018-09-17 at 09:44 -0700, Dylan Baker wrote:
> I feel like for !windows meson is in good enough shape at this point that we
> can start having the discussion about deleting the autotools build. So, is
> there
> anything left that autotools can do that meson cannot (that we actually
Signed-off-by: Jan Vesely
---
.travis.yml | 33 +
1 file changed, 33 insertions(+)
diff --git a/.travis.yml b/.travis.yml
index 895030cc1b..4035a729ff 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -329,6 +329,39 @@ matrix:
- libx11-xcb-dev
SWR takes 17+ minutes to build. Clover builds take ~6-7 minutes.
Signed-off-by: Jan Vesely
---
.travis.yml | 4
1 file changed, 4 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 4035a729ff..78e6d251ae 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -186,7 +186,6 @@ matrix
0_Serie
s_Instruction_Set_Architecture.pdf
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
--
Jan Vesely
signature.asc
Description: This is a digitally signed message part
_
On Tue, Aug 7, 2018 at 12:28 PM, Juan A. Suarez Romero
wrote:
> On Tue, 2018-08-07 at 16:12 +0100, Emil Velikov wrote:
> > On 6 August 2018 at 11:17, Juan A. Suarez Romero
> wrote:
> > > LLVM 6.0 requires GCC 4.9, which is not available in main Travis
> > > repository.
> > >
> >
> > We're not
On Thu, 2018-07-26 at 10:51 -0700, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > It's OK to pass them in memory, which is what kernel invocation needs.
> > Fixes AMDGCN regressions since llvm r337535 ("Reapply "AMDGPU: Fix handling
> > of alignmen
Trigger waiter condition variable.
Passes 'events' CTS on carrizo and turks.
v2: reduce to 0
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/core/event.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/core/event.cpp
b
Trigger waiter condition variable.
Passes 'events' CTS on turks and carrizo.
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/core/event.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/core/event.cpp
b/src/gallium
rithemtic-short
scalar-arithmetic-ushort
scalar-comparison-char
scalar-comparison-uchar
scalar-comparison-short
scalar-comparison-ushort
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/llvm/codegen/common.cpp | 3 +--
src/gallium/state_track
On Tue, 2018-07-17 at 15:22 -0700, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > On Tue, 2018-07-17 at 14:17 -0700, Francisco Jerez wrote:
> > > Jan Vesely writes:
> > >
> > > > Abort all dependent events.
> > > > Signed-off-by: Jan V
On Tue, 2018-07-17 at 14:17 -0700, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > Abort all dependent events.
> > Signed-off-by: Jan Vesely
> > ---
> > src/gallium/state_trackers/clover/core/event.cpp | 7 ++-
> > 1 file changed, 6 insertions(+), 1 d
Waiting for tasks to finish executing
> Checking task status after setting user event status
> Unsuccessful user event case passed.
> test_userevents passed
> HUNG HERE
>
> --Aaron
>
> On Tue, Jul 17, 2018 at 9:38 AM, Jan Vesely wrote:
> > Abort all dependent events.
&g
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/core/kernel.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 4716705323..dab7ef8683 100644
--- a/src/gallium
Abort all dependent events.
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/core/event.cpp | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/core/event.cpp
b/src/gallium/state_trackers/clover/core/event.cpp
index
They might lead to unrecoverable GPU hang.
Signed-off-by: Jan Vesely
---
src/gallium/drivers/radeonsi/si_compute.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_compute.c
b/src/gallium/drivers/radeonsi/si_compute.c
index 5d3341ff61..2349be9584 100644
> > expand
> > -* the pointer into a full descriptor like below. We have to use
> > -* s_load_dword instead. The only case when LLVM 5.0 would
> > select
> > -* s_buffer_load_dword (that we have to prevent)
Use of void * in pointer arithmetic is illegal, use char * instead.
Fixes: cf54bd5e8381dba18d52fe438acda20cc1685bf3 ("drisw: use shared memory when
possible")
Signed-off-by: Jan Vesely
---
src/gallium/winsys/sw/dri/dri_sw_winsys.c | 2 +-
1 file changed, 1 insertion(+), 1 deletio
On Thu, 2018-05-24 at 13:08 -0500, Aaron Watry wrote:
> On Tue, May 22, 2018 at 6:43 PM, Jan Vesely <jan.ves...@rutgers.edu> wrote:
> > r332881 added an extra parameter to the emit function.
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106619
> > Signed-
Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
---
.travis.yml | 35 +++
1 file changed, 35 insertions(+)
diff --git a/.travis.yml b/.travis.yml
index e3471d47ac..33dcbcc8da 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -290,6 +290,41 @@
Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
---
I've confirmed that simple piglits run on turks(llvm-7) and
carrizo(llvm-6,7). Travis says it builds ok for all supported llvm
versions.
.../clover/llvm/codegen/native.cpp| 7 +-
.../state_trackers/clover/llvm/comp
r332881 added an extra parameter to the emit function.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106619
Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
---
.../state_trackers/clover/llvm/codegen/native.cpp | 3 +--
src/gallium/state_trackers/clover/llvm/compat.hpp
; - /* With the temporary file open, we take an exclusive flock on
> -* it. If the flock fails, then another process still has the file
> -* open with the flock held. So just let that file be responsible
> -* for writing the file.
> -*/
> - err = flock(fd, LOCK_EX | LOCK_NB);
On Fri, 2018-05-18 at 21:31 -0500, Aaron Watry wrote:
> On Fri, May 18, 2018 at 1:15 PM, Jan Vesely <jan.ves...@rutgers.edu> wrote:
> > On Thu, 2018-05-17 at 19:20 -0500, Aaron Watry wrote:
> > > They're not used anywhere else, so keep them private
> > >
>
age.
Either way:
Reviewed-by: Jan Vesely <jan.ves...@rutgers.edu>
Jan
> ---
> .../drivers/r600/compute_memory_pool.c| 35 +++
> .../drivers/r600/compute_memory_pool.h| 24 -
> 2 files changed, 29 insertions(+), 30 deletions(-)
>
>
meson Vulkan, Clover, and autotools Vulkan need to be switched to llvm 5
Fixes: f9eb1ef870eba9fdacf9a8cbd815ec3bff81db05
Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
---
.travis.yml | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/.travis.yml b/.trav
On Wed, 2018-05-09 at 15:54 -0400, Jan Vesely wrote:
> Use pipe_refernce to release old RAT surfaces.
> RAT surface adds a reference to pool bo. So use reference counting for
> pool->bo
> as well.
>
> v2: Use the same pattern for both defrag paths
> Drop confusing co
On Thu, 2018-05-10 at 17:10 -0400, Marek Olšák wrote:
> On Thu, May 10, 2018 at 4:39 PM, Jan Vesely <jan.ves...@rutgers.edu> wrote:
>
> > Is this still needed for llvm-6.0.1?
> >
>
> Probably.
was it reproducible on non-ubuntu systems? There's still some time
u
:
> Hello Jan and Kai,
>
> would be nice if this lands (have T-b from me).
>
> Dieter
>
> Am 13.05.2018 19:10, schrieb Jan Vesely:
> > Hi,
> >
> > sorry for the delay. I thought maybe Emil is on holiday. I plan to
> > push this on Monday evening (EDT)
an FTBFS, has two
> T-b
> and one A-b.
>
>
> Kai Wasserbäch wrote on 07.05.2018 16:48:
> > Jan Vesely wrote on 02.05.2018 22:38:
> >> On Wed, 2018-05-02 at 18:38 +0200, Kai Wasserbäch wrote:
> >>> [...]
> >>
> >> Thank for looking into th
Fixes memory leak on module unload.
CC: <mesa-sta...@lists.freedesktop.org>
Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
---
src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amd
Is this still needed for llvm-6.0.1?
On Mon, Apr 16, 2018 at 8:52 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/radeonsi/si_pipe.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git
Use pipe_refernce to release old RAT surfaces.
RAT surface adds a reference to pool bo. So use reference counting for pool->bo
as well.
v2: Use the same pattern for both defrag paths
Drop confusing comment
CC: <mesa-sta...@lists.freedesktop.org>
Signed-off-by: Jan Vesely
1 - 100 of 579 matches
Mail list logo