[PATCH 1/3] drm/vgem: Fix mmaping

Chris Wilson chris at chris-wilson.co.uk
Sat Jun 25 10:39:24 UTC 2016


On Thu, Jun 23, 2016 at 03:35:32PM +0100, Chris Wilson wrote:
> The vGEM mmap code has bitrotted slightly and now immediately BUGs.
> Since vGEM was last updated, there are new core GEM facilities to
> provide more common functions, so let's use those here.
> 
> v2: drm_gem_free_mmap_offset() is performed from
> drm_gem_object_release() so we can remove the redundant call.
> 
> Testcase: igt/vgem_basic/mmap
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96603
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Sean Paul <seanpaul at chromium.org>
> Cc: Zach Reizner <zachr at google.com>
> Cc: Matthew Auld <matthew.auld at intel.com>
> Tested-by: Humberto Israel Perez Rodriguez <humberto.i.perez.rodriguez at intel.com>
> Reviewed-by: Matthew Auld <matthew.auld at intel.com>

Hi Thierry, Daniel recommended to bug you for stewardship of drm-misc in
his absence. Could you please consider this patch? Our CI is now
tripping over this issue.

Sean or Zach could you please ack? And have a look at the following pair
of dma-buf fence patches? I have tests ready for our CI that use the
vgem dma-buf to demonstrate the lack of synchronisation in i915.ko (and
so exercise the fixes without requiring a second piece of hardware).
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the dri-devel mailing list