[dpdk-dev] [PATCH] testpmd: Fix action of operationg invalid port

2015-03-09 Thread Tetsuya Mukawa
return; > + > printf("Closing ports...\n"); > > FOREACH_PORT(pi, ports) { > - if (!port_id_is_invalid(pid, DISABLED_WARN) && pid != pi) > + if (pid != pi && pid != (portid_t)RTE_PORT_ALL) > continue; > > port = [pi]; Acked-by: Tetsuya Mukawa

[dpdk-dev] [PATCH 1/3] BSD: Support Port Hotplug function

2015-03-09 Thread Tetsuya Mukawa
On 2015/03/06 19:32, Iremonger, Bernard wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tetsuya Mukawa >> Sent: Wednesday, March 4, 2015 3:12 AM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH 1/3] BSD:

[dpdk-dev] [PATCH] lib/librte_vhost: exchange kickfd and callfd to avoid confusion

2015-03-09 Thread Tetsuya Mukawa
/* file->index refers to the queue index. The txq is 1, rxq is 0. */ > vq = dev->virtqueue[file->index]; > > - if ((int)vq->kickfd >= 0) > - close((int)vq->kickfd); > + if ((int)vq->callfd >= 0) > + close((int)vq->callfd); > > - vq->kickfd = file->fd; > + vq->callfd = file->fd; > > return 0; > } > @@ -621,10 +621,10 @@ set_vring_kick(struct vhost_device_ctx ctx, struct > vhost_vring_file *file) > /* file->index refers to the queue index. The txq is 1, rxq is 0. */ > vq = dev->virtqueue[file->index]; > > - if ((int)vq->callfd >= 0) > - close((int)vq->callfd); > + if ((int)vq->kickfd >= 0) > + close((int)vq->kickfd); > > - vq->callfd = file->fd; > + vq->kickfd = file->fd; > > return 0; > } Acked-by: Tetsuya Mukawa

[dpdk-dev] [PATCH] test whether file descriptor is valid before close it

2015-03-09 Thread Tetsuya Mukawa
f (dev->virtqueue[VIRTIO_TXQ]->kickfd) > + if ((int)dev->virtqueue[VIRTIO_TXQ]->kickfd >= 0) > close((int)dev->virtqueue[VIRTIO_TXQ]->kickfd); > } > Acked-by: Tetsuya Mukawa

[dpdk-dev] [PATCH] testpmd: Fix port validation code of "port stop all" command

2015-03-09 Thread Tetsuya Mukawa
On 2015/03/06 22:53, De Lara Guarch, Pablo wrote: > Hi Michael, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Qiu, Michael >> Sent: Thursday, March 05, 2015 1:33 PM >> To: Tetsuya Mukawa; dev at dpdk.org >> Subject:

[dpdk-dev] [PATCH] mk: Remove '-fdirectives-only' flag from mk/rte.sdkconfig.mk

2015-03-05 Thread Tetsuya Mukawa
The '-fdirectives-only' flag doesn't exist in cpp of FreeBSD. To remove expanding macros when parsing the config files won't add any issue. Signed-off-by: Tetsuya Mukawa --- mk/rte.sdkconfig.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mk/rte.sdkconfig.mk b/mk

[dpdk-dev] [PATCH] testpmd: Fix port validation code of "port stop all" command

2015-03-05 Thread Tetsuya Mukawa
When "port stop all" is executed, the command doesn't work as it should because of wrong port validation. The patch fixes this issue. Reported-by: Pablo de Lara Signed-off-by: Tetsuya Mukawa --- app/test-pmd/testpmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[dpdk-dev] Build failure on FreeBSD-10.1-RELEASE

2015-03-05 Thread Tetsuya Mukawa
On 2015/03/04 19:15, Bruce Richardson wrote: > On Wed, Mar 04, 2015 at 10:33:14AM +0100, Olivier MATZ wrote: >> Hi Tetsuya, Hi Bruce, >> >> On 03/04/2015 04:34 AM, Tetsuya Mukawa wrote: >>> On 2015/03/02 19:22, Bruce Richardson wrote: >>>> On Mon, Mar 02

[dpdk-dev] [PATCH v15] testpmd: Add port hotplug support

2015-03-05 Thread Tetsuya Mukawa
On 2015/03/04 0:54, De Lara Guarch, Pablo wrote: > Hi Tetsuya, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tetsuya Mukawa >> Sent: Wednesday, February 25, 2015 7:32 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-03-04 Thread Tetsuya Mukawa
2015-02-26 19:57 GMT+09:00 Thomas Monjalon : > 2015-02-26 18:06, Tetsuya Mukawa: >> 2015-02-26 16:03 GMT+09:00 Thomas Monjalon : >> > 2015-02-25 16:49, Stephen Hemminger: >> >> Build fails if HOTPLUG is disabled >> >> Hi Stephen, >> >>

[dpdk-dev] Build failure on FreeBSD-10.1-RELEASE

2015-03-04 Thread Tetsuya Mukawa
On 2015/03/02 19:22, Bruce Richardson wrote: > On Mon, Mar 02, 2015 at 12:47:42PM +0900, Tetsuya Mukawa wrote: >> Hi, >> >> I got a error while building master branch on FreeBSD. >> Here is a log. >> >> $ gmake T=x86_64-native-bsdapp-clang config >> cc

