[dpdk-dev] [PATCH] net/virtio: init MTU in case no control channel

[Yang, Zhiyong]

> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of zhike wang
> Sent: Friday, January 5, 2018 8:54 AM
> To: dev at dpdk.org
> Cc: zhike wang <wangzhike at jd.com>
> Subject: [dpdk-dev] [PATCH] net/virtio: init MTU in case no control channel
> 
> The max_mtu is kept as zero in case no CRTL channel, which leads to failure
> when calling virtio_mtu_set().
> 
> Signed-off-by: zhike wang <wangzhike at jd.com>
> ---
>  drivers/net/virtio/virtio_ethdev.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
> index 21f2131..b7b3364 100644
> --- a/drivers/net/virtio/virtio_ethdev.c
> +++ b/drivers/net/virtio/virtio_ethdev.c
> @@ -1479,6 +1479,8 @@ static int virtio_dev_xstats_get_names(struct
> rte_eth_dev *dev,
>  	} else {
>  		PMD_INIT_LOG(DEBUG, "config->max_virtqueue_pairs=1");
>  		hw->max_queue_pairs = 1;
> +		hw->max_mtu = VIRTIO_MAX_RX_PKTLEN - ETHER_HDR_LEN -
> +			VLAN_TAG_LEN - hw->vtnet_hdr_size;

Good catch!  It looks good to me.

Acked-by: Zhiyong Yang <zhiyong.yang at intel.com>

Thanks
Zhiyong