[PATCH 14/14] drm: remove legacy drm_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We don't have any user of this function anymore, let's remove it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/drm_irq.c | 31 ++- include/drm/drmP.h

[PATCH 12/14] drm/udl: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/udl/udl_modeset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/udl/udl_modeset.c b/drivers/gpu/drm/udl

[PATCH 14/14] drm: remove legacy drm_send_vblank_event()

From: Gustavo Padovan We don't have any user of this function anymore, let's remove it. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/drm_irq.c | 31 ++- include/drm/drmP.h| 2 -- 2 files changed, 6 insertions(+), 27 deletions(-) diff --git a/drivers

[PATCH 10/14] drm/shmobile: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/shmobile/shmob_drm_crtc.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 08/14] drm/radeon: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/radeon/radeon_display.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 10/14] drm/shmobile: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/shmobile/shmob_drm_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.c b/drivers

[PATCH 08/14] drm/radeon: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/radeon/radeon_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu

[PATCH 09/14] drm/rcar-du: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 06/14] drm/qxl: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/qxl/qxl_display.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH 09/14] drm/rcar-du: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_crtc.c b/drivers/gpu

[PATCH 06/14] drm/qxl: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/qxl/qxl_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl

[PATCH 13/14] drm/virtio: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 01/14] drm/amdgpu: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/dce

[PATCH 02/14] drm/armada: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/armada/armada_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH 13/14] drm/virtio: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers

[PATCH 01/14] drm/amdgpu: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 2 +- drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 2 +- 3 files changed, 3

[PATCH 02/14] drm/armada: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/armada/armada_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/armada/armada_crtc.c b/drivers/gpu/drm

[PATCH 07/14] drm/nouveau: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/nouveau/nouveau_display.c | 4 +++- 1 file changed, 3 insertions(+)

[PATCH 05/14] drm/msm: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | 2 +- drivers/gpu/drm/msm/mdp/mdp5/mdp

[PATCH 11/14] drm/tilcdc: use drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH 07/14] drm/nouveau: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/nouveau/nouveau_display.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b

[PATCH 05/14] drm/msm: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c | 2 +- drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 11/14] drm/tilcdc: use drm_crtc_send_vblank_event()

From: Gustavo Padovan Replace the legacy drm_send_vblank_event() with the new helper function. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tilcdc/tilcdc_crtc.c b/drivers/gpu/drm

Re: [RESEND] fence: add missing descriptions for fence

; include/linux/fence.h | 2 ++ > 1 file changed, 2 insertions(+) Reviewed-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Gustavo

Re: [RESEND] fence: add missing descriptions for fence

e Bethencourt > Reviewed-by: Javier Martinez Canillas > --- > Hi, > > Just resending this patch since it hasn't had any reviews in since > March 21st. > > Thanks, > Luis > > include/linux/fence.h | 2 ++ > 1 file changed, 2 insertions(+) Reviewed-by: Gustavo Padovan Gustavo

Re: [PATCH v2] drm/virtio: send vblank event after crtc updates

Hi, Any comment on this? Gustavo 2016-03-22 Gustavo Padovan <gust...@padovan.org>: > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > virtio_gpu was failing to send vblank events when using the atomic IOCTL > with the DRM_MODE_PAGE_FLIP_EVENT flag se

Re: [PATCH v2] drm/virtio: send vblank event after crtc updates

Hi, Any comment on this? Gustavo 2016-03-22 Gustavo Padovan : > From: Gustavo Padovan > > virtio_gpu was failing to send vblank events when using the atomic IOCTL > with the DRM_MODE_PAGE_FLIP_EVENT flag set. This patch fixes each and > enables atomic pageflips updat

Re: [PATCH v10 3/3] staging/android: refactor SYNC IOCTLs

Hi Greg, Any comments on this? Thanks, Gustavo 2016-03-18 Gustavo Padovan <gust...@padovan.org>: > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid > future API breaks and op

