vhost: remove unnecessary memset when enqueueing
authorYuanhan Liu <yuanhan.liu@linux.intel.com>
Wed, 16 Mar 2016 06:44:56 +0000 (14:44 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 17 Mar 2016 20:53:06 +0000 (21:53 +0100)
commitb3869ebebfb1267d81bc4cdc8b12c0d0d484a03d
tree1573f7ce47eaba52b6155b4316ea10d4090ed901
parent9f8d48e56931b0bd145534f5e145fff7e8ef598d
vhost: remove unnecessary memset when enqueueing

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>
lib/librte_vhost/vhost_rxtx.c