[dpdk-dev] [PATCH v2] maintainers: update email address for vhost and null PMD

2016-07-28 Thread Tetsuya Mukawa
Signed-off-by: Tetsuya Mukawa --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index d099131..6536c6b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -373,7 +373,7 @@ F: drivers/net/vmxnet3/ F: doc/guides/nics/vmxnet3.rst Vhost

[dpdk-dev] [PATCH] maintainers: claim responsibility for vhost and null PMD

2016-07-28 Thread Tetsuya Mukawa
ul 28, 2016 at 02:27:00PM +0900, Tetsuya Mukawa wrote: >> Signed-off-by: Tetsuya Mukawa >> --- >> MAINTAINERS | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index d099131..6536c6b 100644 >&g

[dpdk-dev] [PATCH] maintainers: claim responsibility for vhost and null PMD

2016-07-28 Thread Tetsuya Mukawa
I left my current company soon. Until my next email address is assigned, here is my email address. Regards, Tetsuya On 2016/07/28 14:27, Tetsuya Mukawa wrote: > Signed-off-by: Tetsuya Mukawa > --- > MAINTAINERS | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >

[dpdk-dev] [PATCH] maintainers: claim responsibility for vhost and null PMD

2016-07-28 Thread Tetsuya Mukawa
Signed-off-by: Tetsuya Mukawa --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index d099131..6536c6b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -373,7 +373,7 @@ F: drivers/net/vmxnet3/ F: doc/guides/nics/vmxnet3.rst Vhost

[dpdk-dev] SRIOV hot unplug

2016-07-19 Thread Tetsuya Mukawa
Hi Eli, On 2016/07/18 17:47, Eli Britstein wrote: > Hi Bernard, > > Thank you for your answer. > However, to do this, I have to have some communication protocol to the VM's > application in order for it to do this sequence and acknowledge that it is > now safe to proceed with detaching the

[dpdk-dev] [PATCH] vhost: fix Tx error counting of vhost PMD

2016-06-24 Thread Tetsuya Mukawa
On 2016/06/24 11:17, Yuanhan Liu wrote: > On Fri, Jun 24, 2016 at 11:04:20AM +0900, Tetsuya Mukawa wrote: >> According to 'rte_eth_stats' structure comments, 'imissed' >> should represent RX error counting, but currently 'imissed' is >> used to count TX error. >>

[dpdk-dev] [PATCH] vhost: fix Tx error counting of vhost PMD

2016-06-24 Thread Tetsuya Mukawa
According to 'rte_eth_stats' structure comments, 'imissed' should represent RX error counting, but currently 'imissed' is used to count TX error. The patch replaces 'imissed' by 'oerrors'. Fixes: ee584e9710b9 ("vhost: add driver on top of the library") Signed-off-by: Tetsuya Mukawa --

[dpdk-dev] [PATCH v2] eal: Fix resource leak while secondary process maps pci devices

2016-06-20 Thread Tetsuya Mukawa
On 2016/06/17 21:28, David Marchand wrote: > On Thu, Jun 16, 2016 at 4:33 AM, Tetsuya Mukawa wrote: >> This patch fixes resource leak of pci_uio_map_secondary(). >> If pci_map_resource() succeeds but mapped address is different from an >> address primary process mapped,

[dpdk-dev] [PATCH v2] eal: Fix resource leak while secondary process maps pci devices

2016-06-16 Thread Tetsuya Mukawa
This patch fixes resource leak of pci_uio_map_secondary(). If pci_map_resource() succeeds but mapped address is different from an address primary process mapped, this should be error. Then the addresses secondary process mapped should be freed. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal

[dpdk-dev] [PATCH v2] eal: Fix wrong resource release while unmapping pci devices

2016-06-16 Thread Tetsuya Mukawa
This patch fixes wrong resource release of pci_uio_unmap(). The 'path' member of mapped_pci_resource structure is allocated by primary process, but currently it will be freed by both primary and secondary process. The patch fixes to be freed by only primary process. Signed-off-by: Tetsuya Mukawa

[dpdk-dev] [PATCH v2] eal: Fix wrong error checking while parsing device arguments

2016-06-16 Thread Tetsuya Mukawa
This patch fixes wrong error checking of rte_eal_parse_devargs_str(). Currently, a return value of strdup() is wrongly checked. Fixes: 0fe11ec592b2 ("eal: add vdev init and uninit") Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_devargs.c | 2 +- 1 file changed, 1

[dpdk-dev] [PATCH] eal: Fix wrong error checking of rte_eal_parse_devargs_str()

2016-06-16 Thread Tetsuya Mukawa
On 2016/06/15 23:48, David Marchand wrote: > On Wed, Jun 15, 2016 at 5:26 AM, Tetsuya Mukawa wrote: >> Currently, a return value of strdup() isn't checked correctly. >> The patch fixes it. >> >> Signed-off-by: Tetsuya Mukawa > > Idem, the title does not need t

[dpdk-dev] [PATCH] eal: Fix resource leak of pci_uio_map_secondary()

2016-06-16 Thread Tetsuya Mukawa
On 2016/06/15 23:45, David Marchand wrote: > Hello Tetsuya, > > On Wed, Jun 15, 2016 at 5:26 AM, Tetsuya Mukawa wrote: >> If pci_map_resource() succeeds but mapped address is different from an >> address primary process mapped, this should be error. >> Then the addre

