[Mesa-dev] [PATCH v2] dri: add queryDmaBufFormats and queryDmaBufModifiers to DRIimage

2017-04-14 Thread Varad Gautam
From: Varad Gautam these allow querying the driver for supported dmabuf formats and modifiers. v2: move to __DRIimageExtension version 16. Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff

[Mesa-dev] [PATCH v2] egl: implement eglQueryDmaBufFormatsEXT

2017-04-14 Thread Varad Gautam
From: Varad Gautam allow egl clients to query the dmabuf formats supported on this platform. v2: rebase to master. Signed-off-by: Louis-Francis Ratté-Boulianne Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 17 + src/egl/main/eglapi.c | 18

[Mesa-dev] [PATCH v2] st/dri: support format modifier queries

2017-04-14 Thread Varad Gautam
From: Varad Gautam ask the driver for supported modifiers for a given format. bump __DRIimageExtension to 16. v2: move to __DRIimageExtension v16. Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 21 - 1 file changed, 20 insertions(+), 1 deletion

[Mesa-dev] [PATCH v2] st/dri: support format queries

2017-04-14 Thread Varad Gautam
From: Varad Gautam ask the driver for supported dmabuf formats v2: rebase to master. Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium/state_trackers

[Mesa-dev] [PATCH v2] st/dri: implement DRIimage creation from dmabufs with modifiers

2017-04-14 Thread Varad Gautam
From: Varad Gautam support importing dmabufs into DRIimage while taking format modifiers in account, and bump __DRIimageExtension on gallium to version 15. v2: move to __DRIimageExtension v15. Signed-off-by: Varad Gautam --- src/gallium/include/state_tracker/drm_driver.h | 2 ++ src/gallium

[Mesa-dev] [PATCH v2] gallium: introduce dmabuf format and modifier querying

2017-04-14 Thread Varad Gautam
From: Varad Gautam allows drivers to be queried for supported formats and format modifiers for EGL_EXT_image_dma_buf_import_modifiers. drivers that implement format/modifier queries must advertise these under PIPE_CAP_QUERY_DMABUF_ATTRIBS. v2: rebase to master. Signed-off-by: Varad Gautam

[Mesa-dev] [PATCH v2] dri: support DRIimage creation from dmabufs with modifiers

2017-04-14 Thread Varad Gautam
From: Pekka Paalanen add createImageFromDmaBufs2 function which accepts per-plane dmabuf format modifiers. v2: resolve DRIimage version conflict by moving createImageFromDmaBufs2 to v15. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h

[Mesa-dev] [PATCH v2] egl: advertise EGL_EXT_image_dma_buf_import_modifiers

2017-04-14 Thread Varad Gautam
From: Varad Gautam v2: check for both queryDmaBufFormats and queryDmaBufModifiers, rebase to master. Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 6 ++ src/egl/main/eglapi.c | 1 + 2 files changed, 7 insertions(+) diff --git a/src/egl/drivers/dri2

[Mesa-dev] [PATCH v2] egl: implement eglQueryDmaBufModifiersEXT

2017-04-14 Thread Varad Gautam
From: Varad Gautam query and return supported dmabuf format modifiers for EGL_EXT_image_dma_buf_import_modifiers. v2: rebase to master. Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 47 + src/egl/main/eglapi.c | 20

[Mesa-dev] [PATCH v3] st/dri: support format queries

2017-04-26 Thread Varad Gautam
From: Varad Gautam ask the driver for supported dmabuf formats v2: rebase to master. v3: return false on failure. Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/src/gallium/state_trackers/dri/dri2.c

[Mesa-dev] [PATCH v3] gallium: introduce dmabuf format and modifier querying

2017-04-26 Thread Varad Gautam
. Signed-off-by: Varad Gautam --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/drivers/freedreno/freedreno_screen.c | 1 + src/gallium/drivers/i915/i915_screen.c | 1 + src/gallium/drivers/llvmpipe/lp_screen.c | 1 + src/gallium/drivers/nouveau/nv30

[Mesa-dev] [PATCH v3] dri: add queryDmaBufFormats and queryDmaBufModifiers to DRIimage

2017-04-26 Thread Varad Gautam
From: Varad Gautam these allow querying the driver for supported dmabuf formats and modifiers. v2: move to __DRIimageExtension version 16. v3: return GLBoolean for error reporting, document params better. Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 42

[Mesa-dev] [PATCH v3] st/dri: support format modifier queries

2017-04-26 Thread Varad Gautam
From: Varad Gautam ask the driver for supported modifiers for a given format. bump __DRIimageExtension to 16. v2: move to __DRIimageExtension v16. v3: fail if the supplied format is not supported by driver. Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 24

[Mesa-dev] [PATCH v3] egl: implement eglQueryDmaBufFormatsEXT

2017-04-26 Thread Varad Gautam
From: Varad Gautam allow egl clients to query the dmabuf formats supported on this platform. v2: rebase to master. v3: return EGL_FALSE upon failure. Signed-off-by: Louis-Francis Ratté-Boulianne Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 18 ++ src/egl

