Re: [dpdk-dev] [PATCH] vhost: remove unnecessary memset for virtio net hdr

[Thomas Monjalon <thomas.monjalon@6wind.com>]

2016-03-17 01:19, Xie, Huawei:
> On 3/16/2016 2:44 PM, Yuanhan Liu wrote:
> > We have to reset the virtio net hdr at virtio_enqueue_offload()
> > before, due to all mbufs share a single virtio_hdr structure:
> >
> > 	struct virtio_net_hdr_mrg_rxbuf virtio_hdr = {{0, }, 0};
> >
> > 	foreach (mbuf) {
> > 		virtio_enqueue_offload(mbuf, &virtio_hdr.hdr);
> >
> > 		copy net hdr and mbuf to desc buf
> > 	}
> >
> > However, after the vhost rxtx refactor, the code looks like:
> >
> > 	copy_mbuf_to_desc(mbuf)
> > 	{
> > 		struct virtio_net_hdr_mrg_rxbuf virtio_hdr = {{0, }, 0}
> >
> > 		virtio_enqueue_offload(mbuf, &virtio_hdr.hdr);
> >
> > 		copy net hdr and mbuf to desc buf
> > 	}
> >
> > 	foreach (mbuf) {
> > 		copy_mbuf_to_desc(mbuf);
> > 	}
> >
> > Therefore, the memset at virtio_enqueue_offload() is not necessary
> > any more; remove it.
> >
> > Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
> 
> Acked-by: Huawei Xie <huawei.xie@intel.com>

Applied, thanks