[dpdk-dev] [PATCH] eal: Fix wrong resource release of pci_uio_unmap()

2016-06-15 Thread Tetsuya Mukawa
The 'path' member of mapped_pci_resource structure is allocated by primary process, but currenctly it will be freed by both primary and secondary process. The patch fixes to be freed by only primary process. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_pci_uio.c | 4

[dpdk-dev] [PATCH] eal: Fix wrong error checking of rte_eal_parse_devargs_str()

2016-06-15 Thread Tetsuya Mukawa
Currently, a return value of strdup() isn't checked correctly. The patch fixes it. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_devargs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_eal/common/eal_common_devargs.c b/lib/librte_eal/common

[dpdk-dev] [PATCH] eal: Fix resource leak of pci_uio_map_secondary()

2016-06-15 Thread Tetsuya Mukawa
If pci_map_resource() succeeds but mapped address is different from an address primary process mapped, this should be error. Then the address secondary process mapped should be freed. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_pci_uio.c | 3 +++ 1 file changed, 3

[dpdk-dev] packet counting of vhost PMD

2016-06-10 Thread Tetsuya Mukawa
Hi Yuanhan, Can I get your opnion about packet counting of vhost PMD? So far, we may not be able to send all packets at once, the number of failed packets are counted, and when stats_get() is called, the number of the packet is filled like below. stats->imissed = tx_missed_total; I

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-07 Thread Tetsuya Mukawa
On 2016/06/06 19:50, Tan, Jianfeng wrote: >> Please let me make sure how we can invoke many DPDK applications in >> hundreds containers. >> (Do we have a way to do? Or, will we have it in the future?) > > Just to add some option here, we cannot say no to that kind of use case. > To have many

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-06 Thread Tetsuya Mukawa
On 2016/06/06 17:49, Yuanhan Liu wrote: > On Mon, Jun 06, 2016 at 05:33:31PM +0900, Tetsuya Mukawa wrote: >>>> [My solution] >>>> - Pros >>>> Basic principle of my implementation is not to reinvent the wheel. >>> >>> Yes, that's a

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-06 Thread Tetsuya Mukawa
On 2016/06/06 17:03, Tan, Jianfeng wrote: > Hi, > > > On 6/6/2016 1:10 PM, Tetsuya Mukawa wrote: >> Hi Yuanhan, >> >> Sorry for late replying. >> >> On 2016/06/03 13:17, Yuanhan Liu wrote: >>> On Thu, Jun 02, 2016 at 06:30:18PM +0900, Tetsuya

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-06 Thread Tetsuya Mukawa
On 2016/06/06 16:21, Yuanhan Liu wrote: > On Mon, Jun 06, 2016 at 02:10:46PM +0900, Tetsuya Mukawa wrote: >> Hi Yuanhan, >> >> Sorry for late replying. > > Never mind. > >> >> On 2016/06/03 13:17, Yuanhan Liu wrote: >>> On Thu, Jun 02, 2016

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-06 Thread Tetsuya Mukawa
Hi Yuanhan, Sorry for late replying. On 2016/06/03 13:17, Yuanhan Liu wrote: > On Thu, Jun 02, 2016 at 06:30:18PM +0900, Tetsuya Mukawa wrote: >> Hi Yuanhan, >> >> On 2016/06/02 16:31, Yuanhan Liu wrote: >>> But still, I'd ask do we really need 2 virtio fo

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-02 Thread Tetsuya Mukawa
Hi Yuanhan, On 2016/06/02 16:31, Yuanhan Liu wrote: > But still, I'd ask do we really need 2 virtio for container solutions? I appreciate your comments. Let me have time to discuss it with our team. Thanks, Tetsuya

[dpdk-dev] [PATCH v1 2/2] virtio, qtest: Add functionality to handle interrupt

2016-06-02 Thread Tetsuya Mukawa
The patch adds functionality to handle interrupt from pci device of QEMU guest. To handle the interrupts, the patch adds to initialize piix3 pci device. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_ethdev.c | 7 +- drivers/net/virtio/virtio_qtest/qtest.h

[dpdk-dev] [PATCH v1 1/2] virtio: Handle interrupt things under vtpci abstraction

2016-06-02 Thread Tetsuya Mukawa
So far, interrupts from PCI devices are handled in virtio_ethdev directly. The patch changes it, and try to handle it under vtpci abstraction. The patch is needed because virtio-qtest needs to handle interrupts from virtual pci devices. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio

[dpdk-dev] [PATCH v1 0/2] Supplement patches for virtio-qtest to support LSC interrupt

2016-06-02 Thread Tetsuya Mukawa
is handling a virtual virtio-net device. So handling interrupts are a bit different from actual pci devices. In this case, all interrupts are come from unix domain socket connected to QEMU. Tetsuya Mukawa (2): virtio: Handle interrupt things under vtpci abstraction virtio, qtest: Add

[dpdk-dev] [PATCH v5 6/6] virtio: Add QTest support for virtio-net PMD

2016-06-02 Thread Tetsuya Mukawa
isable-hw-vlan --txqflags=0xf00 -i Please specify same unix domain sockets and memory size in both QEMU and DPDK command lines like above. The share memory size should be power of 2, because ivshmem only accepts such memory size. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/Makefile

[dpdk-dev] [PATCH v5 5/6] virtio: Add QTest support to vtpci abstraction