[Mesa-dev] [PATCH v3] egl: implement eglQueryDmaBufModifiersEXT

2017-04-26 Thread Varad Gautam
query and return supported dmabuf format modifiers for EGL_EXT_image_dma_buf_import_modifiers. v2: rebase to master. v3: move format check to the driver instead of making format queries from here and then checking. Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 33

Re: [Mesa-dev] [PATCH v2 00/14] implement EGL_EXT_image_dma_buf_import_modifiers v2

2017-01-04 Thread Varad Gautam
Hi Ben, On Wed, Jan 4, 2017 at 3:25 AM, Ben Widawsky wrote: > On 16-11-24 20:50:37, Varad Gautam wrote: >> >> This is the second revision to the EGL_EXT_image_dma_buf_import_modifiers >> [1] >> series at [2], addressing the comments received. This diverges from v1

[Mesa-dev] [PATCH 01/13] egl: return error for unknown EGLImage attributes

2016-11-15 Thread Varad Gautam
one of the attributes listed in Table bbb, the error EGL_BAD_PARAMETER is generated. Implement this. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam CC: --- src/egl/main/eglimage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/egl/main/eglima

[Mesa-dev] [PATCH 03/13] egl: update eglext.h

2016-11-15 Thread Varad Gautam
From: Varad Gautam update eglext.h to revision 33288 from Khronos Signed-off-by: Varad Gautam --- include/EGL/eglext.h | 60 ++-- 1 file changed, 58 insertions(+), 2 deletions(-) diff --git a/include/EGL/eglext.h b/include/EGL/eglext.h index

[Mesa-dev] [PATCH 00/13] implement EGL_EXT_image_dma_buf_import_modifiers

2016-11-15 Thread Varad Gautam
error for unknown EGLImage attributes egl: introduce DMA_BUF_MAX_PLANES egl/main: add support for fourth plane tokens dri: support DRIimage creation from dmabufs with modifiers egl_dri2: add support for using modifier attributes in eglCreateImageKHR Varad Gautam (8): egl: update eglext.h

[Mesa-dev] [PATCH 04/13] egl/main: add support for fourth plane tokens

2016-11-15 Thread Varad Gautam
From: Pekka Paalanen The EGL_EXT_dma_buf_import_modifiers extension adds support for a fourth plane, just like DRM KMS API does. Bump maximum dma_buf plane count to four. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 2 +- src/egl/main

[Mesa-dev] [PATCH 09/13] dri: add queryDmaBufModifiers to DRIimage

2016-11-15 Thread Varad Gautam
From: Varad Gautam queryDmaBufModifiers function allows querying implementation supported dmabuf format modifiers for a given format. Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/include/GL

[Mesa-dev] [PATCH 02/13] egl: introduce DMA_BUF_MAX_PLANES

2016-11-15 Thread Varad Gautam
From: Pekka Paalanen Rather than hardcoding 3, use a #define. Makes it easier to bump this later to 4. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 8 src/egl/main/eglimage.h | 8 +--- 2 files changed, 9 insertions(+), 7

[Mesa-dev] [PATCH 07/13] egl_dri2: add support for using modifier attributes in eglCreateImageKHR

2016-11-15 Thread Varad Gautam
From: Pekka Paalanen allow creating EGLImages with dmabuf format modifiers when target is EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 68

[Mesa-dev] [PATCH 13/13] egl: advertise EGL_EXT_image_dma_buf_import_modifiers

2016-11-15 Thread Varad Gautam
From: Varad Gautam Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 5 + src/egl/main/eglapi.c | 1 + 2 files changed, 6 insertions(+) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index 443e0a3..a9a6f95 100644 --- a/src/egl

[Mesa-dev] [PATCH 08/13] egl: implement eglQueryDmaBufFormatsEXT

2016-11-15 Thread Varad Gautam
From: Varad Gautam allow egl clients to query the dmabuf formats supported on this platform. Signed-off-by: Louis-Francis Ratté-Boulianne Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 87 + src/egl/main/eglapi.c | 19

[Mesa-dev] [PATCH 05/13] dri: support DRIimage creation from dmabufs with modifiers

2016-11-15 Thread Varad Gautam
From: Pekka Paalanen add createImageFromDmaBufs2 function which accepts per-plane dmabuf format modifiers. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 12/13] egl: implement eglQueryDmaBufModifiersEXT

2016-11-15 Thread Varad Gautam
From: Varad Gautam query and return supported dmabuf format modifiers for EGL_EXT_image_dma_buf_import_modifiers. Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 39 +++ src/egl/main/eglapi.c | 21 + src/egl

[Mesa-dev] [PATCH 11/13] st/dri: support format modifier queries

2016-11-15 Thread Varad Gautam
From: Varad Gautam ask the driver for supported modifiers for a given format. Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium

[Mesa-dev] [PATCH 06/13] st/dri: implement DRIimage creation from dmabufs with modifiers

