diff options
author | Ben Skeggs <bskeggs@redhat.com> | 2012-05-04 06:25:47 (GMT) |
---|---|---|
committer | Ben Skeggs <bskeggs@redhat.com> | 2012-05-24 06:55:58 (GMT) |
commit | 906c033e276877c1374c9159976b05746af3c86d (patch) | |
tree | f7e03b7148cf9f5caad6ac72b33e0354e94aad41 | |
parent | 299bee10fb228fce4a3fc5dd89f32787a6e58fe5 (diff) | |
download | linux-fsl-qoriq-906c033e276877c1374c9159976b05746af3c86d.tar.xz |
drm/nouveau/fence: fix a race where fence->channel can disappear
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
-rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_fence.c | 14 | ||||
-rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_fence.h | 3 | ||||
-rw-r--r-- | drivers/gpu/drm/nouveau/nv04_fence.c | 3 | ||||
-rw-r--r-- | drivers/gpu/drm/nouveau/nv10_fence.c | 8 | ||||
-rw-r--r-- | drivers/gpu/drm/nouveau/nv84_fence.c | 8 | ||||
-rw-r--r-- | drivers/gpu/drm/nouveau/nvc0_fence.c | 5 |
6 files changed, 26 insertions, 15 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.c b/drivers/gpu/drm/nouveau/nouveau_fence.c index 4ba41a4..3c18049 100644 --- a/drivers/gpu/drm/nouveau/nouveau_fence.c +++ b/drivers/gpu/drm/nouveau/nouveau_fence.c @@ -147,15 +147,19 @@ nouveau_fence_wait(struct nouveau_fence *fence, bool lazy, bool intr) int nouveau_fence_sync(struct nouveau_fence *fence, struct nouveau_channel *chan) { - struct nouveau_channel *prev = fence ? fence->channel : NULL; struct drm_device *dev = chan->dev; struct nouveau_fence_priv *priv = nv_engine(dev, NVOBJ_ENGINE_FENCE); + struct nouveau_channel *prev; int ret = 0; - if (unlikely(prev && prev != chan && !nouveau_fence_done(fence))) { - ret = priv->sync(fence, chan); - if (unlikely(ret)) - ret = nouveau_fence_wait(fence, true, false); + prev = fence ? nouveau_channel_get_unlocked(fence->channel) : NULL; + if (prev) { + if (unlikely(prev != chan && !nouveau_fence_done(fence))) { + ret = priv->sync(fence, prev, chan); + if (unlikely(ret)) + ret = nouveau_fence_wait(fence, true, false); + } + nouveau_channel_put_unlocked(&prev); } return ret; diff --git a/drivers/gpu/drm/nouveau/nouveau_fence.h b/drivers/gpu/drm/nouveau/nouveau_fence.h index ec9afa7..82ba733 100644 --- a/drivers/gpu/drm/nouveau/nouveau_fence.h +++ b/drivers/gpu/drm/nouveau/nouveau_fence.h @@ -34,7 +34,8 @@ struct nouveau_fence_chan { struct nouveau_fence_priv { struct nouveau_exec_engine engine; int (*emit)(struct nouveau_fence *); - int (*sync)(struct nouveau_fence *, struct nouveau_channel *); + int (*sync)(struct nouveau_fence *, struct nouveau_channel *, + struct nouveau_channel *); u32 (*read)(struct nouveau_channel *); }; diff --git a/drivers/gpu/drm/nouveau/nv04_fence.c b/drivers/gpu/drm/nouveau/nv04_fence.c index 08bd2ce..abe89db 100644 --- a/drivers/gpu/drm/nouveau/nv04_fence.c +++ b/drivers/gpu/drm/nouveau/nv04_fence.c @@ -51,7 +51,8 @@ nv04_fence_emit(struct nouveau_fence *fence) } static int -nv04_fence_sync(struct nouveau_fence *fence, struct nouveau_channel *chan) +nv04_fence_sync(struct nouveau_fence *fence, + struct nouveau_channel *prev, struct nouveau_channel *chan) { return -ENODEV; } diff --git a/drivers/gpu/drm/nouveau/nv10_fence.c b/drivers/gpu/drm/nouveau/nv10_fence.c index 10831ea..8a1b750 100644 --- a/drivers/gpu/drm/nouveau/nv10_fence.c +++ b/drivers/gpu/drm/nouveau/nv10_fence.c @@ -52,17 +52,19 @@ nv10_fence_emit(struct nouveau_fence *fence) return ret; } + static int -nv10_fence_sync(struct nouveau_fence *fence, struct nouveau_channel *chan) +nv10_fence_sync(struct nouveau_fence *fence, + struct nouveau_channel *prev, struct nouveau_channel *chan) { return -ENODEV; } static int -nv17_fence_sync(struct nouveau_fence *fence, struct nouveau_channel *chan) +nv17_fence_sync(struct nouveau_fence *fence, + struct nouveau_channel *prev, struct nouveau_channel *chan) { struct nv10_fence_priv *priv = nv_engine(chan->dev, NVOBJ_ENGINE_FENCE); - struct nouveau_channel *prev = fence->channel; u32 value; int ret; diff --git a/drivers/gpu/drm/nouveau/nv84_fence.c b/drivers/gpu/drm/nouveau/nv84_fence.c index d23dbc0..0ac98c0 100644 --- a/drivers/gpu/drm/nouveau/nv84_fence.c +++ b/drivers/gpu/drm/nouveau/nv84_fence.c @@ -55,16 +55,18 @@ nv84_fence_emit(struct nouveau_fence *fence) return ret; } + static int -nv84_fence_sync(struct nouveau_fence *fence, struct nouveau_channel *chan) +nv84_fence_sync(struct nouveau_fence *fence, + struct nouveau_channel *prev, struct nouveau_channel *chan) { int ret = RING_SPACE(chan, 7); if (ret == 0) { BEGIN_NV04(chan, 0, NV11_SUBCHAN_DMA_SEMAPHORE, 1); OUT_RING (chan, NvSema); BEGIN_NV04(chan, 0, NV84_SUBCHAN_SEMAPHORE_ADDRESS_HIGH, 4); - OUT_RING (chan, upper_32_bits(fence->channel->id * 16)); - OUT_RING (chan, lower_32_bits(fence->channel->id * 16)); + OUT_RING (chan, upper_32_bits(prev->id * 16)); + OUT_RING (chan, lower_32_bits(prev->id * 16)); OUT_RING (chan, fence->sequence); OUT_RING (chan, NV84_SUBCHAN_SEMAPHORE_TRIGGER_ACQUIRE_GEQUAL); FIRE_RING (chan); diff --git a/drivers/gpu/drm/nouveau/nvc0_fence.c b/drivers/gpu/drm/nouveau/nvc0_fence.c index 41545f1..817228c 100644 --- a/drivers/gpu/drm/nouveau/nvc0_fence.c +++ b/drivers/gpu/drm/nouveau/nvc0_fence.c @@ -60,10 +60,11 @@ nvc0_fence_emit(struct nouveau_fence *fence) } static int -nvc0_fence_sync(struct nouveau_fence *fence, struct nouveau_channel *chan) +nvc0_fence_sync(struct nouveau_fence *fence, + struct nouveau_channel *prev, struct nouveau_channel *chan) { struct nvc0_fence_chan *fctx = chan->engctx[NVOBJ_ENGINE_FENCE]; - u64 addr = fctx->vma.offset + fence->channel->id * 16; + u64 addr = fctx->vma.offset + prev->id * 16; int ret; ret = RING_SPACE(chan, 5); |