[dpdk-dev] [PATCH] mem: Command line option to delete hugepage backing files

2015-10-20 Thread Gonzalez Monroy, Sergio
Hi Shesha, A few format comments: - Spacing/tabbing is all wrong for some reason, need to be fixed. - Signed-off is missing, see 'Contribute by sending patches' in http://dpdk.org/dev. - No capital letters in the commit title - there is a patch submitted with more detailed info regarding patch

[dpdk-dev] RFC - IPsec Sample App

2015-09-16 Thread Gonzalez Monroy, Sergio
On 16/09/2015 09:35, Thomas Monjalon wrote: > 2015-09-16 09:29, Gonzalez Monroy, Sergio: >> We are looking for feedback on the proposal to create a new sample >> application > [...] >> A set of generic APIs for the configuration of IPsec security policies and >>

[dpdk-dev] RFC - IPsec Sample App

2015-09-16 Thread Gonzalez Monroy, Sergio
We are looking for feedback on the proposal to create a new sample application to showcase the new cryptodev library (1) for acceleration in both SW and HW of crypto processing with a real world example of an IPsec implementation on DPDK. In the proposed implementation the core IPsec logic

[dpdk-dev] [PATCH] replaced O(n^2) sort in sort_by_physaddr() with qsort() from standard library

2015-09-09 Thread Gonzalez Monroy, Sergio
Following conversation in http://dpdk.org/ml/archives/dev/2015-September/023230.html : On 17/12/2014 13:31, rolette at infiniteio.com (Jay Rolette) wrote: > Signed-off-by: Jay Rolette > --- Update commit title/description, maybe something like: eal/linux: use qsort for sorting hugepages

[dpdk-dev] [PATCH v1] change hugepage sorting to avoid overlapping memcpy

2015-09-08 Thread Gonzalez Monroy, Sergio
> > On Tue, Sep 8, 2015 at 7:45 AM, Gonzalez Monroy, Sergio > <mailto:sergio.gonzalez.monroy at intel.com>> wrote: > > Hi Ralf, > > Just a few comments/suggestions: > > Add 'eal/linux:' to the commit title, ie: > "eal/linux: change hu

[dpdk-dev] [PATCH v1] change hugepage sorting to avoid overlapping memcpy

2015-09-08 Thread Gonzalez Monroy, Sergio
Hi Ralf, Just a few comments/suggestions: Add 'eal/linux:' to the commit title, ie: "eal/linux: change hugepage sorting to avoid overlapping memcpy" On 04/09/2015 11:14, Ralf Hoffmann wrote: > with only one hugepage or already sorted hugepage addresses, the sort > function called memcpy

[dpdk-dev] [PATCH] examples/l3fwd: fix compilation issue when using exact-match

2015-08-07 Thread Gonzalez Monroy, Sergio
On 07/08/2015 10:08, Pablo de Lara wrote: > L3fwd was trying to use an inexistent function "simple_ipv6_fwd_4pkts", > instead it should be "simple_ipv6_fwd_8pkts". > > Fixes: 80fcb4d4 ("examples/l3fwd: increase lookup burst size to 8") > > Signed-off-by: Pablo de Lara > --- > Acked-by: Sergio

[dpdk-dev] [PATCH 0/2] Warn user if system has more than 64 cores when using VM power manager

2015-08-07 Thread Gonzalez Monroy, Sergio
On 06/08/2015 12:07, Pablo de Lara wrote: > Pablo de Lara (2): >examples/vm_power_mgr: show warning when using systems with more than > 64 cores >doc: add known issue regarding VM power mgr in release notes > > doc/guides/rel_notes/known_issues.rst | 24

[dpdk-dev] [PATCH v2] i40e: fix for ieee15888 with rte_next_abi

2015-07-31 Thread Gonzalez Monroy, Sergio
On 31/07/2015 12:39, John McNamara wrote: > Fixes issue where ieee15888 timestamping doesn't work for the i40e > pmd when RTE_ABI_NEXT is enabled. > > Also refactors repeated ieee15888 flag checking and setting > code into a function. > > Reported-by: Huilong Xu > Signed-off-by: John McNamara >

[dpdk-dev] [PATCH] eal: fix build

2015-07-29 Thread Gonzalez Monroy, Sergio
On 29/07/2015 16:00, Zhang, Helin wrote: > Hi Thomas > > It was just an implicit declaration of function, when set > CONFIG_RTE_PCI_CONFIG=y, as follows. Nobody else cares about it. > > /home/hzhan75/r22/isg_cid-dpdk_org/lib/librte_eal/common/eal_common_pci.c: In > function

[dpdk-dev] [PATCH] hash: fix crash when adding already inserted keys

2015-07-28 Thread Gonzalez Monroy, Sergio
On 28/07/2015 15:54, Xavier Simonart wrote: > When adding with cuckoo hash a key which was already inserted > a new slot is dequeued and then enqueued back, but the enqueue > operation was not done properly. > > Signed-off-by: Xavier Simonart > --- > lib/librte_hash/rte_cuckoo_hash.c | 4 ++-- >

[dpdk-dev] libhugetlbfs

2015-07-23 Thread Gonzalez Monroy, Sergio
On 23/07/2015 09:12, Thomas Monjalon wrote: > 2015-07-23 08:34, Gonzalez Monroy, Sergio: >> On 22/07/2015 11:40, Thomas Monjalon wrote: >>> Sergio, >>> >>> As the maintainer of memory allocation, would you consider using >>> libhugetlbfs in DPDK for Li

[dpdk-dev] iommu on DPDK2.0.0 Error

2015-07-23 Thread Gonzalez Monroy, Sergio
On 23/07/2015 08:09, Rapelly, Varun wrote: > Hi, > > I'm facing problems with "iommu=pt intel_iommu=on" in ProLiant DL380p Gen8 > server[RHEL 7.1]. But on ProLiant DL380 G7[RHEL 7.1] server, not facing this > issue. > > When I add the above options to the kernel boot line and configure OVS

[dpdk-dev] libhugetlbfs

2015-07-23 Thread Gonzalez Monroy, Sergio
On 22/07/2015 11:40, Thomas Monjalon wrote: > Sergio, > > As the maintainer of memory allocation, would you consider using > libhugetlbfs in DPDK for Linux? > It may simplify a part of our memory allocator and avoid some potential > bugs which would be already fixed in the dedicated lib. I did

[dpdk-dev] [PATCH] malloc: fix combined lib build

2015-07-21 Thread Gonzalez Monroy, Sergio
On 21/07/2015 11:18, Sergio Gonzalez Monroy wrote: > Malloc was moved to the EAL and dummy malloc library was left > to not brake apps that had a librte_malloc.so dependency. > Note that the dummy library will be removed in the next release. > > When building a combined library, all objects are

[dpdk-dev] [PATCH] DPDK fails to build into one single library ("CONFIG_RTE_BUILD_COMBINE_LIBS=y").

2015-07-21 Thread Gonzalez Monroy, Sergio
On 21/07/2015 10:49, Rauta, Alin wrote: > The reason why I took into account as a solution the commenting of that line > is that LIBRTE_EAL already has rte_malloc integrated. > So, I'm not sure why the line is needed ... > /Alin That line is needed so when we create shared libs it creates a

[dpdk-dev] [PATCH v11] doc: update malloc documentation

2015-07-16 Thread Gonzalez Monroy, Sergio
On 16/07/2015 11:12, Thomas Monjalon wrote: > 2015-07-16 08:37, Sergio Gonzalez Monroy: >> Update malloc documentation to reflect new implementation details. > Should you reword the memzone chapter and move it after the malloc one? I didn't think that the memzone doc needed to change its wording

[dpdk-dev] App crash when using no-huge option

2015-07-15 Thread Gonzalez Monroy, Sergio
On 15/07/2015 08:34, damu wrote: > Hi, > > Today I tried no-huge option with app/test, the test app crashed. > It seems that memory can not be allocated. > I think it could be this change causing the panic. Can someone check it? > > Hi Damu, Actually it is the following commit causing the panic:

[dpdk-dev] [PATCH] pcap: add support for jumbo frames

2015-07-10 Thread Gonzalez Monroy, Sergio
On 08/07/2015 16:53, John McNamara wrote: > From: Tero Aho > > Extend eth_pcap rx and tx to support jumbo frames. > > On the receive side read large packets into multiple mbufs and > on the transmit side convert them back to a single pcap buffer. > > Signed-off-by: Tero Aho > Signed-off-by: John

[dpdk-dev] [PATCH 0/6] eal/linux: cleanup hugepage code

2015-07-08 Thread Gonzalez Monroy, Sergio
On 07/07/2015 10:00, David Marchand wrote: > Here is a little patchset cleaning eal_hugepage_info.c file and fixing a bug > when having more than two hugepage sizes. > No functional change to be expected. > Acked-by: Sergio Gonzalez Monroy

[dpdk-dev] [PATCH] hash: add missing symbol in version map

2015-07-01 Thread Gonzalez Monroy, Sergio
On 30/06/2015 19:37, Pablo de Lara wrote: > rte_hash_hash is a public function but was not in > rte_hash_version.map > > Signed-off-by: Pablo de Lara > --- > lib/librte_hash/rte_hash_version.map |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git

[dpdk-dev] [PATCHv2 2/2] ABI: Add some documentation

2015-06-25 Thread Gonzalez Monroy, Sergio
On 25/06/2015 08:42, Gonzalez Monroy, Sergio wrote: > On 25/06/2015 08:19, Zhang, Helin wrote: >> >>> -Original Message- >>> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman >>> Sent: Thursday, June 25, 2015 2:35 AM >>&g

[dpdk-dev] [PATCHv2 2/2] ABI: Add some documentation

2015-06-25 Thread Gonzalez Monroy, Sergio
On 25/06/2015 08:19, Zhang, Helin wrote: > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman >> Sent: Thursday, June 25, 2015 2:35 AM >> To: dev at dpdk.org >> Subject: [dpdk-dev] [PATCHv2 2/2] ABI: Add some documentation >> >> People have been

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-23 Thread Gonzalez Monroy, Sergio
On 22/06/2015 08:56, Simon K?gstr?m wrote: > On 2015-06-22 09:44, Gonzalez Monroy, Sergio wrote: >> On 19/06/2015 22:29, Cyril Chemparathy wrote: >>> From: Cyril Chemparathy >>> >>> It is often useful to build with debug enabled, we add a config >&

[dpdk-dev] [PATCH] mk: add support for gdb debug info generation

2015-06-22 Thread Gonzalez Monroy, Sergio
On 19/06/2015 22:29, Cyril Chemparathy wrote: > From: Cyril Chemparathy > > It is often useful to build with debug enabled, we add a config > (CONFIG_RTE_TOOLCHAIN_DEBUG) to do so. > > Note: This patch does not include corresponding changes for ICC. The > author pleads abject ignorance in this

[dpdk-dev] Unsupported --no-as-needed flag

2015-06-12 Thread Gonzalez Monroy, Sergio
On 12/06/2015 11:55, Tom Barbette wrote: > Hi list, > > Patch http://dpdk.org/dev/patchwork/patch/1056/ introduced the > --no-as-needed flag, however on debian, and another ubuntu host, neither > g++ 4.8 or 4.9 know this flag and leads to the error : > > g++: error: unrecognized command line

[dpdk-dev] mk: RTE_BUILD_COMBINE_LIBS doesn't work properly anymore

2015-06-10 Thread Gonzalez Monroy, Sergio
On 10/06/2015 10:38, Dirk-Holger Lenz wrote: > Hello, > since the hardware specific library functions were put into a separate > folder 'drivers' the feature to build a combined library doesn't generate > a combined library including these hardware specific functions anymore. > The reason for this

[dpdk-dev] Shared library build broken

2015-06-09 Thread Gonzalez Monroy, Sergio
On 08/06/2015 22:29, Thomas F Herbert wrote: > Sorry, > > I apologize on behalf of my fingers. I meant combined library build is > broken when PMD_BOND is selected. > > > On 6/8/15 4:14 PM, Thomas F Herbert wrote: >> I just noticed that shared library build is broking. I am building >> current

[dpdk-dev] Build broken with COMBINE_LIBS=y

2015-06-09 Thread Gonzalez Monroy, Sergio
On 09/06/2015 07:31, Li Wei wrote: > Hi list, > > After drivers separation, the following building error was encountered, > it seems the build system build lib/ first and link it into libintel_dpdk.a > and then drivers/ got compiled, so the symbols in drivers never got linked > into

[dpdk-dev] "Re: [RFC PATCH 0/2] dynamic memzones"

2015-06-04 Thread Gonzalez Monroy, Sergio
On 03/06/2015 20:13, Dax Rawal wrote: > Hi Sergio, > >TODOs: > > - Implement memzone_unreserve, simply call rte_malloc_free. > > - Implement mempool_delete, simply call rte_memzone_unreserve. > > - Init heaps with all available memsegs at once. > > - Review symbols in version map. > I do not see

[dpdk-dev] [PATCH] rte_reorder: Allow sequence numbers > 0 as starting point

2015-05-29 Thread Gonzalez Monroy, Sergio
On 20/05/2015 12:02, Simon Kagstrom wrote: > We use sequence numbers from a generator which has potentially started > long before the receiver. Therefore, the first number will typically > be > 0. The rte_reorder code will not work in this case, since the > packet is seen as outside of the buffer.

[dpdk-dev] [PATCH] rte_reorder: Allow sequence numbers > 0 as starting point

2015-05-28 Thread Gonzalez Monroy, Sergio
On 28/05/2015 09:15, Simon K?gstr?m wrote: > Thanks for the review, Sergio! > > On 2015-05-28 09:49, Gonzalez Monroy, Sergio wrote: >>> @@ -325,6 +327,12 @@ rte_reorder_insert(struct rte_reorder_buffer *b, >>> struct rte_mbuf *mbuf) >>>uint32_t

[dpdk-dev] [PATCH] rte_reorder: Allow sequence numbers > 0 as starting point

2015-05-28 Thread Gonzalez Monroy, Sergio
Sorry for the delay :) On 20/05/2015 12:02, Simon Kagstrom wrote: > We use sequence numbers from a generator which has potentially started > long before the receiver. Therefore, the first number will typically > be > 0. The rte_reorder code will not work in this case, since the > packet is seen

[dpdk-dev] freeing memzone

2015-05-22 Thread Gonzalez Monroy, Sergio
On 21/05/2015 17:59, Harish Patil wrote: > Hello dpdk-dev, > > I understand that the reserved memzones cannot be freed, as mentioned in > the DPDK specs. But I would like to know why? Is there any limitations? There should be a few threads in the mailing list related to this topic. Last that comes

[dpdk-dev] How do you setup a VM in Promiscuous Mode using PCI Pass-Through (SR-IOV)?

2015-05-20 Thread Gonzalez Monroy, Sergio
On 19/05/2015 20:23, Assaad, Sami (Sami) wrote: > Hello Michael, > > I've updated the kernel and QEMU. Here are the packages I'm using: > > --> CentOS 7 - 3.10.0-229.4.2.el7.x86_64 > - qemu-kvm-1.5.3-86.el7_1.2.x86_64 > - libvirt-1.2.8-16.el7_1.3.x86_64 > -

[dpdk-dev] Regarding rte_memzone_reserve with len =0

2015-05-13 Thread Gonzalez Monroy, Sergio
On 12/05/2015 15:48, Olivier MATZ wrote: > Hi Sergio, > > On 05/06/2015 06:10 PM, Gonzalez Monroy, Sergio wrote: >> Hi, >> >> I was wondering about the use case of rte_memzone_reserve_ APIs with >> len=0. >> >> From the docs (http://d

[dpdk-dev] [PATCH 1/3] lib: set LDLIBS for each library

2015-05-05 Thread Gonzalez Monroy, Sergio
On 04/05/2015 08:55, Olivier MATZ wrote: > Hi Sergio, > > On 04/15/2015 11:30 AM, Sergio Gonzalez Monroy wrote: >> This patch introduces a new LDLIBS variable to be set per library. >> Its purpose is to especify the library's dependent libraries to >> be explicitly linked against. >> >> Given the

[dpdk-dev] [PATCH v2 2/2] Update Docs for new EXTRA_LDLIBS variable

2015-04-29 Thread Gonzalez Monroy, Sergio
On 29/04/2015 16:16, Wiles, Keith wrote: > > On 4/29/15, 10:04 AM, "Gonzalez Monroy, Sergio" > wrote: > >> On 29/04/2015 16:00, Wiles, Keith wrote: >>> On 4/29/15, 9:55 AM, "Gonzalez Monroy, Sergio" >>> wrote: >>> >>>

[dpdk-dev] [PATCH v2 2/2] Update Docs for new EXTRA_LDLIBS variable

2015-04-29 Thread Gonzalez Monroy, Sergio
On 29/04/2015 16:00, Wiles, Keith wrote: > > On 4/29/15, 9:55 AM, "Gonzalez Monroy, Sergio" > wrote: > >> On 29/04/2015 15:37, Keith Wiles wrote: >>> Signed-off-by: Keith Wiles >>> --- >>>doc/build-sdk-quick.txt

[dpdk-dev] [PATCH v2 2/2] Update Docs for new EXTRA_LDLIBS variable

2015-04-29 Thread Gonzalez Monroy, Sergio
On 29/04/2015 15:37, Keith Wiles wrote: > Signed-off-by: Keith Wiles > --- > doc/build-sdk-quick.txt | 1 + > doc/guides/prog_guide/dev_kit_build_system.rst | 2 ++ > doc/guides/prog_guide/dev_kit_root_make_help.rst | 2 +- > 3 files changed, 4 insertions(+), 1

[dpdk-dev] [PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-29 Thread Gonzalez Monroy, Sergio
On 29/04/2015 11:12, Thomas Monjalon wrote: > Hi Keith, > > Thanks for trying to improve maintainability. > > It seems this is the second version of your patch. > Please add v2 prefix and a changelog to ease review and > patch management. > As you probably know, it is explained here: >

[dpdk-dev] [RFC PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-29 Thread Gonzalez Monroy, Sergio
On 28/04/2015 17:50, Wiles, Keith wrote: > Hi Sergio > > On 4/28/15, 11:25 AM, "Gonzalez Monroy, Sergio" > wrote: > >> On 28/04/2015 16:21, Keith Wiles wrote: >>> Trying to simplify the ifdefs in rte.app.mk to make the code >>> more readable

[dpdk-dev] [RFC PATCH] Simplify the ifdefs in rte.app.mk.

2015-04-28 Thread Gonzalez Monroy, Sergio
On 28/04/2015 16:21, Keith Wiles wrote: > Trying to simplify the ifdefs in rte.app.mk to make the code > more readable and maintainable by moving LDLIBS variable to use > the same style as LDLIBS-y being used in the rest of the code. > > Signed-off-by: Keith Wiles > --- +1 Patch looks good, just

[dpdk-dev] mempool deleting and cache_size

2015-04-16 Thread Gonzalez Monroy, Sergio
On 16/04/2015 10:22, Marc Sune wrote: > > > On 16/04/15 11:03, Gonzalez Monroy, Sergio wrote: >> On 15/04/2015 20:24, Stephen Hemminger wrote: >>> On Wed, 15 Apr 2015 20:15:18 +0100 >>> Zoltan Kiss wrote: >>> >>>> Hi, >>>> &

[dpdk-dev] mempool deleting and cache_size

2015-04-16 Thread Gonzalez Monroy, Sergio
On 16/04/2015 10:03, Gonzalez Monroy, Sergio wrote: > On 15/04/2015 20:24, Stephen Hemminger wrote: >> On Wed, 15 Apr 2015 20:15:18 +0100 >> Zoltan Kiss wrote: >> >>> Hi, >>> >>> I have two questions regarding mempools: >>> >>> -

[dpdk-dev] mempool deleting and cache_size

2015-04-16 Thread Gonzalez Monroy, Sergio
On 15/04/2015 20:24, Stephen Hemminger wrote: > On Wed, 15 Apr 2015 20:15:18 +0100 > Zoltan Kiss wrote: > >> Hi, >> >> I have two questions regarding mempools: >> >> - the first is trivial: how do you delete them? Can you? I can't see a >> function to do that, and none of the examples are doing

[dpdk-dev] [PATCH v3 1/5] mk: remove combined library and related options

2015-04-13 Thread Gonzalez Monroy, Sergio
On 09/04/2015 21:34, Neil Horman wrote: > On Thu, Apr 09, 2015 at 08:00:26PM +0300, Avi Kivity wrote: >> On 04/09/2015 02:19 PM, Neil Horman wrote: >>> On Thu, Apr 09, 2015 at 12:06:47PM +0300, Avi Kivity wrote: >>>> On 04/09/2015 11:33 AM, Gonzalez Monroy, Sergio wr

[dpdk-dev] [PATCH v3 1/5] mk: remove combined library and related options

2015-04-09 Thread Gonzalez Monroy, Sergio
On 09/04/2015 14:33, Thomas Monjalon wrote: > 2015-04-09 07:19, Neil Horman: >> On Thu, Apr 09, 2015 at 12:06:47PM +0300, Avi Kivity wrote: >>> On 04/09/2015 11:33 AM, Gonzalez Monroy, Sergio wrote: >>>> On 08/04/2015 19:26, Stephen Hemminger wrote: >>&g

[dpdk-dev] [PATCH v3 1/5] mk: remove combined library and related options

2015-04-09 Thread Gonzalez Monroy, Sergio
On 09/04/2015 10:06, Avi Kivity wrote: > > > On 04/09/2015 11:33 AM, Gonzalez Monroy, Sergio wrote: >> On 08/04/2015 19:26, Stephen Hemminger wrote: >>> On Wed, 8 Apr 2015 16:07:21 +0100 >>> Sergio Gonzalez Monroy wrote: >>> >>>> C

[dpdk-dev] [PATCH v3 1/5] mk: remove combined library and related options

2015-04-09 Thread Gonzalez Monroy, Sergio
On 08/04/2015 19:26, Stephen Hemminger wrote: > On Wed, 8 Apr 2015 16:07:21 +0100 > Sergio Gonzalez Monroy wrote: > >> Currently, the target/rules to build combined libraries is different >> than the one to build individual libraries. >> >> By removing the combined library option as a build

[dpdk-dev] [PATCH] eal: decrease the memory init time with many hugepages setup

2015-04-03 Thread Gonzalez Monroy, Sergio
On 03/04/2015 10:14, Thomas Monjalon wrote: > 2015-04-03 10:04, Gonzalez Monroy, Sergio: >> On 02/04/2015 14:41, Jay Rolette wrote: >>> On Thu, Apr 2, 2015 at 7:55 AM, Thomas Monjalon >> 6wind.com> >>> wrote: >>> >>>> 2015-04-0

[dpdk-dev] Secondary process crash in rte_eal_init

2015-04-03 Thread Gonzalez Monroy, Sergio
On 01/04/2015 06:00, Dey, Souvik wrote: > Hi All, > We have a single primary application with multiple secondary > applications launched on the same cpu core. When the system boots up for the > first time ,the primary comes up followed by the secondary process everything >

[dpdk-dev] [PATCH] eal: decrease the memory init time with many hugepages setup

2015-04-03 Thread Gonzalez Monroy, Sergio
On 02/04/2015 14:41, Jay Rolette wrote: > On Thu, Apr 2, 2015 at 7:55 AM, Thomas Monjalon > wrote: > >> 2015-04-02 19:30, jerry.lilijun at huawei.com: >>> From: Lilijun >>> >>> In the function map_all_hugepages(), hugepage memory is truly allocated >> by >>> memset(virtaddr, 0, hugepage_sz).

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-26 Thread Gonzalez Monroy, Sergio
On 18/03/2015 12:59, Thomas Monjalon wrote: > Hi Sergio, > > Thank you for explaining the situation. > > 2015-03-18 12:11, Gonzalez Monroy, Sergio: >> Given that the patch to remove combined libraries is not welcome, I'll >> try to explain the current situation so we c

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Gonzalez Monroy, Sergio
On 18/03/2015 15:30, Stefan Puiu wrote: > Hi, > > On Wed, Mar 18, 2015 at 2:59 PM, Thomas Monjalon > wrote: >> Hi Sergio, >> >> Thank you for explaining the situation. >> >> 2015-03-18 12:11, Gonzalez Monroy, Sergio: >>> Given that the patc

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-18 Thread Gonzalez Monroy, Sergio
On 12/03/2015 16:27, Sergio Gonzalez Monroy wrote: > Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. > > Signed-off-by: Sergio Gonzalez Monroy > --- > config/common_bsdapp| 6 -- > config/common_linuxapp | 6 -- >

[dpdk-dev] [PATCH] igb: handle VF LPE mailbox message

2015-03-18 Thread Gonzalez Monroy, Sergio
On 18/03/2015 12:01, Sergio Gonzalez Monroy wrote: > This patch adds the handle function for the LPE mailbox message (VF to > PF) to set maximum packet size, which can be used to enable jumbo > frame support. > > Signed-off-by: Sergio Gonzalez Monroy > --- > lib/librte_pmd_e1000/igb_pf.c | 28

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 16:07, Stephen Hemminger wrote: > On Thu, 12 Mar 2015 16:27:58 + > Sergio Gonzalez Monroy wrote: > >> Remove CONFIG_RTE_BUILD_COMBINE_LIBS and CONFIG_RTE_LIBNAME. >> >> Signed-off-by: Sergio Gonzalez Monroy >> --- > NAK. The combined library is good and useful for those who

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
rrently stands, building combined shared libraries will be broken the moment we have different versions of any API because the linking for the combined lib does not use a version map. Sergio >> Thanks, >> Stefan. >> >> On Fri, Mar 13, 2015 at 3:17 PM, Neil Horman >> w

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 13:16, Kavanagh, Mark B wrote: > >> -Original Message- >> From: Gonzalez Monroy, Sergio >> Sent: Friday, March 13, 2015 11:49 AM >> To: Kavanagh, Mark B >> Cc: dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v2 1/4] mk: Remove c

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 11:34, Kavanagh, Mark B wrote: >> On 13/03/2015 10:49, Kavanagh, Mark B wrote: --- config/common_bsdapp| 6 -- config/common_linuxapp | 6 -- config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - lib/Makefile

[dpdk-dev] [PATCH v2 1/4] mk: Remove combined library and related options

2015-03-13 Thread Gonzalez Monroy, Sergio
On 13/03/2015 10:49, Kavanagh, Mark B wrote: >> --- >> config/common_bsdapp| 6 -- >> config/common_linuxapp | 6 -- >> config/defconfig_ppc_64-power8-linuxapp-gcc | 2 - >> lib/Makefile| 1 - >> mk/rte.app.mk

[dpdk-dev] Not getting statistics for all queues.

2015-03-06 Thread Gonzalez Monroy, Sergio
bit > SFI/SFP+ Network Connection (rev 01) > Subsystem: Hewlett-Packard Company Ethernet 10Gb 2-port 560SFP+ > Adapter/ > > > Thanks, > Alexandra > > On 2 March 2015 at 11:15, Gonzalez Monroy, Sergio > <mailto:sergio.gonzalez.monroy at

[dpdk-dev] [PATCH] librte_eal/common: Fix cast from pointer to integer of different size

2015-03-02 Thread Gonzalez Monroy, Sergio
On 02/03/2015 07:27, Michael Qiu wrote: > /i686-native-linuxapp-gcc/include/rte_memcpy.h:592:23: error: > cast from pointer to integer of different size > [-Werror=pointer-to-int-cast] > >dstofss = 16 - (int)((long long)(void *)dst & 0x0F) + 16; > > Type 'long long' is 64-bit in i686 platform

[dpdk-dev] Not getting statistics for all queues.

2015-03-02 Thread Gonzalez Monroy, Sergio
Hi Alexandra, On 02/03/2015 08:50, Alexandra Sava wrote: > Hi guys, > > Did you have a chance to look over my question? > > > > Thanks, > Alexandra > > > On 24 February 2015 at 13:14, Alexandra Sava > wrote: >> Hi guys, >> >> I'm trying to get statistics per queue (on rx side), therefore I'm >>

[dpdk-dev] [PATCH 0/8] Improve build process

2015-02-23 Thread Gonzalez Monroy, Sergio
On 23/02/2015 13:52, Neil Horman wrote: > On Mon, Feb 23, 2015 at 10:25:01AM +0000, Gonzalez Monroy, Sergio wrote: >> On 22/02/2015 23:37, Neil Horman wrote: >>> On Fri, Feb 20, 2015 at 02:31:36PM +0000, Gonzalez Monroy, Sergio wrote: >>>> On 13/02/2015 12:51, Neil H

[dpdk-dev] [PATCH 0/8] Improve build process

2015-02-23 Thread Gonzalez Monroy, Sergio
On 22/02/2015 23:37, Neil Horman wrote: > On Fri, Feb 20, 2015 at 02:31:36PM +0000, Gonzalez Monroy, Sergio wrote: >> On 13/02/2015 12:51, Neil Horman wrote: >>> On Fri, Feb 13, 2015 at 11:08:02AM +0000, Gonzalez Monroy, Sergio wrote: >>>> On 13/02/2015 10:14, Panu Ma

[dpdk-dev] [PATCH 0/8] Improve build process

2015-02-20 Thread Gonzalez Monroy, Sergio
On 13/02/2015 12:51, Neil Horman wrote: > On Fri, Feb 13, 2015 at 11:08:02AM +0000, Gonzalez Monroy, Sergio wrote: >> On 13/02/2015 10:14, Panu Matilainen wrote: >>> On 02/12/2015 05:52 PM, Neil Horman wrote: >>>> On Thu, Feb 12, 2015 at 04:07:50PM +0200, Panu Matilai

[dpdk-dev] [PATCH v4 0/5] Interrupt mode PMD

2015-02-20 Thread Gonzalez Monroy, Sergio
On 19/02/2015 13:48, Zhou Danny wrote: > v4 changes > - Export interrupt enable/disable functions for shared libraries > - Adjust position of new-added structure fields and functions to > avoid breaking ABI > > v3 changes > - Add return value for interrupt enable/disable functions > - Move spinlok

[dpdk-dev] [PATCH v3 1/5] ethdev: add rx interrupt enable/disable functions

2015-02-19 Thread Gonzalez Monroy, Sergio
On 19/02/2015 08:06, Zhou, Danny wrote: > >> -Original Message- >> From: Neil Horman [mailto:nhorman at tuxdriver.com] >> Sent: Tuesday, February 17, 2015 11:53 PM >> To: Zhou, Danny >> Cc: dev at dpdk.org >> Subject: Re: [dpdk-dev] [PATCH v3 1/5] ethdev: add rx interrupt >>

[dpdk-dev] [PATCH v4 0/5] New Reorder Library

2015-02-18 Thread Gonzalez Monroy, Sergio
On 18/02/2015 14:22, Thomas Monjalon wrote: > Hi Sergio, > > 2015-02-11 13:07, Sergio Gonzalez Monroy: >> This series introduces the new reorder library along with unit tests, >> sample app and a new entry in the programmers guide describing the library. >> >> The library provides reordering of

[dpdk-dev] [PATCH] eal: add missing symbol export for rte_eal_iopl_init()

2015-02-18 Thread Gonzalez Monroy, Sergio
On 18/02/2015 14:17, Panu Matilainen wrote: > Signed-off-by: Panu Matilainen > --- > lib/librte_eal/bsdapp/eal/rte_eal_version.map | 1 + > lib/librte_eal/linuxapp/eal/rte_eal_version.map | 1 + > 2 files changed, 2 insertions(+) > > diff --git

[dpdk-dev] [PATCH 0/2] Removal of RTE_MBUF_REFCNT

2015-02-17 Thread Gonzalez Monroy, Sergio
On 16/02/2015 20:47, Stephen Hemminger wrote: > On Mon, 16 Feb 2015 16:08:31 + > Sergio Gonzalez Monroy wrote: > >> This patch tries to remove the RTE_MBUF_REFCNT config options and >> dependencies >> by introducing a new mbuf flag IND_ATTACHED_MBUF that would indicate when >> the mbuf >>

[dpdk-dev] [PATCH 0/8] Improve build process

2015-02-13 Thread Gonzalez Monroy, Sergio
On 13/02/2015 10:14, Panu Matilainen wrote: > On 02/12/2015 05:52 PM, Neil Horman wrote: >> On Thu, Feb 12, 2015 at 04:07:50PM +0200, Panu Matilainen wrote: >>> On 02/12/2015 02:23 PM, Neil Horman wrote: > [...snip...] >>> So I just realized that I was not having into account a

[dpdk-dev] [PATCH] eal: add rte_eal_iopl_init to version map

2015-02-12 Thread Gonzalez Monroy, Sergio
On 12/02/2015 15:39, Sergio Gonzalez Monroy wrote: > Building shared libraries and using virtio PMD results in undefined > reference to 'rte_eal_iopl_init'. > > Add missing function to eal version map. > > Signed-off-by: Sergio Gonzalez Monroy > --- >

[dpdk-dev] [PATCH] eal: add rte_eal_iopl_init to version map

2015-02-12 Thread Gonzalez Monroy, Sergio
On 12/02/2015 15:12, Thomas Monjalon wrote: > 2015-02-11 13:30, Sergio Gonzalez Monroy: >> Common rte_eal_iopl_init function is missing from eal version map. > How did you see it? > Thanks Building shared libraries and using virtio PMD would result in undefined reference symbol. Should I add it

[dpdk-dev] [PATCH 0/8] Improve build process

2015-02-12 Thread Gonzalez Monroy, Sergio
On 12/02/2015 09:22, Panu Matilainen wrote: > On 02/11/2015 01:11 PM, Gonzalez Monroy, Sergio wrote: >>> From: Neil Horman [mailto:nhorman at tuxdriver.com] >>> Sent: Friday, January 30, 2015 6:13 PM >>> To: Gonzalez Monroy, Sergio >>> Cc: Thomas Monjalon;

[dpdk-dev] [PATCH 0/8] Improve build process

2015-02-12 Thread Gonzalez Monroy, Sergio
On 12/02/2015 05:41, Neil Horman wrote: > On Wed, Feb 11, 2015 at 11:11:13AM +0000, Gonzalez Monroy, Sergio wrote: >>> From: Neil Horman [mailto:nhorman at tuxdriver.com] >>> Sent: Friday, January 30, 2015 6:13 PM >>> To: Gonzalez Monroy, Sergio >>> Cc: Thoma

[dpdk-dev] [PATCH] mk: fix missing link of librte_vhost in shared, non-combined config

2015-02-11 Thread Gonzalez Monroy, Sergio
> From: Panu Matilainen [mailto:pmatilai at redhat.com] > Sent: Wednesday, February 11, 2015 11:26 AM > To: Gonzalez Monroy, Sergio; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] mk: fix missing link of librte_vhost in > shared, non-combined config > > On 02/11/2015 12:5

[dpdk-dev] [PATCH v3 0/5] New Reorder Library

2015-02-11 Thread Gonzalez Monroy, Sergio
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Sunday, February 8, 2015 1:59 PM > To: Gonzalez Monroy, Sergio > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 0/5] New Reorder Library > > On Fri, Feb 06, 2015 at 03:05:59PM +, Sergio