[dpdk-dev] [PATCH 3/3] doc: Remove BSD limitation from hotplug section of programmer's guide

2015-03-04 Thread Tetsuya Mukawa
This patch removes below limitation from hotplug section of programmer's guide. - The framework can only be enabled with Linux. BSD is not supported. Signed-off-by: Tetsuya Mukawa --- doc/guides/prog_guide/port_hotplug_framework.rst | 2 -- 1 file changed, 2 deletions(-) diff --git a/doc

[dpdk-dev] [PATCH 2/3] eal, ethdev: Remove CONFIG_RTE_LIBRTE_EAL_HOTPLUG

2015-03-04 Thread Tetsuya Mukawa
The patch removes CONFIG_RTE_LIBRTE_EAL_HOTPLUG option from DPDK. Signed-off-by: Tetsuya Mukawa --- config/common_bsdapp | 6 -- config/common_linuxapp | 5 - lib/librte_eal/common/eal_common_dev.c | 2 -- lib/librte_eal/common

[dpdk-dev] [PATCH 1/3] BSD: Support Port Hotplug function

2015-03-04 Thread Tetsuya Mukawa
This patch adds Hotplug support to BSD. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/bsdapp/eal/eal_pci.c | 169 +- lib/librte_eal/bsdapp/eal/rte_eal_version.map | 6 + lib/librte_eal/common/include/rte_pci.h | 1 + lib/librte_ether/rte_ethdev.c

[dpdk-dev] [PATCH] testpmd: Fix segmentation fault when portmask is specified

2015-02-27 Thread Tetsuya Mukawa
If testpmd is invoked with portmask option like below, segmentation fault will be occured. This patch fixes the issue. Reported-by: De Lara Guarch, Pablo Signed-off-by: Tetsuya Mukawa --- app/test-pmd/testpmd.c | 37 +++-- 1 file changed, 23 insertions(+), 14

[dpdk-dev] [PATCH v15] testpmd: Add port hotplug support

2015-02-27 Thread Tetsuya Mukawa
On 2015/02/27 3:49, De Lara Guarch, Pablo wrote: > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tetsuya Mukawa >> Sent: Wednesday, February 25, 2015 7:32 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH v15]

[dpdk-dev] [PATCH] eal: Fix build issue of hotplug with icc

2015-02-27 Thread Tetsuya Mukawa
This patch fixes following errors with icc. error #188: enumerated type mixed with another type return -1; Reported-by: Mcnamara, John Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib

[dpdk-dev] [PATCH] librte_pmd_null: Fix build issue with icc

2015-02-27 Thread Tetsuya Mukawa
This patch fixes following errors with icc. rte_eth_null.c(47): error #83: type qualifier specified more than once Reported-by: Mcnamara, John Signed-off-by: Tetsuya Mukawa --- lib/librte_pmd_null/rte_eth_null.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib

[dpdk-dev] [PATCH] librte_pmd_null: Fix build issue with gcc-4.7

2015-02-27 Thread Tetsuya Mukawa
This patch fixes following errors with gcc-4.7. lib/librte_pmd_null/rte_eth_null.c:302:28: error: array subscript is above array bounds Reported-by: Mcnamara, John Reported-by: Stephen Hemminger Signed-off-by: Tetsuya Mukawa --- lib/librte_pmd_null/rte_eth_null.c | 12 +++- 1

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-27 Thread Tetsuya Mukawa
On 2015/02/27 0:36, Mcnamara, John wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mcnamara, John >> Sent: Thursday, February 26, 2015 12:21 PM >> To: Tetsuya Mukawa; Thomas Monjalon >> Cc: dev at dpdk.org >> Su

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Tetsuya Mukawa
2015-02-26 19:57 GMT+09:00 Thomas Monjalon : > 2015-02-26 18:06, Tetsuya Mukawa: >> 2015-02-26 16:03 GMT+09:00 Thomas Monjalon : >> > 2015-02-25 16:49, Stephen Hemminger: >> >> Build fails if HOTPLUG is disabled >> >> Hi Stephen, >> >>

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Tetsuya Mukawa
2015-02-26 16:03 GMT+09:00 Thomas Monjalon : > 2015-02-25 16:49, Stephen Hemminger: >> Build fails if HOTPLUG is disabled Hi Stephen, I appreciate for you reporting. > > OK thanks for reporting. > Actually there is no good reason to disable hotplug on Linux. > Though it means that it's

[dpdk-dev] [PATCH v15] testpmd: Add port hotplug support

2015-02-26 Thread Tetsuya Mukawa
. (Thanks to Iremonger, Bernard) v4: - Fix strings of command help. Signed-off-by: Tetsuya Mukawa --- app/test-pmd/cmdline.c | 137 +++ app/test-pmd/config.c | 102 -- app/test-pmd/parameters.c | 22

[dpdk-dev] [PATCH v15] librte_pmd_pcap: Add port hotplug support

2015-02-26 Thread Tetsuya Mukawa
This patch adds finalization code to free resources allocated by the PMD. v6: - Fix a paramter of rte_eth_dev_free(). v4: - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_pmd_pcap/rte_eth_pcap.c | 40 ++ 1 file changed, 40 insertions

[dpdk-dev] [PATCH v15 13/13] doc: Add port hotplug framework section to programmers guide

