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@xxxxxxxxxxxxxxxxxx> > Cc: Sean Paul <seanpaul@xxxxxxxxxxxx> > Cc: Zach Reizner <zachr@xxxxxxxxxx> > Cc: Matthew Auld <matthew.auld@xxxxxxxxx> > Tested-by: Humberto Israel Perez Rodriguez <humberto.i.perez.rodriguez@xxxxxxxxx> > Reviewed-by: Matthew Auld <matthew.auld@xxxxxxxxx> 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