Re: [PATCH v10 3/3] staging/android: refactor SYNC IOCTLs

Hi Greg, Any comments on this? Thanks, Gustavo 2016-03-18 Gustavo Padovan : > From: Gustavo Padovan > > Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid > future API breaks and optimize buffer allocation. > > Now num_fences can be fi

Re: [PATCH v7 1/5] staging/android: add num_fences field to struct sync_file_info

Hi Greg, 2016-03-30 Greg Kroah-Hartman <gre...@linuxfoundation.org>: > On Thu, Mar 03, 2016 at 04:40:42PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > Gustavo, can you resend both series of your android patc

Re: [PATCH v7 1/5] staging/android: add num_fences field to struct sync_file_info

Hi Greg, 2016-03-30 Greg Kroah-Hartman : > On Thu, Mar 03, 2016 at 04:40:42PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Gustavo, can you resend both series of your android patches so I know I > have the latest ones to work with? Plea

Re: [RFC 0/6] drm/fences: add in-fences to DRM

Hi Inki, 2016-03-24 Inki Dae <inki@samsung.com>: > Hi, > > 2016년 03월 24일 03:47에 Gustavo Padovan 이(가) 쓴 글: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > Hi, > > > > This is a first proposal to discuss the addition of

Re: [RFC 0/6] drm/fences: add in-fences to DRM

Hi Inki, 2016-03-24 Inki Dae : > Hi, > > 2016년 03월 24일 03:47에 Gustavo Padovan 이(가) 쓴 글: > > From: Gustavo Padovan > > > > Hi, > > > > This is a first proposal to discuss the addition of in-fences support > > to DRM. It adds a new struct to fence.

Re: [RFC 0/6] drm/fences: add in-fences to DRM

Hi Maarten, 2016-03-24 Maarten Lankhorst <maarten.lankho...@linux.intel.com>: > Hey, > > Op 23-03-16 om 19:47 schreef Gustavo Padovan: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > Hi, > > > > This is a first proposal to d

Re: [RFC 0/6] drm/fences: add in-fences to DRM

Hi Maarten, 2016-03-24 Maarten Lankhorst : > Hey, > > Op 23-03-16 om 19:47 schreef Gustavo Padovan: > > From: Gustavo Padovan > > > > Hi, > > > > This is a first proposal to discuss the addition of in-fences support > > to DRM. It adds a new struc

[RFC 1/6] drm/fence: add FENCE_FD property to planes

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> FENCE_FD can now be set by the user during an atomic IOCTL, it will be used by atomic_commit to wait until the sync_file is signalled, i.e., the framebuffer is ready for scanout. Signed-off-by: Gustavo Padovan <gus

[RFC 2/6] dma-buf/fence: add struct fence_collection

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> The struct aggregates fences that we need to wait on before proceed with some specific operation. In DRM, for example, we may wait for a group of fences to signal before we scanout the buffers related to those fences. Signed-off-by: G

[RFC 1/6] drm/fence: add FENCE_FD property to planes

From: Gustavo Padovan FENCE_FD can now be set by the user during an atomic IOCTL, it will be used by atomic_commit to wait until the sync_file is signalled, i.e., the framebuffer is ready for scanout. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/drm_atomic.c| 4 drivers/gpu

[RFC 2/6] dma-buf/fence: add struct fence_collection

From: Gustavo Padovan The struct aggregates fences that we need to wait on before proceed with some specific operation. In DRM, for example, we may wait for a group of fences to signal before we scanout the buffers related to those fences. Signed-off-by: Gustavo Padovan --- include/linux

[RFC 4/6] dma-buf/fence: add fence_collection_put()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Put fence_collection data. For that calls fence_put() on all fences and the user put callback. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/dma-buf/fence.c | 17 + include/l

[RFC 4/6] dma-buf/fence: add fence_collection_put()