2015-02-26 Thread Tetsuya Mukawa
This patch adds a new section for describing port hotplug framework. v15: - Fix function names like below. - rte_eal_dev_attach() to rte_eth_dev_attach(). - rte_eal_dev_detach() to rte_eth_dev_detach(). Signed-off-by: Tetsuya Mukawa --- doc/guides/prog_guide/index.rst | 1

[dpdk-dev] [PATCH v15 12/13] ethdev: Add rte_eth_dev_attach/detach() functions

2015-02-26 Thread Tetsuya Mukawa
(). rte_eal_dev_invoke() to rte_eal_vdev_invoke(). - Add code to handle a return value of rte_eal_devargs_remove(). - Fix pci address format in rte_eal_dev_detach(). v4: - Fix comment. - Add error checking. - Fix indent of 'if' statement. - Change function name. Signed-off-by: Tetsuya Mukawa --- lib

[dpdk-dev] [PATCH v15 11/13] eal/pci: Add vdev driver initialization and uninitialization functions

2015-02-26 Thread Tetsuya Mukawa
The patch adds following functions. - rte_eal_vdev_init(); - rte_eal_vdev_uninit(); - rte_eal_parse_devargs_str(). These functions are used for driver initialization and finalization. v15: - Add this patch to hotplug series. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common

[dpdk-dev] [PATCH v15 10/13] ethdev: Add one dev_type parameter to rte_eth_dev_allocate

2015-02-26 Thread Tetsuya Mukawa
commit log. - RTE_ETH_DEV_PHYSICAL is replaced by RTE_ETH_DEV_PCI. (Thanks to Thomas Monjalon) v8: - NONE_TRACE is replaced by NO_TRACE. - Add missing symbol in version map. (Thanks to Qiu, Michael and Iremonger, Bernard) v4: - Fix comments of rte_eth_dev_type. Signed-off-by: Tetsuya Mukawa ---

[dpdk-dev] [PATCH v15 09/13] eal/pci: Add probe and close functions of pci driver

2015-02-26 Thread Tetsuya Mukawa
parameter checking. - Fix indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_pci.c | 98 - lib/librte_eal/common/eal_private.h | 15 lib/librte_eal/common/include/rte_pci.h | 32 lib/librte_eal

[dpdk-dev] [PATCH v15 08/13] eal/linux/pci: Add functions for unmapping igb_uio resources

2015-02-26 Thread Tetsuya Mukawa
ove pci_unmap_device(). It will be implemented in later patch. v8: - Fix typo. (Thanks to Iremonger, Bernard) v5: - Fix pci_unmap_device() to check pt_driver. v4: - Add parameter checking. - Add header file to determine if hotplug can be enabled. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal

[dpdk-dev] [PATCH v15 05/13] eal/pci: Consolidate pci address comparison APIs

2015-02-26 Thread Tetsuya Mukawa
er correctly. v4: - Fix calculation method of eal_compare_pci_addr(). - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/bsdapp/eal/eal_pci.c | 29 -- lib/librte_eal/common/eal_common_pci.c| 2 +- lib/librte_eal/common/include/rte_pci.h

[dpdk-dev] [PATCH v15 04/13] eal/pci, ethdev: Remove assumption that port will not be detached

2015-02-26 Thread Tetsuya Mukawa
Fix indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 2 + lib/librte_ether/rte_ethdev.c | 248 lib/librte_ether/rte_ethdev.h | 5 + 3 files changed, 164 insertions(+), 91 deletions(-)

[dpdk-dev] [PATCH v15 03/13] eal_pci: pci memory map work with driver type

2015-02-26 Thread Tetsuya Mukawa
From: Michael Qiu <michael@intel.com> With the driver type flag in struct rte_pci_dev, we do not need to always map uio devices with vfio related function when vfio enabled. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal/eal_pci.

[dpdk-dev] [PATCH v15 02/13] eal_pci: Add flag to hold kernel driver type

2015-02-26 Thread Tetsuya Mukawa
handle differently in runtime. For example, pci memory map, pot hotplug, and so on. This patch add a flag field for pci device to solve above issue. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 8 + lib/librte_eal/linuxapp/eal/eal_pci.c

[dpdk-dev] [PATCH v15 00/13] Port Hotplug Framework

2015-02-26 Thread Tetsuya Mukawa
l Qiu (2): eal_pci: Add flag to hold kernel driver type eal_pci: pci memory map work with driver type Tetsuya Mukawa (11): eal: Enable port Hotplug framework in Linux eal/pci,ethdev: Remove assumption that port will not be detached eal/pci: Consolidate pci address comparison APIs ethdev

