diff options
author | Shirley Ma <mashirle@us.ibm.com> | 2010-01-18 08:45:23 -0500 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2010-02-23 22:52:27 -0500 |
commit | c021eac4148c16bf53baa0dd14e8ebee6f39dab5 (patch) | |
tree | 558db2ede74488606614d6e7e277f9914e38fbc1 /drivers/virtio | |
parent | 69740c8ba878f58bc3c71f74618fc2cd1da990da (diff) |
virtio: Add ability to detach unused buffers from vrings
There's currently no way for a virtio driver to ask for unused
buffers, so it has to keep a list itself to reclaim them at shutdown.
This is redundant, since virtio_ring stores that information. So
add a new hook to do this.
Signed-off-by: Shirley Ma <xma@us.ibm.com>
Signed-off-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'drivers/virtio')
-rw-r--r-- | drivers/virtio/virtio_ring.c | 25 |
1 files changed, 25 insertions, 0 deletions
diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c index 827f7e042610..782b7292a3d8 100644 --- a/drivers/virtio/virtio_ring.c +++ b/drivers/virtio/virtio_ring.c | |||
@@ -351,6 +351,30 @@ static bool vring_enable_cb(struct virtqueue *_vq) | |||
351 | return true; | 351 | return true; |
352 | } | 352 | } |
353 | 353 | ||
354 | static void *vring_detach_unused_buf(struct virtqueue *_vq) | ||
355 | { | ||
356 | struct vring_virtqueue *vq = to_vvq(_vq); | ||
357 | unsigned int i; | ||
358 | void *buf; | ||
359 | |||
360 | START_USE(vq); | ||
361 | |||
362 | for (i = 0; i < vq->vring.num; i++) { | ||
363 | if (!vq->data[i]) | ||
364 | continue; | ||
365 | /* detach_buf clears data, so grab it now. */ | ||
366 | buf = vq->data[i]; | ||
367 | detach_buf(vq, i); | ||
368 | END_USE(vq); | ||
369 | return buf; | ||
370 | } | ||
371 | /* That should have freed everything. */ | ||
372 | BUG_ON(vq->num_free != vq->vring.num); | ||
373 | |||
374 | END_USE(vq); | ||
375 | return NULL; | ||
376 | } | ||
377 | |||
354 | irqreturn_t vring_interrupt(int irq, void *_vq) | 378 | irqreturn_t vring_interrupt(int irq, void *_vq) |
355 | { | 379 | { |
356 | struct vring_virtqueue *vq = to_vvq(_vq); | 380 | struct vring_virtqueue *vq = to_vvq(_vq); |
@@ -377,6 +401,7 @@ static struct virtqueue_ops vring_vq_ops = { | |||
377 | .kick = vring_kick, | 401 | .kick = vring_kick, |
378 | .disable_cb = vring_disable_cb, | 402 | .disable_cb = vring_disable_cb, |
379 | .enable_cb = vring_enable_cb, | 403 | .enable_cb = vring_enable_cb, |
404 | .detach_unused_buf = vring_detach_unused_buf, | ||
380 | }; | 405 | }; |
381 | 406 | ||
382 | struct virtqueue *vring_new_virtqueue(unsigned int num, | 407 | struct virtqueue *vring_new_virtqueue(unsigned int num, |