2016-11-15 Thread Varad Gautam
From: Varad Gautam support importing dmabufs into DRIimage taking format modifiers in account, as per DRIimage extension version 14. Signed-off-by: Varad Gautam --- src/gallium/include/state_tracker/drm_driver.h | 2 ++ src/gallium/state_trackers/dri/dri2.c | 45

[Mesa-dev] [PATCH 10/13] gallium: introduce format modifier querying

2016-11-15 Thread Varad Gautam
From: Varad Gautam drivers should implement pipe_screen->get_modifiers_for_format and advertise it with PIPE_CAP_QUERY_DMABUF_MODIFIERS to support format modifier queries. Signed-off-by: Varad Gautam --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium/drivers/freedr

[Mesa-dev] [PATCH v2] egl_dri2: add support for using modifier attributes in eglCreateImageKHR

2016-11-16 Thread Varad Gautam
From: Pekka Paalanen allow creating EGLImages with dmabuf format modifiers when target is EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers. v2: clear modifier assembling and error label name (Eric Engestrom) Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed

Re: [Mesa-dev] [PATCH 10/13] gallium: introduce format modifier querying

2016-11-16 Thread Varad Gautam
On Wed, Nov 16, 2016 at 12:25 PM, Michel Dänzer wrote: > On 15/11/16 11:24 PM, Varad Gautam wrote: >> From: Varad Gautam >> >> drivers should implement pipe_screen->get_modifiers_for_format and >> advertise it with PIPE_CAP_QUERY_DMABUF_MODIFIERS to support form

[Mesa-dev] [PATCH v2 00/14] implement EGL_EXT_image_dma_buf_import_modifiers v2

2016-11-24 Thread Varad Gautam
modifiers Varad Gautam (10): egl: update eglext.h st/dri: implement DRIimage creation from dmabufs with modifiers egl_dri2: add support for using modifier attributes in eglCreateImageKHR dri: add queryDmaBufFormats and queryDmaBufModifiers to DRIimage gallium: introduce dmabuf format

[Mesa-dev] [PATCH v2 01/14] egl: return error for unknown EGLImage attributes

2016-11-24 Thread Varad Gautam
one of the attributes listed in Table bbb, the error EGL_BAD_PARAMETER is generated. Implement this. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom CC: --- src/egl/main/eglimage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

[Mesa-dev] [PATCH v2 02/14] egl: introduce DMA_BUF_MAX_PLANES

2016-11-24 Thread Varad Gautam
From: Pekka Paalanen Rather than hardcoding 3, use a #define. Makes it easier to bump this later to 4. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom --- src/egl/drivers/dri2/egl_dri2.c | 8 src/egl/main/eglimage.h | 8 +--- 2

[Mesa-dev] [PATCH v2 04/14] egl/main: add support for fourth plane tokens

2016-11-24 Thread Varad Gautam
) Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom --- src/egl/drivers/dri2/egl_dri2.c | 2 +- src/egl/main/egldisplay.h | 1 + src/egl/main/eglimage.c | 18 ++ src/egl/main/eglimage.h | 5 +++-- 4 files changed, 23

[Mesa-dev] [PATCH v2 05/14] dri: support DRIimage creation from dmabufs with modifiers

2016-11-24 Thread Varad Gautam
From: Pekka Paalanen add createImageFromDmaBufs2 function which accepts per-plane dmabuf format modifiers. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH v2 03/14] egl: update eglext.h

2016-11-24 Thread Varad Gautam
From: Varad Gautam update eglext.h to revision 33288 from Khronos Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom --- include/EGL/eglext.h | 60 ++-- 1 file changed, 58 insertions(+), 2 deletions(-) diff --git a/include/EGL/eglext.h b

[Mesa-dev] [PATCH v2 08/14] dri: add queryDmaBufFormats and queryDmaBufModifiers to DRIimage

2016-11-24 Thread Varad Gautam
From: Varad Gautam these allow querying the driver for supported dmabuf formats and modifiers. Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/include/GL/internal/dri_interface.h b

[Mesa-dev] [PATCH v2 09/14] gallium: introduce dmabuf format and modifier querying

2016-11-24 Thread Varad Gautam
From: Varad Gautam this allows drivers to be queried for supported formats and format modifiers. drivers that implement these queries must expose these under PIPE_CAP_QUERY_DMABUF_ATTRIBS. Signed-off-by: Varad Gautam --- src/gallium/docs/source/screen.rst | 2 ++ src/gallium

[Mesa-dev] [PATCH v2 12/14] st/dri: support format modifier queries

2016-11-24 Thread Varad Gautam
From: Varad Gautam ask the driver for supported modifiers for a given format. bump __DRI_IMAGE implementation version to 15. Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/src

[Mesa-dev] [PATCH v3 07/14] egl_dri2: add support for using modifier attributes in eglCreateImageKHR

2016-11-24 Thread Varad Gautam
From: Varad Gautam allow creating EGLImages with dmabuf format modifiers when target is EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers. v2: - clear modifier assembling and error label name (Eric Engestrom) v3: - remove goto jumps within switch-case (Emil Velikov) - treat