2016-06-02 Thread Tetsuya Mukawa
The patch adds QTest support to vtpci abstraction. With this patch, only modern virtio device will be supported. This implementation will be used by later QTest extension patch of virtio-net PMD. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/Makefile| 1

[dpdk-dev] [PATCH v5 4/6] virtio, qtest: Add misc functions to handle pci information

2016-06-02 Thread Tetsuya Mukawa
The patch adds below functions. - qtest_read_pci_cfg - qtest_get_bar_addr - qtest_get_bar_size These are used for handling pci device information. It will be called by later patches. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_qtest/qtest_utils.c | 77

[dpdk-dev] [PATCH v5 3/6] virtio, qtest: Add functionality to share memory between QTest guest

2016-06-02 Thread Tetsuya Mukawa
The patch adds functionality to share memory between QTest guest and DPDK application using ivshmem device. The shared memory will be all EAL memory on hugepages. This memory will be accessed by QEMU vcpu and DPDK application using same address. Signed-off-by: Tetsuya Mukawa --- drivers/net

[dpdk-dev] [PATCH v5 2/6] virtio, qtest: Add pci device initialization function to qtest utils

2016-06-02 Thread Tetsuya Mukawa
The patch adds general pci device initialization functionality to qtest utils. It initializes pci devices using qtest messaging. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_qtest/qtest_utils.c | 349 +- drivers/net/virtio/virtio_qtest/qtest_utils.h | 114

[dpdk-dev] [PATCH v5 1/6] virtio, qtest: Add QTest utility basic functions

2016-06-02 Thread Tetsuya Mukawa
The patch adds basic functions for accessing to QEMU quest that runs in QTest mode. The functions will be used by virtio container extension that can access to the above guest. Signed-off-by: Tetsuya Mukawa --- config/common_linuxapp| 2 + drivers/net/virtio/Makefile

[dpdk-dev] [PATCH v5 0/6] Virtio-net PMD: QEMU QTest extension for container

2016-06-02 Thread Tetsuya Mukawa
from shared memory, then specify the virtual address of it to virtio-net register, QEMU virtio-net device can understand it without calculating address offset.) Tetsuya Mukawa (6): virtio, qtest: Add QTest utility basic functions virtio, qtest: Add pci device initialization function t

[dpdk-dev] [PATCH v1] null: set port_id in mbufs received from null PMD

2016-05-30 Thread Tetsuya Mukawa
dk-dev] [PATCH v1] null: set port_id in mbufs received from >> null PMD >> >> Ensure that the port field is set in mbufs received from the null PMD. >> >> Signed-off-by: Sean Harte > Acked-by: John McNamara > Acked-by: Tetsuya Mukawa

[dpdk-dev] [PATCH] mk: add build-time library directory to linker path

2016-04-28 Thread Tetsuya Mukawa
gt; O_TO_S_DISP = $(if $(V),"$(O_TO_S_STR)"," LD $(@)") > O_TO_S_DO = @set -e; \ Tested-by: Tetsuya Mukawa

[dpdk-dev] [PATCH v2] vhost: Fix linkage of vhost PMD

2016-04-27 Thread Tetsuya Mukawa
On 2016/04/26 18:35, Panu Matilainen wrote: > On 04/26/2016 08:39 AM, Tetsuya Mukawa wrote: >> Currently, vhost PMD doesn't have linkage for librte_vhost, even though >> it depends on librte_vhost APIs. This causes a linkage error if below >> conditions are fulfilled. &g

[dpdk-dev] [RFC] Link ibrte_vhost to librte_pmd_vhost

2016-04-26 Thread Tetsuya Mukawa
On 2016/04/26 14:48, Yuanhan Liu wrote: > On Tue, Apr 26, 2016 at 02:37:37PM +0900, Tetsuya Mukawa wrote: >> On 2016/04/26 12:47, Yuanhan Liu wrote: >>> On Mon, Apr 25, 2016 at 12:28:37PM +0300, Panu Matilainen wrote: >>> > >>>>> Another way is ap

[dpdk-dev] [PATCH v2] vhost: Fix linkage of vhost PMD

2016-04-26 Thread Tetsuya Mukawa
to link vhost PMD using '-d' DPDK option. The patch adds linkage for librte_vhost to vhost PMD not to cause an above error. Signed-off-by: Tetsuya Mukawa Acked-by: Panu Matilainen --- drivers/net/vhost/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/vhost/Makefile b

[dpdk-dev] [RFC] Link ibrte_vhost to librte_pmd_vhost

2016-04-26 Thread Tetsuya Mukawa
On 2016/04/26 12:47, Yuanhan Liu wrote: > On Mon, Apr 25, 2016 at 12:28:37PM +0300, Panu Matilainen wrote: > > >>> Another way is applying a below patch. >>> --- a/drivers/net/vhost/Makefile >>> +++ b/drivers/net/vhost/Makefile >>> @@ -38,6 +38,7 @@ LIB = librte_pmd_vhost.a >>> >>> CFLAGS += -O3

[dpdk-dev] [PATCH] vhost: Fix linkage of vhost PMD

2016-04-26 Thread Tetsuya Mukawa
to link vhost PMD using '-d' DPDK option. The patch adds linkage for librte_vhost to vhost PMD not to cause an above error. Acked-by: Panu Matilainen Signed-off-by: Tetsuya Mukawa --- drivers/net/vhost/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/vhost/Makefile b