From: Gustavo Padovan Put fence_collection data. For that calls fence_put() on all fences and the user put callback. Signed-off-by: Gustavo Padovan --- drivers/dma-buf/fence.c | 17 + include/linux/fence.h | 2 ++ 2 files changed, 19 insertions(+) diff --git a/drivers/dma

[RFC 5/6] dma-buf/fence: add fence_collection_wait()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Iterate over the array of fences and wait for all of the to finish. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/dma-buf/fence.c | 16 include/linux/fence.h | 1 + 2 files

[RFC 5/6] dma-buf/fence: add fence_collection_wait()

From: Gustavo Padovan Iterate over the array of fences and wait for all of the to finish. Signed-off-by: Gustavo Padovan --- drivers/dma-buf/fence.c | 16 include/linux/fence.h | 1 + 2 files changed, 17 insertions(+) diff --git a/drivers/dma-buf/fence.c b/drivers/dma-buf

[RFC 0/6] drm/fences: add in-fences to DRM

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Hi, This is a first proposal to discuss the addition of in-fences support to DRM. It adds a new struct to fence.c to abstract the use of sync_file in DRM drivers. The new struct fence_collection contains a array with all

[RFC 0/6] drm/fences: add in-fences to DRM

From: Gustavo Padovan Hi, This is a first proposal to discuss the addition of in-fences support to DRM. It adds a new struct to fence.c to abstract the use of sync_file in DRM drivers. The new struct fence_collection contains a array with all fences that a atomic commit needs to wait

[RFC 3/6] dma-buf/sync_file: add sync_file_fences_get()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Creates a function that given an sync file descriptor returns a fence_collection containing all fences in the sync_file. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/dma-buf/sync

[RFC 6/6] drm/fence: support fence_collection on atomic commit

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Let atomic_commit() wait on a collection of fences before proceed with the scanout. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/drm_atomic.c| 9 + drivers/gpu/drm/drm_ato

[RFC 3/6] dma-buf/sync_file: add sync_file_fences_get()

From: Gustavo Padovan Creates a function that given an sync file descriptor returns a fence_collection containing all fences in the sync_file. Signed-off-by: Gustavo Padovan --- drivers/dma-buf/sync_file.c | 36 include/linux/sync_file.h | 8 2

[RFC 6/6] drm/fence: support fence_collection on atomic commit

From: Gustavo Padovan Let atomic_commit() wait on a collection of fences before proceed with the scanout. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/drm_atomic.c| 9 + drivers/gpu/drm/drm_atomic_helper.c | 9 + include/drm/drm_crtc.h | 2 +- 3

Re: [PATCH] dma-buf/sync_file: de-stage sync_file

Hi Sumit, 2016-03-21 Sumit Semwal <sumit.sem...@linaro.org>: > Thanks for the patch, Gustavo! > > On 18 March 2016 at 19:49, Gustavo Padovan <gust...@padovan.org> wrote: > > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > > > sync

Re: [PATCH] dma-buf/sync_file: de-stage sync_file

Hi Sumit, 2016-03-21 Sumit Semwal : > Thanks for the patch, Gustavo! > > On 18 March 2016 at 19:49, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > sync_file is useful to connect one or more fences to the file. The file is > > used by userspa

[PATCH v2] drm/virtio: send vblank event after crtc updates

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> virtio_gpu was failing to send vblank events when using the atomic IOCTL with the DRM_MODE_PAGE_FLIP_EVENT flag set. This patch fixes each and enables atomic pageflips updates. Signed-off-by: Gustavo Padovan <gustavo.pado...@collab

[PATCH v2] drm/virtio: send vblank event after crtc updates

From: Gustavo Padovan virtio_gpu was failing to send vblank events when using the atomic IOCTL with the DRM_MODE_PAGE_FLIP_EVENT flag set. This patch fixes each and enables atomic pageflips updates. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/virtio/virtgpu_display.c | 12