[Mesa-dev] [PATCH v2 06/14] st/dri: implement DRIimage creation from dmabufs with modifiers

2016-11-24 Thread Varad Gautam
From: Varad Gautam support importing dmabufs into DRIimage taking format modifiers in account, as per DRIimage extension version 14. Signed-off-by: Varad Gautam --- src/gallium/include/state_tracker/drm_driver.h | 2 ++ src/gallium/state_trackers/dri/dri2.c | 45

[Mesa-dev] [PATCH v2 10/14] st/dri: support format queries

2016-11-24 Thread Varad Gautam
From: Varad Gautam ask the driver for supported dmabuf formats Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/gallium/state_trackers/dri/dri2.c b/src/gallium/state_trackers/dri/dri2.c index a2b87a7

[Mesa-dev] [PATCH v2 14/14] egl: advertise EGL_EXT_image_dma_buf_import_modifiers

2016-11-24 Thread Varad Gautam
From: Varad Gautam Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 5 + src/egl/main/eglapi.c | 1 + 2 files changed, 6 insertions(+) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index d387e28..e155801 100644 --- a/src/egl

[Mesa-dev] [PATCH v2 11/14] egl: implement eglQueryDmaBufFormatsEXT

2016-11-24 Thread Varad Gautam
From: Varad Gautam allow egl clients to query the dmabuf formats supported on this platform. Signed-off-by: Louis-Francis Ratté-Boulianne Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 17 + src/egl/main/eglapi.c | 19 +++ src/egl

[Mesa-dev] [PATCH v2 13/14] egl: implement eglQueryDmaBufModifiersEXT

2016-11-24 Thread Varad Gautam
From: Varad Gautam query and return supported dmabuf format modifiers for EGL_EXT_image_dma_buf_import_modifiers. Signed-off-by: Varad Gautam --- src/egl/drivers/dri2/egl_dri2.c | 47 + src/egl/main/eglapi.c | 21 ++ src/egl

[Mesa-dev] [PATCH v14 04/36] egl/dri2: Create EGLImages with dmabuf modifiers

2017-05-30 Thread Varad Gautam
From: Varad Gautam Allow creating EGLImages with dmabuf format modifiers when target is EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers. v2: - clear modifier assembling and error label name (Eric Engestrom) v3: - remove goto jumps within switch-case (Emil Velikov) - treat zero

[Mesa-dev] [PATCH v14 01/36] egl: introduce DMA_BUF_MAX_PLANES

2017-05-30 Thread Varad Gautam
From: Pekka Paalanen Rather than hardcoding 3, use a #define. Makes it easier to bump this later to 4. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom Reviewed-by: Daniel Stone --- src/egl/drivers/dri2/egl_dri2.c | 8 src/egl/main/eglimage.h

[Mesa-dev] [PATCH v14 05/36] egl: implement eglQueryDmaBufFormatsEXT

2017-05-30 Thread Varad Gautam
From: Varad Gautam allow egl clients to query the dmabuf formats supported on this platform. v2: return EGLBoolean. v3: Check DRIimageExtension version before querying (Daniel Stone). v4: move to DRIimageExtension version 15, error checking (Jason Ekstrand). Signed-off-by: Louis-Francis Ratté

[Mesa-dev] [PATCH v14 02/36] egl/main: add support for fourth plane tokens

2017-05-30 Thread Varad Gautam
) Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom Reviewed-by: Daniel Stone --- src/egl/drivers/dri2/egl_dri2.c | 2 +- src/egl/main/egldisplay.h | 1 + src/egl/main/eglimage.c | 18 ++ src/egl/main/eglimage.h | 5

[Mesa-dev] [PATCH v14 00/36] EGL_EXT_image_dma_buf_import_modifiers with i915 CCS

2017-05-30 Thread Varad Gautam
ES egl/main: add support for fourth plane tokens Varad Gautam (15): dri: introduce dmabuf format modifier related handles egl/dri2: Create EGLImages with dmabuf modifiers egl: implement eglQueryDmaBufFormatsEXT egl: implement eglQueryDmaBufModifiersEXT egl: advertise EGL_EXT_image_dma_bu

[Mesa-dev] [PATCH v14 06/36] egl: implement eglQueryDmaBufModifiersEXT

2017-05-30 Thread Varad Gautam
From: Varad Gautam query and return supported dmabuf format modifiers for EGL_EXT_image_dma_buf_import_modifiers. v2: move format check to the driver instead of making format queries here and then checking. v3: Check DRIimageExtension version before query (Daniel Stone) v4: - move to

[Mesa-dev] [PATCH v14 07/36] egl: advertise EGL_EXT_image_dma_buf_import_modifiers

2017-05-30 Thread Varad Gautam
From: Varad Gautam v2: check for DRIimageExtension version 15 (Jason Ekstrand) Signed-off-by: Varad Gautam Reviewed-by: Daniel Stone Reviewed-by: Emil Velikov --- src/egl/drivers/dri2/egl_dri2.c | 6 ++ src/egl/main/eglapi.c | 1 + 2 files changed, 7 insertions(+) diff --git

