On 08/12/2016 00:38, Dieter Nützel wrote:
Am 07.12.2016 23:54, schrieb Axel Davy:
This patch adds internal multithreading to gallium nine.
The goal is to offload almost all gallium nine calls (and some other
work) to a worker thread.
Hello Axel,
really nice job!
'But' show us the 'numbers'..
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 14 +-
1 file changed, 5 insertions(+), 9 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index f644980..0ad9a7a 100644
--- a/src/amd/common/ac_nir_to_llvm
While working on spilling, geom shaders and compute queues, I've built
up a bunch of cleanups, so best to dequeue them early.
Dave.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Dave Airlie
Removes some unnecessary functions and pull
some stuff out of the loop.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_bufimage.c | 42 +++--
1 file changed, 7 insertions(+), 35 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_bufim
From: Dave Airlie
This make it easier to add multiple queues later.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_copy.c | 34 +++---
1 file changed, 23 insertions(+), 11 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_copy.c b/src/amd/vulkan/radv_meta
From: Dave Airlie
This code will have use for copy shaders etc.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_pipeline.c | 76 --
1 file changed, 43 insertions(+), 33 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pi
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 5820e62..c2a74d4 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_ni
From: Dave Airlie
This makes it easier to add new params before the array ones.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index
From: Dave Airlie
This isn't needed for later things like geom shader copy shaders,
we won't have NIR.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/comm
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_blit2d.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_blit2d.c
b/src/amd/vulkan/radv_meta_blit2d.c
index 6e92f80..5a42fd8 100644
--- a/src/amd/vulkan/radv_meta_blit2d.c
+
From: Dave Airlie
This just passes the subresource info in and uses it.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_copy.c | 32 +---
1 file changed, 13 insertions(+), 19 deletions(-)
diff --git a/src/amd/vulkan/radv_meta_copy.c b/src/amd/vulkan/radv_me
From: Dave Airlie
Just noticed this file was missing license and any
explaination of what is in it.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_meta_bufimage.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/src/amd/vulkan/radv_meta_bufimage.c
b/src/amd
Give explicit names to the various memory heaps, reducing the risk of
bugs when adding or modifying them.
While at it, use a static_assert to make sure we don't have too many
memory heaps.
Signed-off-by: Eric Engestrom
---
I'm sure there are better names than these, but this patch is here to
su
Following the previous patch, give explicit names to the various memory types,
reducing the risk of bugs when adding or modifying them.
While at it, use a static_assert to make sure we don't have too many
memory types.
Signed-off-by: Eric Engestrom
---
This one isn't really useful yet, but I th
I haven't seen the driver author's opinion on this yet, so it's probably
fair to give him some more time to answer. It's not like this is really
urgent...
Roland
Am 08.12.2016 um 01:11 schrieb Edward O'Callaghan:
> Hi all,
>
> So I'll get right to the crux of this; In summary the consensus would
This series is,
Reviewed-by: Edward O'Callaghan
On 12/08/2016 11:18 AM, Dave Airlie wrote:
> While working on spilling, geom shaders and compute queues, I've built
> up a bunch of cleanups, so best to dequeue them early.
>
> Dave.
>
> ___
> mesa-dev m
No functional change, just rewriting it in an easier-to-understand way.
Signed-off-by: Eric Engestrom
---
src/egl/drivers/dri2/platform_x11.c | 24 ++--
1 file changed, 10 insertions(+), 14 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_x11.c
b/src/egl/drivers/dri2
On 12/08/2016 11:28 AM, Roland Scheidegger wrote:
> I haven't seen the driver author's opinion on this yet, so it's probably
> fair to give him some more time to answer. It's not like this is really
> urgent...
Absolutely!
>
> Roland
>
> Am 08.12.2016 um 01:11 schrieb Edward O'Callaghan:
>> H
Signed-off-by: Eric Engestrom
---
src/egl/main/eglapi.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index 9950a72..925e881 100644
--- a/src/egl/main/eglapi.c
+++ b/src/egl/main/eglapi.c
@@ -890,7 +890,7 @@ eg
Nobody else makes use of this function.
We can always re-export it if someone ever needs it.
Signed-off-by: Eric Engestrom
---
src/egl/main/eglapi.c | 2 +-
src/egl/main/eglapi.h | 3 ---
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
From: Dave Airlie
We don't use the nir shader in here at all.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 15 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index bb52e8f..f5f7
+author again.. no idea why list keeps loosing extra cc's.. it is
kinda annoying..
BR,
-R
On Wed, Dec 7, 2016 at 7:28 PM, Roland Scheidegger wrote:
> I haven't seen the driver author's opinion on this yet, so it's probably
> fair to give him some more time to answer. It's not like this is really
Reviewed-by: Edward O'Callaghan
On 12/08/2016 11:55 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> We don't use the nir shader in here at all.
>
> Signed-off-by: Dave Airlie
> ---
> src/amd/common/ac_nir_to_llvm.c | 15 ++-
> 1 file changed, 6 insertions(+), 9 deletions(-)
>
> d
v2: use fmul(1/65536) instead of fdiv(65535)
---
.../drivers/swr/rasterizer/common/formats.cpp | 104 ++---
.../drivers/swr/rasterizer/common/formats.h| 7 +-
.../drivers/swr/rasterizer/core/format_traits.h| 90 +-
.../drivers/swr/rasterizer/jitt
On 07/12/16 07:16 PM, Nicolai Hähnle wrote:
> On 07.12.2016 08:50, Michel Dänzer wrote:
>> On 06/12/16 10:24 PM, Marek Olšák wrote:
>>> On Mon, Dec 5, 2016 at 10:05 AM, Michel Dänzer
>>> wrote:
On 03/12/16 05:38 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> This fixes random rad
Hi all,
Sorry for the slow response. I think it is fine to drop the driver :(
Not because the driver is currently unmaintained, which is very true
and is a very good reason, but that there is now a Intel Vulkan
driver. Vulkan is somewhat as low-level as Gallium is (or even
lower-level). The dr
Patches 1 to 4 (hopefully) fix
https://bugs.freedesktop.org/show_bug.cgi?id=99010.
Patch 5 and 6 are cosmetic changes.
I build and run tested this with radeonsi, radv and llvmpipe and build tested
with softpipe.
@Emil:
Please test and push if everything is ok.
Tobias Droste (6):
configure.a
this renames MESA_LLVM to FOUND_LLVM and updates the config.log report
to say if LLVM is found or not, to make clear that this does not mean
that it is used.
Signed-off-by: Tobias Droste
---
configure.ac | 21 ++---
1 file changed, 10 insertions(+), 11 deletions(-)
diff --git a/
Gallium code used HAVE_LLVM to check if it needs to compile code for
LLVM in header and source files.
With the new logic HAVE_LLVM is always set. Use extra define to figure
out if LLVM is used.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99010
Signed-off-by: Tobias Droste
---
config
This is actually not needed because the version is checked later.
Line 2609:
if test "x$enable_gallium_llvm" == "xyes"; then
llvm_require_version $LLVM_REQUIRED_GALLIUM "gallium"
llvm_add_default_components "gallium"
HAVE_GALLIUM_LLVM=xyes
DEFINES="${DEFINES} -DHAVE_GALLIUM_LLVM"
Make setting LLVM_LIBS a function, put it to the other LLVM related
functions and call it if LLVM is used.
Signed-off-by: Tobias Droste
---
configure.ac | 109 +--
1 file changed, 54 insertions(+), 55 deletions(-)
diff --git a/configure.ac
This renames llvm_check_version_for to llvm_require_version and let it
set a variable to mark that LLVM will be used.
Use this to make a usefull configure output and to only check if the
libs are found in LLVM if it is actually used.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99010
S
Make sure that HAVE_LLVM compiler define is only set if LLVM is
actually used.
Signed-off-by: Tobias Droste
---
configure.ac | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index a209d87e62..7362ecdfaf 100644
--- a/configure.ac
+++ b/configur
https://bugs.freedesktop.org/show_bug.cgi?id=99010
--- Comment #14 from Tobias Droste ---
Oops.
Fixes sent to mailing list:
https://lists.freedesktop.org/archives/mesa-dev/2016-December/137798.html
You now get a output like this:
llvm found: yes
llvm-config: /llvm-conf
On 08/12/16 09:59 AM, Rob Clark wrote:
> +author again.. no idea why list keeps loosing extra cc's..
Mailman removes addresses from Cc which are subscribed to the list and
have "Avoid duplicate copies of messages?" enabled in the list
Subscription Options.
--
Earthling Michel Dänzer
This handles two cases. The first is where the second ifs
condition is a phi with constant srcs origination from the first if.
The second is where both ifs have the same condtion.
A limitation to merging is that currently this code will only merge
the ifs if there is only a single phi instruction
Reviewed-by: Bruce Cherniak
> On Dec 7, 2016, at 7:16 PM, Tim Rowley wrote:
>
> v2: use fmul(1/65536) instead of fdiv(65535)
> ---
> .../drivers/swr/rasterizer/common/formats.cpp | 104 ++---
> .../drivers/swr/rasterizer/common/formats.h| 7 +-
> .../drivers/swr/ra
Reviewed-by: Bruce Cherniak
> On Dec 7, 2016, at 5:55 PM, Rowley, Timothy O
> wrote:
>
> This one-off specialization was missed.
> ---
> src/gallium/drivers/swr/rasterizer/core/backend.h | 4 +-
> .../drivers/swr/rasterizer/memory/StoreTile.h | 46 +++---
> 2 files change
On 12/07/2016 06:13 PM, Michel Dänzer wrote:
On 08/12/16 09:59 AM, Rob Clark wrote:
+author again.. no idea why list keeps loosing extra cc's..
Mailman removes addresses from Cc which are subscribed to the list and
have "Avoid duplicate copies of messages?" enabled in the list
Subscription O
On 08/12/16 11:53 AM, Pierre-Loup A. Griffais wrote:
> On 12/07/2016 06:13 PM, Michel Dänzer wrote:
>> On 08/12/16 09:59 AM, Rob Clark wrote:
>>> +author again.. no idea why list keeps loosing extra cc's..
>>
>> Mailman removes addresses from Cc which are subscribed to the list and
>> have "Avoid d
On 08/12/16 12:02 AM, Roland Scheidegger wrote:
> The bug in llvm has been fixed, can you confirm lp_test_format passes again?
Yep, it does, thanks!
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X develop
On 11/30/2016 10:44 PM, Christian Gmeiner wrote:
> This a very lightweight library to add basic support for
> renderonly GPUs. It does all the magic regarding in/exporting
> buffers etc. This library will likely break android support and
> hopefully will get replaced with a better solution based on
201 - 242 of 242 matches
Mail list logo