Re: [dpdk-dev] [PATCH v3 1/3] lib/lpm: integrate RCU QSBR

2019-10-12 Thread Honnappa Nagarahalli
> Hi guys, I have tried to consolidate design related questions here. If I have missed anything, please add. > > > > > From: Ruifeng Wang > > > > Currently, the tbl8 group is freed even though the readers might be > > using the tbl8 group entries. The freed tbl8 group can be reallocated > > q

Re: [dpdk-dev] [PATCH v3 2/3] lib/rcu: add resource reclamation APIs

2019-10-12 Thread Honnappa Nagarahalli
> > > > > > Add resource reclamation APIs to make it simple for > > > > > > applications and libraries to integrate rte_rcu library. > > > > > > > > > > > > Signed-off-by: Honnappa Nagarahalli > > > > > > > > > > > > Reviewed-by: Ola Liljedhal > > > > > > Reviewed-by: Ruifeng Wang > > > > > >

Re: [dpdk-dev] [PATCH v3 2/3] lib/rcu: add resource reclamation APIs

2019-10-12 Thread Honnappa Nagarahalli
Hi Vladimir, Apologies for the delayed response, I had to run few experiments. > > Hi Honnappa, > > On 01/10/2019 07:29, Honnappa Nagarahalli wrote: > > Add resource reclamation APIs to make it simple for applications and > > libraries to integrate rte_rcu library. > > > > Signed-off-b

Re: [dpdk-dev] [PATCH v2 1/2] lib/distributor: fix deadlock issue for aarch64

2019-10-12 Thread Honnappa Nagarahalli
Hi Ruifeng, Typically, we have followed the convention of adding comments whenever C11 atomic APIs are used. Can you please add comments indicating why acquire or release semantics are used? > -Original Message- > From: Ruifeng Wang > Sent: Friday, October 11, 2019 9:44 PM > To:

Re: [dpdk-dev] [PATCH 0/8] eBPF arm64 JIT support

2019-10-12 Thread Thomas Monjalon
24/09/2019 19:03, Ananyev, Konstantin: > > Jerin Jacob (8): > > bpf/arm64: add build infrastructure > > bpf/arm64: add prologue and epilogue > > bpf/arm64: add basic arithmetic operations > > bpf/arm64: add logical operations > > bpf/arm64: add byte swap operations > > bpf/arm64: add lo

Re: [dpdk-dev] [PATCH] bpf: hide internal program arg type

2019-10-12 Thread Thomas Monjalon
16/08/2019 15:01, Ananyev, Konstantin: > > From: Jerin Jacob > > > > RTE_BPF_ARG_PTR_STACK is used as internal program > > arg type. Rename to RTE_BPF_ARG_RESERVED to > > avoid exposing internal program type. > > > > Signed-off-by: Jerin Jacob > > Acked-by: Konstantin Ananyev > Tested-by: Kon

Re: [dpdk-dev] [dpdk-stable] [PATCH 08/10] app/testpmd: fix global variable multiple definitions

2019-10-12 Thread Thomas Monjalon
05/09/2019 16:53, Ferruh Yigit: > --- a/app/test-pmd/testpmd.h > +++ b/app/test-pmd/testpmd.h > @@ -507,7 +507,8 @@ struct vxlan_encap_conf { > uint8_t eth_src[RTE_ETHER_ADDR_LEN]; > uint8_t eth_dst[RTE_ETHER_ADDR_LEN]; > }; > -struct vxlan_encap_conf vxlan_encap_conf; > + > +extern st

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] build: avoid --as-needed as it causes overlinking

2019-10-12 Thread Thomas Monjalon
12/09/2019 15:43, Christian Ehrhardt: > > On Wed, Sep 4, 2019 at 11:18 AM Bruce Richardson > > wrote: > > > > On Mon, Sep 2, 2019 at 9:09 AM Christian Ehrhardt > > > > wrote: > > > > > > On Thu, 2019-08-29 at 16:34 +0100, Bruce Richardson wrote: > > > > > > > On Thu, Aug 29, 2019 at 05:30:03PM +0

Re: [dpdk-dev] [PATCH v2 04/10] event/octeontx2: fix maybe-uninitialized warnings for LTO build

