Hi Marek,
On Tuesday, 10 April 2018 20:09:06 CEST Marek Olšák wrote:
> Generally, if you have to loop over all arrays to find common vertex
> buffers, it's better not to do it. The default separate path is going to
> perform best, because it's straightforward and interleaved arrays are super
> rar
Good Morning,
I was thinking to also basically 'move' vbo/vbo_exec_array.c to main/draw.c as
drawing arrays is today largely independent of the vbo module. Its more the
other way round that the vbo module utilizes the draw code path one notch
below the opengl entry points (... validate, set the
From: Roland Scheidegger
If we dump the bitcode for off-line debug purposes, we really want the
pre-optimized bitcode, otherwise it's useless in identifying problems
with IR optimization (if you have a shader which takes an hour to do
IR optimization, it's also nice you don't have to wait that ho
From: Roland Scheidegger
Conversion to int can otherwise overflow if compile times are over
~71min. (Yes this can happen...)
---
src/gallium/auxiliary/gallivm/lp_bld_init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_init.c
b/src/gall
From: Roland Scheidegger
LICM is simply too expensive, even though it presumably can help quite
a bit in some cases.
It was definitely cheaper in llvm 3.3, though as far as I can tell with
llvm 3.3 it failed to do anything in most cases. early-cse also actually
seems to cause licm to be able to m
From: Roland Scheidegger
This pass is quite cheap, and can simplify the IR quite a bit for our
generated IR.
In particular on a variety of shaders I've found the time saved by
other passes due to the simplified IR more than makes up for the cost
of this pass, and on top of that the end result is
https://bugs.freedesktop.org/show_bug.cgi?id=76694
Timothy Arceri changed:
What|Removed |Added
Resolution|--- |NOTABUG
Status|NEW
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=65422
---
src/mesa/Makefile.sources | 4 ++--
src/mesa/main/dlist.c | 2 +-
src/mesa/main/{api_validate.c => draw_validate.c} | 2 +-
src/mesa/main/{api_validate.h => draw_validate.h} | 0
sr
https://bugs.freedesktop.org/show_bug.cgi?id=48837
Timothy Arceri changed:
What|Removed |Added
Component|Mesa core |Drivers/DRI/swrast
--
You are receivin
What's the impact to drm_gralloc?
2018-04-20 5:09 GMT+08:00 Rob Herring :
> Maintaining both flink names and prime fd support which are provided by
> 2 different gralloc implementations is problematic because we have a
> dependency on a specific gralloc implementation header.
>
> This mostly remo
Actually as a followup you might want to deduplicate the
wavefrontsPerSimd in radv_device.c too.
On Mon, Apr 23, 2018 at 2:17 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> ---
> src/amd/common/ac_gpu_info.h | 16
> src/amd/vulkan/radv_shader.c | 12 +---
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Mon, Apr 23, 2018 at 1:59 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> ---
> src/gallium/drivers/radeonsi/si_pipe.c | 33 ++---
> 1 file changed, 2 insertions(+), 31 deletions(-)
>
> diff --git a/src/gallium/drive
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Mon, Apr 23, 2018 at 2:17 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> ---
> src/amd/common/ac_gpu_info.h | 16
> src/amd/vulkan/radv_shader.c | 12 +---
> src/gallium/drivers/radeonsi/si_shad
On 20/04/18 06:08, Vlad Golovkin wrote:
GLSL 4.6 spec describes hex constant as:
hexadecimal-constant:
0x hexadecimal-digit
0X hexadecimal-digit
hexadecimal-constant hexadecimal-digit
Right now if you have a shader with the following structure:
#if 0X1 // or any hex numbe
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Mon, Apr 23, 2018 at 2:43 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This refactors the code out to share it between radv and radeonsi.
> ---
> src/amd/common/ac_gpu_info.c| 113
>
> src/amd/co
On Mon, Apr 23, 2018 at 2:20 AM, Timothy Arceri wrote:
>
>
> On 23/04/18 03:26, Bas Nieuwenhuizen wrote:
>>
>> We seems to use progress for two cases:
>
>
> seems -> seem
>
>> 1) When we lowered some returns.
>> 2) When we remove unreachable code.
>>
>> If just case 2 happens we assert as state->r
From: Dave Airlie
This refactors the code out to share it between radv and radeonsi.
---
src/amd/common/ac_gpu_info.c| 113
src/amd/common/ac_gpu_info.h| 4 ++
src/amd/vulkan/si_cmd_buffer.c | 108 ++
From: Dave Airlie
This follows what radeonsi does.
Ported from radeonsi:
radeonsi: emit PA_SC_RASTER_CONFIG_1 only once
---
src/amd/vulkan/si_cmd_buffer.c | 31 ---
1 file changed, 16 insertions(+), 15 deletions(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/s
https://bugs.freedesktop.org/show_bug.cgi?id=99179
Timothy Arceri changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=99125
Timothy Arceri changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=92877
Timothy Arceri changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
On 23/04/18 03:26, Bas Nieuwenhuizen wrote:
We seems to use progress for two cases:
seems -> seem
1) When we lowered some returns.
2) When we remove unreachable code.
If just case 2 happens we assert as state->return_flag has not
been allocated yet, but we are still trying to do insert all
From: Dave Airlie
---
src/amd/common/ac_gpu_info.h | 16
src/amd/vulkan/radv_shader.c | 12 +---
src/gallium/drivers/radeonsi/si_shader.c | 12 +---
3 files changed, 18 insertions(+), 22 deletions(-)
diff --git a/src/amd/common/ac_gpu_inf
From: Dave Airlie
This just makes this common code between the two drivers.
---
src/amd/common/ac_gpu_info.c| 93 +
src/amd/common/ac_gpu_info.h| 4 +-
src/amd/vulkan/si_cmd_buffer.c | 85 ++
src/galliu
From: Dave Airlie
---
src/amd/common/ac_gpu_info.c | 33 +
src/amd/common/ac_gpu_info.h | 1 +
2 files changed, 34 insertions(+)
diff --git a/src/amd/common/ac_gpu_info.c b/src/amd/common/ac_gpu_info.c
index ef0364b0534..12b03c4ea29 100644
--- a/src/amd/common/a
From: Dave Airlie
---
src/gallium/drivers/radeonsi/si_pipe.c | 33 ++---
1 file changed, 2 insertions(+), 31 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index b3ef098b8ea..327dd7c4242 100644
--- a/src/gal
From: Dave Airlie
---
src/amd/vulkan/radv_device.c | 32 ++--
1 file changed, 2 insertions(+), 30 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index c687b23f27d..8837d6cafde 100644
--- a/src/amd/vulkan/radv_device.c
+++ b/src/
From: Dave Airlie
Missed this on initial radeonsi port, we shouldn't use this value
on gfx9, but also in gfx8 only for when we have a geom shader.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_device.c | 6 +++---
src/amd/vulkan/radv_pipeline.c | 5 +++--
2 files changed, 6 insertions(+
From: Dave Airlie
We can just unreachable here, this aligns with radv code, makes
it easier to move to common code.
---
src/gallium/drivers/radeonsi/si_pipe.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/
https://bugs.freedesktop.org/show_bug.cgi?id=106176
--- Comment #3 from LoneVVolf ---
Thank you for the fast response.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
mesa-dev mailing li
https://bugs.freedesktop.org/show_bug.cgi?id=106180
--- Comment #1 from mercuriete ---
not sure if i forget to put this:
x11-base/xorg-server-1.19.5-r1
x11-base/xorg-drivers-1.19
I think I am using xorg modesetting.
--
You are receiving this mail because:
You are the assignee for the bug.
You
https://bugs.freedesktop.org/show_bug.cgi?id=106180
Bug ID: 106180
Summary: [bisected] radv vulkan smoke test black screen (Add
support for DRI3 v1.2)
Product: Mesa
Version: git
Hardware: Other
OS: All
On Fri, 2018-04-20 at 17:16 -0700, Jason Ekstrand wrote:
> On Fri, Apr 20, 2018 at 5:16 AM, Nicolai Hähnle
> wrote:
> > On 20.04.2018 10:21, Iago Toral wrote:
> >
> > > Hi,
> > >
> > >
> > >
> > > while developing support for Vulkan shaderInt16 on Anvil I came
> > > across
> > >
> > > a featu
https://bugs.freedesktop.org/show_bug.cgi?id=106174
--- Comment #2 from Sylvain BERTRAND ---
On Sun, Apr 22, 2018 at 05:28:45PM +, bugzilla-dae...@freedesktop.org
wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=106174
>
> --- Comment #1 from Bas Nieuwenhuizen ---
> Yeah, I noticed thi
https://bugs.freedesktop.org/show_bug.cgi?id=106174
--- Comment #1 from Bas Nieuwenhuizen ---
Yeah, I noticed this too.
https://patchwork.freedesktop.org/patch/218205/
should fix this.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug
We seems to use progress for two cases:
1) When we lowered some returns.
2) When we remove unreachable code.
If just case 2 happens we assert as state->return_flag has not
been allocated yet, but we are still trying to do insert all
predicates based on it.
This splits the concerns. We only use pr
https://bugs.freedesktop.org/show_bug.cgi?id=106176
Dylan Baker changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=106176
--- Comment #1 from LoneVVolf ---
Created attachment 138989
--> https://bugs.freedesktop.org/attachment.cgi?id=138989&action=edit
increase number in VERSION-file
--
You are receiving this mail because:
You are the assignee for the bug.
You a
On Sun, Apr 22, 2018 at 1:11 PM, Mauro Rossi wrote:
> Fix for build errors in src/gallium/auxiliary/util/u_debug_stack_android.cpp
> error: cast to 'void *' from smaller integer type pid_t
>
> Fixes: ff904978a1 ("gallium/util: Android backtrace support")
>
> Cc: "18.1"
> ---
> src/gallium/auxili
https://bugs.freedesktop.org/show_bug.cgi?id=106176
Bug ID: 106176
Summary: [mesa] git master VERSION not increased when 18.1 was
branched off
Product: Mesa
Version: git
Hardware: All
OS: All
Sta
Fix for build errors in src/gallium/auxiliary/util/u_debug_stack_android.cpp
error: cast to 'void *' from smaller integer type pid_t
Fixes: ff904978a1 ("gallium/util: Android backtrace support")
Cc: "18.1"
---
src/gallium/auxiliary/util/u_debug_stack_android.cpp | 4 ++--
1 file changed, 2 inse
https://bugs.freedesktop.org/show_bug.cgi?id=106174
Bug ID: 106174
Summary: vulkan dota2 broken (segfaulting), found bug commit
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
42 matches
Mail list logo
