On Tue, May 21, 2019 at 02:40:22PM +0200, Daniel Vetter wrote:
On Tue, May 21, 2019 at 01:08:28PM +0200, Thomas Zimmermann wrote:
Replacing drm_gem_vram_push_to_system() moves policy from drivers back to the memory manager. Now, unused BOs are only evicted when the space is required.
The lock/unlock-renaming patch aligns the interface with other names in DRM. No functional changes are done.
Finally, there's now a lockdep assert that ensures we don't call the GEM VRAM _locked() functions with an unlocked BO.
Patches are against a recent drm-tip and tested on mgag200 and ast HW.
Thomas Zimmermann (3): drm: Replace drm_gem_vram_push_to_system() with kunmap + unpin drm: Rename reserve/unreserve to lock/unlock in GEM VRAM helpers drm: Assert that BO is locked in drm_gem_vram_{pin,unpin}_locked()
Awesome, thanks a lot for quickly working on this. On the series:
Acked-by: Daniel Vetter daniel.vetter@ffwll.ch
But definitely get someone with more knowledge of the details to check this all again.
Done & pushed to drm-misc-next.
thanks, Gerd