[dpdk-dev] [PATCH v14 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-25 Thread Tetsuya Mukawa
On 2015/02/25 23:00, Thomas Monjalon wrote: > 2015-02-25 21:32, Tetsuya Mukawa: >> 2015-02-25 20:21 GMT+09:00 Thomas Monjalon : >>> 2015-02-25 13:04, Tetsuya Mukawa: >>>> --- a/lib/librte_eal/common/eal_common_dev.c >>>> +++ b/lib/librte_eal/com

[dpdk-dev] [PATCH v14 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-25 Thread Tetsuya Mukawa
2015-02-25 20:21 GMT+09:00 Thomas Monjalon : > 2015-02-25 13:04, Tetsuya Mukawa: >> --- a/lib/librte_eal/common/eal_common_dev.c >> +++ b/lib/librte_eal/common/eal_common_dev.c >> @@ -32,10 +32,13 @@ >> * OF THIS SOFTWARE, EVEN IF ADVISED OF THE

[dpdk-dev] [PATCH v1 0/2] eal: fix symbol missing in version map

2015-02-25 Thread Tetsuya Mukawa
On 2015/02/25 12:39, Cunming Liang wrote: > These two patches are the fixing for the compling error when > CONFIG_RTE_BUILD_SHARED_LIB=y. > The root cause is *per_lcore__socket_id* and *rte_sys_gettid* are missing in > the version map. > Thanks for the notification from

[dpdk-dev] [PATCH v14] testpmd: Add port hotplug support

2015-02-25 Thread Tetsuya Mukawa
) - port_id: port identifier v7: - Fix doc. (Thanks to Iremonger, Bernard) - Fix port checking implementation of star_port(); (Thanks to Qiu, Michael) v5: - Add testpmd documentation. (Thanks to Iremonger, Bernard) v4: - Fix strings of command help. Signed-off-by: Tetsuya Mukawa --- app/test

[dpdk-dev] [PATCH v14] librte_pmd_pcap: Add port hotplug support

2015-02-25 Thread Tetsuya Mukawa
This patch adds finalization code to free resources allocated by the PMD. v6: - Fix a paramter of rte_eth_dev_free(). v4: - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_pmd_pcap/rte_eth_pcap.c | 40 ++ 1 file changed, 40 insertions

[dpdk-dev] [PATCH v14 13/13] doc: Add port hotplug framework section to programmers guide

2015-02-25 Thread Tetsuya Mukawa
This patch adds a new section for describing port hotplug framework. Signed-off-by: Tetsuya Mukawa --- doc/guides/prog_guide/index.rst | 1 + doc/guides/prog_guide/port_hotplug_framework.rst | 110 +++ 2 files changed, 111 insertions(+) create mode 100644

[dpdk-dev] [PATCH v14 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-25 Thread Tetsuya Mukawa
of rte_eal_devargs_remove(). - Fix pci address format in rte_eal_dev_detach(). v4: - Fix comment. - Add error checking. - Fix indent of 'if' statement. - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_dev.c | 278 ++-- lib/librte_eal

[dpdk-dev] [PATCH v14 11/13] ethdev: Add one dev_type parameter to rte_eth_dev_allocate

2015-02-25 Thread Tetsuya Mukawa
y NO_TRACE. - Add missing symbol in version map. (Thanks to Qiu, Michael and Iremonger, Bernard) v4: - Fix comments of rte_eth_dev_type. Signed-off-by: Tetsuya Mukawa --- app/test/virtual_pmd.c | 2 +- lib/librte_ether/rte_ethdev.c| 25

[dpdk-dev] [PATCH v14 09/13] eal/linux/pci: Add functions for unmapping igb_uio resources

2015-02-25 Thread Tetsuya Mukawa
ove pci_unmap_device(). It will be implemented in later patch. v8: - Fix typo. (Thanks to Iremonger, Bernard) v5: - Fix pci_unmap_device() to check pt_driver. v4: - Add parameter checking. - Add header file to determine if hotplug can be enabled. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal

[dpdk-dev] [PATCH v14 08/13] ethdev: Add functions that will be used by port hotplug functions

2015-02-25 Thread Tetsuya Mukawa
. rte_eth_dev_get_changed_port(). rte_eth_dev_get_port_by_addr(). v4: - Add parameter checking. v3: - Fix if-condition bug while comparing pci addresses. - Add error checking codes. Reported-by: Mark Enright Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 103

[dpdk-dev] [PATCH v14 07/13] eal, ethdev: Add a function and function pointers to close ether device

2015-02-25 Thread Tetsuya Mukawa
Monjalon) v6: - Fix rte_eth_dev_uninit() to handle a return value of uninit function of PMD. v4: - Add parameter checking. - Change function names. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 6 lib/librte_ether/rte_ethdev.c | 64

[dpdk-dev] [PATCH v14 06/13] ethdev: Add rte_eth_dev_release_port to release specified port

2015-02-25 Thread Tetsuya Mukawa
of rte_eth_dev_free(). v4: - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 11 +++ lib/librte_ether/rte_ethdev.h | 12 2 files changed, 23 insertions(+) diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c index b702039

[dpdk-dev] [PATCH v14 05/13] eal/pci: Consolidate pci address comparison APIs

2015-02-25 Thread Tetsuya Mukawa
er correctly. v4: - Fix calculation method of eal_compare_pci_addr(). - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/bsdapp/eal/eal_pci.c | 29 -- lib/librte_eal/common/eal_common_pci.c| 2 +- lib/librte_eal/common/include/rte_pci.h

[dpdk-dev] [PATCH v14 04/13] eal/pci, ethdev: Remove assumption that port will not be detached

2015-02-25 Thread Tetsuya Mukawa
Fix indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 2 + lib/librte_ether/rte_ethdev.c | 248 lib/librte_ether/rte_ethdev.h | 5 + 3 files changed, 164 insertions(+), 91 deletions(-)

[dpdk-dev] [PATCH v14 03/13] eal_pci: pci memory map work with driver type

2015-02-25 Thread Tetsuya Mukawa
From: Michael Qiu <michael@intel.com> With the driver type flag in struct rte_pci_dev, we do not need to always map uio devices with vfio related function when vfio enabled. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal/eal_pci.

[dpdk-dev] [PATCH v14 02/13] eal_pci: Add flag to hold kernel driver type

2015-02-25 Thread Tetsuya Mukawa
handle differently in runtime. For example, pci memory map, pot hotplug, and so on. This patch add a flag field for pci device to solve above issue. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 8 + lib/librte_eal/linuxapp/eal/eal_pci.c

[dpdk-dev] [PATCH v14 01/13] eal: Enable port Hotplug framework in Linux

2015-02-25 Thread Tetsuya Mukawa
The patch adds CONFIG_RTE_LIBRTE_EAL_HOTPLUG in Linux and BSD configuration. So far, Hotplug functions only support linux. v9: - Move this patch at the top of this patch series. (Thanks to Thomas Monjalon) Signed-off-by: Tetsuya Mukawa --- config/common_bsdapp | 6 ++ config

[dpdk-dev] Missing symbol error

2015-02-25 Thread Tetsuya Mukawa
Hi, I've got following error when I enable CONFIG_RTE_BUILD_SHARED_LIB. dpdk/x86_64-native-linuxapp-gcc/lib/libethdev.so: undefined reference to `per_lcore__socket_id' collect2: error: ld returned 1 exit status make[5]: *** [dump_cfg] Error 1 make[4]: *** [dump_cfg] Error 2 make[4]: *** Waiting

[dpdk-dev] Cannot compile l2fwd-jobstats example

2015-02-25 Thread Tetsuya Mukawa
Hi, I cannot compile l2fwd-jobstats using master branch. Here is log $ T=x86_64-native-linuxapp-gcc make examples == Build examples for x86_64-native-linuxapp-gcc == bond == cmdline == distributor == exception_path == helloworld == ip_pipeline == ip_reassembly == ipv4_multicast

[dpdk-dev] [PATCH v13 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-24 Thread Tetsuya Mukawa
2015-02-24 22:25 GMT+09:00 Maxime Leroy : > Hi Tetsuya, > > On Tue, Feb 24, 2015 at 5:49 AM, Tetsuya Mukawa wrote: >> These functions are used for attaching or detaching a port. > [...] >> +static int >> +rte_eal_vdev_init(const char *name, const char *args) >

[dpdk-dev] [PATCH] devargs: restore empty devargs as ""

2015-02-24 Thread Tetsuya Mukawa
ng the empty argument as "" and add a note in the api. > Restore associated tests. > > Fixes: c07691ae1089 ("devargs: remove limit on parameters length") > Reported-by: Tetsuya Mukawa > Signed-off-by: David Marchand > --- > app/test/test_devargs.c

[dpdk-dev] [PATCH] eal: mmap uio resources using resourceX files

2015-02-24 Thread Tetsuya Mukawa
On 2015/02/24 0:00, Bruce Richardson wrote: > On Mon, Feb 23, 2015 at 02:57:24PM +, Bruce Richardson wrote: >> Instead of distinguishing the BAR mappings via offset within a single >> file, originally /dev/uioX, switch to mapping each individual bar via >> the appropriately numbered resourceX

[dpdk-dev] [PATCH v13 13/13] doc: Add port hotplug framework section to programmers guide

2015-02-24 Thread Tetsuya Mukawa
This patch adds a new section for describing port hotplug framework. Signed-off-by: Tetsuya Mukawa --- doc/guides/prog_guide/index.rst | 1 + doc/guides/prog_guide/port_hotplug_framework.rst | 110 +++ 2 files changed, 111 insertions(+) create mode 100644

[dpdk-dev] [PATCH v13 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-24 Thread Tetsuya Mukawa
in rte_eal_dev_detach(). v4: - Fix comment. - Add error checking. - Fix indent of 'if' statement. - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_dev.c | 284 ++-- lib/librte_eal/common/eal_common_devargs.c | 46 ++-- lib/librte_eal

[dpdk-dev] [PATCH v13 11/13] ethdev: Add one dev_type parameter to rte_eth_dev_allocate

2015-02-24 Thread Tetsuya Mukawa
y NO_TRACE. - Add missing symbol in version map. (Thanks to Qiu, Michael and Iremonger, Bernard) v4: - Fix comments of rte_eth_dev_type. Signed-off-by: Tetsuya Mukawa --- app/test/virtual_pmd.c | 2 +- lib/librte_ether/rte_ethdev.c| 25

[dpdk-dev] [PATCH v13 10/13] eal/pci: Add probe and close functions of pci driver

2015-02-24 Thread Tetsuya Mukawa
indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_pci.c | 98 - lib/librte_eal/common/eal_private.h | 15 + lib/librte_eal/common/include/rte_pci.h | 32 +++ lib/librte_eal/linuxapp/eal/eal_pci.c | 94

[dpdk-dev] [PATCH v13 09/13] eal/linux/pci: Add functions for unmapping igb_uio resources

2015-02-24 Thread Tetsuya Mukawa
ove pci_unmap_device(). It will be implemented in later patch. v8: - Fix typo. (Thanks to Iremonger, Bernard) v5: - Fix pci_unmap_device() to check pt_driver. v4: - Add parameter checking. - Add header file to determine if hotplug can be enabled. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal

[dpdk-dev] [PATCH v13 07/13] eal, ethdev: Add a function and function pointers to close ether device

2015-02-24 Thread Tetsuya Mukawa
Monjalon) v6: - Fix rte_eth_dev_uninit() to handle a return value of uninit function of PMD. v4: - Add parameter checking. - Change function names. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 6 lib/librte_ether/rte_ethdev.c | 64

[dpdk-dev] [PATCH v13 06/13] ethdev: Add rte_eth_dev_release_port to release specified port

2015-02-24 Thread Tetsuya Mukawa
of rte_eth_dev_free(). v4: - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 11 +++ lib/librte_ether/rte_ethdev.h | 12 2 files changed, 23 insertions(+) diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c index b702039

[dpdk-dev] [PATCH v13 05/13] eal/pci: Consolidate pci address comparison APIs

2015-02-24 Thread Tetsuya Mukawa
er correctly. v4: - Fix calculation method of eal_compare_pci_addr(). - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/bsdapp/eal/eal_pci.c | 29 -- lib/librte_eal/common/eal_common_pci.c| 2 +- lib/librte_eal/common/include/rte_pci.h

[dpdk-dev] [PATCH v13 04/13] eal/pci, ethdev: Remove assumption that port will not be detached

2015-02-24 Thread Tetsuya Mukawa
Fix indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 2 + lib/librte_ether/rte_ethdev.c | 248 lib/librte_ether/rte_ethdev.h | 5 + 3 files changed, 164 insertions(+), 91 deletions(-)

[dpdk-dev] [PATCH v13 03/13] eal_pci: pci memory map work with driver type

2015-02-24 Thread Tetsuya Mukawa
From: Michael Qiu <michael@intel.com> With the driver type flag in struct rte_pci_dev, we do not need to always map uio devices with vfio related function when vfio enabled. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal/eal_pci.

[dpdk-dev] [PATCH v13 01/13] eal: Enable port Hotplug framework in Linux

2015-02-24 Thread Tetsuya Mukawa
The patch adds CONFIG_RTE_LIBRTE_EAL_HOTPLUG in Linux and BSD configuration. So far, Hotplug functions only support linux. v9: - Move this patch at the top of this patch series. (Thanks to Thomas Monjalon) Signed-off-by: Tetsuya Mukawa --- config/common_bsdapp | 6 ++ config

[dpdk-dev] [PATCH v12] testpmd: Add port hotplug support

2015-02-23 Thread Tetsuya Mukawa
) - port_id: port identifier v7: - Fix doc. (Thanks to Iremonger, Bernard) - Fix port checking implementation of star_port(); (Thanks to Qiu, Michael) v5: - Add testpmd documentation. (Thanks to Iremonger, Bernard) v4: - Fix strings of command help. Signed-off-by: Tetsuya Mukawa --- app/test

[dpdk-dev] [PATCH v12] librte_pmd_pcap: Add port hotplug support

2015-02-23 Thread Tetsuya Mukawa
This patch adds finalization code to free resources allocated by the PMD. v6: - Fix a paramter of rte_eth_dev_free(). v4: - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_pmd_pcap/rte_eth_pcap.c | 40 ++ 1 file changed, 40 insertions

[dpdk-dev] [PATCH v12 13/13] doc: Add port hotplug framework section to programmers guide

2015-02-23 Thread Tetsuya Mukawa
This patch adds a new section for describing port hotplug framework. Signed-off-by: Tetsuya Mukawa --- doc/guides/prog_guide/index.rst | 1 + doc/guides/prog_guide/port_hotplug_framework.rst | 110 +++ 2 files changed, 111 insertions(+) create mode 100644

[dpdk-dev] [PATCH v12 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-23 Thread Tetsuya Mukawa
comment. - Add error checking. - Fix indent of 'if' statement. - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_dev.c | 285 ++-- lib/librte_eal/common/eal_common_devargs.c | 46 ++-- lib/librte_eal/common/eal_private.h

[dpdk-dev] [PATCH v12 11/13] ethdev: Add one dev_type parameter to rte_eth_dev_allocate

2015-02-23 Thread Tetsuya Mukawa
y NO_TRACE. - Add missing symbol in version map. (Thanks to Qiu, Michael and Iremonger, Bernard) v4: - Fix comments of rte_eth_dev_type. Signed-off-by: Tetsuya Mukawa --- app/test/virtual_pmd.c | 2 +- lib/librte_ether/rte_ethdev.c| 25

[dpdk-dev] [PATCH v12 10/13] eal/pci: Add probe and close functions of pci driver

2015-02-23 Thread Tetsuya Mukawa
indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_pci.c | 98 - lib/librte_eal/common/eal_private.h | 15 + lib/librte_eal/common/include/rte_pci.h | 32 +++ lib/librte_eal/linuxapp/eal/eal_pci.c | 94

[dpdk-dev] [PATCH v12 09/13] eal/linux/pci: Add functions for unmapping igb_uio resources

2015-02-23 Thread Tetsuya Mukawa
ove pci_unmap_device(). It will be implemented in later patch. v8: - Fix typo. (Thanks to Iremonger, Bernard) v5: - Fix pci_unmap_device() to check pt_driver. v4: - Add parameter checking. - Add header file to determine if hotplug can be enabled. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal

[dpdk-dev] [PATCH v12 08/13] ethdev: Add functions that will be used by port hotplug functions

2015-02-23 Thread Tetsuya Mukawa
. rte_eth_dev_get_changed_port(). rte_eth_dev_get_port_by_addr(). v4: - Add parameter checking. v3: - Fix if-condition bug while comparing pci addresses. - Add error checking codes. Reported-by: Mark Enright Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 103

[dpdk-dev] [PATCH v12 07/13] eal, ethdev: Add a function and function pointers to close ether device

2015-02-23 Thread Tetsuya Mukawa
Monjalon) v6: - Fix rte_eth_dev_uninit() to handle a return value of uninit function of PMD. v4: - Add parameter checking. - Change function names. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 6 lib/librte_ether/rte_ethdev.c | 64

[dpdk-dev] [PATCH v12 06/13] ethdev: Add rte_eth_dev_release_port to release specified port

2015-02-23 Thread Tetsuya Mukawa
of rte_eth_dev_free(). v4: - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 11 +++ lib/librte_ether/rte_ethdev.h | 12 2 files changed, 23 insertions(+) diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c index 0e1e5c9

[dpdk-dev] [PATCH v12 05/13] eal/pci: Consolidate pci address comparison APIs

2015-02-23 Thread Tetsuya Mukawa
er correctly. v4: - Fix calculation method of eal_compare_pci_addr(). - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/bsdapp/eal/eal_pci.c | 29 -- lib/librte_eal/common/eal_common_pci.c| 2 +- lib/librte_eal/common/include/rte_pci.h

[dpdk-dev] [PATCH v12 04/13] eal/pci, ethdev: Remove assumption that port will not be detached

2015-02-23 Thread Tetsuya Mukawa
Fix indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 2 + lib/librte_ether/rte_ethdev.c | 248 lib/librte_ether/rte_ethdev.h | 5 + 3 files changed, 164 insertions(+), 91 deletions(-)

[dpdk-dev] [PATCH v12 03/13] eal_pci: pci memory map work with driver type

2015-02-23 Thread Tetsuya Mukawa
From: Michael Qiu <michael@intel.com> With the driver type flag in struct rte_pci_dev, we do not need to always map uio devices with vfio related function when vfio enabled. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal/eal_pci.

[dpdk-dev] [PATCH v12 02/13] eal_pci: Add flag to hold kernel driver type

2015-02-23 Thread Tetsuya Mukawa
handle differently in runtime. For example, pci memory map, pot hotplug, and so on. This patch add a flag field for pci device to solve above issue. Signed-off-by: Michael Qiu Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 8 + lib/librte_eal/linuxapp/eal/eal_pci.c

[dpdk-dev] [PATCH v12 01/13] eal: Enable port Hotplug framework in Linux

2015-02-23 Thread Tetsuya Mukawa
The patch adds CONFIG_RTE_LIBRTE_EAL_HOTPLUG in Linux and BSD configuration. So far, Hotplug functions only support linux. v9: - Move this patch at the top of this patch series. (Thanks to Thomas Monjalon) Signed-off-by: Tetsuya Mukawa --- config/common_bsdapp | 6 ++ config

[dpdk-dev] [PATCH v12 00/13] Port Hotplug Framework

2015-02-23 Thread Tetsuya Mukawa
ove 'rte_eth_dev_validate_port()', and cleanup codes. Michael Qiu (2): eal_pci: Add flag to hold kernel driver type eal_pci: pci memory map work with driver type Tetsuya Mukawa (11): eal: Enable port Hotplug framework in Linux eal/pci,ethdev: Remove assumption that port will not be detac

[dpdk-dev] [PATCH v11 08/13] ethdev: Add functions that will be used by port hotplug functions

2015-02-23 Thread Tetsuya Mukawa
On 2015/02/23 20:39, Iremonger, Bernard wrote: the pointer diff --git a/lib/librte_ether/rte_ether_version.map b/lib/librte_ether/rte_ether_version.map index f66fd2d..099c769 100644 --- a/lib/librte_ether/rte_ether_version.map +++ b/lib/librte_ether/rte_ether_version.map

[dpdk-dev] Testpmd returns error.

2015-02-23 Thread Tetsuya Mukawa
On 2015/02/23 20:12, Bruce Richardson wrote: > On Mon, Feb 23, 2015 at 11:33:45AM +0900, Tetsuya Mukawa wrote: >> On 2015/02/23 5:46, Bruce Richardson wrote: >>> On Sun, Feb 22, 2015 at 02:30:02PM +0900, Tetsuya Mukawa wrote: >>>> Hi, >>>> >>>&

[dpdk-dev] [PATCH v11 08/13] ethdev: Add functions that will be used by port hotplug functions

2015-02-23 Thread Tetsuya Mukawa
On 2015/02/23 20:01, Iremonger, Bernard wrote: >> -Original Message- >> From: Tetsuya Mukawa [mailto:mukawa at igel.co.jp] >> Sent: Monday, February 23, 2015 5:09 AM >> To: dev at dpdk.org >> Cc: Qiu, Michael; Iremonger, Bernard; maxime.leroy at 6wind.com;

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-23 Thread Tetsuya Mukawa
This patch adds port hotplug support to Null PMD. v9: - Use rte_eth_dev_release_port() instead of rte_eth_dev_free(). v7: - Add parameter checkings. (Thanks to Iremonger, Bernard) v6: - Fix a parameter of rte_eth_dev_free(). v4: - Fix commit title. Signed-off-by: Tetsuya Mukawa --- lib

[dpdk-dev] [PATCH v11 1/2] librte_pmd_null: Add Null PMD

2015-02-23 Thread Tetsuya Mukawa
parameter checkings. (Thanks to Iremonger, Bernard) - Remove needless "__rte_unused". v4: - Fix memory leak. (Thanks to Iremonger, Bernard) Signed-off-by: Tetsuya Mukawa --- config/common_bsdapp | 5 + config/common_linuxapp |

[dpdk-dev] [PATCH v11] testpmd: Add port hotplug support

2015-02-23 Thread Tetsuya Mukawa
) - port_id: port identifier v7: - Fix doc. (Thanks to Iremonger, Bernard) - Fix port checking implementation of star_port(); (Thanks to Qiu, Michael) v5: - Add testpmd documentation. (Thanks to Iremonger, Bernard) v4: - Fix strings of command help. Signed-off-by: Tetsuya Mukawa --- app/test