[dpdk-dev] [PATCH] mk: fix missing link of librte_vhost in shared, non-combined config

2015-02-11 Thread Gonzalez Monroy, Sergio
Hi Panu, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Panu Matilainen > Sent: Wednesday, February 11, 2015 8:53 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] mk: fix missing link of librte_vhost in shared, > non-combined config > > When building

[dpdk-dev] [PATCH] testpmd: remove incorrect parameter limits in help command line

2015-02-11 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > Sent: Friday, February 6, 2015 2:23 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] testpmd: remove incorrect parameter limits in > help command line > > Ring threshold parameters an RX/TX queue (pthresh, wthresh and

[dpdk-dev] [PATCH] testpmd: remove duplicated parameter parsing

2015-02-11 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Pablo de Lara > Sent: Friday, February 6, 2015 2:15 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] testpmd: remove duplicated parameter parsing > > Several parameters were being parsed twice in testpmd, so this patch gets > rid of

[dpdk-dev] [PATCH] MAINTAINERS: claim IP fragmentation and ACL

2015-02-06 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Konstantin > Ananyev > Sent: Friday, February 6, 2015 12:36 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] MAINTAINERS: claim IP fragmentation and ACL > > Signed-off-by: Konstantin Ananyev > --- > MAINTAINERS | 2 ++ > 1 file

