Re: [PATCH] tools/virtio: Remove unsupported --batch option from vhost_net_test

[Eugenio Perez Martin]

On Thu, Jun 18, 2026 at 12:03 PM Yichong Chen <chenyichong@xxxxxxxxxxxxx> wrote:
>
> vhost_net_test has --batch in longopts, but not in help.
>
> The parser never handles 'b', so --batch hits assert(0).
>
> Remove the unsupported option.
>
> Signed-off-by: Yichong Chen <chenyichong@xxxxxxxxxxxxx>

Acked-by: Eugenio Pérez <eperezma@xxxxxxxxxx>

Thanks!

> ---
>  tools/virtio/vhost_net_test.c | 5 -----
>  1 file changed, 5 deletions(-)
>
> diff --git a/tools/virtio/vhost_net_test.c b/tools/virtio/vhost_net_test.c
> index 389d99a6d7c7..566e15420bb6 100644
> --- a/tools/virtio/vhost_net_test.c
> +++ b/tools/virtio/vhost_net_test.c
> @@ -450,11 +450,6 @@ static const struct option longopts[] = {
>                 .val = 'n',
>                 .has_arg = required_argument,
>         },
> -       {
> -               .name = "batch",
> -               .val = 'b',
> -               .has_arg = required_argument,
> -       },
>         {
>         }
>  };
> --
> 2.51.0
>