[02/11] vhost: fix build with clang 15
Checks
Commit Message
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(-)
Comments
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
@@ -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);