[PATCH 01/11] virtio_ring: Merge identical variables

Heinrich Schuchardt xypron.glpk at gmx.de
Thu Apr 7 09:03:16 CEST 2022


On 3/31/22 12:09, Andrew Scull wrote:
> The variables `total_sg` and `descs_used` have the same value. Replace
> the few uses of `total_sg` with `descs_used` to simplify the situation.
>
> Signed-off-by: Andrew Scull <ascull at google.com>

Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>

> ---
>   drivers/virtio/virtio_ring.c | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c
> index 7f1cbc5932..a6922ce1b8 100644
> --- a/drivers/virtio/virtio_ring.c
> +++ b/drivers/virtio/virtio_ring.c
> @@ -20,17 +20,16 @@ int virtqueue_add(struct virtqueue *vq, struct virtio_sg *sgs[],
>   		  unsigned int out_sgs, unsigned int in_sgs)
>   {
>   	struct vring_desc *desc;
> -	unsigned int total_sg = out_sgs + in_sgs;
> -	unsigned int i, n, avail, descs_used, uninitialized_var(prev);
> +	unsigned int descs_used = out_sgs + in_sgs;
> +	unsigned int i, n, avail, uninitialized_var(prev);
>   	int head;
>
> -	WARN_ON(total_sg == 0);
> +	WARN_ON(descs_used == 0);
>
>   	head = vq->free_head;
>
>   	desc = vq->vring.desc;
>   	i = head;
> -	descs_used = total_sg;
>
>   	if (vq->num_free < descs_used) {
>   		debug("Can't add buf len %i - avail = %i\n",



More information about the U-Boot mailing list