[dpdk-dev] No probed ethernet devices with shared library

2015-05-18 Thread Panu Matilainen
On 05/18/2015 12:55 AM, Stuart Andrews wrote: > Hello, > > I've been trying to create an app which uses the DPDK shared library and > therefore I have > > CONFIG_RTE_BUILD_SHARED_LIB=y > > However, when I try to run 'test-pmd' I get > > EAL: No probed ethernet devices > > This is strange because wh

[dpdk-dev] [PATCH] vhost: make vhost lockless enqueue configurable

2015-04-29 Thread Panu Matilainen
On 04/29/2015 02:29 PM, Huawei Xie wrote: > vhost enabled vSwitch could have their own thread-safe vring enqueue policy. > Add the RTE_LIBRTE_VHOST_LOCKLESS_ENQ macro for vhost lockless enqueue. > Turn it off by default. > > Signed-off-by: Huawei Xie > --- > config/common_linuxapp| 1 +

[dpdk-dev] Did we reduce unnecessary linkage too well?

2016-09-30 Thread Panu Matilainen
On 09/30/2016 01:15 PM, Bruce Richardson wrote: > On Thu, Sep 29, 2016 at 09:26:48AM +0200, Christian Ehrhardt wrote: >> On Thu, Sep 29, 2016 at 9:20 AM, Panu Matilainen >> wrote: >> >>> >>> Yup. Set CONFIG_RTE_EAL_PMD_PATH to the path where your PMDs are &g

[dpdk-dev] [PATCH] dpdk_procinfo: check for primary process

2016-09-07 Thread Panu Matilainen
On 09/06/2016 08:12 PM, Maryam Tahhan wrote: > Add a check to see if the primary process is running and exit gracefully if it > is not. > > Suggested-by: Patrick Kutch > Signed-off-by: Maryam Tahhan > --- > app/proc_info/main.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/app/pr

[dpdk-dev] [PATCH v3 12/15] ether: extract function eth_dev_get_intr_handle