[dpdk-dev] Git repository might be down

2016-04-25 Thread Tetsuya Mukawa
Hi, I've faced below error while connecting to DPDK git repository. $ git clone git://dpdk.org/dpdk Cloning into 'dpdk'... fatal: read error: Connection reset by peer It might be caused by my environment, but report it just in case. Thanks, Tetsuya

[dpdk-dev] [RFC] Link ibrte_vhost to librte_pmd_vhost

2016-04-25 Thread Tetsuya Mukawa
On 2016/04/25 18:28, Panu Matilainen wrote: > On 04/25/2016 12:05 PM, Tetsuya Mukawa wrote: >> Hi Yuanhan, >> >> I want to apply a patch to vhost PMD. >> Before submitting, could you please let me know your guess about the >> patch? >> >> Here is m

[dpdk-dev] [RFC] Link ibrte_vhost to librte_pmd_vhost

2016-04-25 Thread Tetsuya Mukawa
Hi Yuanhan, I want to apply a patch to vhost PMD. Before submitting, could you please let me know your guess about the patch? Here is my problem. I am using below shared library configuration to build my application. CONFIG_RTE_BUILD_SHARED_LIB=y Normally, My application doesn't need vhost

[dpdk-dev] [Announce] A new tree for vhost/virtio

2016-04-19 Thread Tetsuya Mukawa
On 2016/04/19 3:55, Yuanhan Liu wrote: > Hi, > > Here I'd like to announce a new tree for vhost/virtio[0], and I'm > going to be the maintainer. > > [0]: http://dpdk.org/browse/next/dpdk-next-virtio/ > > This is done by a private request to Thomas few days ago (well, I'd > confess this should

[dpdk-dev] [PATCH] vhost: call rte_vhost_enable_guest_notification only on enabled queues

2016-04-08 Thread Tetsuya Mukawa
On 2016/04/08 15:14, Yuanhan Liu wrote: > On Fri, Apr 08, 2016 at 10:45:33AM +0900, Tetsuya Mukawa wrote: >> On 2016/04/08 2:20, Thomas Monjalon wrote: >>>>> If the vhost PMD were configured with more queues than the guest, the old >>>>> code would segfault

[dpdk-dev] [PATCH] vhost: call rte_vhost_enable_guest_notification only on enabled queues

2016-04-08 Thread Tetsuya Mukawa
On 2016/04/08 2:20, Thomas Monjalon wrote: >>> If the vhost PMD were configured with more queues than the guest, the old >>> code would segfault in rte_vhost_enable_guest_notification due to a NULL >>> virtqueue pointer. >>> >>> Fixes: ee584e9710b9 ("vhost: add driver on top of the library") >>>

[dpdk-dev] [PATCH] vhost: Fix retrieval of numa information in PMD

2016-04-06 Thread Tetsuya Mukawa
On 2016/04/06 16:17, Yuanhan Liu wrote: > On Wed, Apr 06, 2016 at 03:49:25PM +0900, Tetsuya Mukawa wrote: >> On 2016/04/06 1:09, Ciara Loftus wrote: >>> After some testing, it was found that retrieving numa information >>> about a vhost device via a call to get_mempolic

[dpdk-dev] [PATCH] vhost: Fix retrieval of numa information in PMD

2016-04-06 Thread Tetsuya Mukawa
On 2016/04/06 1:09, Ciara Loftus wrote: > After some testing, it was found that retrieving numa information > about a vhost device via a call to get_mempolicy is more > accurate when performed during the new_device callback versus > the vring_state_changed callback, in particular upon initial boot

[dpdk-dev] [PATCH] vhost PMD: Fix wrong handling of maximum value of rx/tx queues

2016-03-23 Thread Tetsuya Mukawa
figure(). >> >> To fix the issue, the patch prepare one more variable to keep the >> number of maximum supported queues in vhost PMD. >> >> Signed-off-by: Tetsuya Mukawa >> --- >> drivers/net/vhost/rte_eth_vhost.c | 14 -- >> 1 file

[dpdk-dev] [PATCH] vhost PMD: Fix wrong handling of maximum value of rx/tx queues

2016-03-22 Thread Tetsuya Mukawa
the maximum value of current enabled queues and supported queues will be '2' after calling first rte_eth_dev_configure(). To fix the issue, the patch prepare one more variable to keep the number of maximum supported queues in vhost PMD. Signed-off-by: Tetsuya Mukawa --- drivers/net/vhost

[dpdk-dev] [PATCH v13 2/2] vhost: Add VHOST PMD

