Re: [PATCH 02/11] vhost: fix build with clang 15

[Maxime Coquelin <maxime.coquelin@redhat.com>]

On 11/18/22 09:53, David Marchand wrote:
> This variable is not used.
> 
> Fixes: abeb86525577 ("vhost: remove copy threshold for async path")
> Cc: stable@dpdk.org
> 
> Signed-off-by: David Marchand <david.marchand@redhat.com>
> ---
>   lib/vhost/virtio_net.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/lib/vhost/virtio_net.c b/lib/vhost/virtio_net.c
> index 4358899718..9abf752f30 100644
> --- a/lib/vhost/virtio_net.c
> +++ b/lib/vhost/virtio_net.c
> @@ -1877,7 +1877,6 @@ virtio_dev_rx_async_submit_packed(struct virtio_net *dev, struct vhost_virtqueue
>   	struct rte_mbuf **pkts, uint32_t count, int16_t dma_id, uint16_t vchan_id)
>   {
>   	uint32_t pkt_idx = 0;
> -	uint32_t remained = count;
>   	uint16_t n_xfer;
>   	uint16_t num_buffers;
>   	uint16_t num_descs;
> @@ -1903,7 +1902,6 @@ virtio_dev_rx_async_submit_packed(struct virtio_net *dev, struct vhost_virtqueue
>   		pkts_info[slot_idx].mbuf = pkts[pkt_idx];
>   
>   		pkt_idx++;
> -		remained--;
>   		vq_inc_last_avail_packed(vq, num_descs);
>   	} while (pkt_idx < count);
>   

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime