> -----Original Message----- > From: dev [mailto:[email protected]] On Behalf Of zhike wang > Sent: Friday, January 5, 2018 8:54 AM > To: [email protected] > Cc: zhike wang <[email protected]> > 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 <[email protected]> > --- > 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 <[email protected]> Thanks Zhiyong