[PATCH 1/2] drm/virtio: use new drm_crtc_send_vblank_event()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Simplify code by using the new vblank crtc helpers. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

[PATCH 1/2] drm/virtio: use new drm_crtc_send_vblank_event()

From: Gustavo Padovan Simplify code by using the new vblank crtc helpers. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/virtio/virtgpu_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_display.c b/drivers/gpu/drm/virtio

[PATCH 2/2] drm/virtio: send vblank event on plane atomic update

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> virtio_gpu was failing to send vblank events when using the atomic IOCTL with the DRM_MODE_PAGE_FLIP_EVENT flag set. This patch fixes each and enables atomic pageflips updates. Signed-off-by: Gustavo Padovan <gustavo.pado...@collab

[PATCH 2/2] drm/virtio: send vblank event on plane atomic update

From: Gustavo Padovan virtio_gpu was failing to send vblank events when using the atomic IOCTL with the DRM_MODE_PAGE_FLIP_EVENT flag set. This patch fixes each and enables atomic pageflips updates. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/virtio/virtgpu_plane.c | 9 - 1

Re: [PATCH] powerpc: rename to_user_ptr to __to_user_ptr

Hi, 2016-03-17 Gustavo Padovan <gust...@padovan.org>: > From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> > > to_user_ptr() is a local macro defined by signal_32.c, rename it to > __to_user_ptr() as now we will have a global to_user_ptr() defined by > kernel.h t

Re: [PATCH] powerpc: rename to_user_ptr to __to_user_ptr

Hi, 2016-03-17 Gustavo Padovan : > From: Gustavo Padovan > > to_user_ptr() is a local macro defined by signal_32.c, rename it to > __to_user_ptr() as now we will have a global to_user_ptr() defined by > kernel.h that has a different meaning from this one. > > Cc: Benjami

[PATCH v9 2/3] kernel.h: add to_user_ptr()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> This function had copies in 3 different files. Unify them in kernel.h. Cc: Andrew Morton <a...@linux-foundation.org> Cc: David Airlie <airl...@linux.ie> Cc: Daniel Vetter <daniel.vet...@intel.com> Cc: Rob Clark <

[PATCH v9 2/3] kernel.h: add to_user_ptr()

From: Gustavo Padovan This function had copies in 3 different files. Unify them in kernel.h. Cc: Andrew Morton Cc: David Airlie Cc: Daniel Vetter Cc: Rob Clark Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 5 - drivers/gpu/drm/i915/i915_drv.h

[PATCH] powerpc: rename to_user_ptr to __to_user_ptr

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> to_user_ptr() is a local macro defined by signal_32.c, rename it to __to_user_ptr() as now we will have a global to_user_ptr() defined by kernel.h that has a different meaning from this one. Cc: Benjamin Herrenschm

[PATCH] powerpc: rename to_user_ptr to __to_user_ptr

From: Gustavo Padovan to_user_ptr() is a local macro defined by signal_32.c, rename it to __to_user_ptr() as now we will have a global to_user_ptr() defined by kernel.h that has a different meaning from this one. Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Signed-off

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Gustavo Padovan <gustavo.pado...@collabora.co.uk>: > 2016-03-17 Joe Perches <j...@perches.com>: > > > On Thu, 2016-03-17 at 14:30 -0300, Gustavo Padovan wrote: > > > This function had copies in 3 different files. Unify them in > > > kerne

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Gustavo Padovan : > 2016-03-17 Joe Perches : > > > On Thu, 2016-03-17 at 14:30 -0300, Gustavo Padovan wrote: > > > This function had copies in 3 different files. Unify them in > > > kernel.h. > > > > This is only used by gpu/drm. > >

[PATCH v9 3/3] staging/android: refactor SYNC IOCTLs

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the

[PATCH v9 3/3] staging/android: refactor SYNC IOCTLs

