Revert "Revert "drm/i810: cleanup reclaim_buffers""
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 11 Jun 2012 19:50:23 +0000 (21:50 +0200)
committerDave Airlie <airlied@redhat.com>
Fri, 20 Jul 2012 02:49:49 +0000 (22:49 -0400)
commitd5346b3727bad3a033cb0a5b45a4a2da45119ec3
treecbf3fb93d8478d03d1bc14434bcedb5ea0eb3f24
parent3ae6b64400cc92530bcab73d13c6e1b7a5cfd915
Revert "Revert "drm/i810: cleanup reclaim_buffers""

This reverts commit 6e877b576ddf7cde5db2e9a6dcb56fef0ea77e64,
reinstating the original commit:

commit 87499ffdcb1c70f66988cd8febc4ead0ba2f9118
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Tue Oct 25 23:51:24 2011 +0200

    drm/i810: cleanup reclaim_buffers

    My dear old i815 always hits the deadlocked on reclaim_buffers
    warning. Switch over to the idlelock duct-tape on hope that
    works better. I've fired up my i815 and now closing glxgears doesn't
    take 5 seconds anymore. \o/

The original problem with that was that I've moved it ahead in the
series so that it could be included despite some patches not being
ready quite yet. The little problem is that this patch required some
of the previous rework to work correctly.

Now that everything is in the right order again, this actually works
on my i810 and does speed up closing gl apps as the original commit
claimed. Without hanging the machine, as the revert says.

Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/i810/i810_dma.c
drivers/gpu/drm/i810/i810_drv.c
drivers/gpu/drm/i810/i810_drv.h