[dpdk-dev] [PATCH] MAINTAINERS: claim metering, sched and pkt framework

2015-02-06 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cristian Dumitrescu > Sent: Wednesday, February 4, 2015 3:53 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] MAINTAINERS: claim metering, sched and pkt > framework > > As original author of these DPDK components, I am volunteering

[dpdk-dev] [PATCH 0/8] Improve build process

2015-01-30 Thread Gonzalez Monroy, Sergio
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Thursday, January 29, 2015 7:46 PM > To: Gonzalez Monroy, Sergio > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 0/8] Improve build process > > On Thu, Jan 29, 2015 at 05:04:20PM +0000, Gonzalez Monroy, Ser

[dpdk-dev] [PATCH 1/3] librte_reorder: New reorder library

2015-01-30 Thread Gonzalez Monroy, Sergio
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Thursday, January 29, 2015 8:40 PM > To: Gonzalez Monroy, Sergio > Cc: Thomas Monjalon; Pattan, Reshma; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/3] librte_reorder: New reorder library > > On Thu, Jan 2

[dpdk-dev] [PATCH 1/3] librte_reorder: New reorder library

2015-01-29 Thread Gonzalez Monroy, Sergio
Hi Thomas, > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Tuesday, January 20, 2015 8:01 AM > To: Pattan, Reshma > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/3] librte_reorder: New reorder library > > Hi, > > 2015-01-07 16:39, Reshma Pattan: > >