From: Gustavo Padovan Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the kernel. If the num_fences passed is greater than

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Joe Perches <j...@perches.com>: > On Thu, 2016-03-17 at 14:30 -0300, Gustavo Padovan wrote: > > This function had copies in 3 different files. Unify them in > > kernel.h. > > This is only used by gpu/drm. > > I think this is a poor name fo

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Joe Perches : > On Thu, 2016-03-17 at 14:30 -0300, Gustavo Padovan wrote: > > This function had copies in 3 different files. Unify them in > > kernel.h. > > This is only used by gpu/drm. > > I think this is a poor name for a generic function > that wo

[PATCH] de-stage android sync framework

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Hi, This is the first step and the most important one of the de-stage of the the sync framework, it de-stage the sync_file part which is used to send/receive fence file descriptors with the userspace. These patches sits on top of th

[PATCH] de-stage android sync framework

From: Gustavo Padovan Hi, This is the first step and the most important one of the de-stage of the the sync framework, it de-stage the sync_file part which is used to send/receive fence file descriptors with the userspace. These patches sits on top of the sync ABI changes that I sent earlier

[PATCH v10 2/3] kernel.h: add u64_to_user_ptr()

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> This function had copies in 3 different files. Unify them in kernel.h. Cc: Joe Perches <j...@perches.com> Cc: Andrew Morton <a...@linux-foundation.org> Cc: David Airlie <airl...@linux.ie> Cc: Daniel Vetter <danie

[PATCH v10 2/3] kernel.h: add u64_to_user_ptr()

From: Gustavo Padovan This function had copies in 3 different files. Unify them in kernel.h. Cc: Joe Perches Cc: Andrew Morton Cc: David Airlie Cc: Daniel Vetter Cc: Rob Clark Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 11 +++ drivers/gpu

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Joe Perches <j...@perches.com>: > On Thu, 2016-03-17 at 18:19 -0300, Gustavo Padovan wrote: > > 2016-03-17 Joe Perches <j...@perches.com>: > > > On Thu, 2016-03-17 at 16:50 -0400, Rob Clark wrote: > > > > On Thu, Mar 17, 2016 at 4:4

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Joe Perches : > On Thu, 2016-03-17 at 18:19 -0300, Gustavo Padovan wrote: > > 2016-03-17 Joe Perches : > > > On Thu, 2016-03-17 at 16:50 -0400, Rob Clark wrote: > > > > On Thu, Mar 17, 2016 at 4:40 PM, Joe Perches wrote: > > > [] > >

[PATCH v10 1/3] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

[PATCH v10 1/3] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/uapi/sync.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Joe Perches : > On Thu, 2016-03-17 at 16:50 -0400, Rob Clark wrote: > > On Thu, Mar 17, 2016 at 4:40 PM, Joe Perches wrote: > [] > > > It's a name that seems like it should be a straightforward > > > cast of a kernel pointer to a __user pointer

Re: [PATCH v9 2/3] kernel.h: add to_user_ptr()

2016-03-17 Joe Perches : > On Thu, 2016-03-17 at 16:50 -0400, Rob Clark wrote: > > On Thu, Mar 17, 2016 at 4:40 PM, Joe Perches wrote: > [] > > > It's a name that seems like it should be a straightforward > > > cast of a kernel pointer to a __user pointer like: > > > > > > static inline void

[PATCH] dma-buf/sync_file: de-stage sync_file

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> sync_file is useful to connect one or more fences to the file. The file is used by userspace to track fences. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> --- drivers/Kconfig

[PATCH] dma-buf/sync_file: de-stage sync_file

From: Gustavo Padovan sync_file is useful to connect one or more fences to the file. The file is used by userspace to track fences. Signed-off-by: Gustavo Padovan --- drivers/Kconfig| 2 + drivers/dma-buf/Kconfig| 11

[PATCH v10 3/3] staging/android: refactor SYNC IOCTLs

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the

[PATCH v10 3/3] staging/android: refactor SYNC IOCTLs