[Mesa-dev] [PATCH v14 09/36] st/dri: enable DRIimage modifier queries

2017-05-30 Thread Varad Gautam
From: Varad Gautam return the modifier selected by the driver when creating this image. v2: since we can use winsys_handle->modifier to serve these, remove DRIimage->modifier from v1. use DRM_API_HANDLE_TYPE_KMS instead of DRM_API_HANDLE_TYPE_FD to avoid ownership transfer.

[Mesa-dev] [PATCH v14 03/36] dri: introduce dmabuf format modifier related handles

2017-05-30 Thread Varad Gautam
) - pass a single modifier form createImageFromDmaBufs2 since all planes have the same modifier (Jason Ekstrand) Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Daniel Stone Reviewed-by: Emil Velikov --- include/GL/internal/dri_interface.h | 63

[Mesa-dev] [PATCH v14 14/36] st/dri: support format modifier queries

2017-05-30 Thread Varad Gautam
From: Varad Gautam ask the driver for supported modifiers for a given format. v2: move to __DRIimageExtension v16. v3: fail if the supplied format is not supported by driver. v4: purge PIPE_CAP_QUERY_DMABUF_ATTRIBS. v5: - move to __DRIimageExtension v15, pass external_only to the driver

[Mesa-dev] [PATCH v14 16/36] i965: Invert image modifier/tiling inference

2017-05-30 Thread Varad Gautam
From: Daniel Stone When allocating images, we record a tiling mode and then work backwards to infer the modifier. Unfortunately this is the wrong way around, since it is a one:many mapping (e.g. TILING_Y can be plain Y-tiling, or Y-tiling with CCS). Invert the mapping, so we record a modifier fi

[Mesa-dev] [PATCH v14 08/36] gallium/winsys/drm: introduce modifier field to winsys_handle

2017-05-30 Thread Varad Gautam
From: Varad Gautam we use this to import resources with format modifiers, and to support per-resource modifier queries. Signed-off-by: Varad Gautam Cc: Lucas Stach --- src/gallium/include/state_tracker/drm_driver.h | 6 ++ src/gallium/state_trackers/dri/dri2.c | 7 +++ 2

[Mesa-dev] [PATCH v14 15/36] i965: Improve same-buffer restriction for imports

2017-05-30 Thread Varad Gautam
even if the underlying buffer is the same. Instead of checking the FDs for equality, we must check if they actually point to the same buffer (Jason). Signed-off-by: Daniel Stone Signed-off-by: Varad Gautam --- src/mesa/drivers/dri/i965/intel_screen.c | 24 ++-- 1 file changed

[Mesa-dev] [PATCH v14 17/36] i965: Support images with offset aux buffers

2017-05-30 Thread Varad Gautam
From: Ben Widawsky Previously our aux buffers (MCS, and HiZ) never had an offset because they were in their own buffer object. When using the CCS lossless compression feature, it's desirable to store the data at an offset from the main framebuffer, ie. share a buffer object. This patch just makes

[Mesa-dev] [PATCH v14 13/36] gallium: introduce format modifier querying

2017-05-30 Thread Varad Gautam
From: Varad Gautam format modifiers tokens are driver specific, and hence, need to come in from the driver. this allows drivers to be queried for supported format modifiers for EGL_EXT_image_dma_buf_import_modifiers. v2: rebase to master. v3: drivers must return false on query failure. v4: use

[Mesa-dev] [PATCH v14 18/36] i965/miptree: Add a helper function for image creation

2017-05-30 Thread Varad Gautam
From: Ben Widawsky This provides a common function or creating miptrees when there is an existing DRIimage to use. That provides an easy way to add CCS allocation. v2: Make the new function assume there are always no layout flags. This will be adjusted later. Signed-off-by: Ben Widawsky Acked-

[Mesa-dev] [PATCH v14 11/36] st/dri: implement DRIimage creation from dmabufs with modifiers

2017-05-30 Thread Varad Gautam
From: Varad Gautam support importing dmabufs into DRIimage while taking format modifiers in account, as per DRIimage extension version 15. v2: initialize winsys modifier to DRM_FORMAT_MOD_INVALID (Daniel Stone) v3: do not bump DRIimageExtension version. split out winsys changes. Signed-off-by

[Mesa-dev] [PATCH v14 12/36] st/dri: support format queries

2017-05-30 Thread Varad Gautam
From: Varad Gautam ask the driver for supported dmabuf formats v2: rebase to master. v3: return false on failure. v4: use pscreen->is_format_supported instead of adding a new query. (Lucas Stach) v5: stylefix to conform to formatting rules (Brian Paul). add fourcc list here instead

[Mesa-dev] [PATCH v14 23/36] i965: Add logic for allocating BO with CCS

