Hi,
On Mon, 2017-11-13 at 14:20 -0800, Jason Ekstrand wrote:
> On Mon, Nov 6, 2017 at 2:02 PM, Louis-Francis Ratté-Boulianne ollabora.com> wrote:
> > From: Daniel Stone
> >
> > Not currently used.
> >
> > Signed-off-by: Daniel Stone
> > ---
>
Allow the winsys to provide a set of acceptable modifiers to the driver
when creating WSI images.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 3 ++
src/intel/vulkan/anv_image.c | 9 +++-
src/intel/vulkan/anv_private.h | 3 ++
src/intel/vulkan/anv_wsi.c | 101
When it is detected that a window could have been flipped
but has been copied because of suboptimal format/modifier.
The Vulkan client should then re-create the swapchain.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common_x11.c | 53
From: Daniel Stone
Adds support for multiple planes and buffer modifiers.
v4: Rename "has_dri3_v1_1" to "has_dri3_modifiers"
---
src/vulkan/wsi/wsi_common_x11.c | 238 +---
1 file changed, 199 insertions(+), 39 deletions(-)
diff --git a/src/vulkan/wsi/wsi_co
From: Daniel Stone
That's what it actually means; the fact it generally means a linear copy
is requires is incidental.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 6 +++---
src/intel/vulkan/anv_wsi.c | 2 +-
src/vulkan/wsi/wsi_common.h | 2 +-
3 files changed, 5 insertions(+
From: Daniel Stone
zwp_linux_dmabuf_v1 lets us use multi-planar images and buffer
modifiers.
Signed-off-by: Daniel Stone
---
src/vulkan/Makefile.am | 10 +++
src/vulkan/Makefile.sources | 4 +-
src/vulkan/wsi/wsi_common_wayland.c | 157 +++---
From: Daniel Stone
Given a tiling mode and an aux usage, return the DRM modifier.
Signed-off-by: Daniel Stone
---
src/intel/isl/isl.h | 6 +-
src/intel/isl/isl_drm.c | 17 +
2 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/src/intel/isl/isl.h b/src/intel/
From: Daniel Stone
The only use for this boolean was to decide whether or not it should
export a dmabuf FD. Simplify things a bit by giving that directly.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 6 --
src/intel/vulkan/anv_wsi.c | 21 +
The caller might want to discriminate between two possible
sets of tiling flags. For example, some tiling modes might
allow direct scanout, and so should be preferred even if
GPU operations are a little less performant.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/intel/isl/isl.c | 33
Add support for DRI3 v1.1, which allows pixmaps to be backed by
multi-planar buffers, or those with format modifiers. This is both
for allocating render buffers, as well as EGLImage imports from a
native pixmap (EGL_NATIVE_PIXMAP_KHR).
Signed-off-by: Louis-Francis Ratté-Boulianne
Reviewed-by
From: Daniel Stone
Not yet used anywhere.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/Makefile.am | 1 +
src/amd/vulkan/radv_wsi.c | 2 ++
src/intel/vulkan/anv_wsi.c | 9 +
src/vulkan/wsi/wsi_common.h | 2 ++
4 files changed, 14 insertions(+)
diff --git a/src/amd/vulkan/Makefil
From: Daniel Stone
Not currently used.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 13 +++--
src/intel/vulkan/anv_wsi.c | 9 +
src/vulkan/wsi/wsi_common.h | 9 +
src/vulkan/wsi/wsi_common_wayland.c | 11 +++
src/vulka
If PresentCompleteNotify event says the pixmap was presented
with mode PresentCompleteModeSuboptimalCopy, it means the pixmap
could possibly have been flipped instead if allocated with a
different format/modifier.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/egl/drivers/dri2/egl_dri2.c
It does the same as createImagewithModifiers but allow multiple
modifiers set to be given. The modifier used to create the image
should be selected from the first tranche if possible. If not,
then the subsequent tranches should be used.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
include
From: Daniel Stone
Provide a hook to inform the driver that implicit synchronization should
be suppressed.
---
include/GL/internal/dri_interface.h | 10 +-
src/egl/drivers/dri2/egl_dri2.c | 27 +++
2 files changed, 36 insertions(+), 1 deletion(-)
diff --git a
From: Daniel Stone
This is used to hold information about the allocated image, rather than
an ever-growing function argument list.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 31 ++
src/intel/vulkan/anv_wsi.c | 25 +++---
src/v
From: Daniel Stone
---
src/mesa/drivers/dri/i965/brw_bufmgr.h | 3 +++
src/mesa/drivers/dri/i965/intel_screen.c | 17 +++--
2 files changed, 18 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.h
b/src/mesa/drivers/dri/i965/brw_bufmgr.h
index ee9132
It was assumed that fromPlanar() could return NULL to mean
that the planar image is the same as the parent DRI image.
That assumption wasn't made everywhere though.
Let's fix things and make sure that all callers understand
a NULL result
Signed-off-by: Louis-Francis Ratté-Boulianne
--
From: Daniel Stone
Signed-off-by: Daniel Stone
---
src/egl/main/eglapi.c | 1 +
src/egl/main/egldisplay.h | 1 +
src/egl/main/eglimage.c | 26 ++
src/egl/main/eglimage.h | 3 +++
4 files changed, 31 insertions(+)
diff --git a/src/egl/main/eglapi.c b/src/egl/m
The planar_format image property was always set even for
non-planar formats. This was breaking CCS support as
intel_from_planar is now making sure we can't have both
a modifier and an planar format.
Signed-off-by: Louis-Francis Ratté-Boulianne
Reviewed-by: Daniel Stone
---
src/mesa/driver
Hi,
With full support for modifiers in DRIimage, this patch series adds
support for fully plumbing them through X11. This is the fifth
revision, more context can be found here:
https://lists.freedesktop.org/archives/mesa-dev/2017-June/158457.html
https://lists.freedesktop.org/archives/mesa-dev/20
From: Daniel Stone
Signed-off-by: Daniel Stone
---
src/intel/vulkan/anv_wsi.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/src/intel/vulkan/anv_wsi.c b/src/intel/vulkan/anv_wsi.c
index 97f8e10afc..607d2aa49d 100644
--- a/src/intel/vulkan/anv_wsi.c
+++ b/src/i
From: Daniel Stone
Adds support for multiple planes and buffer modifiers.
v4: Rename "has_dri3_v1_1" to "has_dri3_modifiers"
---
src/vulkan/wsi/wsi_common_x11.c | 180
1 file changed, 162 insertions(+), 18 deletions(-)
diff --git a/src/vulkan/wsi/wsi_co
From: Daniel Stone
Not yet used anywhere.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/Makefile.am | 1 +
src/amd/vulkan/radv_wsi.c | 2 ++
src/intel/vulkan/anv_wsi.c | 8
src/vulkan/wsi/wsi_common.h | 1 +
4 files changed, 12 insertions(+)
diff --git a/src/amd/vulkan/Makefile.
When it is detected that a window could have been flipped
but has been copied because of suboptimal format/modifier.
The Vulkan client should then re-create the swapchain.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common_x11.c | 30 +-
1
The caller might want to discriminate between two possible
sets of tiling flags. For example, some tiling modes might
allow direct scanout, and so should be preferred even if
GPU operations are a little less performant.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/intel/isl/isl.c | 33
From: Daniel Stone
zwp_linux_dmabuf_v1 lets us use multi-planar images and buffer
modifiers.
Signed-off-by: Daniel Stone
---
src/vulkan/Makefile.am | 10 +++
src/vulkan/Makefile.sources | 4 +-
src/vulkan/wsi/wsi_common_wayland.c | 139 +++---
From: Daniel Stone
Allow the winsys to provide a set of acceptable modifiers to the driver
when creating WSI images.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 36 +--
src/intel/vulkan/anv_image.c| 9 ++--
src/intel/vulkan/anv
From: Daniel Stone
Provide a hook to inform the driver that implicit synchronization should
be suppressed.
---
include/GL/internal/dri_interface.h | 10 +-
src/egl/drivers/dri2/egl_dri2.c | 27 +++
2 files changed, 36 insertions(+), 1 deletion(-)
diff --git a
From: Daniel Stone
If we fail in the alloc, just use the (mostly) identical
radv_wsi_image_free() to clean up after us.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 39 ---
1 file changed, 16 insertions(+), 23 deletions(-)
diff --git a/src/am
If PresentCompleteNotify event says the pixmap was presented
with mode PresentCompleteModeSuboptimalCopy, it means the pixmap
could possibly have been flipped instead if allocated with a
different format/modifier.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/loader/loader_dri3_helper.c
From: Daniel Stone
The only use for this boolean was to decide whether or not it should
export a dmabuf FD. Simplify things a bit by giving that directly.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 6 --
src/intel/vulkan/anv_wsi.c | 21 +
From: Daniel Stone
Create a new helper function which handles the actual image/memory
allocations for a swapchain image, removing the bulk of
radv_wsi_image_create. This will make it easier to move linear handling
into radv in the next patch.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv
From: Daniel Stone
This is pretty much radv-specific anyway.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 49 +++--
src/intel/vulkan/anv_wsi.c | 23 -
src/vulkan/wsi/wsi_common.h | 4 ++-
src/vulkan/wsi/
From: Daniel Stone
Not currently used.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 13 +++--
src/intel/vulkan/anv_wsi.c | 9 +
src/vulkan/wsi/wsi_common.h | 9 +
src/vulkan/wsi/wsi_common_wayland.c | 11 +++
src/vulka
From: Daniel Stone
That's what it actually means; the fact it generally means a linear copy
is requires is incidental.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 10 +-
src/intel/vulkan/anv_wsi.c | 2 +-
src/vulkan/wsi/wsi_common.h | 2 +-
3 files changed, 7 inser
From: Daniel Stone
Given a tiling mode and an aux usage, return the DRM modifier.
Signed-off-by: Daniel Stone
---
src/intel/isl/isl.h | 6 +-
src/intel/isl/isl_drm.c | 17 +
2 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/src/intel/isl/isl.h b/src/intel/
Add support for DRI3 v1.1, which allows pixmaps to be backed by
multi-planar buffers, or those with format modifiers. This is both
for allocating render buffers, as well as EGLImage imports from a
native pixmap (EGL_NATIVE_PIXMAP_KHR).
Signed-off-by: Louis-Francis Ratté-Boulianne
Reviewed-by
From: Daniel Stone
This is used to hold information about the allocated image, rather than
an ever-growing function argument list.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 31 ++
src/intel/vulkan/anv_wsi.c | 25 +++---
src/v
-by: Louis-Francis Ratté-Boulianne
---
src/gbm/backends/dri/gbm_dri.c | 12 ++--
src/mesa/drivers/dri/i965/intel_screen.c | 11 ---
2 files changed, 14 insertions(+), 9 deletions(-)
diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_dri.c
index
It does the same as createImagewithModifiers but allow multiple
modifiers set to be given. The modifier used to create the image
should be selected from the first tranche if possible. If not,
then the subsequent tranches should be used.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
include
Hi,
With full support for modifiers in DRIimage, this patch series adds
support for fully plumbing them through X11. This is the fourth
revision, more context can be found here:
https://lists.freedesktop.org/archives/mesa-dev/2017-June/158457.html
https://lists.freedesktop.org/archives/mesa-dev/20
From: Daniel Stone
---
src/mesa/drivers/dri/i965/brw_bufmgr.h | 3 +++
src/mesa/drivers/dri/i965/intel_screen.c | 17 +++--
2 files changed, 18 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.h
b/src/mesa/drivers/dri/i965/brw_bufmgr.h
index de0ba1
The planar_format image property was always set even for
non-planar formats. This was breaking CCS support as
intel_from_planar is now making sure we can't have both
a modifier and an planar format.
Signed-off-by: Louis-Francis Ratté-Boulianne
Reviewed-by: Daniel Stone
---
src/mesa/driver
From: Daniel Stone
Signed-off-by: Daniel Stone
---
src/egl/main/eglapi.c | 1 +
src/egl/main/egldisplay.h | 1 +
src/egl/main/eglimage.c | 26 ++
src/egl/main/eglimage.h | 3 +++
4 files changed, 31 insertions(+)
diff --git a/src/egl/main/eglapi.c b/src/egl/m
Hi,
On Thu, 2017-09-28 at 17:00 +0100, Emil Velikov wrote:
> Hi Louis-Francis,
>
> On 28 September 2017 at 09:25, Louis-Francis Ratté-Boulianne
> wrote:
> > This patchset makes use of Jason Ekstrand's recent work on SYNC_FD
> > semaphores and DRI3 v1.2 supp
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common.h | 3 +
src/vulkan/wsi/wsi_common_x11.c | 152 +++-
2 files changed, 121 insertions(+), 34 deletions(-)
diff --git a/src/vulkan/wsi/wsi_common.h b/src/vulkan/wsi/wsi_common.h
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common_x11.c | 46 +
1 file changed, 46 insertions(+)
diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_x11.c
index 4569c928c8..d9467092d2 100644
--- a/src/vulkan
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/egl/main/eglapi.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index 41a2a1a9d8..ca2e90cdf0 100644
--- a/src/egl/main/eglapi.c
+++ b/src/egl/main/eglapi.c
@@ -1769,9
Allow drivers to add a list of semaphores to wait on
when queuing a image to present.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/amd/vulkan/radv_wsi.c | 2 +-
src/intel/vulkan/anv_wsi.c | 2 ++
src/vulkan/wsi/wsi_common.h | 6 +
src/vulkan/wsi
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/intel/vulkan/anv_wsi.c | 121 -
1 file changed, 119 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_wsi.c b/src/intel/vulkan/anv_wsi.c
index 2e05c3b07b..1e8e507d99 100644
--- a/src
This patchset makes use of Jason Ekstrand's recent work on SYNC_FD
semaphores and DRI3 v1.2 support for DMA fences:
https://lists.x.org/archives/xorg-devel/2017-September/054770.html
This is the second version of this patchset. For more context, see:
https://lists.freedesktop.org/archives/mesa-dev
From: Daniel Stone
If we fail in the alloc, just use the (mostly) identical
radv_wsi_image_free() to clean up after us.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 39 ---
1 file changed, 16 insertions(+), 23 deletions(-)
diff --git a/src/am
From: Daniel Stone
Signed-off-by: Daniel Stone
---
src/intel/vulkan/anv_wsi.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/src/intel/vulkan/anv_wsi.c b/src/intel/vulkan/anv_wsi.c
index f59d349b50..d8ffdfbab2 100644
--- a/src/intel/vulkan/anv_wsi.c
+++ b/src/i
optimal modifiers instead of just assuming they have
changed.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common_x11.c | 57 -
1 file changed, 50 insertions(+), 7 deletions(-)
diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan
From: Daniel Stone
Not currently used.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 13 +++--
src/intel/vulkan/anv_wsi.c | 9 +
src/vulkan/wsi/wsi_common.h | 9 +
src/vulkan/wsi/wsi_common_wayland.c | 11 +++
src/vulka
From: Daniel Stone
This is used to hold information about the allocated image, rather than
an ever-growing function argument list.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 31 ++
src/intel/vulkan/anv_wsi.c | 25 +++---
src/v
From: Daniel Stone
That's what it actually means; the fact it generally means a linear copy
is requires is incidental.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 10 +-
src/intel/vulkan/anv_wsi.c | 2 +-
src/vulkan/wsi/wsi_common.h | 2 +-
3 files changed, 7 inser
From: Daniel Stone
Create a new helper function which handles the actual image/memory
allocations for a swapchain image, removing the bulk of
radv_wsi_image_create. This will make it easier to move linear handling
into radv in the next patch.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv
From: Daniel Stone
zwp_linux_dmabuf_v1 lets us use multi-planar images and buffer
modifiers.
Signed-off-by: Daniel Stone
---
src/vulkan/Makefile.am | 10 +++
src/vulkan/Makefile.sources | 4 +-
src/vulkan/wsi/wsi_common_wayland.c | 139 +++---
The optimal modifier depends on the current CRTC as some modifiers
might not allow direct scanout. When the window is moved to a
different CRTC, available modifiers should be re-fetched and the
buffers re-allocated.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/egl/drivers/dri2/egl_dri2
From: Daniel Stone
The only use for this boolean was to decide whether or not it should
export a dmabuf FD. Simplify things a bit by giving that directly.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 6 --
src/intel/vulkan/anv_wsi.c | 21 +
From: Daniel Stone
Adds support for multiple planes and buffer modifiers.
---
src/vulkan/wsi/wsi_common_x11.c | 179
1 file changed, 161 insertions(+), 18 deletions(-)
diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_x11.c
index 8
From: Daniel Stone
Given a tiling mode and an aux usage, return the DRM modifier.
Signed-off-by: Daniel Stone
---
src/intel/isl/isl.h | 6 +-
src/intel/isl/isl_drm.c | 17 +
2 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/src/intel/isl/isl.h b/src/intel/
From: Daniel Stone
Allow the winsys to provide a set of acceptable modifiers to the driver
when creating WSI images.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 36 +--
src/intel/vulkan/anv_image.c| 9 ++--
src/intel/vulkan/anv
From: Daniel Stone
Not yet used anywhere.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/Makefile.am | 1 +
src/amd/vulkan/radv_wsi.c | 2 ++
src/intel/vulkan/anv_wsi.c | 8
src/vulkan/wsi/wsi_common.h | 1 +
4 files changed, 12 insertions(+)
diff --git a/src/amd/vulkan/Makefile.
From: Daniel Stone
---
src/mesa/drivers/dri/i965/brw_bufmgr.h | 3 +++
src/mesa/drivers/dri/i965/intel_screen.c | 16 ++--
2 files changed, 17 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.h
b/src/mesa/drivers/dri/i965/brw_bufmgr.h
index de0ba1d
It does the same as createImagewithModifiers but allow multiple
modifiers set to be given. The modifier used to create the image
should be selected from the first tranche if possible. If not,
then the subsequent tranches should be used.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
include
Add support for DRI3 v1.1, which allows pixmaps to be backed by
multi-planar buffers, or those with format modifiers. This is both
for allocating render buffers, as well as EGLImage imports from a
native pixmap (EGL_NATIVE_PIXMAP_KHR).
Signed-off-by: Louis-Francis Ratté-Boulianne
Reviewed-by
From: Daniel Stone
Provide a hook to inform the driver that implicit synchronization should
be suppressed.
---
include/GL/internal/dri_interface.h | 10 +-
src/egl/drivers/dri2/egl_dri2.c | 27 +++
2 files changed, 36 insertions(+), 1 deletion(-)
diff --git a
-by: Louis-Francis Ratté-Boulianne
---
src/gbm/backends/dri/gbm_dri.c | 12 ++--
src/mesa/drivers/dri/i965/intel_screen.c | 11 ---
2 files changed, 14 insertions(+), 9 deletions(-)
diff --git a/src/gbm/backends/dri/gbm_dri.c b/src/gbm/backends/dri/gbm_dri.c
index
From: Daniel Stone
Signed-off-by: Daniel Stone
---
src/egl/main/eglapi.c | 1 +
src/egl/main/egldisplay.h | 1 +
src/egl/main/eglimage.c | 26 ++
src/egl/main/eglimage.h | 3 +++
4 files changed, 31 insertions(+)
diff --git a/src/egl/main/eglapi.c b/src/egl/m
The caller might want to discriminate between two possible
sets of tiling flags. For example, some tiling modes might
allow direct scanout, and so should be preferred even if
GPU operations are a little less performant.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/intel/isl/isl.c | 33
From: Daniel Stone
This is pretty much radv-specific anyway.
Signed-off-by: Daniel Stone
---
src/amd/vulkan/radv_wsi.c | 49 +++--
src/intel/vulkan/anv_wsi.c | 23 -
src/vulkan/wsi/wsi_common.h | 4 ++-
src/vulkan/wsi/
The planar_format image property was always set even for
non-planar formats. This was breaking CCS support as
intel_from_planar is now making sure we can't have both
a modifier and an planar format.
Signed-off-by: Louis-Francis Ratté-Boulianne
Reviewed-by: Daniel Stone
---
src/mesa/driver
Hi,
With full support for modifiers in DRIimage, this patch series adds
support for fully plumbing them through X11. This is the third
revision, more context can be found here:
https://lists.freedesktop.org/archives/mesa-dev/2017-June/158457.html
https://lists.freedesktop.org/archives/mesa-dev/201
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common.h | 3 +
src/vulkan/wsi/wsi_common_x11.c | 143 ++--
2 files changed, 113 insertions(+), 33 deletions(-)
diff --git a/src/vulkan/wsi/wsi_common.h b/src/vulkan/wsi/wsi_common.h
Xorg server uses the OpenGL API by default and we still want
to support fencing in that situation. Fences seem to work
properly even is not using the OpenGL ES API.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/egl/main/eglapi.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions
Allow drivers to add a list of semaphores to wait on
when queuing a image to present.
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/amd/vulkan/radv_wsi.c | 2 +-
src/intel/vulkan/anv_wsi.c | 2 ++
src/vulkan/wsi/wsi_common.h | 6 ++
src/vulkan/wsi
Hi,
This patchset makes use of Jason Ekstrand's recent work on SYNC_FD
semaphores and DRI3 v1.1 support for DMA fences:
https://lists.freedesktop.org/archives/mesa-dev/2017-August/168154.html
It allows Vulkan WSI clients (X11 only for now) to use explicit fencing.
That means vkQueuePresent can ha
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/intel/vulkan/anv_wsi.c | 121 -
1 file changed, 119 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_wsi.c b/src/intel/vulkan/anv_wsi.c
index 2c6c4db4eb..38eec76a97 100644
--- a/src
Signed-off-by: Louis-Francis Ratté-Boulianne
---
src/vulkan/wsi/wsi_common_x11.c | 31 +++
1 file changed, 31 insertions(+)
diff --git a/src/vulkan/wsi/wsi_common_x11.c b/src/vulkan/wsi/wsi_common_x11.c
index ecdaf91434..d132541156 100644
--- a/src/vulkan/wsi
From: Chad Versace
Update vulkan.h and vk.xml from [1]:
git://github.com/chadversary/vulkan-spec
refs/tags/chadv/test/2017-03-06-VK_MESAX_external
This pulls in the following extensions:
- VK_EXT_get_image_properties
- VK_MESAX_external_memory_dma_buf
- VK_MESAX_external_image_dma_buf
From: Chad Versace
When creating a VkSurface for X11, we created the underlying VkImage by
calling anv_image_create() and passing driver-private info to it. We can
now accomplish the same thing without driver-private info, using
VK_MESAX_external_image_dma_buf.
In anv_wsi.c, replace the combo o
From: Chad Versace
When creating a VkSurface for X11, we created the underlying VkImage by
calling anv_image_create() and passing driver-private info to it. We can
now accomplish the same thing without driver-private info, using
VK_MESAX_external_image_dma_buf.
In anv_wsi.c, replace the combo o
From: Daniel Stone
Allow the WSI to provide a set of modifiers to be used along with the
format.
For now, no winsys provides any modifier support. Add a fallback to the
previous default (X-tiling) inside ANV.
RADV remains somewhat broken in the presence of a winsys which will
suggest modifiers,
From: Chad Versace
This "extension" function predates Vulkan 1.0 and Vulkan WSI. We created
it during the early days of bringing up the Vulkan driver. We used it
for testing the early driver because, in those early days, there was no
other way to import/export a tiled VkImage and examine it. No
V
From: Chad Versace
---
src/intel/vulkan/anv_image.c | 26 ++
src/intel/vulkan/anv_private.h | 5 -
2 files changed, 6 insertions(+), 25 deletions(-)
diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c
index 241b7c9fe0..a99f0e4b29 100644
--- a/
From: Chad Versace
---
src/intel/vulkan/anv_image.c | 26 ++
src/intel/vulkan/anv_private.h | 5 -
2 files changed, 6 insertions(+), 25 deletions(-)
diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c
index 241b7c9fe0..a99f0e4b29 100644
--- a/
From: Chad Versace
- Rename the anv_image_create_info vars from 'create_info' to 'anv_info'.
- Rename the VkImageCreateInfo vars from 'pCreateInfo' and 'vk_info' to
'base_info'.
This reduces the diff and eliminates ambiguity in follow-up patches for
upcoming extensions, because the extensions
From: Daniel Stone
Adds support for multiple planes and buffer modifiers.
---
src/intel/vulkan/anv_wsi.c | 7 ++-
src/vulkan/wsi/wsi_common_x11.c | 118
2 files changed, 112 insertions(+), 13 deletions(-)
diff --git a/src/intel/vulkan/anv_wsi.c b/
From: Daniel Stone
Add a simple conversion from ISL tiling to GEM BO tiling flags, and use
that instead of hardcoding I915_TILING_X in the ANV WSI.
Signed-off-by: Daniel Stone
---
src/intel/isl/isl.c| 28
src/intel/isl/isl.h| 8
src/intel/
From: Chad Versace
For now, we support dma_buf images for only a single format,
VK_FORMAT_R8G8B8A8_UNORM. And the image must be a "simple" image: 2D,
single-sample, non-mipmappped, non-array, non-cube.
---
src/intel/vulkan/anv_extensions.py | 1 +
src/intel/vulkan/anv_formats.c | 144
From: Chad Versace
This "extension" function predates Vulkan 1.0 and Vulkan WSI. We created
it during the early days of bringing up the Vulkan driver. We used it
for testing the early driver because, in those early days, there was no
other way to import/export a tiled VkImage and examine it. No
V
From: Chad Versace
Like anv_get_format(), but the returned format is not adjusted, not even
for the aspect.
Add anv_get_raw_isl_format() too, to match anv_get_isl_format().
---
src/intel/vulkan/anv_formats.c | 7 +++
src/intel/vulkan/anv_private.h | 8
2 files changed, 15 insertion
From: Daniel Stone
Adds support for multiple planes and buffer modifiers.
---
src/intel/vulkan/anv_wsi.c | 7 ++-
src/vulkan/wsi/wsi_common_x11.c | 118
2 files changed, 112 insertions(+), 13 deletions(-)
diff --git a/src/intel/vulkan/anv_wsi.c b/
From: Chad Versace
Now that anv_image_create() is gone, there is no need for struct
anv_image_create_info. The struct just adds an unused layer of
indirection.
---
src/intel/vulkan/anv_image.c | 38 +++---
src/intel/vulkan/anv_private.h | 9 -
2 files c
From: Daniel Stone
Allow the WSI to provide a set of modifiers to be used along with the
format.
For now, no winsys provides any modifier support. Add a fallback to the
previous default (X-tiling) inside ANV.
RADV remains somewhat broken in the presence of a winsys which will
suggest modifiers,
From: Chad Versace
---
src/intel/vulkan/anv_private.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index 7b4e9dfbe0..55840d5325 100644
--- a/src/intel/vulkan/anv_private.h
+++ b/src/intel/vulkan/anv_privat
From: Chad Versace
make_surface() always returns VK_SUCCESS, so change its return type to
'void'. This simplifies the return-code handling in
anv_image_create_info() and removes a goto.
---
src/intel/vulkan/anv_image.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff
1 - 100 of 155 matches
Mail list logo
