Re: [Intel-gfx] [PATCH] drm/i915: Deny wrapping an userptr into a framebuffer

2015-10-13 Thread Tvrtko Ursulin
and display server, where the lifetime and hardware access is even more confusing. Deny it. Reviewed-by: Tvrtko Ursulin <tvrtko.ursu...@intel.com> Regards, Tvrtko -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majord...@vger

Re: [Intel-gfx] [PATCH] drm/i915: Deny wrapping an userptr into a framebuffer

2015-10-13 Thread Tvrtko Ursulin
On 13/10/15 15:00, Tvrtko Ursulin wrote: On 13/10/15 14:22, Chris Wilson wrote: Pinning a userptr onto the hardware raises interesting questions about the lifetime of such a surface as the framebuffer extends that life beyond the client's address space. That is the hardware will need to keep

[PATCH] drm: Use userspace compatible type in fourcc_mod_code macro

2015-09-23 Thread Tvrtko Ursulin
From: Tvrtko Ursulin <tvrtko.ursu...@intel.com> __u64 should be used instead of u64. Feature originally added in: commit e3eb3250d84ef97b766312345774367b6a310db8 Author: Rob Clark <robdcl...@gmail.com> Date: Thu Feb 5 14:41:52 2015 + drm: add support for tiled/compressed/

Re: [Intel-gfx] [PATCH v3 2/3] drm/i915: Fix userptr deadlock with aliased GTT mmappings

2015-09-10 Thread Tvrtko Ursulin
On 09/09/2015 04:03 PM, Chris Wilson wrote: On Wed, Sep 09, 2015 at 02:56:16PM +0100, Tvrtko Ursulin wrote: Hi, On 08/10/2015 09:51 AM, Chris Wilson wrote: +out: drm_free_large(pvec); return ret; + +err: + /* No pages here, no need for the mmu-notifier to wake us

Re: [Intel-gfx] [PATCH v4 2/3] drm/i915: Fix userptr deadlock with aliased GTT mmappings

2015-09-10 Thread Tvrtko Ursulin
On 09/10/2015 11:25 AM, Tvrtko Ursulin wrote: + +active = false; +if (pinned < 0) +ret = pinned; +else if (pinned < num_pages) +ret = __i915_gem_userptr_get_pages_schedule(obj, ); +else ret = __i915_gem_userptr_set_pages(obj, pvec, num

Re: [Intel-gfx] [PATCH v4 2/3] drm/i915: Fix userptr deadlock with aliased GTT mmappings

2015-09-10 Thread Tvrtko Ursulin
: Michał Winiarski <michal.winiar...@intel.com> Testcase: igt/gem_userptr_blits/map-fixed* Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Michał Winiarski <michal.winiar...@intel.com> Cc: Tvrtko Ursulin <tvrtko.ursu...@intel.com> Cc: stable@vger.kernel.org --- dri

Re: [Intel-gfx] [PATCH v3 2/3] drm/i915: Fix userptr deadlock with aliased GTT mmappings

2015-09-09 Thread Tvrtko Ursulin
t; v4: Take the active->flag early so we know in the mmu-notifier when we > have to cancel a pending gup-worker. > > Reported-by: Michał Winiarski <michal.winiar...@intel.com> > Testcase: igt/gem_userptr_blits/map-fixed* > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>

[PATCH 2/2 v3] drm: Provide compat ioctl for addfb2.1

2015-07-15 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com Frame buffer modifiers extensions provided in; commit e3eb3250d84ef97b766312345774367b6a310db8 Author: Rob Clark robdcl...@gmail.com Date: Thu Feb 5 14:41:52 2015 + drm: add support for tiled/compressed/etc modifier in addfb2

[PATCH 2/2] drm: Provide compat ioctl for addfb2.1

2015-07-14 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com Frame buffer modifiers extensions provided in; commit e3eb3250d84ef97b766312345774367b6a310db8 Author: Rob Clark robdcl...@gmail.com Date: Thu Feb 5 14:41:52 2015 + drm: add support for tiled/compressed/etc modifier in addfb2

[PATCH 1/2] drm/i915: Forward all core DRM ioctls to core compat handling

2015-07-14 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com Previously only core DRM ioctls under the DRM_COMMAND_BASE were being forwarded, but the drm.h header suggests (and reality confirms) ones after (and including) DRM_COMMAND_END should be forwarded as well. Signed-off-by: Tvrtko Ursulin tvrtko.ursu

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Fix userptr deadlock with MAP_FIXED

2015-07-01 Thread Tvrtko Ursulin
obj-pages. Reported-by: Michał Winiarski michal.winiar...@intel.com Testcase: igt/gem_userptr_blits/map-fixed* Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Cc: Michał Winiarski michal.winiar...@intel.com Cc: Tvrtko Ursulin tvrtko.ursu...@intel.com Cc: stable@vger.kernel.org --- drivers/gpu

Re: [Intel-gfx] [PATCH v2] drm/i915: Fix userptr deadlock with MAP_FIXED

2015-06-30 Thread Tvrtko Ursulin
michal.winiar...@intel.com Testcase: igt/gem_userptr_blits/map-fixed* Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Cc: Michał Winiarski michal.winiar...@intel.com Cc: Tvrtko Ursulin tvrtko.ursu...@intel.com Cc: stable@vger.kernel.org --- drivers/gpu/drm/i915/i915_gem_userptr.c | 43

Re: [Intel-gfx] [PATCH] drm/i915: Always reset vma-ggtt_view.pages cache on unbinding

2015-06-11 Thread Tvrtko Ursulin
Ursulin tvrtko.ursu...@intel.com Date: Wed Dec 10 17:27:58 2014 + drm/i915: Infrastructure for supporting different GGTT views per object Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1227892 Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Cc: Tvrtko Ursulin tvrtko.ursu

Re: [Intel-gfx] [PATCH] drm/i915: Do not store the error pointer for a failed userptr registration

2014-09-26 Thread Tvrtko Ursulin
; } static int Reviewed-by: Tvrtko Ursulin tvrtko.ursu...@intel.com -- To unsubscribe from this list: send the line unsubscribe stable in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html