On Wednesday, 31 August 2016, 15:53:05 CEST, Serge Martin wrote:
> On Wednesday 31 August 2016 12:39:23 Vedran Miletić wrote:
> > On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote:
> > > This is required by the OpenCL standard.
> > >
> > > Signed-o
This is required by the OpenCL standard.
Signed-off-by: Niels Ole Salscheider <niels_...@salscheider-online.de>
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/g
This fixes a crash in pb_cache_release_all_buffers.
Signed-off-by: Niels Ole Salscheider <niels_...@salscheider-online.de>
---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/winsys/rade
Since 8e7df519bd8556591794b2de08a833a67e34d526, we initialise all targets in
clover. This fixes bug 85380.
v2: Mention correct bug in commit message
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On Saturday 24 January 2015, 18:24:16, Jan Vesely wrote:
On Sat, 2015-01-24 at 22:49 +0100, Niels Ole Salscheider wrote:
Since 8e7df519bd8556591794b2de08a833a67e34d526, we initialise all targets
in clover. This fixes bug 85380.
v2: Mention correct bug in commit message
Signed-off
On Thursday 22 January 2015, 13:46:14, Jan Vesely wrote:
On Thu, 2015-01-22 at 16:45 +, Emil Velikov wrote:
On 15/01/15 21:38, Tom Stellard wrote:
On Thu, Jan 15, 2015 at 07:25:56PM +0100, Niels Ole Salscheider wrote:
Since 8e7df519bd8556591794b2de08a833a67e34d526, we initialise all
Since 8e7df519bd8556591794b2de08a833a67e34d526, we initialise all targets in
clover. This fixes bug 85189.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index
not regress
any piglit tests for me.
Can someone with write access please push this?
Thanks,
Ole
On Mon, 2014-09-08 at 20:10 +0200, Niels Ole Salscheider wrote:
v2: Do not demote items that are already in the pool
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From 9714d3ee55ee0ddb0bcf63934b552df641b866a2 Mon Sep 17 00:00:00 2001
From: Niels Ole Salscheider niels_...@salscheider-online.de
Date: Mon, 22 Sep
On Tuesday 09 September 2014, 11:40:49, Bruno Jimenez wrote:
On Mon, 2014-09-08 at 18:30 +0200, Niels Ole Salscheider wrote:
On Monday 08 September 2014, 15:19:15, Bruno Jimenez wrote:
Hi,
I'm not sure if this will work. Imagine this case:
We have an item in the pool, and we
in it.
- Bruno
On Sun, 2014-09-07 at 18:32 +0200, Niels Ole Salscheider wrote:
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/evergreen_compute.c | 27 ---
src/gallium/drivers/r600/evergreen_compute.h | 1 +
src/gallium
v2: Do not demote items that are already in the pool
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/evergreen_compute.h | 1 +
src/gallium/drivers/r600/r600_blit.c | 59
2 files changed, 43 insertions(+), 17
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/evergreen_compute.c | 27 ---
src/gallium/drivers/r600/evergreen_compute.h | 1 +
src/gallium/drivers/r600/r600_blit.c | 40
3 files changed, 41
not offer any
performance improvements for buffer downloads.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeon/r600_buffer_common.c | 20
1 file changed, 8 insertions(+), 12 deletions(-)
diff --git a/src/gallium/drivers
Without this patch I get the following during DMA transfers:
[drm:radeon_cs_ib_chunk] *ERROR* Invalid command stream !
radeon :01:00.0: CP DMA dst buffer too small (21475829792 4096)
This is a fixup for e878e154cdfd4dbb5474f776e0a6d86fcb983098.
Signed-off-by: Niels Ole Salscheider niels_
improvements.
Would it now be OK to change u_upload_mgr or do you have a better proposal?
Ole
Marek
On Mon, Mar 3, 2014 at 9:29 PM, Niels Ole Salscheider
niels_...@salscheider-online.de wrote:
Using the DMA engine for buffer downloads vastly improves performance.
This is because reads from
On Wednesday 16 July 2014, 16:49:08, Tom Stellard wrote:
Also change the wait parameter from false to true.
---
I'm really not sure what is correct here, but this patch fixes event
profiling on SI.
I think you should call end_query in the constructor right after the call to
create_query.
Reviewed-by: Niels Ole Salscheider niels_...@salscheider-online.de
On Wednesday 16 July 2014, 17:37:48, Tom Stellard wrote:
v2:
- Move the end_query() call into the timestamp constructor.
- Still pass false as the wait parameter to get_query_result().
---
src/gallium/state_trackers
This fixes automake builds which are broken since
b52a530ce2aada1967bc8fefa83ab53e6a737dae.
v2: This patch also adds the FEATURE_* defines back to targets/egl-static for
Android and Scons that have been removed in the mentioned commit.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider
This fixes automake builds which are broken since
b52a530ce2aada1967bc8fefa83ab53e6a737dae.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/egl/Makefile.am | 20
1 file changed, 20 insertions(+)
diff --git a/src/gallium
On Tuesday 10 June 2014, 16:18:56, Emil Velikov wrote:
On 10/06/14 15:17, Niels Ole Salscheider wrote:
This fixes automake builds which are broken since
b52a530ce2aada1967bc8fefa83ab53e6a737dae.
Not sure what I was smoking with the above mentioned patch.
Seem like I've completely
be renamed.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/Makefile.sources | 1 +
src/gallium/drivers/radeonsi/si_dma.c | 352 ++
src/gallium/drivers/radeonsi/si_pipe.h| 9 +
src/gallium/drivers
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/evergreen_hw_context.c | 2 +-
src/gallium/drivers/r600/evergreen_state.c | 2 +-
src/gallium/drivers/r600/r600_hw_context.c | 12 +---
src/gallium/drivers/r600/r600_pipe.h
also my latest DMA patches for r600g.
Thanks.
Marek
On Thu, Mar 13, 2014 at 8:45 AM, Niels Ole Salscheider
niels_...@salscheider-online.de wrote:
This code is a slightly modified version of evergreen_dma_blit (and
evergreen_dma_copy as well as evergreen_dma_copy_tile).
It would
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/si_hw_context.c | 3 +++
src/gallium/drivers/radeonsi/si_pipe.c | 22 ++
2 files changed, 25 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_hw_context.c
b
be renamed.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/si_hw_context.c | 65 +++
src/gallium/drivers/radeonsi/si_pipe.h | 7 +
src/gallium/drivers/radeonsi/si_state.c | 265 ++-
src/gallium/drivers
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/r600_pipe.c | 30 -
src/gallium/drivers/r600/r600_pipe.h | 1 -
src/gallium/drivers/radeon/r600_pipe_common.c | 32 +++
src/gallium
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/si_pipe.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index 827e9fe..401bf6a 100644
--- a/src
On Sunday 09 March 2014, 02:24:51, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/r600/evergreen_state.c | 37 +---
src/gallium/drivers/r600/r600_state.c | 41 ++---
src/gallium/drivers/radeon/r600_buffer_common.c |
You are right, r600-r700 require dword alignment while linear copies can be
byte aligned on EG+.
Apart from that, patch 1 and 2 look good to me...
Ole
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
Using DMA for reads is much faster.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeon/r600_buffer_common.c | 74 +++--
1 file changed, 56 insertions(+), 18 deletions(-)
diff --git a/src/gallium/drivers/radeon
, the patch looks good.
CP DMA != async DMA (dma_copy). CP DMA is actually a feature of the
graphics ring.
Marek
On Tue, Mar 4, 2014 at 6:23 PM, Niels Ole Salscheider
niels_...@salscheider-online.de wrote:
Using DMA for reads is much faster.
Signed-off-by: Niels Ole Salscheider niels_
Using DMA for reads is much faster.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeon/r600_buffer_common.c | 78 +++--
1 file changed, 60 insertions(+), 18 deletions(-)
diff --git a/src/gallium/drivers/radeon
Could you please do this without changing u_upload_mgr? You can still
use u_upload_alloc to allocate buffer memory in the driver and the map
buffer read/write flags are not important with persistent coherent
buffer mappings anyway.
I have sent an updated patch to the list.
Ole
Multiply the dw value by 4 in order to map the complete buffer.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/compute_memory_pool.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/r600
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/auxiliary/hud/hud_context.c | 3 +-
src/gallium/auxiliary/util/u_blitter.c| 3 +-
src/gallium/auxiliary/util/u_upload_mgr.c | 49 +++
src/gallium/auxiliary/util
)
1841.07 MB/s(DH)
Judging by these numbers, it might even make sense to use the DMA engine for
larger buffer downloads...
Niels Ole Salscheider (2):
util/u_upload_mgr: Allow to also use it for downloads
radeon: Use transfer manager for buffer downloads
src/gallium/auxiliary/hud/hud_context.c
Using DMA for reads is much faster.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeon/r600_buffer_common.c | 78 +++--
src/gallium/drivers/radeon/r600_pipe_common.c | 11
src/gallium/drivers/radeon/r600_pipe_common.h
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 20 +---
1 file changed, 5 insertions(+), 15 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
b/src/gallium/winsys/radeon/drm
Hi Tom,
this has been on my todo list for quite a while.
Your patch looks good to me, but in my experience a block with approximately
the same size for each dimension gives slightly better performance in many
cases when compared to one where one dimension is significantly larger.
Maybe you
Am Donnerstag, 3. Oktober 2013, 11:08:26 schrieb Francisco Jerez:
Niels Ole Salscheider niels_...@salscheider-online.de writes:
Do you have any example of a real world application that relies on this?
Or at least some reasonable use case?
The problem is that the queue is only cleared
I don't think this is right, with this patch we remove *all* events from
the command queue, signalled or not, every time the command queue is
flushed.
You are right, I got the logic wrong here (see also
http://lists.freedesktop.org/archives/mesa-dev/2013-September/044363.html).
The problem
Do you have any example of a real world application that relies on this?
Or at least some reasonable use case?
The problem is that the queue is only cleared from already signalled events
when we flush it. And we might not do this if the user only calls
clWaitForEvents once the corresponding
Ping
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
object.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/clover/core/event.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/event.cpp
b/src/gallium/state_trackers/clover/core
While unreferencing fences as early as possible is not a bad idea, this patch
hides the underlying problem. That is, events are never deleted from the
queued_events list of the queue object if their fences are signalled before
the queue is flushed.
I will send a patch that fixes the problem
Events that are already signalled can be removed from the queue, too.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/clover/core/queue.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/r600.h| 1 +
src/gallium/drivers/radeonsi/r600_hw_context.c | 28 ++
src/gallium/drivers/radeonsi/r600_query.c | 7 +--
src/gallium/drivers/radeonsi
This makes sure that there are not too many concurrent fences.
Also, simplify status handling by keeping track of the current state.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/clover/core/event.cpp | 29 +++-
src
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c | 9 +
1 Datei geändert, 9 Zeilen hinzugefügt(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_pipe.c
b/src/gallium/drivers/radeonsi/radeonsi_pipe.c
index 3ba8232
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/r600.h| 1 +
src/gallium/drivers/radeonsi/r600_hw_context.c | 31 ++
src/gallium/drivers/radeonsi/r600_query.c | 14 +++-
src/gallium/drivers
The command is submitted once the event has been triggered, but it might not
have completed yet. Therefore, we have to add it to deps in order to wait on it.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/clover/core/event.cpp | 2 +-
1 Datei
This patch adds support for:
PIPE_COMPUTE_CAP_MAX_INPUT_SIZE
PIPE_COMPUTE_CAP_MAX_LOCAL_SIZE
Return the values reported by the closed source driver for now.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c | 15
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/clover/api/event.cpp | 26 -
src/gallium/state_trackers/clover/core/event.cpp | 116 ---
src/gallium/state_trackers/clover/core/event.hpp | 18 +++-
3 Dateien geändert
---
lib/Target/R600/SIInstructions.td | 8 ++--
test/CodeGen/R600/fma.ll | 31 +++
2 Dateien geändert, 37 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)
create mode 100644 test/CodeGen/R600/fma.ll
diff --git a/lib/Target/R600/SIInstructions.td
---
lib/Target/R600/SIISelLowering.cpp | 18 ++
lib/Target/R600/SIISelLowering.h | 1 +
test/CodeGen/R600/fmuladd.ll | 31 +++
3 Dateien geändert, 50 Zeilen hinzugefügt(+)
create mode 100644 test/CodeGen/R600/fmuladd.ll
diff --git
Clover needs the option component of llvm.
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
configure.ac | 4
1 Datei geändert, 4 Zeilen hinzugefügt(+)
diff --git a/configure.ac b/configure.ac
index 62d06e0..0dcd2a5 100644
--- a/configure.ac
+++ b/configure.ac
---
lib/Target/R600/SIInstrInfo.td| 6 ++
lib/Target/R600/SIInstructions.td | 8 ++--
test/CodeGen/R600/fp64_to_sint.ll | 9 +
test/CodeGen/R600/sint_to_fp64.ll | 9 +
4 Dateien geändert, 30 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)
create mode 100644
---
lib/Target/R600/SIISelLowering.cpp | 3 +++
lib/Target/R600/SIInstructions.td | 8 ++--
test/CodeGen/R600/fpext.ll | 9 +
test/CodeGen/R600/fptrunc.ll | 9 +
4 Dateien geändert, 27 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)
create mode 100644
in patch #2 by a vector move since
there is probably no point in having a floating point value in a scalar
register.
Kind regards,
OleFrom 4224b314cf2d97cdf2ac99564d6155fa04fbb971 Mon Sep 17 00:00:00 2001
From: Niels Ole Salscheider niels_...@salscheider-online.de
Date: Sat, 1 Jun 2013 16:48:56
Pass cl_khr_fp64 preprocessor definition to clang
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 1 +
1 Datei geändert, 1 Zeile hinzugefügt(+)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b
to double values.
Regards,
OleFrom 4224b314cf2d97cdf2ac99564d6155fa04fbb971 Mon Sep 17 00:00:00 2001
From: Niels Ole Salscheider niels_...@salscheider-online.de
Date: Sat, 1 Jun 2013 16:48:56 +0200
Subject: [PATCH 1/6] R600/SI: Add initial double precision support for SI
---
lib/Target/R600
Am Freitag, 24. Mai 2013, 14:07:29 schrieb Tom Stellard:
From: Tom Stellard thomas.stell...@amd.com
---
src/gallium/drivers/radeonsi/radeonsi_compute.c | 38
++--- 1 file changed, 27 insertions(+), 11 deletions(-)
diff --git
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/drivers/r600/r600_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/r600_shader.c
b/src/gallium/drivers/r600/r600_shader.c
index 4e5af70..4d74db0 100644
Am Dienstag, 9. April 2013, 11:17:39 schrieb Michel Dänzer:
From: Michel Dänzer michel.daen...@amd.com
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/targets/opencl/Makefile.am | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Am Sonntag, 24. Februar 2013, 15:02:33 schrieb Matt Turner:
On Sun, Feb 24, 2013 at 2:00 PM, Niels Ole Salscheider
niels_...@salscheider-online.de wrote:
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/targets/opencl/Makefile.am | 4 ++--
1 file
Clover needs the irreader component of llvm
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index 81d4a3f..bfba1b3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1649,7 +1649,7 @@ if test x$enable_gallium_llvm = xyes; then
Clover needs the irreader component of llvm
v2: Check for irreader component
irreader is only available with LLVM 3.3 = 177971
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
configure.ac | 4
1 file changed, 4 insertions(+)
diff --git a/configure.ac b
Signed-off-by: Niels Ole Salscheider niels_...@salscheider-online.de
---
src/gallium/targets/opencl/Makefile.am | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/targets/opencl/Makefile.am
b/src/gallium/targets/opencl/Makefile.am
index c5c3003..709112f 100644
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 39717b6..9f58312 100644
--- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
+++
70 matches
Mail list logo