i965: Remove clearing of bo->map_gtt after failure
authorChris Wilson <chris@chris-wilson.co.uk>
Sat, 1 Jul 2017 10:14:00 +0000 (11:14 +0100)
committerKenneth Graunke <kenneth@whitecape.org>
Sat, 1 Jul 2017 18:46:34 +0000 (11:46 -0700)
commitd8382d6889d139db8f7f720aaba4cdfb0a0c8f11
treea3bd178f5ef70c635e36daf58d33503deeead752
parentf78aa2c9864d26e50df0729b94deb24701d95871
i965: Remove clearing of bo->map_gtt after failure

With the conversion to storing the result of drm_mmap to a local and not
directly to bo->map_gtt itself, we no longer should clear bo->map_gtt.
In the best the operation is redundant as we know bo->map_gtt to already
be NULL, but in the worst case we overwrite a concurrent thread that
successfully mmaped the GTT.

Fixes: 314647c4c206 ("i965: Drop global bufmgr lock from brw_bo_map_* functions.")
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_bufmgr.c