From: Gustavo Padovan Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the kernel. If the num_fences passed is greater than

[PATCH v9 1/3] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

[PATCH v9 1/3] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/uapi/sync.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH v8 2/2] staging/android: refactor SYNC IOCTLs

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the

[PATCH v8 2/2] staging/android: refactor SYNC IOCTLs

From: Gustavo Padovan Change SYNC_IOC_FILE_INFO (former SYNC_IOC_FENCE_INFO) behaviour to avoid future API breaks and optimize buffer allocation. Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the kernel. If the num_fences passed is greater than

[PATCH v8 1/2] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

[PATCH v8 1/2] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/uapi/sync.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH] staging/android: change IOCTLs opcode after ABI change

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Burn the old opcode to avoid any potential old userspace running the old API to get weird errors. Changing the opcodes will make them fail right away. This is just a precaution, there no upstream users of these interfaces yet and th

[PATCH] staging/android: change IOCTLs opcode after ABI change

From: Gustavo Padovan Burn the old opcode to avoid any potential old userspace running the old API to get weird errors. Changing the opcodes will make them fail right away. This is just a precaution, there no upstream users of these interfaces yet and the only user is Android, but we don't

[PATCH v7 2/5] staging/android: rename SYNC_IOC_FENCE_INFO

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> We don't use the 'fence' name to refer to sync_file anymore. So rename it to SYNC_IOC_FILE_INFO. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

Re: [PATCH] staging/android: add flags member to sync ioctl structs

2016-03-03 Gustavo Padovan <gustavo.pado...@collabora.co.uk>: > Hi Greg, > > 2016-03-03 Greg Kroah-Hartman <gre...@linuxfoundation.org>: > > > On Thu, Mar 03, 2016 at 11:37:17AM -0300, Gustavo Padovan wrote: > > > From: Gustavo Padovan <gustavo.pado.

[PATCH v7 2/5] staging/android: rename SYNC_IOC_FENCE_INFO

From: Gustavo Padovan We don't use the 'fence' name to refer to sync_file anymore. So rename it to SYNC_IOC_FILE_INFO. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/sync.c | 2 +- drivers/staging/android/uapi/sync.h | 2 +- 2 files changed, 2

Re: [PATCH] staging/android: add flags member to sync ioctl structs

2016-03-03 Gustavo Padovan : > Hi Greg, > > 2016-03-03 Greg Kroah-Hartman : > > > On Thu, Mar 03, 2016 at 11:37:17AM -0300, Gustavo Padovan wrote: > > > From: Gustavo Padovan > > > > > > Play safe and add flags member to all structs. So we don'

[PATCH v7 5/5] staging/android: add flags member to sync ioctl structs

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Play safe and add flags member to all structs. So we don't need to break API or create new IOCTL in the future if new features that requires flags arises. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed

[PATCH v7 5/5] staging/android: add flags member to sync ioctl structs

From: Gustavo Padovan Play safe and add flags member to all structs. So we don't need to break API or create new IOCTL in the future if new features that requires flags arises. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- v2: check if flags are valid (zero, in this case

[PATCH v7 3/5] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Maarten Lankhorst <maarten.lankho...@

[PATCH v7 4/5] staging/android: refactor SYNC_IOC_FILE_INFO

From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Change SYNC_IOC_FILE_INFO behaviour to avoid future API breaks and optimize buffer Now num_fences can be filled by the caller to inform how many fences it wants to retrieve from the kernel. If the num_fences passed is greater than zer

[PATCH v7 3/5] staging/android: remove redundant comments on sync_merge_data

From: Gustavo Padovan struct sync_merge_data already have documentation on top of the struct definition. No need to duplicate it. Signed-off-by: Gustavo Padovan Reviewed-by: Maarten Lankhorst --- drivers/staging/android/uapi/sync.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

<    6   7   8   9   10   11   12   13   14   15   >