diff options
| author | zhangdongchuan@eswincomputing.com <zhangdongchuan@eswincomputing.com> | 2025-11-26 06:40:16 +0300 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2025-12-26 23:00:00 +0300 |
| commit | 4b7bf8d5503287ed3bd661207b9d061999ac494e (patch) | |
| tree | c4d168659a1027920b66c694aee1d85db92756fa | |
| parent | 3c4629b68dbe18e454cce4b864c530268cffbeed (diff) | |
| download | linux-4b7bf8d5503287ed3bd661207b9d061999ac494e.tar.xz | |
virtio_ring: code cleanup in detach_buf_split
Since the return value of vring_unmap_one_split() is exactly
vq->split.desc_extra[i].next, 'i = vq->split.desc_extra[i].next' is
redundant. Assign vring_unmap_one_split() to i instead.
Since vq->split.desc_extra is assigned to extra, use extra[i].next
instead of vq->split.desc_extra[i].next to improve readability.
No change in functionality.
Signed-off-by: zhangdongchuan <zhangdongchuan@eswincomputing.com>
Acked-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Message-Id: <202511261140162936986@eswincomputing.com>
| -rw-r--r-- | drivers/virtio/virtio_ring.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c index ddab68959671..560e132f1f19 100644 --- a/drivers/virtio/virtio_ring.c +++ b/drivers/virtio/virtio_ring.c @@ -764,13 +764,12 @@ static void detach_buf_split(struct vring_virtqueue *vq, unsigned int head, i = head; while (vq->split.vring.desc[i].flags & nextflag) { - vring_unmap_one_split(vq, &extra[i]); - i = vq->split.desc_extra[i].next; + i = vring_unmap_one_split(vq, &extra[i]); vq->vq.num_free++; } vring_unmap_one_split(vq, &extra[i]); - vq->split.desc_extra[i].next = vq->free_head; + extra[i].next = vq->free_head; vq->free_head = head; /* Plus final descriptor */ |