2016-09-16 Thread Panu Matilainen
On 09/15/2016 05:05 PM, Thomas Monjalon wrote: > 2016-09-15 14:02, Hunt, David: >> On 9/9/2016 9:43 AM, Shreyansh Jain wrote: >>> +static inline >>> +struct rte_intr_handle *eth_dev_get_intr_handle(struct rte_eth_dev *dev) >>> +{ >>> + if (dev->pci_dev) { >>> + return &dev->pci_dev->int

[dpdk-dev] [PATCH v3 02/15] eal/soc: add rte_eal_soc_register/unregister logic

2016-09-16 Thread Panu Matilainen
On 09/15/2016 05:09 PM, Thomas Monjalon wrote: > 2016-09-15 15:09, Jan Viktorin: >> On Thu, 15 Sep 2016 14:00:25 +0100 >> "Hunt, David" wrote: >> new file mode 100644 index 000..56135ed --- /dev/null +++ b/lib/librte_eal/common/eal_common_soc.c @@ -0,0 +1,56 @@ +/

[dpdk-dev] [PATCH 3/3] drivers/net:build support for new tap device driver

2016-09-16 Thread Panu Matilainen
On 09/15/2016 05:10 PM, Keith Wiles wrote: > Signed-off-by: Keith Wiles > --- > config/common_linuxapp | 3 +++ > drivers/net/Makefile | 1 + > mk/rte.app.mk | 1 + > 3 files changed, 5 insertions(+) > > diff --git a/config/common_linuxapp b/config/common_linuxapp > index 2483dfa..704c

[dpdk-dev] [PATCH] net/mlx: fix compile errors with ignore pedantic pragma

2016-09-21 Thread Panu Matilainen
- Original Message - > On Tue, Sep 20, 2016 at 02:51:27PM +0200, Adrien Mazarguil wrote: > > On Mon, Sep 19, 2016 at 04:26:05PM +0100, Bruce Richardson wrote: > > > On Mon, Sep 19, 2016 at 04:59:59PM +0200, Adrien Mazarguil wrote: > > > > Hi Bruce, > > > > > > > > On Mon, Sep 19, 2016 at 0

[dpdk-dev] Did we reduce unnecessary linkage too well?

2016-09-29 Thread Panu Matilainen
On 09/29/2016 09:58 AM, Christian Ehrhardt wrote: > Hi, > I was finally getting to more deeply re-validate Openvswitch 2.6 together > with DPDK 16.07. And I think I found a whiplash of our effort to reduce > unnecessary hard linkage. > > Trying to avoid cross-posting, picking DPDK list and the main

[dpdk-dev] [PATCH v4 03/12] vhost: update version map file

2015-08-12 Thread Panu Matilainen
On 08/12/2015 11:02 AM, Ouyang Changchun wrote: > From: Changchun Ouyang > > it is added in v4. > > Signed-off-by: Changchun Ouyang > --- > lib/librte_vhost/rte_vhost_version.map | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_vhost/rte_vhost_version.map >

[dpdk-dev] [PATCH] mk: fix the combined library problems by replacing it with a linker script

2015-12-01 Thread Panu Matilainen
On 11/30/2015 06:41 PM, Stephen Hemminger wrote: > On Mon, 30 Nov 2015 10:03:43 -0500 > Neil Horman wrote: > >> On Wed, Nov 25, 2015 at 08:08:37AM -0800, Stephen Hemminger wrote: >>> On Wed, 25 Nov 2015 10:38:48 +0200 >>> Panu Matilainen wrote: >>> >

[dpdk-dev] 2.3 Roadmap

2015-12-01 Thread Panu Matilainen
On 12/01/2015 12:03 PM, Bruce Richardson wrote: > On Mon, Nov 30, 2015 at 05:16:55PM -0800, Stephen Hemminger wrote: >> On Mon, 30 Nov 2015 22:53:50 + >> Kyle Larose wrote: >> >>> Hi Tim, >>> >>> On Mon, Nov 30, 2015 at 3:50 PM, O'Driscoll, Tim >> intel.com> wrote: >>> Tcpdump Support: Su

[dpdk-dev] [PATCH] mk: bump minimum march in default machine

2015-12-01 Thread Panu Matilainen
On 12/01/2015 04:26 PM, Christian Ehrhardt wrote: > While playing with building 2.2-rc2 I found that our usual way didn't work > anymore. > We usually configured "make config T=x86_64-native-linuxapp-gcc" but then > set CONFIG_RTE_MACHINE="default" to get something like the "lowest acceptable > bui

[dpdk-dev] 2.3 Roadmap

2015-12-01 Thread Panu Matilainen
On 12/01/2015 04:48 PM, Vincent JARDIN wrote: > On 01/12/2015 15:27, Panu Matilainen wrote: >> The problem with that (unless I'm missing something here) is that KNI >> requires using out-of-tree kernel modules which makes it pretty much a >> non-option for distros. &

[dpdk-dev] [PATCH 00/10] standard make install

2015-12-02 Thread Panu Matilainen
On 12/02/2015 05:57 AM, Thomas Monjalon wrote: > Following the recent discussions, this is a proposal to have a standard > installation process while keeping compatibility with most of the old > behaviours. > Thank you Mario and Bruce for having submitted other proposals. > I hope there will be a s

[dpdk-dev] [PATCH v8 00/11] Add installation rules for dpdk files.

2015-12-02 Thread Panu Matilainen
On 12/01/2015 09:39 PM, Mario Carrillo wrote: > DPDK package lacks of a mechanism to install libraries, headers > applications, kernel modules and sdk files to a file system tree. > This patch set allows to install files based on the next > proposal: > http://www.freedesktop.org/software/systemd/ma

[dpdk-dev] [PATCH 00/10] standard make install

2015-12-02 Thread Panu Matilainen
On 12/02/2015 11:25 AM, Thomas Monjalon wrote: > 2015-12-02 09:44, Panu Matilainen: >> That aside, a bigger problem is that it doesn't seem to work. >> >> make clean >> make config T=x86_64-native-linuxapp-gcc >> make >> make install DESTDIR=/tmp/dpdk-r

[dpdk-dev] [PATCH 06/10] mk: install kernel modules

2015-12-02 Thread Panu Matilainen
On 12/02/2015 05:57 AM, Thomas Monjalon wrote: > Add kernel modules to "make install". > Nothing is done if there is no kernel module compiled. > > On native Linux, this path is suggested: > kerneldir=/lib/modules/$(uname -r)/extra/dpdk > > Suggested-by: Mario Carrillo > Signed-off-by: Thoma

[dpdk-dev] [PATCH 07/10] mk: install binding tool in sbin directory

2015-12-02 Thread Panu Matilainen
On 12/02/2015 05:57 AM, Thomas Monjalon wrote: > sbin/dpdk_nic_bind is a symbolic link to tools/dpdk_nic_bind.py > where some python objects may be generated. > > Signed-off-by: Thomas Monjalon > --- > mk/rte.sdkinstall.mk | 4 > 1 file changed, 4 insertions(+) > > diff --git a/mk/rte.sdki

[dpdk-dev] [PATCH 03/10] mk: install a standard cutomizable tree

2015-12-02 Thread Panu Matilainen
On 12/02/2015 05:57 AM, Thomas Monjalon wrote: > The rule "install" follows these conventions: > https://www.gnu.org/prep/standards/html_node/Directory-Variables.html > https://www.gnu.org/prep/standards/html_node/DESTDIR.html > > The variable sdkdir has been added to the more standards ones, > to

[dpdk-dev] [PATCH 03/10] mk: install a standard cutomizable tree

2015-12-02 Thread Panu Matilainen
On 12/02/2015 01:25 PM, Thomas Monjalon wrote: > 2015-12-02 12:27, Panu Matilainen: >> On 12/02/2015 05:57 AM, Thomas Monjalon wrote: >>> The old installed tree was static and always had .config, includes and >>> libs in a RTE_TARGET subdirectory. There is no such

[dpdk-dev] [PATCH 03/10] mk: install a standard cutomizable tree

2015-12-02 Thread Panu Matilainen
On 12/02/2015 03:05 PM, Thomas Monjalon wrote: > 2015-12-02 14:54, Panu Matilainen: >> On 12/02/2015 01:25 PM, Thomas Monjalon wrote: >>> 2015-12-02 12:27, Panu Matilainen: >>>> $(prefix)/share is supposed to be shareable across different >>>> architec

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-02 Thread Panu Matilainen
On 12/02/2015 05:43 AM, Yuanhan Liu wrote: > VHOST_USER_SET_LOG_BASE request is used to tell the backend (dpdk > vhost-user) where we should log dirty pages, and how big the log > buffer is. > > This request introduces a new payload: > > typedef struct VhostUserLog { > uint64_t

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-02 Thread Panu Matilainen
On 12/02/2015 04:31 PM, Yuanhan Liu wrote: > On Wed, Dec 02, 2015 at 03:53:45PM +0200, Panu Matilainen wrote: >> On 12/02/2015 05:43 AM, Yuanhan Liu wrote: >>> VHOST_USER_SET_LOG_BASE request is used to tell the backend (dpdk >>> vhost-user) where we should log dirty

[dpdk-dev] [PATCH] scripts: support any legal git revisions as abi validation range

2015-12-02 Thread Panu Matilainen
In addition to git tags, support validating abi between any legal gitrevisions(7) syntaxes, such as "validate-abi.sh . -1 " "validate-abi.sh master mybrach " etc in addition to validating between tags. Makes it easier to run the validator for in-development work. Signed-off

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-02 Thread Panu Matilainen
On 12/02/2015 05:09 PM, Yuanhan Liu wrote: > On Wed, Dec 02, 2015 at 04:48:14PM +0200, Panu Matilainen wrote: > ... >>>>> diff --git a/lib/librte_vhost/rte_virtio_net.h >>>>> b/lib/librte_vhost/rte_virtio_net.h >>>>> index 5687452..416dac2 1

[dpdk-dev] [PATCH v4 0/2] Add support for driver directories

2015-12-03 Thread Panu Matilainen
r invalid plugin paths >>>>> - support directories via the existing -d option instead of adding >>>>> new >>>>> >>>>> v3: - merge the first commits >>>>> >>>>> v2: - move code to eal/common >>>>

[dpdk-dev] [PATCH v2] mk: fix compile error and ABI versioning for combined shared library

2015-12-03 Thread Panu Matilainen
On 12/03/2015 03:22 AM, Ferruh Yigit wrote: > Fixes following error (observed when versioning macros used): >LD libdpdk.so >/usr/bin/ld: /root/dpdk/build/lib/libdpdk.so: version node not found >for symbol @DPDK_x.y > > Also resulting combined library contains symbol version information:

[dpdk-dev] [PATCH v2 10/12] mk: install examples

2015-12-03 Thread Panu Matilainen
On 12/03/2015 07:02 AM, Thomas Monjalon wrote: > The examples are part of the installed documentation. > > Signed-off-by: Thomas Monjalon > --- > mk/rte.sdkinstall.mk | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/mk/rte.sdkinstall.mk b/mk/rte.sdkinstall.mk > index 902a933..13fa819 10

[dpdk-dev] [PATCH v2 00/12] standard make install

2015-12-03 Thread Panu Matilainen
On 12/03/2015 07:01 AM, Thomas Monjalon wrote: > Following the recent discussions, this is a proposal to have a standard > installation process while keeping compatibility with most of the old > behaviours. > > v2 changes: > - fix default build dir > - RTE_TARGET subdir in $(sdkdir). > - better ker

[dpdk-dev] [PATCH v2 10/12] mk: install examples

2015-12-03 Thread Panu Matilainen
On 12/03/2015 03:32 PM, Thomas Monjalon wrote: > 2015-12-03 15:19, Panu Matilainen: >> On 12/03/2015 07:02 AM, Thomas Monjalon wrote: >>> The examples are part of the installed documentation. >>> >>> Signed-off-by: Thomas Monjalon >>> --- >>>

[dpdk-dev] [PATCH] scripts: support any legal git revisions as abi validation range

2015-12-03 Thread Panu Matilainen
On 12/03/2015 02:14 PM, Mcnamara, John wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen >> Sent: Wednesday, December 2, 2015 4:51 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH] scripts: supp

[dpdk-dev] [PATCH] scripts: support any legal git revisions as abi validation range

2015-12-03 Thread Panu Matilainen
On 12/03/2015 03:28 PM, Thomas Monjalon wrote: > 2015-12-03 12:14, Mcnamara, John: >> Also, if someone has some bandwidth it would be good to add an option >> to pass -j with an optional number to "make" in the script. > > We can use -j without any number: > "make will not limit the number of jobs

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-03 Thread Panu Matilainen
In addition to git tags, support validating abi between any legal gitrevisions(7) syntaxes, such as "validate-abi.sh -1 . " "validate-abi.sh master mybranch " etc in addition to validating between tags. Makes it easier to run the validator for in-development work. Signed-off

[dpdk-dev] [announce] driverctl: utility for persistent alternative driver binding

2015-12-04 Thread Panu Matilainen
Hi all, While this is not directly related to DPDK or OVS, it is potentially useful for users of both, so excuse me for cross-posting. Quoting from the project README (for the full text see http://laiskiainen.org/git/?p=driverctl.git;a=blob_plain;f=README) > driverctl is a tool for manipulatin

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Panu Matilainen
On 12/07/2015 01:07 AM, Thomas Monjalon wrote: > 2015-12-02 15:53, Panu Matilainen: >> This (and other changes in patch 2 breaks the librte_vhost ABI again, so >> you'd need to at least add a deprecation note to 2.2 to be able to do it >> in 2.3 at all according to the A

[dpdk-dev] [PATCH v2] mk: pass EXTRA_CFLAGS to AUTO_CPUFLAGS to enable local modifications

2015-12-07 Thread Panu Matilainen
On 12/04/2015 08:53 PM, Thomas Monjalon wrote: We have encountered a CPU where the AES-NI instruction set is disabled due to export restrictions. Since the build machine and target machine is different, using -native configs doesn't work, and on this CPU, the application refuses

[dpdk-dev] [PATCH v4] ip_pipeline: add flow actions pipeline

2015-12-07 Thread Panu Matilainen
On 12/07/2015 03:17 AM, Thomas Monjalon wrote: > 2015-11-18 17:09, Fan Zhang: >> Flow actions pipeline is an extension of flow-classification pipeline. >> Some of the operations of flow classification pipeline such as traffic >> metering/marking(for e.g. Single Rate Three Color Marker (srTCM), Two

[dpdk-dev] [PATCH] mk: fix external shared library dependencies of drivers, round 2

2015-12-07 Thread Panu Matilainen
Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a, but for qat and mpipe drivers. The former did not exist when the previous patch was sent and latter I just missed. Fixes: 5f9115e58cc6 ("mk: fix shared library dependencies of drivers") Signed-off-by: Panu Matilainen --

[dpdk-dev] [PATCH v2] mk: fix external shared library dependencies of drivers, round 2

2015-12-07 Thread Panu Matilainen
Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a, but for qat and mpipe drivers. The former did not exist when the previous patch was sent and latter I just missed. Fixes: 5f9115e58cc6 ("mk: fix shared library dependencies of drivers") Signed-off-by: Panu Matilainen --- v

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Panu Matilainen
On 12/07/2015 01:28 PM, Thomas Monjalon wrote: > 2015-12-07 08:29, Panu Matilainen: >> On 12/07/2015 01:07 AM, Thomas Monjalon wrote: >>> 2015-12-02 15:53, Panu Matilainen: >>>> This (and other changes in patch 2 breaks the librte_vhost ABI again, so >>>>

[dpdk-dev] [PATCH] i40e: remove redundant compiler warning disablers

2015-12-07 Thread Panu Matilainen
These may have been required at some point but current i40e base driver compiles cleanly without them, at least with clang 3.7.0 and gcc 5.1.1. Signed-off-by: Panu Matilainen --- drivers/net/i40e/Makefile | 13 - 1 file changed, 13 deletions(-) diff --git a/drivers/net/i40e

[dpdk-dev] [PATCH] mk: fix warning spew when EXTRA_CFLAGS specifies warning flags

2015-12-07 Thread Panu Matilainen
ty ignored without -Wformat [-Wformat-security] Passing WERROR_FLAGS to AUTO_CPU helper makes the warning flag usage consistent throughout the codebase, silencing the warnings. Fixes: 9aa2053c6e81 ("mk: influence CPU flags with user input") Signed-off-by: Panu Matilainen --- mk/rte.c

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-07 Thread Panu Matilainen
On 12/03/2015 04:05 PM, Panu Matilainen wrote: > In addition to git tags, support validating abi between any legal > gitrevisions(7) syntaxes, such as "validate-abi.sh -1 . " > "validate-abi.sh master mybranch " etc in addition to > validating between tags. Make

[dpdk-dev] [PATCH v2] scripts: support any legal git revisions as abi validation range

2015-12-07 Thread Panu Matilainen
On 12/07/2015 04:32 PM, Thomas Monjalon wrote: > 2015-12-07 16:09, Panu Matilainen: >> On 12/03/2015 04:05 PM, Panu Matilainen wrote: >>> In addition to git tags, support validating abi between any legal >>> gitrevisions(7) syntaxes, such as "validate-abi.sh -1

[dpdk-dev] [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

2015-12-07 Thread Panu Matilainen
On 12/07/2015 03:55 PM, Thomas Monjalon wrote: > 2015-12-07 13:41, Panu Matilainen: >> On 12/07/2015 01:28 PM, Thomas Monjalon wrote: >>> 2015-12-07 08:29, Panu Matilainen: >>>> On 12/07/2015 01:07 AM, Thomas Monjalon wrote: >>>>> 2015-12-02 15:53,

[dpdk-dev] [PATCH v3 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-08 Thread Panu Matilainen
On 12/07/2015 10:55 PM, Stephen Hemminger wrote: > On Mon, 7 Dec 2015 19:36:05 +0100 > Kamil Rytarowski wrote: > >> +/* Check if there is sysfs mounted */ >> +if (stat("/sys/module", &st) != 0) { >> +RTE_LOG(DEBUG, EAL, "Open /sys/module failed: %s\n", >> +s

[dpdk-dev] [PATCH] mk: fix external shared library dependencies of libraries

2015-12-08 Thread Panu Matilainen
operation. Linking with the combined library (whether shared or not) still requires knowing the internal dependencies, and intra-dpdk dependencies are also not currently recorded. Signed-off-by: Panu Matilainen --- Note: I haven't tested on FreeBSD, it should be straightforward but it wou

[dpdk-dev] [PATCH] mk: fix external shared library dependencies of libraries

2015-12-08 Thread Panu Matilainen
On 12/08/2015 12:11 PM, Thomas Monjalon wrote: > Hi Panu, > > 2015-12-08 10:30, Panu Matilainen: >> --- a/lib/librte_vhost/Makefile >> +++ b/lib/librte_vhost/Makefile >> @@ -44,10 +44,12 @@ CFLAGS += -I vhost_user >> else >> CFLAGS += -I vhost_cuse -

[dpdk-dev] [PATCH] mk: fix external shared library dependencies of libraries

2015-12-08 Thread Panu Matilainen
On 12/08/2015 01:19 PM, Panu Matilainen wrote: > On 12/08/2015 12:11 PM, Thomas Monjalon wrote: >> Hi Panu, >> >> 2015-12-08 10:30, Panu Matilainen: >>> --- a/lib/librte_vhost/Makefile >>> +++ b/lib/librte_vhost/Makefile >>> @@ -44,10 +44,12 @@ CFLAG

[dpdk-dev] [PATCH v2] mk: fix external shared library dependencies of libraries

2015-12-08 Thread Panu Matilainen
, shared libraries should always know their own dependencies for sane operation. Linking with the combined library (whether shared or not) still requires knowing the internal dependencies, and intra-dpdk dependencies are also not currently recorded. Signed-off-by: Panu Matilainen --- v2: - clean up

[dpdk-dev] [ovs-discuss] [announce] driverctl: utility for persistent alternative driver binding

2015-12-08 Thread Panu Matilainen
On 12/04/2015 05:44 PM, Gray, Mark D wrote: > I welcome this initiative, one question below: > >> -Original Message- >> From: discuss [mailto:discuss-bounces at openvswitch.org] On Behalf Of Panu >> Matilainen >> Sent: Friday, December 4, 2015 10:54 AM &g

[dpdk-dev] [PATCH v4 2/2] eal/linux: Add support for handling built-in kernel modules

2015-12-09 Thread Panu Matilainen
On 12/08/2015 05:33 PM, Kamil Rytarowski wrote: > Currently rte_eal_check_module() detects Linux kernel modules via reading > /proc/modules. Built-in ones aren't listed there and therefore they are not > being found by the script. > > Add support for checking built-in modules with parsing the sysfs

[dpdk-dev] [PATCH v2] mk: fix external shared library dependencies of libraries

2015-12-09 Thread Panu Matilainen
On 12/08/2015 06:28 PM, Sergio Gonzalez Monroy wrote: > On 08/12/2015 11:47, Panu Matilainen wrote: >> Similar to commit 5f9115e58cc6f304ff4ade694cf5823d32887d1a etc, but >> for libraries. Clean up librte_vhost CFLAGS/LDFLAGS/LDLIBS confusion >> while at it. >> >&g

[dpdk-dev] [PATCH v2] examples/vhost: reduce number of hugepages needed

2015-12-11 Thread Panu Matilainen
On 12/10/2015 04:50 PM, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Bernard Iremonger >> Sent: Thursday, December 10, 2015 1:53 PM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCH v2] examples/vhost: reduce number of hu

[dpdk-dev] [PATCH] doc: announce API change for rte_ether.h

2015-12-11 Thread Panu Matilainen
On 12/11/2015 01:27 AM, Stephen Hemminger wrote: > Plan to change to version of struct ether_addr in > DPDK 2.3. The change in DPDK source is trivial but it will impact > source compatablilty therefore notification is necessary. > > Signed-off-by: Stephen Hemminger > --- > doc/guides/rel_notes/

[dpdk-dev] [PATCH] scripts: fix relpath.sh output when $prefix is set in environment

2015-12-11 Thread Panu Matilainen
../../include/dpdk lib -> /usr../../../lib Initialize the local variable to empty to avoid side-effects from environment. Signed-off-by: Panu Matilainen --- scripts/relpath.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/relpath.sh b/scripts/relpath.sh index 7d2f48f..4ff4

[dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols

2015-12-15 Thread Panu Matilainen
On 12/15/2015 12:47 PM, Thomas Monjalon wrote: > The API has been removed but the symbols were still declared in the map. > > Fixes: a421b86a4a02 ("ethdev: remove old flow director API") > > Signed-off-by: Thomas Monjalon > --- > lib/librte_ether/rte_ether_version.map | 8 > 1 file cha

[dpdk-dev] [PATCH] scripts: fix abi-validator regression when revision is a tag

2015-12-15 Thread Panu Matilainen
t any git revisions as ABI validation range") Signed-off-by: Panu Matilainen --- scripts/validate-abi.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/validate-abi.sh b/scripts/validate-abi.sh index 8d7be24..c36ad61 100755 --- a/scripts/validate-abi.sh +++

[dpdk-dev] [PATCH v2 4/6] bond mode 4: allow external state machine

2016-02-22 Thread Panu Matilainen
On 02/19/2016 09:17 PM, Eric Kinzie wrote: > From: Eric Kinzie > > Provide functions to allow an external 802.3ad state machine to transmit > and recieve LACPDUs and to set the collection/distribution flags on > slave interfaces. > > Signed-off-by: Eric Kinzie > Signed-off-by: Stephen Hemminger

[dpdk-dev] [PATCH] config: remove duplicate configuration information

2016-02-22 Thread Panu Matilainen
On 02/22/2016 06:02 PM, Wiles, Keith wrote: >> Hi Keith, >> >> What makes a param common? >> >> e.g. cryptodev QAT PMD is supported in linux, but currently not supported >> in bsd. >> So typically I disable it in the bsd file and enable it in the linux file. >> >> Couldn't the same apply to any o

[dpdk-dev] including rte.app.mk from a Makefile.am

2016-02-24 Thread Panu Matilainen
On 02/24/2016 04:24 AM, Stefan Puiu wrote: > Hi, > > I'm working on a Linux project that uses the DPDK and (unfornately, > IMO) automake; so we have a Makefile.am where we include rte.extapp.mk > and rte.vars.mk from the DPDK, add LDLIBS to the linker > > However, I've tried building against DPDK 2

[dpdk-dev] [PATCH] mk: fix the combined library problems by replacing it with a linker script

2016-02-24 Thread Panu Matilainen
ah it seemed to me most (if not everybody) had converged on the side of the linker script approach. > > Comments below: > > 2015-11-24 16:31, Panu Matilainen: >> The physically linked-together combined library has been an increasing >> source of problems, as was pred

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-24 Thread Panu Matilainen
On 02/23/2016 07:35 AM, Xie, Huawei wrote: > On 2/22/2016 10:52 PM, Xie, Huawei wrote: >> On 2/4/2016 1:24 AM, Olivier MATZ wrote: >>> Hi, >>> >>> On 01/27/2016 02:56 PM, Panu Matilainen wrote: >>>> Since rte_pktmbuf_alloc_bulk() is an inline func

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-02-29 Thread Panu Matilainen
On 02/24/2016 03:23 PM, Ananyev, Konstantin wrote: > Hi Panu, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen >> Sent: Wednesday, February 24, 2016 12:12 PM >> To: Xie, Huawei; Olivier MATZ; dev at dpdk.o

[dpdk-dev] [PATCH v5 01/11] ethdev: add API to query packet type filling info

2016-02-29 Thread Panu Matilainen
On 02/26/2016 09:34 AM, Jianfeng Tan wrote: > Add a new API rte_eth_dev_get_ptype_info to query whether/what packet > type can be filled by given pmd rx burst function. > > Signed-off-by: Jianfeng Tan > --- > lib/librte_ether/rte_ethdev.c | 26 ++ > lib/librte_ether/rte_

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Panu Matilainen
On 02/29/2016 01:35 PM, Ferruh Yigit wrote: > On 2/29/2016 11:06 AM, Thomas Monjalon wrote: >> Hi, >> I totally agree with Avi's comments. >> This topic is really important for the future of DPDK. >> So I think we must give some time to continue the discussion >> and have netdev involved in the cho

[dpdk-dev] [PATCH 1/3] kcp: add kernel control path kernel module

2016-02-29 Thread Panu Matilainen
On 02/29/2016 05:27 PM, Thomas Monjalon wrote: > 2016-02-29 17:19, Panu Matilainen: >> On 02/29/2016 01:35 PM, Ferruh Yigit wrote: >>> On 2/29/2016 11:06 AM, Thomas Monjalon wrote: >>>> Hi, >>>> I totally agree with Avi's comments. >>>

[dpdk-dev] [PATCH 4/4] virtio: check if any kernel driver is manipulating the device

2016-01-05 Thread Panu Matilainen
On 01/04/2016 11:02 AM, Xie, Huawei wrote: > On 12/25/2015 6:33 PM, Xie, Huawei wrote: >> virtio PMD could use IO port to configure the virtio device without >> using uio driver. >> >> There are two issues with previous implementation: >> 1) virtio PMD will take over each virtio device blindly even

[dpdk-dev] [PATCH v2 4/4] virtio: check if any kernel driver is manipulating the virtio device

2016-01-07 Thread Panu Matilainen
On 01/03/2016 07:56 PM, Huawei Xie wrote: > v2 changes: > change LOG level from ERR to INFO > > virtio PMD could use IO port to configure the virtio device without > using uio driver. > > There are two issues with previous implementation: > 1) virtio PMD will take over each virtio device blindly

[dpdk-dev] [PATCH v2 1/3] cmdline: increase command line buffer

2016-01-12 Thread Panu Matilainen
On 01/12/2016 12:49 PM, Nelio Laranjeiro wrote: > Allow long command lines in testpmd (like flow director with IPv6, ...). > > Signed-off-by: John McNamara > Signed-off-by: Nelio Laranjeiro > --- > doc/guides/rel_notes/deprecation.rst | 5 - > lib/librte_cmdline/cmdline_rdline.h | 2 +- >

[dpdk-dev] [PATCH] pci: Add the class_id support in pci probe

2016-01-13 Thread Panu Matilainen
On 01/13/2016 01:55 PM, Bruce Richardson wrote: > On Thu, Dec 31, 2015 at 09:12:14AM -0800, Stephen Hemminger wrote: >> On Tue, 29 Dec 2015 10:53:26 +0800 >> Ziye Yang wrote: >> >>> This patch is used to add the class_id support >>> for pci_probe since some devices need the class_info >>> (class_c

[dpdk-dev] [PATCH v2 1/3] cmdline: increase command line buffer

2016-01-15 Thread Panu Matilainen
On 01/15/2016 10:44 AM, N?lio Laranjeiro wrote: > On Tue, Jan 12, 2016 at 02:46:07PM +0200, Panu Matilainen wrote: >> On 01/12/2016 12:49 PM, Nelio Laranjeiro wrote: >>> Allow long command lines in testpmd (like flow director with IPv6, ...). >>> >>> Signed-off-

[dpdk-dev] [PATCH] cfgfile: support looking up sections by index

2016-01-18 Thread Panu Matilainen
On 01/17/2016 05:58 AM, Rich Lane wrote: > This is useful when sections have duplicate names. > > Signed-off-by: Rich Lane > --- > lib/librte_cfgfile/rte_cfgfile.c | 16 > lib/librte_cfgfile/rte_cfgfile.h | 23 +++ > 2 files changed, 39 insertions(+) > Thi

[dpdk-dev] [RFC 0/3] Use common Linux tools to control DPDK ports

2016-01-19 Thread Panu Matilainen
On 01/19/2016 11:59 AM, Ferruh Yigit wrote: > On Mon, Jan 18, 2016 at 11:20:02AM -0500, Aaron Conole wrote: >> Ferruh Yigit writes: >>> This work is to make DPDK ports more visible and to enable using common >>> Linux tools to configure DPDK ports. >> >> This is a good goal. Only question - why us

[dpdk-dev] [PKTGEN] fixing weird termio issues that complicate debugging

2016-01-20 Thread Panu Matilainen
On 01/20/2016 08:32 AM, Matthew Hall wrote: > Hello, > > Since the pktgen code is reindented I am finding time to read through it > and experiment and see if I can get it working. > > I have issues with the init process of pktgen. It is difficult to debug > it because the init code does a lot of ve

[dpdk-dev] [PATCH v2] cfgfile: support looking up sections by index

2016-01-21 Thread Panu Matilainen
On 01/19/2016 06:41 AM, Rich Lane wrote: > This is useful when sections have duplicate names. > > Signed-off-by: Rich Lane > --- > v1->v2: > - Added new symbol to version script. > > lib/librte_cfgfile/rte_cfgfile.c | 16 > lib/librte_cfgfile/rte_cfgfile.h |

[dpdk-dev] [PKTGEN] fixing weird termio issues that complicate debugging

2016-01-21 Thread Panu Matilainen
On 01/20/2016 06:26 PM, Wiles, Keith wrote: > On 1/20/16, 12:32 AM, "dev on behalf of Matthew Hall" dpdk.org on behalf of mhall at mhcomputing.net> wrote: > >> Hello, >> >> Since the pktgen code is reindented I am finding time to read through it >> and experiment and see if I can get it working. >

[dpdk-dev] [PATCH v3] Patch introducing API to read/write Intel Architecture Model Specific Registers (MSR)...

2016-01-21 Thread Panu Matilainen
On 01/21/2016 10:18 AM, Wojciech Andralojc wrote: > Patch reworked. > > Signed-off-by: Wojciech Andralojc > --- > lib/librte_eal/common/include/arch/x86/rte_msr.h | 88 + > lib/librte_eal/linuxapp/eal/Makefile | 1 + > lib/librte_eal/linuxapp/eal/arch/x86/rte_msr

[dpdk-dev] [PKTGEN] fixing weird termio issues that complicate debugging

2016-01-22 Thread Panu Matilainen
On 01/21/2016 05:03 PM, Wiles, Keith wrote: > On 1/21/16, 2:46 AM, "Panu Matilainen" wrote: > >> On 01/20/2016 06:26 PM, Wiles, Keith wrote: >>> On 1/20/16, 12:32 AM, "dev on behalf of Matthew Hall" >> dpdk.org on behalf of mhall at mhcomputing.

[dpdk-dev] [PATCH v3] Patch introducing API to read/write Intel Architecture Model Specific Registers (MSR)...

2016-01-22 Thread Panu Matilainen
On 01/21/2016 12:51 PM, Ananyev, Konstantin wrote: > Hi Panu, > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen >> Sent: Thursday, January 21, 2016 10:39 AM >> To: Andralojc, WojciechX >> Cc: dev at dpdk.o

[dpdk-dev] [PATCH V1 1/1] jobstats: added function abort for job

2016-01-27 Thread Panu Matilainen
On 01/26/2016 06:15 PM, Marcin Kerlin wrote: > This patch adds new function rte_jobstats_abort. It marks *job* as finished > and time of this work will be add to management time instead of execution > time. > This function should be used instead of rte_jobstats_finish if condition > occure, > con

[dpdk-dev] [PATCH v6 1/2] mbuf: provide rte_pktmbuf_alloc_bulk API

2016-01-27 Thread Panu Matilainen
On 01/26/2016 07:03 PM, Huawei Xie wrote: > v6 changes: > reflect the changes in release notes and library version map file > revise our duff's code style a bit to make it more readable > > v5 changes: > add comment about duff's device and our variant implementation > > v3 changes: > move w

[dpdk-dev] [PATCH V1 1/1] jobstats: added function abort for job

2016-01-28 Thread Panu Matilainen
On 01/27/2016 05:57 PM, Jastrzebski, MichalX K wrote: >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen >> Sent: Wednesday, January 27, 2016 2:38 PM >> To: Kerlin, MarcinX ; dev at dpdk.org >> Subject: Re: [dpd

[dpdk-dev] [PATCH v2] fix checkpatch errors

2016-01-28 Thread Panu Matilainen
On 01/28/2016 10:38 AM, Xie, Huawei wrote: > On 1/28/2016 4:06 PM, Thomas Monjalon wrote: >> 2016-01-28 03:09, Xie, Huawei: >>> On 1/28/2016 2:17 AM, Thomas Monjalon wrote: 2016-01-27 01:26, Huawei Xie: > v2 changes: > add missed commit message in v1 > > fix the error reporte

[dpdk-dev] [PATCH v3 4/4] virtio: check if kernel driver is manipulating the virtio device

2016-01-28 Thread Panu Matilainen
On 01/27/2016 05:21 PM, Huawei Xie wrote: > v3 changes: > change log message to tell user that the virtio device is skipped > due to it is managed by kernel driver, instead of asking user to > unbind it from kernel driver. > > v2 changes: > change LOG level from ERR to INFO > > virtio PMD could

[dpdk-dev] [PATCH v6 1/2] tools: Add support for handling built-in kernel modules

2016-01-28 Thread Panu Matilainen
On 01/28/2016 01:17 PM, Kamil Rytarowski wrote: > > > W dniu 26.01.2016 o 16:23, Thomas Monjalon pisze: >> 2016-01-20 10:48, krytarowski at caviumnetworks.com: >>> --- a/tools/dpdk_nic_bind.py >>> +++ b/tools/dpdk_nic_bind.py >>> -for line in loaded_mods: >>> +try: >>> +# Get list o

[dpdk-dev] [PATCH 3/4] lib/librte_port: add packet dumping to PCAP file support in sink port

2016-01-28 Thread Panu Matilainen
On 01/27/2016 07:39 PM, Fan Zhang wrote: > Originally, sink ports in librte_port releases received mbufs back to > mempool. This patch adds optional packet dumping to PCAP feature in sink > port: the packets will be dumped to user defined PCAP file for storage or > debugging. The user may also choo

[dpdk-dev] [PATCH 1/4] lib/librte_port: add PCAP file support to source port

2016-01-28 Thread Panu Matilainen
On 01/27/2016 07:39 PM, Fan Zhang wrote: > Originally, source ports in librte_port is an input port used as packet > generator. Similar to Linux kernel /dev/zero character device, it > generates null packets. This patch adds optional PCAP file support to > source port: instead of sending NULL packe

[dpdk-dev] [PATCH] pci: Add the class_id support in pci probe

2016-01-29 Thread Panu Matilainen
On 01/28/2016 11:38 PM, Thomas Monjalon wrote: > 2016-01-13 14:22, Panu Matilainen: >> On 01/13/2016 01:55 PM, Bruce Richardson wrote: >>> On Thu, Dec 31, 2015 at 09:12:14AM -0800, Stephen Hemminger wrote: >>>> On Tue, 29 Dec 2015 10:53:26 +0800 >>>> Ziye

[dpdk-dev] [PATCH] pci: Add the class_id support in pci probe

2016-01-29 Thread Panu Matilainen
On 01/29/2016 11:34 AM, Thomas Monjalon wrote: > 2016-01-29 11:21, Panu Matilainen: >> On 01/28/2016 11:38 PM, Thomas Monjalon wrote: >>> 2016-01-13 14:22, Panu Matilainen: >>>> On 01/13/2016 01:55 PM, Bruce Richardson wrote: >>>>> On Thu, Dec 31, 2015

[dpdk-dev] [PATCH] mem: skip memory locking on failure

2016-06-14 Thread Panu Matilainen
ot;mem: lock pages when not using hugepages") > Reported-by: Panu Matilainen > Signed-off-by: Olivier Matz > --- > lib/librte_eal/linuxapp/eal/eal_memory.c | 9 + > 1 file changed, 9 insertions(+) > > diff --git a/lib/librte_eal/linuxapp/eal/eal_memory.c >

[dpdk-dev] [PATCH v2 1/1] eal: fix resource leak of mapped memory

2016-06-15 Thread Panu Matilainen
On 06/15/2016 12:35 PM, Kerlin, MarcinX wrote: > Hi Sergio, > > Thanks for tips, I removed double casting and I leave (void *) casting > because pointer hp is const and compiler returns warnings. If hp is something that needs freeing then it shouldn't be const in the first place. Please fix that

[dpdk-dev] random pkt generator PMD

2016-06-15 Thread Panu Matilainen
On 06/15/2016 02:10 PM, Yerden Zhumabekov wrote: > > > On 15.06.2016 16:43, Dumitrescu, Cristian wrote: >> >>> -Original Message- >>> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Yerden >>> Zhumabekov >>> Sent: Wednesday, June 15, 2016 10:44 AM >>> To: dev at dpdk.org >>> Subject

[dpdk-dev] random pkt generator PMD

2016-06-15 Thread Panu Matilainen
On 06/15/2016 03:14 PM, Yerden Zhumabekov wrote: > > > On 15.06.2016 17:25, Panu Matilainen wrote: >> On 06/15/2016 02:10 PM, Yerden Zhumabekov wrote: >>> >>> >>> On 15.06.2016 16:43, Dumitrescu, Cristian wrote: >>>> >>>>> >

[dpdk-dev] [PATCH v4] eal: out-of-bounds write

2016-06-16 Thread Panu Matilainen
On 06/15/2016 04:25 PM, Slawomir Mrozowicz wrote: > Overrunning array mcfg->memseg of 256 44-byte elements > at element index 257 using index j. > Fixed by add condition with message information. > > Fixes: af75078fece3 ("first public release") > Coverity ID 13282 > > Signed-off-by: Slawomir Mrozow

[dpdk-dev] [PATCHv7 1/6] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-16 Thread Panu Matilainen
y: Neil Horman > CC: Bruce Richardson > CC: Thomas Monjalon > CC: Stephen Hemminger > CC: Panu Matilainen > --- Unlike earlier versions, pmdinfogen ends up installed in bindir during "make install". Is that intentional, or just a side-effect from using rte.hostapp.mk? If its intentional it probably should be prefixed with dpdk_ like the other tools. - Panu -

[dpdk-dev] [PATCHv7 5/6] pmdinfo.py: Add tool to query binaries for hw and other support information

2016-06-16 Thread Panu Matilainen
ting, where the raw JSON strings are dumped out > b) table format (default) where hex pci ids are dumped in a table format > c) pretty, where a user supplied pci.ids file is used to print out vendor > and device strings > > Signed-off-by: Neil Horman > CC: Bruce Richardson

[dpdk-dev] [PATCHv7 1/6] pmdinfogen: Add buildtools and pmdinfogen utility

2016-06-16 Thread Panu Matilainen
On 06/16/2016 04:33 PM, Neil Horman wrote: > On Thu, Jun 16, 2016 at 03:29:57PM +0300, Panu Matilainen wrote: >> On 06/09/2016 08:46 PM, Neil Horman wrote: >>> pmdinfogen is a tool used to parse object files and build json strings for >>> use in later determining h

[dpdk-dev] [PATCH] dropping librte_ivshmem - was log: deprecate history dump

2016-06-21 Thread Panu Matilainen
On 06/10/2016 12:26 AM, Thomas Monjalon wrote: > Looking a bit more into librte_ivshmem, the documentation says we need > a Qemu patch but the URL doesn't exist anymore: > https://01.org/packet-processing/intel%C2%AE-ovdk > -> 404 Oops, we couldn't find that page > > I've never understo

  1   2   3   4   5   >