[dpdk-dev] [PATCH v11] librte_pmd_pcap: Add port hotplug support

2015-02-23 Thread Tetsuya Mukawa
This patch adds finalization code to free resources allocated by the PMD. v6: - Fix a paramter of rte_eth_dev_free(). v4: - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_pmd_pcap/rte_eth_pcap.c | 40 ++ 1 file changed, 40 insertions

[dpdk-dev] [PATCH v11 13/13] doc: Add port hotplug framework section to programmers guide

2015-02-23 Thread Tetsuya Mukawa
This patch adds a new section for describing port hotplug framework. Signed-off-by: Tetsuya Mukawa --- doc/guides/prog_guide/index.rst | 1 + doc/guides/prog_guide/port_hotplug_framework.rst | 110 +++ 2 files changed, 111 insertions(+) create mode 100644

[dpdk-dev] [PATCH v11 12/13] eal/pci: Add rte_eal_dev_attach/detach() functions

2015-02-23 Thread Tetsuya Mukawa
comment. - Add error checking. - Fix indent of 'if' statement. - Change function name. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_dev.c | 285 ++-- lib/librte_eal/common/eal_common_devargs.c | 46 ++-- lib/librte_eal/common/eal_private.h

[dpdk-dev] [PATCH v11 11/13] ethdev: Add one dev_type parameter to rte_eth_dev_allocate