2016-03-22 Thread Tetsuya Mukawa
On 2016/03/22 0:40, Loftus, Ciara wrote: >> + >> +static void >> +eth_dev_info(struct rte_eth_dev *dev, >> + struct rte_eth_dev_info *dev_info) >> +{ >> +dev_info->driver_name = drivername; >> +dev_info->max_mac_addrs = 1; >> +dev_info->max_rx_pktlen = (uint32_t)-1; >> +

[dpdk-dev] [PATCH v13 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-21 Thread Tetsuya Mukawa
2016-03-21 20:05 GMT+09:00 Bruce Richardson : > On Mon, Mar 21, 2016 at 06:24:36PM +0900, Tetsuya Mukawa wrote: >> 2016-03-21 17:37 GMT+09:00 Thomas Monjalon : >> > 2016-03-21 14:45, Tetsuya Mukawa: >> >> This event will be occured when some queues are enabled or disa

[dpdk-dev] [PATCH v13 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-21 Thread Tetsuya Mukawa
2016-03-21 17:37 GMT+09:00 Thomas Monjalon : > 2016-03-21 14:45, Tetsuya Mukawa: >> This event will be occured when some queues are enabled or disabled. >> So far, only vhost PMD supports the event, and it indicates some queues >> are enabled or disabled by virtio-net

[dpdk-dev] [PATCH v13 2/2] vhost: Add VHOST PMD

2016-03-21 Thread Tetsuya Mukawa
To connect above testpmd, here is qemu command example. $ qemu-system-x86_64 \ -chardev socket,id=chr0,path=/tmp/sock0 \ -netdev vhost-user,id=net0,chardev=chr0,vhostforce,queues=1 \ -device virtio-net-pci,netdev=net0,mq=on Signed-off-by: Tetsuya Mukawa Acked

[dpdk-dev] [PATCH v13 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-21 Thread Tetsuya Mukawa
-net device may not enable all queues vhost PMD prepare. Because only vhost PMD uses the event so far, it isn't an actual hardware interrupt but a simple software event. Signed-off-by: Tetsuya Mukawa Acked-by: Ferruh Yigit Acked-by: Yuanhan Liu Acked-by: Rich Lane Tested-by: Rich Lane

[dpdk-dev] [PATCH v13 0/2] Add VHOST PMD

2016-03-21 Thread Tetsuya Mukawa
s. - Add code to support vhost multiple queues. Actually, multiple queues functionality is not enabled so far. RFC PATCH v2 changes: - Fix issues reported by checkpatch.pl (Thanks to Stephen Hemminger) Tetsuya Mukawa (2): ethdev: Add a new event type to n

[dpdk-dev] [PATCH v12 2/2] vhost: Add VHOST PMD

2016-03-21 Thread Tetsuya Mukawa
On 2016/03/18 21:27, Bruce Richardson wrote: > On Tue, Mar 15, 2016 at 05:31:41PM +0900, Tetsuya Mukawa wrote: >> The patch introduces a new PMD. This PMD is implemented as thin wrapper >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. >&g

[dpdk-dev] [PATCH v12 2/2] vhost: Add VHOST PMD

2016-03-19 Thread Tetsuya Mukawa
On 2016/03/18 23:13, Bruce Richardson wrote: > On Fri, Mar 18, 2016 at 11:03:56PM +0900, Tetsuya Mukawa wrote: >> On 2016/03/18 22:52, Thomas Monjalon wrote: >>> 2016-03-18 22:41, Tetsuya Mukawa: >>>> 2016/03/18 ??9:27 "Bruce Richardson" : >>>>&g

[dpdk-dev] [PATCH v12 2/2] vhost: Add VHOST PMD

2016-03-19 Thread Tetsuya Mukawa
On 2016/03/18 22:52, Thomas Monjalon wrote: > 2016-03-18 22:41, Tetsuya Mukawa: >> 2016/03/18 ??9:27 "Bruce Richardson" : >>> I hope to get this set merged for RC2 very soon. Can you provide an >>> update for the nic overview.rst doc listing out the features

[dpdk-dev] [PATCH v12 2/2] vhost: Add VHOST PMD

2016-03-18 Thread Tetsuya Mukawa
2016/03/18 ??9:27 "Bruce Richardson" : > > On Tue, Mar 15, 2016 at 05:31:41PM +0900, Tetsuya Mukawa wrote: > > The patch introduces a new PMD. This PMD is implemented as thin wrapper > > of librte_vhost. It means librte_vhost is also needed to compile the

[dpdk-dev] [PATCH v12 2/2] vhost: Add VHOST PMD

2016-03-15 Thread Tetsuya Mukawa
To connect above testpmd, here is qemu command example. $ qemu-system-x86_64 \ -chardev socket,id=chr0,path=/tmp/sock0 \ -netdev vhost-user,id=net0,chardev=chr0,vhostforce,queues=1 \ -device virtio-net-pci,netdev=net0,mq=on Signed-off-by: Tetsuya Mukawa Acked

[dpdk-dev] [PATCH v12 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-15 Thread Tetsuya Mukawa
This patch adds a below event type. - RTE_ETH_EVENT_QUEUE_STATE_CHANGE This event is used for notifying a queue state changed event. Signed-off-by: Tetsuya Mukawa Acked-by: Ferruh Yigit Acked-by: Yuanhan Liu Acked-by: Rich Lane Tested-by: Rich Lane --- lib/librte_ether/rte_ethdev.h | 2

[dpdk-dev] [PATCH v12 0/2] Add VHOST PMD

2016-03-15 Thread Tetsuya Mukawa
s doesn't use vhost PMD, can fully use vhost library APIs. - Add code to support vhost multiple queues. Actually, multiple queues functionality is not enabled so far. RFC PATCH v2 changes: - Fix issues reported by checkpatch.pl (Thanks to Stephen Hemminger) Tetsuya Mukawa (2): ethdev: A

[dpdk-dev] [PATCH v11 2/2] vhost: Add VHOST PMD

2016-03-15 Thread Tetsuya Mukawa
On 2016/03/14 21:02, Bruce Richardson wrote: > On Mon, Mar 07, 2016 at 11:07:14AM +0900, Tetsuya Mukawa wrote: >> The patch introduces a new PMD. This PMD is implemented as thin wrapper >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. >&g

[dpdk-dev] [PATCH v3] vhost: Fix default value of kickfd and callfd

2016-03-14 Thread Tetsuya Mukawa
specified in QEMU command line. It means we cannot treat -1 as uninitialized state. The patch defines -1 and -2 as VIRTIO_INVALID_EVENTFD and VIRTIO_UNINITIALIZED_EVENTFD, and uses VIRTIO_UNINITIALIZED_EVENTFD for the default values of kickfd and callfd. Signed-off-by: Tetsuya Mukawa --- lib/li

[dpdk-dev] [PATCH v2] vhost: Fix default value of kickfd and callfd

2016-03-14 Thread Tetsuya Mukawa
On 2016/03/14 17:21, Yuanhan Liu wrote: > On Mon, Mar 14, 2016 at 04:54:00PM +0900, Tetsuya Mukawa wrote: >> On 2016/03/14 11:08, Yuanhan Liu wrote: >>> On Mon, Mar 14, 2016 at 10:54:14AM +0900, Tetsuya Mukawa wrote: >>>> On 2016/03/11 16:19, Yuanhan Liu wrote: >&

[dpdk-dev] [PATCH v2] vhost: Fix default value of kickfd and callfd

2016-03-14 Thread Tetsuya Mukawa
On 2016/03/14 11:08, Yuanhan Liu wrote: > On Mon, Mar 14, 2016 at 10:54:14AM +0900, Tetsuya Mukawa wrote: >> On 2016/03/11 16:19, Yuanhan Liu wrote: >>> On Thu, Mar 10, 2016 at 04:06:05PM +0900, Tetsuya Mukawa wrote: >>>> Currently, default values of kickfd and cal

[dpdk-dev] [PATCH v2] vhost: Fix default value of kickfd and callfd

2016-03-14 Thread Tetsuya Mukawa
On 2016/03/11 16:19, Yuanhan Liu wrote: > On Thu, Mar 10, 2016 at 04:06:05PM +0900, Tetsuya Mukawa wrote: >> Currently, default values of kickfd and callfd are -1. >> If the values are -1, current code guesses kickfd and callfd haven't >> been initialized yet. And vho

[dpdk-dev] [PATCH v2] vhost: Fix default value of kickfd and callfd

2016-03-10 Thread Tetsuya Mukawa
specified in QEMU command line. It means we cannot treat -1 as uninitialized state. The patch changes default values to -2. And the patch defines -2 as VIRTIO_UNINITIALIZED_EVENTFD. Signed-off-by: Tetsuya Mukawa --- lib/librte_vhost/rte_virtio_net.h | 1 + lib/librte_vhost/vhost_user/

[dpdk-dev] [PATCH] vhost: Fix default value of kickfd and callfd

2016-03-10 Thread Tetsuya Mukawa
On 2016/03/10 15:39, Tan, Jianfeng wrote: > > On 3/10/2016 2:34 PM, Tetsuya Mukawa wrote: >> On 2016/03/10 15:25, Tan, Jianfeng wrote: >>> >>> On 3/10/2016 2:14 PM, Tetsuya Mukawa wrote: >>>> Currently, default value of kickfd and callfd is -1. >>

[dpdk-dev] [PATCH] vhost: Fix default value of kickfd and callfd

2016-03-10 Thread Tetsuya Mukawa
On 2016/03/10 15:25, Tan, Jianfeng wrote: > > > On 3/10/2016 2:14 PM, Tetsuya Mukawa wrote: >> Currently, default value of kickfd and callfd is -1. >> If the value is -1, current code guess kickfd and callfd hasn't been >> initialized yet. And vhost library will gues

[dpdk-dev] [PATCH v4 12/12] docs: add release note for qtest virtio container support

2016-03-09 Thread Tetsuya Mukawa
Signed-off-by: Tetsuya Mukawa --- doc/guides/rel_notes/release_16_04.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/guides/rel_notes/release_16_04.rst b/doc/guides/rel_notes/release_16_04.rst index e3142f2..1c8c6b2 100644 --- a/doc/guides/rel_notes/release_16_04.rst +++ b/doc

[dpdk-dev] [PATCH v4 11/12] virtio: Add QTest support for virtio-net PMD

2016-03-09 Thread Tetsuya Mukawa
=0xf00 -i Please specify same unix domain sockets and memory size in both QEMU and DPDK command lines like above. The share memory size should be power of 2, because ivshmem only accepts such memory size. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/qtest.h | 55 +

[dpdk-dev] [PATCH v4 10/12] virtio: Add QTest support to vtpci abstraction

2016-03-09 Thread Tetsuya Mukawa
The patch adds QTest support to vtpci abstraction. With this patch, only modern virtio device will be supported. This QTest support will be used by later QTest extension patch of virtio-net PMD. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/qtest.h | 39 drivers/net/virtio

[dpdk-dev] [PATCH v4 09/12] virtio, qtest: Add misc functions to handle pci information

2016-03-09 Thread Tetsuya Mukawa
The patch adds below functions. - qtest_read_pci_cfg - qtest_get_bar - qtest_get_bar_addr - qtest_get_bar_size These are used for handling pci device information. It will be called by later patches. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/qtest_utils.c | 77

[dpdk-dev] [PATCH v4 08/12] virtio, qtest: Add functionality to handle interrupt

2016-03-09 Thread Tetsuya Mukawa
The patch adds functionality to handle interrupt from pci device of QEMU guest. To handle the interrupts, the patch adds to initialize piix3 pci device. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/qtest_utils.c | 225 ++- drivers/net/virtio

[dpdk-dev] [PATCH v4 07/12] virtio, qtest: Add functionality to share memory between QTest guest

2016-03-09 Thread Tetsuya Mukawa
The patch adds functionality to share memory between QTest guest and DPDK application using ivshmem device. The shared memory will be all EAL memory on hugepages. This memory will be accessed by QEMU vcpu and DPDK application using same address. Signed-off-by: Tetsuya Mukawa --- drivers/net

[dpdk-dev] [PATCH v4 06/12] virtio, qtest: Add pci device initialization function to qtest utils

2016-03-09 Thread Tetsuya Mukawa
The patch adds general pci device initialization functionality to qtest utils. It initializes pci devices using qtest messaging. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/qtest_utils.c | 349 ++- drivers/net/virtio/qtest_utils.h | 114

[dpdk-dev] [PATCH v4 05/12] virtio, qtest: Add QTest utility basic functions

2016-03-09 Thread Tetsuya Mukawa
The patch adds basic functions for accessing to QEMU quest that runs in QTest mode. The functions will be used by virtio container extension that can access to the above guest. Signed-off-by: Tetsuya Mukawa --- config/common_base | 1 + drivers/net/virtio/Makefile | 4

[dpdk-dev] [PATCH v4 04/12] EAL: Add a new "--align-memsize" option

2016-03-09 Thread Tetsuya Mukawa
The option will work with "--range-virtaddr", and if the option is specified, mapped address will be align by EAL memory size. Such an alignment is required for using virtio-net PMD extension on container that uses QEMU QTest framework. Signed-off-by: Tetsuya Mukawa --- lib/librte_

[dpdk-dev] [PATCH v4 03/12] EAL: Add a new "--range-virtaddr" option

2016-03-09 Thread Tetsuya Mukawa
will not work with '--base-virtaddr'. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_options.c | 9 lib/librte_eal/common/eal_internal_cfg.h | 2 + lib/librte_eal/common/eal_options.h| 2 + lib/librte_eal/linuxapp/eal/eal.c | 39 ++ lib/librte_eal

[dpdk-dev] [PATCH v4 02/12] vhost: Add a function to check virtio device type

2016-03-09 Thread Tetsuya Mukawa
The patch adds below function to cleanup virtio code. - virtio_dev_check() Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_ethdev.c | 52 ++ drivers/net/virtio/virtio_ethdev.h | 32 +++ 2 files changed, 57 insertions(+), 27

[dpdk-dev] [PATCH v4 01/12] virtio: Retrieve driver name from eth_dev

2016-03-09 Thread Tetsuya Mukawa
Currently, virtio_dev_info_get() retrieves driver name from pci_drv. If the driver is virtual PMD, pci_drv will be invalid. So retrieves the name from eth_dev. Signed-off-by: Tetsuya Mukawa --- drivers/net/virtio/virtio_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[dpdk-dev] [PATCH v4 00/12] Virtio-net PMD: QEMU QTest extension for container

2016-03-09 Thread Tetsuya Mukawa
then specify the virtual address of it to virtio-net register, QEMU virtio-net device can understand it without calculating address offset.) Tetsuya Mukawa (12): virtio: Retrieve driver name from eth_dev vhost: Add a function to check virtio device type EAL: Add a new "--range-vi

[dpdk-dev] [PATCH v11 2/2] vhost: Add VHOST PMD

2016-03-07 Thread Tetsuya Mukawa
To connect above testpmd, here is qemu command example. $ qemu-system-x86_64 \ -chardev socket,id=chr0,path=/tmp/sock0 \ -netdev vhost-user,id=net0,chardev=chr0,vhostforce,queues=1 \ -device virtio-net-pci,netdev=net0,mq=on Signed-off-by: Tetsuya Mukawa Acked

[dpdk-dev] [PATCH v11 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-07 Thread Tetsuya Mukawa
This patch adds a below event type. - RTE_ETH_EVENT_QUEUE_STATE_CHANGE This event is used for notifying a queue state changed event. Signed-off-by: Tetsuya Mukawa Acked-by: Ferruh Yigit Acked-by: Yuanhan Liu Acked-by: Rich Lane Tested-by: Rich Lane --- lib/librte_ether/rte_ethdev.h | 2

[dpdk-dev] [PATCH v11 0/2] Add VHOST PMD

2016-03-07 Thread Tetsuya Mukawa
port vhost multiple queues. Actually, multiple queues functionality is not enabled so far. RFC PATCH v2 changes: - Fix issues reported by checkpatch.pl (Thanks to Stephen Hemminger) Tetsuya Mukawa (2): ethdev: Add a new event type to notify a queue state changed event vhost: Add V

[dpdk-dev] [PATCH v10 2/2] vhost: Add VHOST PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/04 17:39, Yuanhan Liu wrote: > On Fri, Mar 04, 2016 at 01:17:42PM +0900, Tetsuya Mukawa wrote: >> The patch introduces a new PMD. This PMD is implemented as thin wrapper >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. >> The vhost mes

[dpdk-dev] [PATCH v3 5/6] virtio: Add support for qtest virtio-net PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/04 15:10, Tan, Jianfeng wrote: > Hi Tetsuya, > > On 3/4/2016 1:05 PM, Tetsuya Mukawa wrote: >> On 2016/03/04 11:18, Tan, Jianfeng wrote: >>> Hi Tetsuya, >>> >>> Seems that this patch is too long. Is it possible to split into >>> mu

[dpdk-dev] [PATCH v3 5/6] virtio: Add support for qtest virtio-net PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/04 11:18, Tan, Jianfeng wrote: > Hi Tetsuya, > > Seems that this patch is too long. Is it possible to split into > multiple commits? Hi Jianfeng, Sure, will do. > > On 2/22/2016 4:17 PM, Tetsuya Mukawa wrote: >> The patch adds a new virtio-net PMD configura

[dpdk-dev] [PATCH v10 2/2] vhost: Add VHOST PMD

2016-03-04 Thread Tetsuya Mukawa
To connect above testpmd, here is qemu command example. $ qemu-system-x86_64 \ -chardev socket,id=chr0,path=/tmp/sock0 \ -netdev vhost-user,id=net0,chardev=chr0,vhostforce,queues=1 \ -device virtio-net-pci,netdev=net0,mq=on Signed-off-by: Tetsuya Mukawa Acked

[dpdk-dev] [PATCH v10 1/2] ethdev: Add a new event type to notify a queue state changed event

2016-03-04 Thread Tetsuya Mukawa
This patch adds a below event type. - RTE_ETH_EVENT_QUEUE_STATE_CHANGE This event is used for notifying a queue state changed event. Signed-off-by: Tetsuya Mukawa Acked-by: Ferruh Yigit --- lib/librte_ether/rte_ethdev.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/librte_ether

[dpdk-dev] [PATCH v10 0/2] Add VHOST PMD

2016-03-04 Thread Tetsuya Mukawa
vdev() To support this functionality, vhost library is also changed. Anyway, if users doesn't use vhost PMD, can fully use vhost library APIs. - Add code to support vhost multiple queues. Actually, multiple queues functionality is not enabled so far. RFC PATCH v2 changes: - Fix issues

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-03-04 Thread Tetsuya Mukawa
On 2016/03/02 11:24, Qiu, Michael wrote: > On 3/1/2016 10:19 AM, Tetsuya Mukawa wrote: >> On 2016/03/01 11:00, Qiu, Michael wrote: >>> On 2/26/2016 4:36 PM, Tetsuya Mukawa wrote: >>>> On 2016/02/26 13:29, Tetsuya Mukawa wrote: > [...] > >>>>>

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-03-01 Thread Tetsuya Mukawa
On 2016/03/01 11:00, Qiu, Michael wrote: > On 2/26/2016 4:36 PM, Tetsuya Mukawa wrote: >> On 2016/02/26 13:29, Tetsuya Mukawa wrote: >>> On 2016/02/25 16:51, Qiu, Michael wrote: >>>> On 2/24/2016 1:10 PM, Tetsuya Mukawa wrote: >>>>> On 2016/02/24 1

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-02-26 Thread Tetsuya Mukawa
On 2016/02/26 13:29, Tetsuya Mukawa wrote: > On 2016/02/25 16:51, Qiu, Michael wrote: >> On 2/24/2016 1:10 PM, Tetsuya Mukawa wrote: >>> On 2016/02/24 11:45, Qiu, Michael wrote: >>>> Hi, Tetsuya >>>> >>>> When I applied your v6 patch, I c

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-02-26 Thread Tetsuya Mukawa
On 2016/02/25 16:51, Qiu, Michael wrote: > On 2/24/2016 1:10 PM, Tetsuya Mukawa wrote: >> On 2016/02/24 11:45, Qiu, Michael wrote: >>> Hi, Tetsuya >>> >>> When I applied your v6 patch, I could reach 9.5Mpps with 64B packet. >>> >>> But whe

[dpdk-dev] [PATCH v9 0/2] Add VHOST PMD

2016-02-24 Thread Tetsuya Mukawa
t, but I don't see the drop on my environment. (My cpu is Xeon E5-2697-v2, and the performances of v6 and v9 patch are almost 5.9Mpps) Did you use totally same code except for vhost PMD? Thanks, Tetsuya > Thanks, > Michael > On 2/9/2016 5:38 PM, Tetsuya Mukawa wrote: >> The pat

[dpdk-dev] [PATCH v3 6/6] docs: add release note for qtest virtio container support

2016-02-24 Thread Tetsuya Mukawa
On 2016/02/23 19:28, Mcnamara, John wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mcnamara, John >> Sent: Monday, February 22, 2016 3:41 PM >> To: Tetsuya Mukawa ; dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH

[dpdk-dev] [PATCH v3 6/6] docs: add release note for qtest virtio container support

2016-02-22 Thread Tetsuya Mukawa
Signed-off-by: Tetsuya Mukawa --- doc/guides/rel_notes/release_16_04.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/guides/rel_notes/release_16_04.rst b/doc/guides/rel_notes/release_16_04.rst index 197d4e1..4c6d528 100644 --- a/doc/guides/rel_notes/release_16_04.rst +++ b/doc

  1   2   3   4   5   6   7   8   9   10   >