Re: [PATCH 1/3] net/tap: add tap_fd_get_mtu() to query host tap MTU

["Eugenio Pérez" <eperezma@redhat.com>]

From: Eugenio Perez Martin <eperezma@redhat.com>


Good feature! I'm sorry, all of the series fell through the cracks.

On Sun, 16 Aug 2026 at 07:34:49 +0300, Nariman Sayed wrote:
> +    if (n->net_conf.mtu_from_tap) {
> +        NetClientState *tap_peer = n->nic_conf.peers.ncs[0];
> +        if (tap_peer && tap_peer->info->type == NET_CLIENT_DRIVER_TAP) {
> +            int mtu = tap_get_mtu(tap_peer);
> +            if (mtu > 0) {
> +                n->net_conf.mtu = mtu;
> +                n->host_features |= (1ULL << VIRTIO_NET_F_MTU);
> +            }
> +        }
> +    }

The series looks good, but this part should not check for the backend type.
Instead, rename the callback as get_mtu and call peer->get_mtu, so if we add
the feature to other backends like vhost-user or vhost-vdpa this part of the
code does not need to be modified.

Good addition by the way :).

Thanks!