2017-05-30 Thread Varad Gautam
From: Ben Widawsky This patch provides the support (and comments) for allocating the BO with space for the CCS buffer just underneath it. This patch was originally titled: "i965: Create correctly sized mcs for an image" In order to make things more bisectable, reviewable, and to have the CCS_MO

[Mesa-dev] [PATCH v14 22/36] i965: Allocate tile aligned height

2017-05-30 Thread Varad Gautam
From: Ben Widawsky This patch shouldn't actually do anything because the libdrm function should already do this alignment. However, it preps us for a future patch where we add in the CCS AUX size, and in the process it serves as a good place to find bisectable issues if libdrm or kernel does some

[Mesa-dev] [PATCH v14 21/36] i965: add a no_aux field to identify buffers without aux data

2017-05-30 Thread Varad Gautam
From: Varad Gautam v2: split this into a separate patch (Jason) Signed-off-by: Ben Widawsky Signed-off-by: Daniel Stone Cc: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_context.c | 3 +++ src/mesa/drivers/dri/i965/intel_fbo.h | 7 +++ src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v14 32/36] i965: Remove scanout restriction from lossless compression

2017-05-30 Thread Varad Gautam
From: Ben Widawsky v2: Try to keep the assert as recommended by Topi. This requires modifying the num_samples check to be <= 1 because internally created buffers set num_samples = 0. v3: Buffers are proactively marked as scanout, often, and so checking is_scanout in whether or not the buffer sup

[Mesa-dev] [PATCH v14 19/36] i965/miptree: Allocate mcs_buf for an image's CCS_E

2017-05-30 Thread Varad Gautam
From: Ben Widawsky This code will disable actually creating these buffers for the scanout, but it puts the allocation in place. Primarily this patch is split out for review, it can be squashed in later if preferred. v2: assert(mt->offset == 0) in ccs creation (as requested by Topi) Remove bogus

[Mesa-dev] [PATCH v14 10/36] st/dri: implement createImageWithModifiers in DRIimage

2017-05-30 Thread Varad Gautam
From: Varad Gautam adds a pscreen->resource_create_with_modifiers() to create textures with modifier. v2: - stylefixes (Emil Velikov) - don't return selected modifier from resource_create_with_modifiers. we can use the winsys_handle to get this. Signed-off-by: Varad Gautam Rev

[Mesa-dev] [PATCH v14 24/36] i965/miptree: Add a return for updating of winsys

2017-05-30 Thread Varad Gautam
From: Ben Widawsky There is nothing particularly useful to do currently if the update fails, but there is no point carrying on either. As a result, this has a behavior change. v2: Make the return type a bool (Topi) v3: Don't leak the bo if update_winsys_renderbuffer fails. (Jason) Signed-off-b

[Mesa-dev] [PATCH v14 27/36] i965: Change resolve flags to enum

2017-05-30 Thread Varad Gautam
From: Ben Widawsky In the foreseeable future it doesn't seem to make sense to have multiple resolve flags. What does make sense is to have the caller give an indication to the lower layers what it things should be done for resolve. The enum change distinguishes this binary selection. v2: Make se

[Mesa-dev] [PATCH v14 30/36] i965: Use partial resolves for CCS buffers being scanned out

2017-05-30 Thread Varad Gautam
From: Ben Widawsky On Gen9 hardware, the display engine is able to scanout a compressed framebuffer by providing an offset to auxiliary compression information. Unfortunately, the hardware is incapable of doing the same thing for the fast clear color. To mitigate this, the hardware introduced a

[Mesa-dev] [PATCH v14 26/36] i965: Pretend that CCS modified images are two planes

2017-05-30 Thread Varad Gautam
From: Ben Widawsky v2: move is_aux into if block. (Jason) Use else block instead of goto (Jason) v3: Fix up logic for is_aux (Ben) Fix up size calculations and add FIXME (Ben) Cc: Jason Ekstrand Signed-off-by: Ben Widawsky Acked-by: Daniel Stone --- src/mesa/drivers/dri/i965/intel_screen.c

[Mesa-dev] [PATCH v14 31/36] i965/miptree: Remove dead code assertion

2017-05-30 Thread Varad Gautam
From: Ben Widawsky We no longer allocate a miptree for the mcs_buf, so this is not a useful assertion. Recommended-by: Topi Pohjolainen Signed-off-by: Ben Widawsky Reviewed-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 8 1 file changed, 8 deletions(-) di

[Mesa-dev] [PATCH v14 25/36] i965/miptree: Allocate mt earlier in update winsys

2017-05-30 Thread Varad Gautam
From: Ben Widawsky Allows us to continue utilizing common miptree creation using __DRIimage without creating a new DRIimage (for the intel_process_dri2_buffer() case). This is a bit ugly, but I think it's the best one can do. v2: This patch let's us remove the temporary no_aux variable since mt

[Mesa-dev] [PATCH v14 20/36] i965: Restructure CCS disabling

