This adds support for the KHR_display extension to the radv Vulkan
driver. The driver now attempts to open the master DRM node when the
KHR_display extension is requested so that the common winsys code can
perform the necessary operations.
v2:
* Simplify addition of
This adds support for the KHR_display extension to the anv Vulkan
driver. The driver now attempts to open the master DRM node when the
KHR_display extension is requested so that the common winsys code can
perform the necessary operations.
v2: Make sure primary fd is usable
When
This extension adds the ability to borrow an X RandR output for
temporary use directly by a Vulkan application. For DRM, we use the
Linux resource leasing mechanism.
v2:
Clean up xlib_lease detection
* Use separate temporary '_xlib_lease' variable to hold the
option
Add support for the EXT_direct_mode_display extension. This just
provides the vkReleaseDisplayEXT function.
Signed-off-by: Keith Packard
---
src/amd/vulkan/radv_extensions.py | 1 +
src/amd/vulkan/radv_wsi_display.c | 11 +++
2 files changed, 12 insertions(+)
diff --git
Add support for the EXT_direct_mode_display extension. This just
provides the vkReleaseDisplayEXT function.
v2:
Adopt Jason Ekstrand's coding conventions
Declare variables at first use, eliminate extra whitespace
between types and names. Wrap lines to 80 columns.
This extension adds the ability to borrow an X RandR output for
temporary use directly by a Vulkan application to the anv driver.
v2:
Simplify addition of VK_USE_PLATFORM_XLIB_XRANDR_KHR to
vulkan_wsi_args
Suggested-by: Eric Engestrom
v3:
Add extension to list
https://bugs.freedesktop.org/show_bug.cgi?id=106881
--- Comment #3 from Tapani Pälli ---
IMO this looks like a shader cache issue that got fixed by following commit.
Please pull the latest Mesa and test again.
--- 8< ---
commit e266b320590ebbeadf7c98b0b493d89886534ccb
Author: Jordan Justen
This adds support for the KHR_display extension support to the vulkan
WSI layer. Driver support will be added separately.
v2:
* fix double ;; in wsi_common_display.c
* Move mode list from wsi_display to wsi_display_connector
* Fix scope for wsi_display_mode
Here's an updated patch series for DRM lease support using KHR_display
and other standard Vulkan extensions. This series incorporates review
from Eric Engestrom and Jason Ekstrom. Thanks to both of them!
-keith
___
mesa-dev mailing list
This extension adds the ability to borrow an X RandR output for
temporary use directly by a Vulkan application to the radv driver.
v2:
Simplify addition of VK_USE_PLATFORM_XLIB_XRANDR_KHR to
vulkan_wsi_args
Suggested-by: Eric Engestrom
Signed-off-by: Keith Packard
---
On Mon, Jun 11, 2018 at 9:21 AM, Dylan Baker wrote:
> Quoting Eric Engestrom (2018-06-11 02:10:37)
> > On Friday, 2018-06-08 10:14:58 -0700, Dylan Baker wrote:
> > > Quoting Jason Ekstrand (2018-06-08 10:04:50)
> > > > On Fri, Jun 8, 2018 at 9:27 AM, Eric Engestrom <
> eric.engest...@intel.com>
On GFX8+, there is a bug that affects TC-compatible depth surfaces
when the ZRange is not reset after LateZ kills pixels.
The workaround is to always set DB_Z_INFO.ZRANGE_PRECISION to match
the last fast clear value. Because the value is set to 1 by default,
we only need to update it when
v2: - Use -mpower8-vector in compiler test for altivec
- rename altivec option to power8
- reword power8 option description to be more clear, originally I
had made it a boolean, but replaced it with an auto option.
Signed-off-by: Dylan Baker
---
meson.build | 31
meson 0.43 gained support for optional modules, which clover wold like
to use. Since we require 0.44.1 now we can rely on them being available
for clover.
compile tested only.
Signed-off-by: Dylan Baker
---
meson.build | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
Some of these are done already, some of them are things we're not going
to do, like have a toggle for GLX_TLS.
v2: - Don't remove USE_MGL_NAMESPACE or with_glx_direct
Signed-off-by: Dylan Baker
---
meson.build | 8
src/egl/meson.build | 2 --
src/gbm/meson.build | 2 --
3
On Monday, 2018-06-11 09:42:08 -0700, Jason Ekstrand wrote:
> On Mon, Jun 11, 2018 at 9:21 AM, Dylan Baker wrote:
>
> > Quoting Eric Engestrom (2018-06-11 02:10:37)
> > > On Friday, 2018-06-08 10:14:58 -0700, Dylan Baker wrote:
> > > > Quoting Jason Ekstrand (2018-06-08 10:04:50)
> > > > > On
Series looks good to me. Compile tested on ppc64 and ppc64le and
confirmed that the POWER8 intrinsics are enabled only on ppc64le.
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
v2: - split this from the next patch
- Only include x86-64 and not x86 when buiding x86_64
Signed-off-by: Dylan Baker
---
src/mesa/meson.build | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mesa/meson.build b/src/mesa/meson.build
index
Not tested at all (not even compiled), just a blind port from autotools.
Signed-off-by: Dylan Baker
---
meson.build| 5 +
src/mapi/glapi/meson.build | 3 ++-
src/mesa/meson.build | 8 +++-
3 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/meson.build
This just makes using cc and cpp easier.
v2: - Add this patch to fix altivec
---
meson.build | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/meson.build b/meson.build
index 160bbfa30ba..16866d77ed3 100644
--- a/meson.build
+++ b/meson.build
@@ -29,6 +29,9 @@ project(
https://bugs.freedesktop.org/show_bug.cgi?id=106810
Jordan Justen changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=106877
--- Comment #3 from Bas Nieuwenhuizen ---
Any indication to devs trying to get to the same spot for testing, what level
this is?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the
We can have arrays of images or samplers. But I forgot to handle that
case long ago. Suprised no one complained yet.
Signed-off-by: Rob Clark
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=106893
Bug ID: 106893
Summary: Potential mem leak with radv
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Priority:
Eric Engestrom writes:
> I just pushed it; your `if (result != VK_SUCCESS)` should now just be
> `goto fail;` :)
Yup, I've update my drm-lease branch to match.
--
-keith
signature.asc
Description: PGP signature
___
mesa-dev mailing list
Hi Bas,
commit [1] removed a check on 'supported' attribute in
src/intel/vulkan/anv_entrypoints_gen.py
Should the check on 'supported' attribute be removed also in
src/amd/vulkan/radv_entrypoints_gen.py ?
Thanks for your feedback
Mauro
[1]
The UBO push analysis pass incorrectly assumed that all values would fit
within a 32B chunk, and only recorded a bit for the 32B chunk containing
the starting offset.
For example, if a UBO contained the following, tightly packed:
vec4 a; // [0, 16)
float b; // [16, 20)
vec4 c; // [20,
https://bugs.freedesktop.org/show_bug.cgi?id=106836
Yogish Kulkarni changed:
What|Removed |Added
CC||kyle.brenne...@gmail.com
--- Comment
Patch sets additional formats renderable and enables the extension
when OpenGL ES 3.1 is supported.
Signed-off-by: Tapani Pälli
---
src/mesa/main/extensions_table.h | 1 +
src/mesa/main/fbobject.c | 20 +++-
src/mesa/main/glformats.c| 9 +
3 files
On Friday, 2018-06-08 10:14:58 -0700, Dylan Baker wrote:
> Quoting Jason Ekstrand (2018-06-08 10:04:50)
> > On Fri, Jun 8, 2018 at 9:27 AM, Eric Engestrom
> > wrote:
> >
> > On Friday, 2018-06-08 09:06:06 -0700, Jason Ekstrand wrote:
> > > On Fri, Jun 8, 2018 at 8:57 AM, Eric Engestrom
https://bugs.freedesktop.org/show_bug.cgi?id=106877
Samuel Pitoiset changed:
What|Removed |Added
Component|Drivers/Gallium/radeonsi|Drivers/Vulkan/radeon
QA
https://bugs.freedesktop.org/show_bug.cgi?id=106595
--- Comment #11 from Samuel Pitoiset ---
You have to grab and build renderdoc from https://github.com/baldurk/renderdoc
Then, you can do "export ENABLE_VULKAN_RENDERDOC_CAPTURE=1", start the game and
uses F12 for capturing a trace.
--
You
https://bugs.freedesktop.org/show_bug.cgi?id=106877
--- Comment #2 from Samuel Pitoiset ---
Can you try to reproduce the issue with latest mesa trunk?
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the
This warning is misleading: When a struct is partially initialized without
assigning to the structure members by name, then the remaining fields
will be zeroed out, and this warning will be issued (if enabled). If, on the
other hand, the partial initialization is done by assigning to named
Handle the failure to read from /proc/self/cmdline by printing an error
message and fix the -Wsign-compare warning:
In file included from ./util/u_memory.h:39:0,
from os/os_process.c:31:
os/os_process.c: In function 'os_get_command_line':
os/os_process.c:140:16: warning:
Am Freitag, den 08.06.2018, 16:28 +0100 schrieb Emil Velikov:
> On 8 June 2018 at 12:02, Gert Wollny wrote:
> > From: Gert Wollny
> >
> > This warning is misleading: When a struct is partially initialized
> > without
> > assigning to the struture members by name, then the un-initialized
> >
On Monday, 2018-06-11 11:33:20 +0200, Gert Wollny wrote:
> This warning is misleading: When a struct is partially initialized without
> assigning to the structure members by name, then the remaining fields
> will be zeroed out, and this warning will be issued (if enabled). If, on the
> other hand,
Previously, TargetNVC0::insnCanLoadOffset() returned whether the offset
could be set to a specific value. The IndirectPropagation pass expected
it to return whether the offset could be increased.
Signed-off-by: Rhys Perry
---
src/gallium/drivers/nouveau/codegen/nv50_ir_target_nvc0.cpp | 1 +
1
Ping to those who seem appropriate for this patch in case it was forgotten
or missed.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 2018-06-09 05:16 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> This improves performance for certain games.
>
> Cc: 18.1
> ---
> src/gallium/drivers/radeonsi/si_buffer.c | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_buffer.c
https://bugs.freedesktop.org/show_bug.cgi?id=106881
xinghua changed:
What|Removed |Added
Assignee|intel-3d-bugs@lists.freedes |mesa-dev@lists.freedesktop.
On Tue, 05 Jun 2018 19:22:47 +0200
"Juan A. Suarez Romero" wrote:
> On Tue, 2018-06-05 at 12:41 +0100, Daniel Stone wrote:
> > Hi Juan,
> >
> > On 5 June 2018 at 09:51, Juan A. Suarez Romero wrote:
> >
> > > On Mon, 2018-06-04 at 13:22 +0100, Daniel Stone wrote:
> > > > Yes, that's
https://bugs.freedesktop.org/show_bug.cgi?id=106881
--- Comment #1 from Lionel Landwerlin ---
This test passes on Firefox, could this be a chrome bug?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the
https://bugs.freedesktop.org/show_bug.cgi?id=106881
Lionel Landwerlin changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #2 from
https://bugs.freedesktop.org/show_bug.cgi?id=106836
Eric Engestrom changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
An alternative solution to the problem fixed in
0bd83d0 ("nv50/ir: move LateAlgebraicOpt to the very end"). Should be
useful in the future and seems to make dolphin ubershaders a bit smaller.
total instructions in shared programs : 226722 -> 226464 (-0.11%)
total gprs used in shared programs:
From: Eric Engestrom
v2: intel_miptree_release() already takes care of the planes, no need
to hand-code the loop (Lionel)
Coverity ID: 1436909
Cc: Lionel Landwerlin
Fixes: 3352f2d746d3959b22ca4 "i965: Create multiple miptrees for planar YUV
images"
Signed-off-by: Eric Engestrom
---
From: Eric Engestrom
v2: intel_miptree_release() already takes care of the planes, no need
to hand-code the loop (Lionel)
Coverity ID: 1436909
Cc: Lionel Landwerlin
Fixes: 3352f2d746d3959b22ca4 "i965: Create multiple miptrees for planar YUV
images"
Signed-off-by: Eric Engestrom
---
NAK'ing my own patch, this is causing the symbols to no longer be
exported; I'll look at this, but for now let's just ignore this patch :)
On Sunday, 2018-06-10 09:35:52 +0100, Eric Engestrom wrote:
> Signed-off-by: Eric Engestrom
> ---
> include/KHR/khrplatform.h | 20
>
On 11/06/18 14:22, Eric Engestrom wrote:
From: Eric Engestrom
v2: intel_miptree_release() already takes care of the planes, no need
to hand-code the loop (Lionel)
Coverity ID: 1436909
Cc: Lionel Landwerlin
Fixes: 3352f2d746d3959b22ca4 "i965: Create multiple miptrees for planar YUV
On Mon, Jun 11, 2018 at 6:54 AM, Michel Dänzer wrote:
> On 2018-06-09 05:16 AM, Marek Olšák wrote:
> > From: Marek Olšák
> >
> > This improves performance for certain games.
> >
> > Cc: 18.1
> > ---
> > src/gallium/drivers/radeonsi/si_buffer.c | 6 +-
> > 1 file changed, 5 insertions(+),
The series is OK with me, even though radeonsi can't support the new
opcodes.
Marek
On Mon, Jun 11, 2018 at 6:24 AM, Rhys Perry
wrote:
> Ping to those who seem appropriate for this patch in case it was forgotten
> or missed.
>
___
mesa-dev mailing
Run this pass late (after opt loop) to move load_const instructions back
into the basic blocks which use the result, in cases where a load_const
is only consumed in a single block.
This helps reduce register usage in cases where the backend driver
cannot lower the load_const to a uniform.
v2:
https://bugs.freedesktop.org/show_bug.cgi?id=106843
Alex Granni changed:
What|Removed |Added
CC||jfons...@vmware.com
--
You are
Am Montag, den 11.06.2018, 11:05 +0100 schrieb Eric Engestrom:
> On Monday, 2018-06-11 11:33:20 +0200, Gert Wollny wrote:
> > This warning is misleading: When a struct is partially initialized
> > without
> > assigning to the structure members by name, then the remaining
> > fields
> > will be
Quoting Gert Wollny (2018-06-11 09:05:52)
> Am Montag, den 11.06.2018, 11:05 +0100 schrieb Eric Engestrom:
> > On Monday, 2018-06-11 11:33:20 +0200, Gert Wollny wrote:
> > > This warning is misleading: When a struct is partially initialized
> > > without
> > > assigning to the structure members by
Quoting Eric Engestrom (2018-06-11 02:10:37)
> On Friday, 2018-06-08 10:14:58 -0700, Dylan Baker wrote:
> > Quoting Jason Ekstrand (2018-06-08 10:04:50)
> > > On Fri, Jun 8, 2018 at 9:27 AM, Eric Engestrom
> > > wrote:
> > >
> > > On Friday, 2018-06-08 09:06:06 -0700, Jason Ekstrand wrote:
>
This warning is misleading: When a struct is partially initialized without
assigning to the structure members by name, then the remaining fields
will be zeroed out, and this warning will be issued (if enabled). If, on the
other hand, the partial initialization is done by assigning to named
Quoting Eric Anholt (2018-06-08 11:29:41)
> Dylan Baker writes:
>
> > [ Unknown signature status ]
> > Quoting Eric Anholt (2018-06-07 14:29:19)
> >> Dylan Baker writes:
> >>
> >> > Some of these are done already, some of them are things we're not going
> >> > to do, like have a toggle for
101 - 159 of 159 matches
Mail list logo
