[PATCH drm 4/6] drm: add drm_gem_prime_page_flip() helper

2015-10-23 Thread Daniel Stone
Hi, On 22 October 2015 at 21:25, Daniel Vetter wrote: > Maarten Lankhorst had patches to do that for i915, but they just didn't go > anywhere since i915 gem locking is a bit ... antique. But if your goal is > to only fix up the page_flip path, and only for i915 as the display > driver, then the o

[PATCH drm 4/6] drm: add drm_gem_prime_page_flip() helper

2015-10-23 Thread Daniel Vetter
On Thu, Oct 22, 2015 at 11:31 PM, Alexander Goins wrote: >>This looks really strange - you force-complete the fence already attached >>(which might be from any driver attached to this dma-buf)? >>The creator of the fence is supposed to complete it when whatever async >>operation that is schedule

[PATCH drm 4/6] drm: add drm_gem_prime_page_flip() helper

2015-10-22 Thread Daniel Vetter
On Thu, Oct 22, 2015 at 01:00:57PM -0700, Alex Goins wrote: > From: agoins > > Adds drm_gem_prime_page_flip(), a helper implementation of > prime_page_flip() to be used by DRM drivers. > > Signed-off-by: Alex Goins > --- > drivers/gpu/drm/drm_prime.c | 147 > ++

[PATCH drm 4/6] drm: add drm_gem_prime_page_flip() helper

2015-10-22 Thread Alexander Goins
Thanks for the quick feedback, Daniel! >This looks really strange - you force-complete the fence already attached >(which might be from any driver attached to this dma-buf)? >The creator of the fence is supposed to complete it when whatever async >operation that is scheduled and which is using t

[PATCH drm 4/6] drm: add drm_gem_prime_page_flip() helper

2015-10-22 Thread Alex Goins
From: agoins Adds drm_gem_prime_page_flip(), a helper implementation of prime_page_flip() to be used by DRM drivers. Signed-off-by: Alex Goins --- drivers/gpu/drm/drm_prime.c | 147 include/drm/drmP.h | 3 + 2 files changed, 150 insertion