https://bugs.freedesktop.org/show_bug.cgi?id=96872
Tapani Pälli changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Reviewed-by: Bas Nieuwenhuizen
On Mon, Jan 30, 2017, at 07:14, Dave Airlie wrote:
> From: Dave Airlie
>
> We weren't adding the fragment position properly.
>
> Signed-off-by: Dave Airlie
> ---
> src/amd/common/ac_nir_to_llvm.c | 11 ++-
> 1 file changed, 6 insertions(+), 5 deletions(
Reviewed-by: Bas Nieuwenhuizen
On Mon, Jan 30, 2017, at 04:28, Dave Airlie wrote:
> From: Dave Airlie
>
> It appears we can get image barriers like:
> srcStageMask: VkPipelineStageFlags = 4096
> (VK_PIPELINE_STAGE_TRANSFER_BIT)
> dstStageMask: VkP
On 30 January 2017 at 16:02, Dave Airlie wrote:
> From: Dave Airlie
>
> This adds gs copy shader support to the pipeline cache, and few
> geometry related changes.
There is a minor rebase bug in this, I've fixed it in my
radv-geom-shader-submit branch,.
>
> v2: rebase for spill changes.
>
> Si
From: Dave Airlie
We weren't adding the fragment position properly.
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 f83d0e1..8f
From: Dave Airlie
This enables geometry shader support on radv.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_device.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index fd4e7f5..06fefb3 100644
--- a/sr
From: Dave Airlie
This just adds some defines needed for geom shaders.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index fadbb55..7c70709 100644
---
From: Dave Airlie
This uses the scratch infrastructure to handle the esgs
and gsvs rings.
(this replaces the old code that did this with patching).
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 28 ++
src/amd/vulkan/radv_device.c | 199
From: Dave Airlie
This just adds the API and fixes up the callers.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 22 ++
1 file changed, 14 insertions(+), 8 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
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 10 +-
src/amd/common/ac_nir_to_llvm.h | 2 ++
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 4139b0a..f83d0e1 1006
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index b5c3e90..b66dff9 100644
--- a/src/amd/vulkan/radv_cmd_
From: Dave Airlie
This handles geometry shader inputs written by the vertex (es) shader
to the esgs ring.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 56 +
1 file changed, 56 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_pipeline.c | 32
src/amd/vulkan/radv_private.h | 2 ++
2 files changed, 34 insertions(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index da8f561..200ba8d 1
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index b66dff9..6b34d29 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
+++ b/src/amd/vulkan/r
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 592ee99..664c5c4 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/am
From: Dave Airlie
This handles emitting things to the gsvs ring, and sending the
correct GS msgs.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 126
1 file changed, 126 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src
From: Dave Airlie
This gets hit with some geom shaders.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 6f7d8a0..4139b0a 100644
--- a/s
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 38 +-
src/amd/common/ac_nir_to_llvm.h | 1 +
2 files changed, 38 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
From: Dave Airlie
This adds gs copy shader support to the pipeline cache, and few
geometry related changes.
v2: rebase for spill changes.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_pipeline.c | 128 +--
src/amd/vulkan/radv_pipeline_cache.c | 4 +
From: Dave Airlie
This create the gs copy shader and compiles it.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 81 +
src/amd/common/ac_nir_to_llvm.h | 7
2 files changed, 88 insertions(+)
diff --git a/src/amd/common/ac_nir_to_l
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 88959b8..85881b7 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/commo
From: Dave Airlie
We need this info to program some registers.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 5 +
src/amd/common/ac_nir_to_llvm.h | 2 ++
2 files changed, 7 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
inde
From: Dave Airlie
This sets up the rings and adds the variables
needed to make them work.
v2: rework for sharing ring and scratch
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 63 +++--
src/amd/common/ac_nir_to_llvm.h | 3 +-
2 files chan
From: Dave Airlie
This emits the shader to the ES block in the correct case.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 21 -
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buff
From: Dave Airlie
This just sets up the gpr inputs.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 39 +++
src/amd/common/ac_nir_to_llvm.h | 2 ++
2 files changed, 41 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/co
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 49 +++--
1 file changed, 32 insertions(+), 17 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 3e922b0..217f1a0 100644
--- a/
From: Dave Airlie
This emits the compiled geometry shader and other state registers.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 98 +++-
1 file changed, 96 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/
From: Dave Airlie
This enables the paths for setting up user ptrs to vs/es and gs.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 30 ++
1 file changed, 22 insertions(+), 8 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulk
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_device.c | 32
src/amd/vulkan/radv_private.h | 3 +++
2 files changed, 35 insertions(+)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 6071421..af16c89 100644
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_private.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/amd/vulkan/radv_private.h b/src/amd/vulkan/radv_private.h
index fac5b97..68c161e 100644
--- a/src/amd/vulkan/radv_private.h
+++ b/src/amd/vulkan/radv_priva
From: Dave Airlie
This is to later allow ES shaders to be emitted.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_cmd_buffer.c | 69 +++-
1 file changed, 40 insertions(+), 29 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/ra
From: Dave Airlie
This just places the flag into the shader info so we can use it from
the driver after we create the shader.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 3 +++
src/amd/common/ac_nir_to_llvm.h | 1 +
2 files changed, 4 insertions(+)
diff --git a/src/amd/co
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 6 ++
src/amd/common/ac_nir_to_llvm.h | 6 ++
2 files changed, 12 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index dedea65..fadbb55 100644
--- a/src/amd/
From: Dave Airlie
This fixes up afew of the commented out blocks.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/si_cmd_buffer.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
index e2025b
This is rebased onto the scratch buffer in CS preamble work
that Bas has done, makes this all a lot cleaner for emitting
the rings.
Otherwise there shouldn't be any changes.
Dave.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.f
On Sunday, January 29, 2017 6:20:10 PM PST Matt Turner wrote:
> This partially reverts commit 97217a40f97cdeae0304798b607f704deb0c3558.
> It leaves ES 2.0 support in place per Ian's suggestion, because ES 2.0
> is designed to work on hardware like i915.
Your commit message should mention why dropp
...and provide a better citation for the existing one.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_pipe_control.c | 32 +---
1 file changed, 24 insertions(+), 8 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_pipe_control.c
b/src/mesa/driver
From: Dave Airlie
It appears we can get image barriers like:
srcStageMask: VkPipelineStageFlags = 4096
(VK_PIPELINE_STAGE_TRANSFER_BIT)
dstStageMask: VkPipelineStageFlags = 4096
(VK_PIPELINE_STAGE_TRANSFER_BIT)
dependencyFlags:VkDe
Patches 4&5 LGTM,
Reviewed-by: Edward O'Callaghan
On 01/30/2017 12:19 PM, Bas Nieuwenhuizen wrote:
> Patch 1-3, 6-7 are
>
> Reviewed-by: Bas Nieuwenhuizen
>
> The other two you'll need to find someone else to review as I'm not
> going to review my own patches.
>
> - Bas
>
> On Fri, Jan 27, 2
https://bugs.freedesktop.org/show_bug.cgi?id=99517
Kenneth Graunke changed:
What|Removed |Added
Depends on||99339
Referenced Bugs:
https://bugs.
This partially reverts commit 97217a40f97cdeae0304798b607f704deb0c3558.
It leaves ES 2.0 support in place per Ian's suggestion, because ES 2.0
is designed to work on hardware like i915.
The piglit results look like:
name: before-revert-i915 after-revert-i915
--
On 27.01.2017 12:26, Andy Furniss wrote:
Peter Frühberger wrote:
just tell us when we can remove:
https://github.com/FernetMenta/kodi-agile/blob/master/xbmc/cores/VideoPlayer/DVDCodecs/Video/VAAPI.cpp#L534
Maybe I mis-remember, but doesn't AMD also need some EGL work for
VAAPI display wi
Patch 1-3, 6-7 are
Reviewed-by: Bas Nieuwenhuizen
The other two you'll need to find someone else to review as I'm not
going to review my own patches.
- Bas
On Fri, Jan 27, 2017, at 06:03, Andres Rodriguez wrote:
> ---
> src/amd/vulkan/radv_device.c | 4
> 1 file changed, 4 insertions(+)
From: Marek Olšák
the value from the kernel is wrong
---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
index 278d4f3..a8d
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_state_draw.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index 01340af..6b6613c 100644
--- a/src/gallium/drivers/radeon
From: Marek Olšák
This fixes a bug uncovered by the 17-part patch series, specifically:
"gallium/radeon: merge dirty_fb_counter and dirty_tex_descriptor_counter"
If dirty_tex_counter has been updated and set_shader_image invokes DCC
decompression, the DCC decompression itself checks the counte
On 30 January 2017 at 03:26, Bas Nieuwenhuizen wrote:
> v2: Create the descriptor BO with CPU access.
> Signed-off-by: Bas Nieuwenhuizen
Tom has pushed the spill to llvm 4.0 as well now, so with the checks
changed to 4.0
for the series:
Reviewed-by: Dave Airlie
> ---
> src/amd/vulkan/radv_de
On Sun, Jan 29, 2017 at 11:14 PM, Tobias Droste wrote:
> Am Sonntag, 29. Januar 2017, 22:31:55 CET schrieb Marek Olšák:
>> On Sat, Jan 28, 2017 at 9:29 PM, Tobias Droste wrote:
>> > Am Samstag, 28. Januar 2017, 16:09:29 CET schrieb Marek Olšák:
>> >> On Sat, Jan 28, 2017 at 3:31 PM, Ilia Mirkin
On 29/01/17 14:22, Andy Furniss wrote:
> Tested with ffmpeg and gst-vaapi. Without this bits per
> frame is set way too low for fractional framerates.
>
> v2: Mark Thompson: simplify calculation.
> Use float.
>
> Signed-off-by: Andy Furniss
> ---
> src/gallium/state_trackers/va/picture.c |
Am Sonntag, 29. Januar 2017, 22:31:55 CET schrieb Marek Olšák:
> On Sat, Jan 28, 2017 at 9:29 PM, Tobias Droste wrote:
> > Am Samstag, 28. Januar 2017, 16:09:29 CET schrieb Marek Olšák:
> >> On Sat, Jan 28, 2017 at 3:31 PM, Ilia Mirkin
wrote:
> >> > Can you explain why it's a desirable goal to b
On Sat, Jan 28, 2017 at 9:29 PM, Tobias Droste wrote:
> Am Samstag, 28. Januar 2017, 16:09:29 CET schrieb Marek Olšák:
>> On Sat, Jan 28, 2017 at 3:31 PM, Ilia Mirkin wrote:
>> > Can you explain why it's a desirable goal to be able to build radv without
>> > --enable-gallium-llvm? Perhaps it's ob
---
src/gallium/drivers/r600/sb/sb_valtable.cpp | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/sb/sb_valtable.cpp
b/src/gallium/drivers/r600/sb/sb_valtable.cpp
index a8b7b49cd4..d31a1b76d5 100644
--- a/src/gallium/drivers/r600/sb/sb_valtable.cp
v2: Create the descriptor BO with CPU access.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_device.c | 186 -
src/amd/vulkan/radv_pipeline.c | 11 +--
src/amd/vulkan/radv_private.h | 8 ++
3 files changed, 199 insertions(+), 6 deletions(-)
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_device.c | 187 -
src/amd/vulkan/radv_pipeline.c | 11 +--
src/amd/vulkan/radv_private.h | 8 ++
3 files changed, 200 insertions(+), 6 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/s
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_device.c | 6 ++-
src/amd/vulkan/radv_radeon_winsys.h | 1 +
src/amd/vulkan/radv_wsi.c | 2 +-
src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 62 +--
4 files changed
Based on code written by Dave Airlie.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_cmd_buffer.c | 22 +++-
src/amd/vulkan/radv_device.c | 23
src/amd/vulkan/radv_pipeline.c | 75
src/amd/vulkan/radv_private.h| 8
Based on code written by Dave Airlie.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/common/ac_binary.c | 30 +++---
src/amd/common/ac_binary.h | 4 +++-
src/amd/common/ac_llvm_util.c | 4 ++--
src/amd/common/ac_llvm_util.h | 2 +-
src/amd/common/ac_nir_to_
Build mesa 3295 completed
Commit b8ee45ebdc by Bas Nieuwenhuizen on 1/29/2017 4:03 PM:
llvmpipe: Use LLVMDumpModule, not DumpModule.\n\nForgot the prefix ...\n\nFixes: 0fca80b3db64dc1d004f78e22b9de86a07e9de96\nSigned-off-by: Bas Nieuwenhuizen
Configu
For the series:
Tested-by: Edmondo Tommasina
The Talos Principle benchmark seems to be about 0.5% faster.
Thanks
edmondo
On Fri, Jan 27, 2017 at 12:02 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> also remove the BIND flags
> ---
> src/gallium/drivers/r600/r600_state_common.c| 14 +
Build mesa 3294 failed
Commit 0fca80b3db by Bas Nieuwenhuizen on 1/28/2017 4:32 PM:
various: Fix missing DumpModule with recent LLVM.\n\nSince LLVM revision 293359 DumpModule gets only implemented when\neither a debug build or LLVM_ENABLE_DUMP is set.\n\nThis p
Christian König wrote:
That looks correct to me, but I'm not deeply enough into the H264
encoding any more.
Adding Leo and Boyuan to comment as well.
I just sent v2 to list and tried to cc Boyuan and Leo but it failed.
git send-email -1 -v2 --to=mesa-dev@lists.freedesktop.org
--cc=christian.
Mark Thompson wrote:
On 27/01/17 22:06, Andy Furniss wrote:
Could use floats I guess like I did here.
https://cgit.freedesktop.org/mesa/mesa/commit/?id=a5993022275c20061ac025d9adc26c5f9d02afee
I don't know what the preference is between float and int.
I think float is fine here? I don't k
https://bugs.freedesktop.org/show_bug.cgi?id=99591
Jure Repinc changed:
What|Removed |Added
Summary|Segmentation fault when |Segmentation fault when
Tested with ffmpeg and gst-vaapi. Without this bits per
frame is set way too low for fractional framerates.
v2: Mark Thompson: simplify calculation.
Use float.
Signed-off-by: Andy Furniss
---
src/gallium/state_trackers/va/picture.c | 19 +--
1 file changed, 13 insertions(+),
https://bugs.freedesktop.org/show_bug.cgi?id=99591
Bug ID: 99591
Summary: Segmentation fault when running vulkaninfo with rafv
Radeon Vulkan driver
Product: Mesa
Version: 13.0
Hardware: x86-64 (AMD64)
OS: Li
https://bugs.freedesktop.org/show_bug.cgi?id=99467
--- Comment #9 from Bogomil Vasilev ---
Has there been any communication made yet with the game devs or any approach to
workaround (hack) and bypass the specs to make this work?
--
You are receiving this mail because:
You are the assignee for t
66 matches
Mail list logo