2019-10-12 Thread Pavan Nikhilesh Bhagavatula
>During LTO build compiler reports some 'false positive' warnings about >variables being possibly used uninitialized. This patch silences these >warnings. > >Signed-off-by: Andrzej Ostruszka Acked-by: Pavan Nikhilesh >--- > drivers/event/octeontx2/otx2_tim_worker.h | 2 +- > 1 file changed, 1 i

Re: [dpdk-dev] [PATCH 00/17] Series short description

2019-10-12 Thread Stephen Hemminger
On Sat, 12 Oct 2019 02:26:26 +0200 Alfredo Cardigliano wrote: > The patch series provides an initial version of a > poll mode driver for Pensando network adapters. > The driver name is ionic. > > --- > > Alfredo Cardigliano (17): > net/ionic: add skeleton > net/ionic: add hardware s

Re: [dpdk-dev] [PATCH 06/17] net/ionic: add basic lif support

2019-10-12 Thread Stephen Hemminger
On Sat, 12 Oct 2019 02:26:57 +0200 Alfredo Cardigliano wrote: > @@ -37,6 +37,7 @@ ifeq ($(shell test $(GCC_VERSION) -ge 44 && echo 1), 1) > CFLAGS += -Wno-deprecated -Wno-unused-parameter -Wno-unused-value > CFLAGS += -Wno-strict-aliasing -Wno-format-extra-args > CFLAGS += -Wno-missing-field-i

Re: [dpdk-dev] [PATCH 03/17] net/ionic: add log

2019-10-12 Thread Stephen Hemminger
On Sat, 12 Oct 2019 02:26:41 +0200 Alfredo Cardigliano wrote: > +#ifdef RTE_LIBRTE_IONIC_DEBUG_RX > +#define ionic_rx_print(level, fmt, args...) RTE_LOG(level, PMD, \ > + "%s(): " fmt "\n", __func__, ## args) > +#else > +#define ionic_rx_print(level, fmt, args...) do { } while (0) > +

Re: [dpdk-dev] [PATCH v4 16/19] net/hinic: add hinic PMD doc files

2019-10-12 Thread Wangxiaoyun (Cloud, Network Chip Application Development Dept)
Hi Ferruh, Thanks for your comments and modify. Best Regards Xiaoyun Wang On 10/10/2019 3:52 PM, Xiaoyun wang wrote: Add new supported features to rst file and add features to ini file. Signed-off-by: Xiaoyun wang --- doc/guides/nics/features/hinic.ini | 2 ++ doc/guides/rel_no

Re: [dpdk-dev] [PATCH v2 03/10] app/eventdev: fix maybe-uninitialized warnings for LTO build

2019-10-12 Thread Jerin Jacob
On Tue, Sep 17, 2019 at 1:28 PM Andrzej Ostruszka wrote: > > During LTO build compiler reports some 'false positive' warnings about > variables being possibly used uninitialized. This patch silences these > warnings. > > Signed-off-by: Andrzej Ostruszka # Please fix check-git-log.sh errors. Mis

Re: [dpdk-dev] [PATCH v2 02/10] eventdev: fix possible use of uninitialized var

2019-10-12 Thread Jerin Jacob
On Tue, Sep 17, 2019 at 1:28 PM Andrzej Ostruszka wrote: > > Fix the logic for the case of event queue allowing all schedule types. > > Signed-off-by: Andrzej Ostruszka This fix looks good to me. # Please fix check-git-log.sh errors. Missing 'Fixes' tag: eventdev: fix possible use of un

Re: [dpdk-dev] [PATCH v1] doc: update cross build tool links and fix compiling issue

2019-10-12 Thread Jerin Jacob
On Sat, Oct 12, 2019 at 8:10 AM Joyce Kong wrote: > > Update cross build tool links as newer cross build tools > version are provided on Linaro. > And remove the command 'git checkout' which results in > compiling error when cross building. > > Fixes: 01add9da25cd ("doc: add cross compiling guide"

[dpdk-dev] [PATCH] maintainers: update for armv8 crypto library

2019-10-12 Thread jerinj
From: Jerin Jacob Update the armv8 crypto PMD maintainership. https://github.com/caviumnetworks/armv8_crypto external crypto the library will not be maintained and probably removed soon therefor updating the PMD documentation to reflect the same. Signed-off-by: Jerin Jacob --- This patch is b