drm: Fix error handling in drm_mm eviction kselftest
authorChris Wilson <chris@chris-wilson.co.uk>
Tue, 10 Jan 2017 14:40:31 +0000 (14:40 +0000)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 11 Jan 2017 07:53:34 +0000 (08:53 +0100)
commit95b8c64afad824014178df6b396c6ba0f4b1b80a
tree9519aa49a997e6e17073c0478885aeaa3099503b
parentfc36ec76f3f6a6578086b5c045322805df0b9d83
drm: Fix error handling in drm_mm eviction kselftest

        drivers/gpu/drm/selftests/test-drm_mm.c:1277 evict_everything()
        warn: calling list_del() inside list_for_each

The list_del() inside the error handling in the eviction loop is
overkill. We have to undo the eviction scan to return the drm_mm back to
a recoverable state, so have to iterate over the full list, but we only
want to report the error once and once we have an error we can return
early.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: 560b32842912 ("drm: kselftest for drm_mm and eviction")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20170110144031.7609-1-chris@chris-wilson.co.uk
drivers/gpu/drm/selftests/test-drm_mm.c