Re: [PATCH] virtio_ring: code cleanup in detach_buf_split

From: Jason Wang

Date: Thu Nov 27 2025 - 21:24:51 EST


On Wed, Nov 26, 2025 at 11:41 AM zhangdongchuan@xxxxxxxxxxxxxxxxxx
<zhangdongchuan@xxxxxxxxxxxxxxxxxx> wrote:
>
> 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@xxxxxxxxxxxxxxxxxx>
> ---

Acked-by: Jason Wang <jasowang@xxxxxxxxxx>

Thanks