Re: [PATCH] drm/i915: use drm vma accounting functions to make sure VMs don't get lost

[Jesse B. <jb...@vi...>]

On Monday, December 29, 2008 1:08 am Dave Airlie wrote:
> From: Dave Airlie <ai...@li...>
>
> The gem gtt mapping code didn't use the drm_vm.c accounting open/close,
> it did call the initial drm_vm_open_locked, so it should always do this.
>
> I'm not 100% sure it doesn't need a special open/close pair, but this at
> least makes /proc/dri/0/vma not end up with lots of crap in it. I'm still
> not getting kms + EXA on i915 working at all.
>
> Signed-off-by: Dave Airlie <ai...@re...>

At first I thought you meant core VM accounting, but this is purely DRM code.  
Anyway, it looks fine, you can add my 
Acked-by: Jesse Barnes <jb...@vi...>

If you want to use EXA and not UXA you'll want my EXA pixmap patch (old 
version attached, refreshing now to current xf86-video-intel).

Thanks,
-- 
Jesse Barnes, Intel Open Source Technology Center