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 `PKG
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
libc
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
(and uses 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:
>
> &g
On Tue, Aug 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(-)
> >
> >
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.
> > >
program base
> Kernel compilation of base finished in 4.55s.
>
> Compiling OpenCL program denoising
> Kernel compilation of denoising finished in 2.08s.
>
> blender: ../src/gallium/drivers/radeonsi/si_compute.c:319:
> si_set_global_binding: Assertion `first + n <= MAX_
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
x27;s just few lines 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.
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
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.protec
ld be
interested in 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 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
lect ICD drivers.
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
it clear that volunteer 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 curren
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
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
>
;flags;
> +
> + if (!sctx->has_graphics) {
> + /* Only 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_GLOBAL_L2 |
> + SI_CONTEXT_INV_L2_METADATA |
> + SI_CONTEXT_CS_PARTIAL_FLUSH;
> + }
> +
> uint32_t cp_coher_cntl = 0;
> uint32_t flush_cb_db = flags & (SI_CONTEXT_FLUSH_AND_INV_CB |
> SI_CONTEXT_FLUSH_AND_INV_DB);
>
> if (flags & SI_CONTEXT_FLUSH_AND_INV_CB)
> sctx->num_cb_cache_flushes++;
> if (flags & SI_CONTEXT_FLUSH_AND_INV_DB)
> sctx->num_db_cache_flushes++;
>
> /* SI has a bug that it always flushes ICACHE and KCACHE if either
> @@ -1061,25 +1073,26 @@ void si_emit_cache_flush(struct si_context *sctx)
> EOP_DATA_SEL_VALUE_32BIT,
> sctx->wait_mem_scratch, va,
> sctx->wait_mem_number, SI_NOT_QUERY);
> si_cp_wait_mem(sctx, cs, va, sctx->wait_mem_number, 0x,
> WAIT_REG_MEM_EQUAL);
> }
>
> /* Make sure ME is idle (it executes most packets) before continuing.
>* This prevents read-after-write hazards between PFP and ME.
>*/
> - if (cp_coher_cntl ||
> - (flags & (SI_CONTEXT_CS_PARTIAL_FLUSH |
> - SI_CONTEXT_INV_VMEM_L1 |
> - SI_CONTEXT_INV_GLOBAL_L2 |
> - SI_CONTEXT_WRITEBACK_GLOBAL_L2))) {
> + if (sctx->has_graphics &&
> + (cp_coher_cntl ||
> + (flags & (SI_CONTEXT_CS_PARTIAL_FLUSH |
> +SI_CONTEXT_INV_VMEM_L1 |
> +SI_CONTEXT_INV_GLOBAL_L2 |
> +SI_CONTEXT_WRITEBACK_GLOBAL_L2 {
> radeon_emit(cs, PKT3(PKT3_PFP_SYNC_ME, 0, 0));
> radeon_emit(cs, 0);
> }
>
> /* SI-CI-VI only:
>* When one of the CP_COHER_CNTL.DEST_BASE flags is set, SURFACE_SYNC
>* waits for idle, so it should be last. SURFACE_SYNC is done in PFP.
>*
>* cp_coher_cntl should contain all necessary flags except TC flags
>* at this point.
> diff --git a/src/gallium/drivers/radeonsi/si_texture.c
> b/src/gallium/drivers/radeonsi/si_texture.c
> index a50088d2d8f..581f90a7b2f 100644
> --- a/src/gallium/drivers/radeonsi/si_texture.c
> +++ b/src/gallium/drivers/radeonsi/si_texture.c
> @@ -457,20 +457,23 @@ static bool si_texture_discard_dcc(struct si_screen
> *sscreen,
> * compressed tiled
> *
> * \param sctx the current context if you have one, or sscreen->aux_context
> * if you don't.
> */
> bool si_texture_disable_dcc(struct si_context *sctx,
> struct si_texture *tex)
> {
> struct si_screen *sscreen = sctx->screen;
>
> + if (!sctx->has_graphics)
> + return si_texture_discard_dcc(sscreen, tex);
> +
> if (!si_can_disable_dcc(tex))
> return false;
>
> if (&sctx->b == sscreen->aux_context)
> mtx_lock(&sscreen->aux_context_lock);
>
> /* Decompress DCC. */
> si_decompress_dcc(sctx, tex);
> sctx->b.flush(&sctx->b, NULL, 0);
>
--
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
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/
n some hours.
>
> 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
>>>> 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?
> >>>>>
> >>>>> i
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 auto
thanks,
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
true -Dgallium-opencl=icd
> -Dglvnd=true -Dgallium-va=false -Dgallium-xvmc=false
> -Dgallium-omx=disabled -Dgallium-xa=false
>
> Only -Dgallium-opencl=disabled works.
>
> Thanks,
> Dieter
> ___
> mesa-dev mailing list
>
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
> > > compute ring.
&g
t; 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 clove
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 was
ICE_TYPE_ALL) No devices found
> in platform
>
> ICD loader properties
> ICD loader Name OpenCL ICD Loader
> ICD loader Vendor OCL Icd free software
> ICD loader Version 2.2.12
> I
rd plugged in.
>
> 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
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,
>
k
Jan
On Mon, Dec 24, 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&
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.
> >
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/mes
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
> > >
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 any
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 12:09 +, Em
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 r6
ass to 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 va
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 Franc
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 detai
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 mac
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'
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 provid
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 w
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
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
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 bu
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 ali
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/ev
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/s
lar-arithemtic-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_
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
unsuccessful result
> 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
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
ight now.
> > -*
> > -* LLVM 5.0 on SI doesn't insert a required s_nop between SALU
> > setting
> > -* a descriptor and s_buffer_load_dword using it, so we can't
> > expand
> > - * the pointer into a full descriptor like below. We have to use
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 wrote:
> > r332881 added an extra parameter to the emit function.
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106619
> > Signed-off-by: Jan Vesely
> >
Signed-off-by: Jan Vesely
---
.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 @@ matrix:
- libx11-xcb-dev
Signed-off-by: Jan Vesely
---
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/compat.hpp
r332881 added an extra parameter to the emit function.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106619
Signed-off-by: Jan Vesely
---
.../state_trackers/clover/llvm/codegen/native.cpp | 3 +--
src/gallium/state_trackers/clover/llvm/compat.hpp | 10 ++
2 files
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);
> - if (err == -1)
> - goto done;
> -
>
On Fri, 2018-05-18 at 21:31 -0500, Aaron Watry wrote:
> On Fri, May 18, 2018 at 1:15 PM, Jan Vesely wrote:
> > On Thu, 2018-05-17 at 19:20 -0500, Aaron Watry wrote:
> > > They're not used anywhere else, so keep them private
> > >
> > > Signed-off-by: Aa
On Thu, 2018-05-17 at 19:20 -0500, Aaron Watry wrote:
> They're not used anywhere else, so keep them private
>
> Signed-off-by: Aaron Watry
I'm not sure what the original purpose of the removed functions was. If
you recall please add it to the commit message.
Either way
meson Vulkan, Clover, and autotools Vulkan need to be switched to llvm 5
Fixes: f9eb1ef870eba9fdacf9a8cbd815ec3bff81db05
Signed-off-by: Jan Vesely
---
.travis.yml | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index c8b68a6696
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 wrote:
>
> > Is this still needed for llvm-6.0.1?
> >
>
> Probably.
was it reproducible on non-ubuntu systems? There's still some time
until 6.0.1 is out, but I do
er Nützel wrote:
> 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
d 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 this. We probably need CLANG_LIBS handl
Fixes memory leak on module unload.
CC:
Signed-off-by: Jan Vesely
---
src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
index
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 a/src/gallium/drivers/radeonsi/si_pipe.c b/src/gallium/drivers/
> radeonsi/si_
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:
Signed-off-by: Jan Vesely
---
src/gallium/drivers/r600/compute_memory_poo
1 - 100 of 581 matches
Mail list logo