2015-02-23 Thread Tetsuya Mukawa
Iremonger, Bernard) v4: - Fix comments of rte_eth_dev_type. Signed-off-by: Tetsuya Mukawa --- app/test/virtual_pmd.c | 2 +- lib/librte_ether/rte_ethdev.c| 25 +++-- lib/librte_ether/rte_ethdev.h| 25

[dpdk-dev] [PATCH v11 10/13] eal/pci: Add probe and close functions of pci driver

2015-02-23 Thread Tetsuya Mukawa
indent of 'if' statement. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/eal_common_pci.c | 98 - lib/librte_eal/common/eal_private.h | 15 + lib/librte_eal/common/include/rte_pci.h | 32 +++ lib/librte_eal/linuxapp/eal/eal_pci.c | 94

[dpdk-dev] [PATCH v11 09/13] eal/linux/pci: Add functions for unmapping igb_uio resources

2015-02-23 Thread Tetsuya Mukawa
ove pci_unmap_device(). It will be implemented in later patch. v8: - Fix typo. (Thanks to Iremonger, Bernard) v5: - Fix pci_unmap_device() to check pt_driver. v4: - Add parameter checking. - Add header file to determine if hotplug can be enabled. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/linuxapp/eal

[dpdk-dev] [PATCH v11 08/13] ethdev: Add functions that will be used by port hotplug functions

