[dpdk-dev] [PATCH 4/4] virtio/vdev: add a new vdev named eth_cvio

2016-01-27 Thread Qiu, Michael
On 1/11/2016 2:43 AM, Tan, Jianfeng wrote: > Add a new virtual device named eth_cvio, it can be used just like > eth_ring, eth_null, etc. > > Configured parameters include: > - rx (optional, 1 by default): number of rx, only allowed to be > 1 for now. > -

[dpdk-dev] [PATCH 4/4] virtio/vdev: add a new vdev named eth_cvio

2016-01-12 Thread Tan, Jianfeng
Hi Fedin, On 1/12/2016 4:39 PM, Tan, Jianfeng wrote: > Hi Fedin, > > On 1/12/2016 3:45 PM, Pavel Fedin wrote: >> Hello! >> >> See inline >> >>> ... >>> } >>> >>> +struct rte_mbuf *m = NULL; >>> +if (dev->dev_type == RTE_ETH_DEV_PCI) >>> +vq->offset =

[dpdk-dev] [PATCH 4/4] virtio/vdev: add a new vdev named eth_cvio

2016-01-12 Thread Tan, Jianfeng
Hi Fedin, On 1/12/2016 3:45 PM, Pavel Fedin wrote: > Hello! > > See inline > >> ... >> } >> >> +struct rte_mbuf *m = NULL; >> +if (dev->dev_type == RTE_ETH_DEV_PCI) >> +vq->offset = (uintptr_t)>buf_addr; >> +#ifdef RTE_VIRTIO_VDEV >> +else { >> +

[dpdk-dev] [PATCH 4/4] virtio/vdev: add a new vdev named eth_cvio

2016-01-12 Thread Yuanhan Liu
On Tue, Jan 12, 2016 at 10:45:59AM +0300, Pavel Fedin wrote: > Hello! > > See inline Hi, Please strip unrelated context, so that people could reach to your comments as quick as possible, otherwise, people could easily get lost from the long patch. > > > -Original Message- > > From:

[dpdk-dev] [PATCH 4/4] virtio/vdev: add a new vdev named eth_cvio

2016-01-12 Thread Pavel Fedin
Hello! See inline > -Original Message- > From: Jianfeng Tan [mailto:jianfeng.tan at intel.com] > Sent: Sunday, January 10, 2016 2:43 PM > To: dev at dpdk.org > Cc: rich.lane at bigswitch.com; yuanhan.liu at linux.intel.com; mst at > redhat.com; > nakajima.yoshihiro at lab.ntt.co.jp;

[dpdk-dev] [PATCH 4/4] virtio/vdev: add a new vdev named eth_cvio

2016-01-10 Thread Jianfeng Tan
Add a new virtual device named eth_cvio, it can be used just like eth_ring, eth_null, etc. Configured parameters include: - rx (optional, 1 by default): number of rx, only allowed to be 1 for now. - tx (optional, 1 by default): number of tx, only allowed