drm/i915: Fix use of uninitialized var in 40a5f0de
authorFlorian Mickler <florian@mickler.org>
Mon, 6 Apr 2009 20:55:41 +0000 (22:55 +0200)
committerEric Anholt <eric@anholt.net>
Wed, 8 Apr 2009 17:18:19 +0000 (10:18 -0700)
commit2bc43b5cf5158a26fa1328234795abed2dff5275
tree0965b8d142a1a1c70d65be61f5478807fb8daff0
parent5b40f871158da7aaccff442645dae8b97c2e4d50
drm/i915: Fix use of uninitialized var in 40a5f0de

i915_gem_put_relocs_to_user returned an uninitialized value which
got returned to userspace. This caused libdrm in my setup to never
get out of a do{}while() loop retrying i915_gem_execbuffer.

result was hanging X, overheating of cpu and 2-3gb of logfile-spam.

This patch adresses the issue by
 1. initializing vars in this file where necessary
 2. correcting wrongly interpreted return values of copy_[from/to]_user

Signed-off-by: Florian Mickler <florian@mickler.org>
[anholt: cleanups of unnecessary changes, consistency in APIs]
Signed-off-by: Eric Anholt <eric@anholt.net>
drivers/gpu/drm/i915/i915_gem.c