2017-05-30 Thread Varad Gautam
From: Ben Widawsky Make the code only disable CCS when it has to, unlike before where it disabled CCS and enabled it when it could. This is much more inline with how it should work in a few patches, where we have fewer restrictions as to when we disable CCS. v2: Change CCS disabling to an assert

[Mesa-dev] [PATCH v14 28/36] i965: Plumb resolve hints from miptrees to blorp

2017-05-30 Thread Varad Gautam
From: Ben Widawsky Signed-off-by: Ben Widawsky Acked-by: Daniel Stone Reviewed-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_blorp.c | 3 ++- src/mesa/drivers/dri/i965/brw_blorp.h | 3 ++- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 4 ++-- 3 files changed, 6 inse

[Mesa-dev] [PATCH v14 33/36] i965: Handle compression modifier

2017-05-30 Thread Varad Gautam
From: Ben Widawsky v2: Rename modifier to be more smart (Jason) FINISHME: Use the kernel's final choice for the fb modifier bwidawsk@norris2:~/intel-gfx/kmscube (modifiers $) ~/scripts/measure_bandwidth.sh ./kmscube none Read bandwidth: 603.91 MiB/s Write bandwidth: 615.28 MiB/s bwidawsk@norri

[Mesa-dev] [PATCH v14 36/36] i965: Add format/modifier advertising

2017-05-30 Thread Varad Gautam
From: Varad Gautam v2: Rebase and reuse tiling/modifier map. (Daniel Stone) v3: bump DRIimageExtension to version 15, fill external_only array. Signed-off-by: Varad Gautam Signed-off-by: Daniel Stone --- src/mesa/drivers/dri/i965/intel_screen.c | 79 +--- 1 file

[Mesa-dev] [PATCH v14 35/36] i965: Support dmabuf import with modifiers

2017-05-30 Thread Varad Gautam
From: Varad Gautam Add support for createImageFromDmaBufs2, adding a modifier to the original, and allow importing CCS resources with auxiliary data from dmabufs. v2: avoid DRIimageExtension version bump, pass single modifier to createImageFromDmaBufs2. Signed-off-by: Varad Gautam Signed

[Mesa-dev] [PATCH v14 29/36] i965: Add new resolve hints full and partial

2017-05-30 Thread Varad Gautam
From: Ben Widawsky Upper layers of the code will have the need to specify full or partial resolves (more on this in the next patch). This code simply adds the new enums and plumbs it in as minimally as necessary. Signed-off-by: Ben Widawsky Acked-by: Daniel Stone Reviewed-by: Topi Pohjolainen

[Mesa-dev] [PATCH v14 34/36] i965: Add tiling mode to BO import

2017-05-30 Thread Varad Gautam
From: Daniel Stone When importing a dmabuf, verify that the tiling mode matches what was expected. Signed-off-by: Daniel Stone Signed-off-by: Varad Gautam --- src/mesa/drivers/dri/i965/brw_bufmgr.c | 12 ++-- src/mesa/drivers/dri/i965/brw_bufmgr.h | 3 ++- src/mesa/drivers/dri

Re: [Mesa-dev] [PATCH v14 15/36] i965: Improve same-buffer restriction for imports

