diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2010-02-01 15:59:17 +0300 |
|---|---|---|
| committer | Eric Anholt <eric@anholt.net> | 2010-02-22 19:54:42 +0300 |
| commit | 10ae9bd25acf394c8fa2f9d795dfa9cec4d19ed6 (patch) | |
| tree | d5d67859fab5c64963551a37f95228dde93287ad /drivers/gpu/drm/i915/i915_gem_tiling.c | |
| parent | 4a7266123fce399f695b62b7f87b467b317f1487 (diff) | |
| download | linux-10ae9bd25acf394c8fa2f9d795dfa9cec4d19ed6.tar.xz | |
drm/i915: blow away userspace mappings before fence change
This aligns it with the other user of i915_gem_clear_fence_reg,
which blows away the mapping before changing the fence reg.
Only affects userspace if it races against itself when changing
tiling parameters, i.e. behaviour is undefined, anyway.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Eric Anholt <eric@anholt.net>
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem_tiling.c')
| -rw-r--r-- | drivers/gpu/drm/i915/i915_gem_tiling.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem_tiling.c b/drivers/gpu/drm/i915/i915_gem_tiling.c index f308fdf43725..b0cbe3a62f84 100644 --- a/drivers/gpu/drm/i915/i915_gem_tiling.c +++ b/drivers/gpu/drm/i915/i915_gem_tiling.c @@ -371,12 +371,6 @@ i915_gem_set_tiling(struct drm_device *dev, void *data, goto err; } - /* If we've changed tiling, GTT-mappings of the object - * need to re-fault to ensure that the correct fence register - * setup is in place. - */ - i915_gem_release_mmap(obj); - obj_priv->tiling_mode = args->tiling_mode; obj_priv->stride = args->stride; } |