On Thu, 9 Feb 2012 00:19:27 +0100, Ben Widawsky <[email protected]> wrote: > From: Adam Jackson <[email protected]> > +static void vgem_gem_free_object(struct drm_gem_object *obj) > +{ > + if (obj->map_list.map) > + drm_gem_free_mmap_offset(obj); > + > + drm_gem_object_release(obj); > +}
drm_gem_free_mmap_offset() should be pushed down into drm_gem_object_release(). Out of the 6 distinct users of drm_gem_object_release(), 3 call drm_gem_free_mmap_offset() just prior and gma500 forgets to. I do accept that it is a separate patch though ;-) -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ dri-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/dri-devel