2017-06-05 Thread Varad Gautam
Hi Daniel, On Mon, 2017-06-05 at 14:53 +0100, Daniel Stone wrote: > Hi Varad, > > On 30 May 2017 at 12:53, Varad Gautam wrote: > > > > +   /* We only support all planes from the same bo. > > +* brw_bo_gem_create_from_prime() should return the same pointer for

[Mesa-dev] [PATCH v3] i965: Support dmabuf import with modifiers

2017-06-06 Thread Varad Gautam
n for imports' v2 Signed-off-by: Varad Gautam Signed-off-by: Daniel Stone --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 ++ src/mesa/drivers/dri/i965/intel_screen.c | 114 +++--- 2 files changed, 107 insertions(+), 13 deletions(-) diff --git a/src/mesa/d

[Mesa-dev] [PATCH v2] i965: Add tiling mode to BO import

2017-06-06 Thread Varad Gautam
From: Daniel Stone When importing a dmabuf, verify that the tiling mode matches what was expected. v2: rebase to 'i965: Improve same-buffer restriction for imports' v2. Signed-off-by: Daniel Stone Signed-off-by: Varad Gautam --- src/mesa/drivers/dri/i965/brw_bufmg

[Mesa-dev] [PATCH v2] i965: Improve same-buffer restriction for imports

2017-06-06 Thread Varad Gautam
even if the underlying buffer is the same. Instead of checking the FDs for equality, we must check if they actually point to the same buffer (Jason). v2: stop leaking bo references (Daniel Stone). buildfix. Signed-off-by: Daniel Stone Signed-off-by: Varad Gautam --- src/mesa/drivers/dri/i965

Re: [Mesa-dev] [PATCH] st/dri: Add support for PIPE_FORMAT_RGBX8888_UNORM

2017-06-19 Thread Varad Gautam
pf = PIPE_FORMAT_RGBX_UNORM; > + break; > case __DRI_IMAGE_FORMAT_ABGR: > pf = PIPE_FORMAT_RGBA_UNORM; >break; Reviewed-by: Varad Gautam > -- > 2.13.1.518.g3df882009-goog > > ___ > mesa-dev mailing list

[Mesa-dev] [PATCH v3 02/15] dri: Add an image creation with modifiers

2017-05-10 Thread Varad Gautam
. v2: Don't allow usage + modifiers v3: Make NAND actually NAND. Bug introduced in v2. (Jason) v4: Bump DRIimage version. (Varad) Cc: Kristian Høgsberg Cc: Jason Ekstrand Signed-off-by: Ben Widawsky Reviewed-by: Eric Engestrom (v1) Acked-by: Daniel Stone Signed-off-by: Varad Gautam --- s

[Mesa-dev] [PATCH v3 04/15] egl/main: add support for fourth plane tokens

2017-05-10 Thread Varad Gautam
) Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom --- src/egl/drivers/dri2/egl_dri2.c | 2 +- src/egl/main/egldisplay.h | 1 + src/egl/main/eglimage.c | 18 ++ src/egl/main/eglimage.h | 5 +++-- 4 files changed, 23

[Mesa-dev] [PATCH v3 06/15] dri: support DRIimage creation from dmabufs with modifiers

2017-05-10 Thread Varad Gautam
From: Pekka Paalanen add createImageFromDmaBufs2 function which accepts per-plane dmabuf format modifiers. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH v3 01/15] st/dri: refactor multi-planar YUV import path

2017-05-10 Thread Varad Gautam
From: Varad Gautam we currently ignore the plane count when converting from __DRI_IMAGE_FORMAT* tokens to __DRI_IMAGE_FOURCC* for multiplanar images, and only return the first plane's simplified fourcc. this adds a fourcc to __DRI_IMAGE_FORMAT_* mapping to dri, allowing us to return the co

[Mesa-dev] [PATCH v3 05/15] st/dri: enable DRIimage modifier queries

2017-05-10 Thread Varad Gautam
From: Varad Gautam introduce modifier field in DRIimage and set it to DRM_FORMAT_MOD_INVALID for now. support DRIimage modifier queries. Suggested-by: Daniel Stone Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c | 15 +++ src/gallium/state_trackers/dri

[Mesa-dev] [PATCH v3 07/15] st/dri: implement DRIimage creation from dmabufs with modifiers

2017-05-10 Thread Varad Gautam
From: Varad Gautam support importing dmabufs into DRIimage while taking format modifiers in account, as per DRIimage extension version 15. bump __DRIimageExtension to 15. v2: initialize winsys modifier to DRM_FORMAT_MOD_INVALID (Daniel Stone) Signed-off-by: Varad Gautam --- src/gallium

[Mesa-dev] [PATCH v3 00/15] EGL_EXT_image_dma_buf_import_modifiers v3

2017-05-10 Thread Varad Gautam
th plane tokens dri: support DRIimage creation from dmabufs with modifiers Varad Gautam (11): st/dri: refactor multi-planar YUV import path st/dri: enable DRIimage modifier queries st/dri: implement DRIimage creation from dmabufs with modifiers egl_dri2: add support fo

[Mesa-dev] [PATCH v3 08/15] egl_dri2: add support for using modifier attributes in eglCreateImageKHR

2017-05-10 Thread Varad Gautam
From: Varad Gautam allow creating EGLImages with dmabuf format modifiers when target is EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers. v2: - clear modifier assembling and error label name (Eric Engestrom) v3: - remove goto jumps within switch-case (Emil Velikov) - treat

[Mesa-dev] [PATCH v3 03/15] egl: introduce DMA_BUF_MAX_PLANES

2017-05-10 Thread Varad Gautam
From: Pekka Paalanen Rather than hardcoding 3, use a #define. Makes it easier to bump this later to 4. Signed-off-by: Pekka Paalanen Signed-off-by: Varad Gautam Reviewed-by: Eric Engestrom --- src/egl/drivers/dri2/egl_dri2.c | 8 src/egl/main/eglimage.h | 8 +--- 2

[Mesa-dev] [PATCH v3 09/15] dri: add queryDmaBufFormats and queryDmaBufModifiers to DRIimage

2017-05-10 Thread Varad Gautam
From: Varad Gautam these allow querying the driver for supported dmabuf formats and modifiers. v2: move to __DRIimageExtension version 16. v3: return GLBoolean for error reporting, document params better. Signed-off-by: Varad Gautam --- include/GL/internal/dri_interface.h | 42

[Mesa-dev] [PATCH v3 10/15] st/dri: support format queries

2017-05-10 Thread Varad Gautam
From: Varad Gautam ask the driver for supported dmabuf formats v2: rebase to master. v3: return false on failure. v4: use pscreen->is_format_supported instead of adding a new query. (Lucas Stach) Signed-off-by: Varad Gautam --- src/gallium/state_trackers/dri/dri2.c |

  1   2   >