2015-02-23 Thread Tetsuya Mukawa
. rte_eth_dev_get_changed_port(). rte_eth_dev_get_port_by_addr(). v4: - Add parameter checking. v3: - Fix if-condition bug while comparing pci addresses. - Add error checking codes. Reported-by: Mark Enright Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 103

[dpdk-dev] [PATCH v11 07/13] eal, ethdev: Add a function and function pointers to close ether device

2015-02-23 Thread Tetsuya Mukawa
Monjalon) v6: - Fix rte_eth_dev_uninit() to handle a return value of uninit function of PMD. v4: - Add parameter checking. - Change function names. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/common/include/rte_pci.h | 6 lib/librte_ether/rte_ethdev.c | 64

[dpdk-dev] [PATCH v11 06/13] ethdev: Add rte_eth_dev_release_port to release specified port

2015-02-23 Thread Tetsuya Mukawa
of rte_eth_dev_free(). v4: - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_ether/rte_ethdev.c | 11 +++ lib/librte_ether/rte_ethdev.h | 12 2 files changed, 23 insertions(+) diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c index 0e1e5c9

[dpdk-dev] [PATCH v11 05/13] eal/pci: Consolidate pci address comparison APIs

2015-02-23 Thread Tetsuya Mukawa
er correctly. v4: - Fix calculation method of eal_compare_pci_addr(). - Add parameter checking. Signed-off-by: Tetsuya Mukawa --- lib/librte_eal/bsdapp/eal/eal_pci.c | 29 -- lib/librte_eal/common/eal_common_pci.c| 2 +- lib/librte_eal/common/include/rte_pci.h

<    1   2   3   4   5   6   7   8   9   10   >