[dpdk-dev] [PATCH 0/8] Improve build process

2015-01-29 Thread Gonzalez Monroy, Sergio
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Thursday, January 29, 2015 4:39 PM > To: Gonzalez Monroy, Sergio > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 0/8] Improve build process > > On Thu, Jan 29, 2015 at 03:20:03PM +, Sergio

[dpdk-dev] [PATCH] mk: add support for ICC 15 compiler

2015-01-23 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Daniel Mrzyglod > Sent: Thursday, January 22, 2015 9:54 AM > > This patch add Support for ICC 15. > > ICC 15 changed inline-max-size and inline-max-total-size default values, so > for ICC 15 flags -no-inline-max-size

[dpdk-dev] [PATCH RFC 00/13] Update build system

2015-01-22 Thread Gonzalez Monroy, Sergio
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, January 22, 2015 10:39 AM > To: Gonzalez Monroy, Sergio > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH RFC 00/13] Update build system > > 2015-01-22 10:03, Gonzalez Monroy, Sergio: > &

[dpdk-dev] [PATCH RFC 00/13] Update build system

2015-01-22 Thread Gonzalez Monroy, Sergio
> From: Gonzalez Monroy, Sergio > Sent: Monday, January 12, 2015 5:22 PM > To: Thomas Monjalon > Subject: Re: [dpdk-dev] [PATCH RFC 00/13] Update build system > > Hi Thomas, > > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Monday, Ja

