> -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of zhike wang > Sent: Friday, January 5, 2018 8:54 AM > To: dev@dpdk.org > Cc: zhike wang <wangzh...@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 <wangzh...@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.y...@intel.com> Thanks Zhiyong