[dpdk-dev] [PATCH RFC 00/13] Update build system

2015-01-12 Thread Gonzalez Monroy, Sergio
Hi Thomas, > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, January 12, 2015 4:52 PM > > Hi Sergio, > > 2015-01-12 16:33, Sergio Gonzalez Monroy: > > This patch series updates the DPDK build system. > > Thanks for proposing such rework. > We need discussions on

[dpdk-dev] [PATCH v3 3/4] Add library version extenstion

2014-12-23 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > Sent: Tuesday, December 23, 2014 3:52 PM > > a/mk/rte.lib.mk b/mk/rte.lib.mk index 1d3b646..7326b8e 100644 > --- a/mk/rte.lib.mk > +++ b/mk/rte.lib.mk > @@ -163,9 +168,13 @@ endif > # install lib in $(RTE_OUTPUT)/lib > # >

[dpdk-dev] [PATCH v2 3/4] Add library version extenstion

2014-12-23 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > Sent: Monday, December 22, 2014 8:24 PM > > a/mk/rte.lib.mk b/mk/rte.lib.mk index 7949254..ad058b5 100644 > --- a/mk/rte.lib.mk > +++ b/mk/rte.lib.mk > @@ -166,9 +171,11 @@ endif > # install lib in $(RTE_OUTPUT)/lib > # >

[dpdk-dev] [PATCH 1/4] compat: Add infrastructure to support symbol versioning

2014-12-22 Thread Gonzalez Monroy, Sergio
> From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Monday, December 22, 2014 4:35 PM > > On Mon, Dec 22, 2014 at 02:01:10PM +0000, Gonzalez Monroy, Sergio wrote: > > > > > From: Neil Horman [mailto:nhorman at tuxdriver.com] > > > Sent: Saturday, De

[dpdk-dev] [PATCH 1/4] compat: Add infrastructure to support symbol versioning

2014-12-22 Thread Gonzalez Monroy, Sergio
Hi Neil, Just a couple of minor comments. > From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Saturday, December 20, 2014 9:02 PM > Subject: [PATCH 1/4] compat: Add infrastructure to support symbol > versioning > > Add initial pass header files to support symbol versioning. > [...]

[dpdk-dev] [PATCH v2] ixgbe_vf: Fix getting link state

2014-12-19 Thread Gonzalez Monroy, Sergio
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Balazs Nemeth > Sent: Thursday, December 18, 2014 5:57 PM > To: dev at dpdk.org > Cc: Nemeth, Balazs > Subject: [dpdk-dev] [PATCH v2] ixgbe_vf: Fix getting link state > > This patch fixes checking the link state of a virtual function. If

[dpdk-dev] [PATCH v2] mk: link combined shared lib using CC

2014-12-17 Thread Gonzalez Monroy, Sergio
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, December 16, 2014 11:43 PM > > 2014-12-16 19:48, Thomas Monjalon: > > 2014-10-28 15:48, Sergio Gonzalez Monroy: > > > If we set EXTRA_CFLAGS=-O0, build fails with following error: > > > > > > /usr/bin/ld: test: hidden

[dpdk-dev] [PATCH v2] doc: add known issue for iommu and igb_uio

2014-12-12 Thread Gonzalez Monroy, Sergio
> From: Nicolas Dichtel [mailto:nicolas.dichtel at 6wind.com] > Sent: Friday, December 12, 2014 1:28 PM > > Le 12/12/2014 14:20, Gonzalez Monroy, Sergio a ?crit : > >> From: Nicolas Dichtel [mailto:nicolas.dichtel at 6wind.com] > >> Sent: Friday, December 12, 2014 1

  1   2   >