RE: [PATCH v2] app/eventdev: add crypto producer mode

2022-01-03 Thread Shijith Thotton
--+ >> >| 64 | 2048| 1| 1 | -0.5% >> > | >> >++-----+------+-+--+ >> > >> >To view detailed results, visit: >&g

[PATCH v3] app/eventdev: add crypto producer mode

2022-01-04 Thread Shijith Thotton
--crypto_adptr_mode 1 --test=perf_atq \ --stlist=a --wlcores 1 --plcores 2 Signed-off-by: Shijith Thotton --- v3: * Reduce dereference inside loop. v2: * Fix RHEL compilation warning. app/test-eventdev/evt_common.h | 3 + app/test-eventdev/evt_main.c | 13 +- app/test-eventdev

RE: [EXT] Re: [PATCH v2] app/eventdev: add crypto producer mode

2022-01-04 Thread Shijith Thotton
>> >> In crypto producer mode, producer core enqueues cryptodev with >> software generated crypto ops and worker core dequeues crypto >> completion events from the eventdev. Event crypto metadata used for >> above processing is pre- populated in each crypto session. >>

[dpdk-dev] [RFC PATCH] eventdev: use shared memory to store timer adapter

2021-02-12 Thread Shijith Thotton
It is not possible for secondary process to arm timer as timer adapter is not stored in shared memory. Using shared memory allows the secondary to lookup adapters and arm them. Signed-off-by: Shijith Thotton --- drivers/event/octeontx/timvf_evdev.c | 20 +- drivers/event/octeontx

RE: [EXT] [PATCH] drivers: skip build of sub-libs not supporting IOVA mode

2023-03-02 Thread Shijith Thotton
need to be skipped earlier >to avoid defining the sub-libraries. > >Fixes: a986c2b7973d ("build: add option to configure IOVA mode as PA") >Cc: sta...@dpdk.org > >Signed-off-by: Thomas Monjalon >--- Acked-by: Shijith Thotton

RE: [PATCH v2 1/3] event/cnxk: use LMTST for enqueue new burst

2023-05-18 Thread Shijith Thotton
>From: Pavan Nikhilesh > >Use LMTST when all events in the burst are enqueue with >rte_event:op as RTE_EVENT_OP_NEW i.e. events are enqueued >with the `rte_event_enqueue_new_burst` API. > >Signed-off-by: Pavan Nikhilesh Acked-by: Shijith Thotton >--- >v2

RE: [EXT] [PATCH v2 2/3] app/eventdev: use enqueue new event burst routine

2023-05-18 Thread Shijith Thotton
>From: Pavan Nikhilesh > >Use the `rte_event_enqueue_new_burst` routine to enqueue events >with rte_event::op as RTE_EVENT_OP_NEW. This allows PMDs to use >optimized enqueue routines. > >Signed-off-by: Pavan Nikhilesh [] Acked-by: Shijith Thotton >--- > app/

RE: [EXT] [PATCH v2 3/3] app/eventdev: prevent mempool exhaustion

2023-05-18 Thread Shijith Thotton
>From: Pavan Nikhilesh > >Prevent mempool exhaustion due to elements being stuck in lcore >local caches. > >Signed-off-by: Pavan Nikhilesh Acked-by: Shijith Thotton >--- > app/test-eventdev/test_perf_common.c | 11 ++- > 1 file changed, 6 insertions(+), 5 d

[PATCH] mbuf: fix API to copy mbuf dynamic fields

2024-06-26 Thread Shijith Thotton
mbufs. This patch fixes the same. see https://bugs.dpdk.org/show_bug.cgi?id=1472 Bugzilla ID: 1472 Fixes: 03b57eb7ab9a ("mbuf: add second dynamic field member") Cc: sta...@dpdk.org Signed-off-by: Shijith Thotton --- lib/mbuf/rte_mbuf.h | 3 +++ 1 file changed, 3 insertions(+) diff -

[PATCH v2] mbuf: fix API to copy mbuf dynamic fields

2024-06-27 Thread Shijith Thotton
mbufs. This patch fixes the same. see https://bugs.dpdk.org/show_bug.cgi?id=1472 Bugzilla ID: 1472 Fixes: 03b57eb7ab9a ("mbuf: add second dynamic field member") Cc: sta...@dpdk.org Signed-off-by: Shijith Thotton Reviewed-by: Morten Brørup Acked-by: Stephen Hemminger --- v2: - Mov

[PATCH v1] common/cnxk: fix initialization of timer LF count

2023-03-13 Thread Shijith Thotton
uot;) Signed-off-by: Shijith Thotton --- drivers/common/cnxk/roc_tim.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/common/cnxk/roc_tim.c b/drivers/common/cnxk/roc_tim.c index 6f256c60fa..f8607b2852 100644 --- a/drivers/common/cnxk/roc_tim.c +++ b/drivers/common/cnxk/

[PATCH v1] eventdev/timer: use loop to check for timeout events

2023-03-15 Thread Shijith Thotton
Replaced sleep with a function to dequeue timer expiry events until either the expected number of events have been dequeued or the specified time has elapsed. This change closely esembles the behavior of the tested systems. Signed-off-by: Shijith Thotton --- app/test/test_event_timer_adapter.c

[PATCH v2] eventdev/timer: fix timeout event wait behavior

2023-03-20 Thread Shijith Thotton
behavior based on the type of timer being used (software or hardware). Fixes: d1f3385d0076 ("test: add event timer adapter auto-test") Signed-off-by: Shijith Thotton --- v2: - Updated commit message and added fixed tag. app/test/test_event_timer_adapter.c | 169 +++---

[PATCH v1] test/event_crypto_adapter: fix runtime parameter test

2023-04-17 Thread Shijith Thotton
rypto: support runtime set/get parameters") Signed-off-by: Shijith Thotton --- app/test/test_event_crypto_adapter.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/app/test/test_event_crypto_adapter.c b/app/test/test_event_crypto_adapter.c index 264d6f731e..

[PATCH v1] event/cnxk: fix setting event attributes in empty get work

2023-04-26 Thread Shijith Thotton
Even when no events are dequeued, it is important to update event attributes as they provide valuable information for checking the work slot state and other attributes. Fixes: c0a9774f030d ("event/cnxk: move post-processing to separate function") Signed-off-by: Shijith Thotton --

[PATCH v2 0/5] mbuf dynamic field expansion

2022-09-07 Thread Shijith Thotton
first cacheline if RTE_IOVA_AS_VA = 1. Shijith Thotton (5): build: add meson option to configure IOVA mode as VA mbuf: add second dynamic field member for VA only build lib: move mbuf next pointer to first cache line drivers: mark Marvell cnxk PMDs work with IOVA as VA drivers: mark software

[PATCH v2 2/5] mbuf: add second dynamic field member for VA only build

2022-09-07 Thread Shijith Thotton
mbuf physical address field is not used in builds which only uses VA. It is used to expand the dynamic field area. Signed-off-by: Shijith Thotton --- lib/mbuf/rte_mbuf_core.h | 26 +- lib/mbuf/rte_mbuf_dyn.c | 2 ++ 2 files changed, 19 insertions(+), 9 deletions

[PATCH v2 1/5] build: add meson option to configure IOVA mode as VA

2022-09-07 Thread Shijith Thotton
file to enable build. mbuf structure holds the physical (PA) and virtual address (VA) of a buffer. if IOVA mode is set to VA, PA is redundant as it is the same as VA. So PA field need not be updated and marked invalid if the build is configured to use only VA. Signed-off-by: Shijith Thotton

[PATCH v2 3/5] lib: move mbuf next pointer to first cache line

2022-09-07 Thread Shijith Thotton
Swapped position of mbuf next pointer and second dynamic field (dynfield2) if the build is configured to use IOVA as VA. This is to move the mbuf next pointer to first cache line. kni library is disabled for this change as it depends on the offset value of next pointer. Signed-off-by: Shijith

[PATCH v2 4/5] drivers: mark Marvell cnxk PMDs work with IOVA as VA

2022-09-07 Thread Shijith Thotton
Enabled the flag pmd_iova_as_va in cnxk driver build files as they work with IOVA as VA. Updated cn9k and cn10k soc build configurations to enable the IOVA as VA build by default. Signed-off-by: Shijith Thotton --- config/arm/meson.build | 8 ++-- drivers/common/cnxk

[PATCH v2 5/5] drivers: mark software PMDs work with IOVA as VA

2022-09-07 Thread Shijith Thotton
Enabled software PMDs in IOVA as VA build as they work with IOVA as VA. Signed-off-by: Shijith Thotton --- drivers/crypto/armv8/meson.build| 1 + drivers/crypto/ipsec_mb/meson.build | 1 + drivers/crypto/null/meson.build | 1 + drivers/crypto/openssl/meson.build | 1 + drivers/dma

RE: [EXT] Re: [PATCH v1 2/4] mbuf: add second dynamic field member for VA only build

2022-09-07 Thread Shijith Thotton
>> > >> > -- >> > On Mon, Aug 29, 2022 at 08:32:20PM +0200, Morten Brørup wrote: >> > > >> > > > From: Shijith Thotton [mailto:sthot...@marvell.com] >> > > &g

RE: [EXT] Re: [PATCH] mbuf: add mbuf physical address field to dynamic field

2022-09-12 Thread Shijith Thotton
>On 7/1/2022 1:24 PM, Shijith Thotton wrote: >>>> If all devices are configured to run in IOVA mode as VA, physical >>>> address field of mbuf (buf_iova) won't be used. In such cases, buf_iova >>>> space is free to use as a dynamic field. So a new dynam

[PATCH] drivers: remove support to limit XAQ in cnxk event driver

2022-09-19 Thread Shijith Thotton
Removed support to limit XAQ from devargs. If XAQ is limited, new add works could run out of XAQ entries and disable the queue. Signed-off-by: Shijith Thotton --- doc/guides/eventdevs/cnxk.rst | 5 ++--- drivers/common/cnxk/roc_mbox.h | 2 +- drivers/common/cnxk/roc_sso.c | 2

[PATCH v2] drivers: remove support to limit XAQ in cnxk event driver

2022-09-20 Thread Shijith Thotton
Removed support to limit XAQ from devargs. If XAQ is limited, new add works could run out of XAQ entries and disable the queue. Signed-off-by: Shijith Thotton --- v2: * Removed used function parameter. doc/guides/eventdevs/cnxk.rst | 5 ++--- drivers/common/cnxk/roc_mbox.h | 2

[PATCH v3 0/5] mbuf dynamic field expansion

2022-09-21 Thread Shijith Thotton
rte_is_iova_as_va_build(). * Moved mbuf next pointer to first cacheline if RTE_IOVA_AS_VA = 1. Shijith Thotton (5): build: add meson option to configure IOVA mode as VA mbuf: add second dynamic field member for VA only build lib: move mbuf next pointer to first cache line drivers: mark Marvell cnxk PMDs

[PATCH v3 1/5] build: add meson option to configure IOVA mode as VA

2022-09-21 Thread Shijith Thotton
file to enable build. mbuf structure holds the physical (PA) and virtual address (VA) of a buffer. if IOVA mode is set to VA, PA is redundant as it is the same as VA. So PA field need not be updated and marked invalid if the build is configured to use only VA. Signed-off-by: Shijith Thotton

[PATCH v3 2/5] mbuf: add second dynamic field member for VA only build

2022-09-21 Thread Shijith Thotton
mbuf physical address field is not used in builds which only uses VA. It is used to expand the dynamic field area. Signed-off-by: Shijith Thotton --- lib/mbuf/rte_mbuf_core.h | 26 +- lib/mbuf/rte_mbuf_dyn.c | 2 ++ 2 files changed, 19 insertions(+), 9 deletions

[PATCH v3 3/5] lib: move mbuf next pointer to first cache line

2022-09-21 Thread Shijith Thotton
Swapped position of mbuf next pointer and second dynamic field (dynfield2) if the build is configured to use IOVA as VA. This is to move the mbuf next pointer to first cache line. kni library is disabled for this change as it depends on the offset value of next pointer. Signed-off-by: Shijith

[PATCH v3 4/5] drivers: mark Marvell cnxk PMDs work with IOVA as VA

2022-09-21 Thread Shijith Thotton
Enabled the flag pmd_iova_as_va in cnxk driver build files as they work with IOVA as VA. Updated cn9k and cn10k soc build configurations to enable the IOVA as VA build by default. Signed-off-by: Shijith Thotton --- config/arm/meson.build | 8 +++- drivers/common/cnxk

[PATCH v3 5/5] drivers: mark software PMDs work with IOVA as VA

2022-09-21 Thread Shijith Thotton
Enabled software PMDs in IOVA as VA build as they work with IOVA as VA. Signed-off-by: Shijith Thotton --- drivers/crypto/armv8/meson.build| 1 + drivers/crypto/ipsec_mb/meson.build | 1 + drivers/crypto/null/meson.build | 1 + drivers/crypto/openssl/meson.build | 1 + drivers/dma

RE: [EXT] Re: [PATCH v2 0/5] mbuf dynamic field expansion

2022-09-21 Thread Shijith Thotton
>> >> This is a continuation of the discussions[1] to add mbuf physical address >> field to >dynamic field. >> Previous version was to add PA field to dynamic field area based on the EAL >IOVA mode option. It was >> deemed unsafe as some components could still use the PA field without >checking IO

RE: [PATCH v3 0/5] mbuf dynamic field expansion

2022-09-27 Thread Shijith Thotton
: > * Cleared use of buf_iova from cnxk PMD. > >v2: > * Used RTE_IOVA_AS_VA instread of rte_is_iova_as_va_build(). > * Moved mbuf next pointer to first cacheline if RTE_IOVA_AS_VA = 1. > >Shijith Thotton (5): > build: add meson option to configure IOVA mode as VA > mbuf

RE: [EXT] Re: [PATCH v3 0/5] mbuf dynamic field expansion

2022-09-28 Thread Shijith Thotton
Hi Olivier, Thanks for the review. >On Wed, Sep 21, 2022 at 07:26:16PM +0530, Shijith Thotton wrote: >> This is a continuation of the discussions[1] to add mbuf physical address >> field to >dynamic field. >> Previous version was to add PA field to dynamic field area bas

RE: [EXT] Re: [PATCH v3 1/5] build: add meson option to configure IOVA mode as VA

2022-09-28 Thread Shijith Thotton
d invalid if the build is >> configured to use only VA. >> >> Signed-off-by: Shijith Thotton >> --- >> app/test-bbdev/test_bbdev_perf.c | 2 +- >> app/test-crypto-perf/cperf_test_common.c | 5 +-- >> app/test/test_bpf.c |

RE: [EXT] Re: [PATCH v3 2/5] mbuf: add second dynamic field member for VA only build

2022-09-28 Thread Shijith Thotton
>> > > mbuf physical address field is not used in builds which only uses VA. It >> > > is used to expand the dynamic field area. >> > > >> > > Signed-off-by: Shijith Thotton >> > >> > We cannot condition the use of the dynamic f

RE: [EXT] Re: [PATCH v3 3/5] lib: move mbuf next pointer to first cache line

2022-09-28 Thread Shijith Thotton
xt pointer. >> >> Signed-off-by: Shijith Thotton >> --- >> lib/mbuf/rte_mbuf_core.h | 29 + >> lib/meson.build | 3 +++ >> 2 files changed, 24 insertions(+), 8 deletions(-) >> >> diff --git a/lib/mbuf/rte_mbuf_

RE: [EXT] Re: [PATCH v3 4/5] drivers: mark Marvell cnxk PMDs work with IOVA as VA

2022-09-28 Thread Shijith Thotton
>> Enabled the flag pmd_iova_as_va in cnxk driver build files as they work >> with IOVA as VA. Updated cn9k and cn10k soc build configurations to >> enable the IOVA as VA build by default. >> >> Signed-off-by: Shijith Thotton >> --- >> con

RE: [EXT] Re: [PATCH v3 4/5] drivers: mark Marvell cnxk PMDs work with IOVA as VA

2022-09-29 Thread Shijith Thotton
>> >> Enabled the flag pmd_iova_as_va in cnxk driver build files as they work >> >> with IOVA as VA. Updated cn9k and cn10k soc build configurations to >> >> enable the IOVA as VA build by default. >> >> >> >> Signed-off-

[PATCH v4 0/7] mbuf dynamic field expansion

2022-10-07 Thread Shijith Thotton
. * Updated release notes. v3: * Cleared use of buf_iova from cnxk PMD. v2: * Used RTE_IOVA_AS_VA instread of rte_is_iova_as_va_build(). * Moved mbuf next pointer to first cacheline if RTE_IOVA_AS_VA = 1. Shijith Thotton (7): mbuf: add API to get and set mbuf physical address test/dma: use

[PATCH v4 1/7] mbuf: add API to get and set mbuf physical address

2022-10-07 Thread Shijith Thotton
Added APIs rte_mbuf_iova_set and rte_mbuf_iova_get to set and get the physical address of an mbuf respectively. Updated applications and library to use the same. Signed-off-by: Shijith Thotton --- app/test-bbdev/test_bbdev_perf.c | 2 +- app/test-crypto-perf/cperf_test_common.c | 5

[PATCH v4 2/7] test/dma: use API to get mbuf data physical address

2022-10-07 Thread Shijith Thotton
Used rte_mbuf_data_iova API to get the physical address of mbuf data. Signed-off-by: Shijith Thotton --- app/test/test_dmadev.c | 33 ++--- 1 file changed, 14 insertions(+), 19 deletions(-) diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c index

[PATCH v4 3/7] build: add meson option to configure IOVA mode as PA

2022-10-07 Thread Shijith Thotton
pmd_supports_disable_iova_as_pa in its build file. mbuf structure holds the physical (PA) and virtual address (VA) of a buffer. if IOVA as PA is disabled at compile time, PA field (buf_iova) of mbuf is redundant as it is the same as VA and is replaced by a dummy field. Signed-off-by: Shijith Thotton --- app/test

[PATCH v4 4/7] mbuf: add second dynamic field member

2022-10-07 Thread Shijith Thotton
If IOVA as PA is disabled during build, mbuf physical address field is undefined. This space is used to add the second dynamic field. Signed-off-by: Shijith Thotton --- lib/mbuf/rte_mbuf_core.h | 6 +- lib/mbuf/rte_mbuf_dyn.c | 3 +++ 2 files changed, 8 insertions(+), 1 deletion(-) diff

[PATCH v4 5/7] lib: move mbuf next pointer to first cache line

2022-10-07 Thread Shijith Thotton
Swapped position of mbuf next pointer and second dynamic field (dynfield2) if the build is configured to disable IOVA as PA. This is to move the mbuf next pointer to first cache line. Signed-off-by: Shijith Thotton --- doc/guides/rel_notes/release_22_11.rst | 3 +++ lib/mbuf/rte_mbuf_core.h

[PATCH v4 6/7] drivers: mark cnxk PMDs work with IOVA as PA disabled

2022-10-07 Thread Shijith Thotton
Enabled the flag pmd_supports_disable_iova_as_pa in cnxk driver build files as they work with IOVA as VA. Updated cn9k and cn10k soc build configurations to disable the IOVA as PA build by default. Signed-off-by: Shijith Thotton --- config/arm/meson.build | 8 +++- doc/guides

[PATCH v4 7/7] drivers: mark software PMDs work with IOVA as PA disabled

2022-10-07 Thread Shijith Thotton
Enabled software PMDs in IOVA as PA disabled build as they work with IOVA as VA. Signed-off-by: Shijith Thotton --- drivers/crypto/armv8/meson.build| 1 + drivers/crypto/ipsec_mb/meson.build | 1 + drivers/crypto/null/meson.build | 1 + drivers/crypto/openssl/meson.build | 1 + drivers

RE: [EXT] Re: [PATCH v3 0/5] mbuf dynamic field expansion

2022-10-07 Thread Shijith Thotton
>> This is a continuation of the discussions[1] to add mbuf physical address >> field to >dynamic field. >> Previous version was to add PA field to dynamic field area based on the EAL >IOVA mode option. It was >> deemed unsafe as some components could still use the PA field without >checking IOVA

RE: [EXT] [PATCH v3 5/5] examples/l3fwd: use em vector path for event vector

2022-10-07 Thread Shijith Thotton
>Use em vector path to process event vector. > >Signed-off-by: Pavan Nikhilesh >--- > examples/l3fwd/l3fwd_em.c| 12 +++-- > examples/l3fwd/l3fwd_em.h| 29 +-- > examples/l3fwd/l3fwd_em_hlm.h| 72 +--- > examples/l3fwd/l3fwd_em_sequentia

RE: [EXT] [PATCH v3 4/5] examples/l3fwd: fix event vector processing in fib

2022-10-07 Thread Shijith Thotton
> >Fix stack overflow when event vector size is greater than >MAX_BURST_SIZE. >Add missing mac swap and rfc1812 stage. > >Fixes: e8adca1951d4 ("examples/l3fwd: support event vector") > >Signed-off-by: Pavan Nikhilesh >--- > examples/l3fwd/l3fwd_fib.c | 123 ++--- > 1

RE: [EXT] Re: [PATCH v4 1/7] mbuf: add API to get and set mbuf physical address

2022-10-07 Thread Shijith Thotton
Hi Olivier, >On Sat, Oct 08, 2022 at 01:00:23AM +0530, Shijith Thotton wrote: >> Added APIs rte_mbuf_iova_set and rte_mbuf_iova_get to set and get the >> physical address of an mbuf respectively. Updated applications and >> library to use the same. >> >&g

RE: [EXT] Re: [PATCH v3 4/5] drivers: mark Marvell cnxk PMDs work with IOVA as VA

2022-10-07 Thread Shijith Thotton
>> Enabled the flag pmd_iova_as_va in cnxk driver build files as they work >> with IOVA as VA. Updated cn9k and cn10k soc build configurations to >> enable the IOVA as VA build by default. >> >> Signed-off-by: Shijith Thotton >> --- >> con

[PATCH v5 0/7] mbuf dynamic field expansion

2022-10-07 Thread Shijith Thotton
. * Moved to #if scheme instead of union. * Updated release notes. v3: * Cleared use of buf_iova from cnxk PMD. v2: * Used RTE_IOVA_AS_VA instread of rte_is_iova_as_va_build(). * Moved mbuf next pointer to first cacheline if RTE_IOVA_AS_VA = 1. Shijith Thotton (7): mbuf: add API to get and set

[PATCH v5 1/7] mbuf: add API to get and set mbuf physical address

2022-10-07 Thread Shijith Thotton
Added APIs rte_mbuf_iova_set and rte_mbuf_iova_get to set and get the physical address of an mbuf respectively. Updated applications and library to use the same. Signed-off-by: Shijith Thotton Acked-by: Olivier Matz --- app/test-bbdev/test_bbdev_perf.c | 2 +- app/test-crypto-perf

[PATCH v5 2/7] test/dma: use API to get mbuf data physical address

2022-10-07 Thread Shijith Thotton
Used rte_mbuf_data_iova API to get the physical address of mbuf data. Signed-off-by: Shijith Thotton Acked-by: Olivier Matz --- app/test/test_dmadev.c | 33 ++--- 1 file changed, 14 insertions(+), 19 deletions(-) diff --git a/app/test/test_dmadev.c b/app/test

[PATCH v5 3/7] build: add meson option to configure IOVA mode as PA

2022-10-07 Thread Shijith Thotton
pmd_supports_disable_iova_as_pa in its build file. mbuf structure holds the physical (PA) and virtual address (VA) of a buffer. if IOVA as PA is disabled at compile time, PA field (buf_iova) of mbuf is redundant as it is the same as VA and is replaced by a dummy field. Signed-off-by: Shijith Thotton Acked-by: Olivier

[PATCH v5 4/7] mbuf: add second dynamic field member

2022-10-07 Thread Shijith Thotton
If IOVA as PA is disabled during build, mbuf physical address field is undefined. This space is used to add the second dynamic field. Signed-off-by: Shijith Thotton Acked-by: Olivier Matz --- lib/mbuf/rte_mbuf_core.h | 6 +- lib/mbuf/rte_mbuf_dyn.c | 3 +++ 2 files changed, 8 insertions

[PATCH v5 5/7] lib: move mbuf next pointer to first cache line

2022-10-07 Thread Shijith Thotton
Swapped position of mbuf next pointer and second dynamic field (dynfield2) if the build is configured to disable IOVA as PA. This is to move the mbuf next pointer to first cache line. Signed-off-by: Shijith Thotton Acked-by: Olivier Matz --- doc/guides/rel_notes/release_22_11.rst | 3 +++ lib

[PATCH v5 6/7] drivers: mark cnxk PMDs work with IOVA as PA disabled

2022-10-07 Thread Shijith Thotton
Enabled the flag pmd_supports_disable_iova_as_pa in cnxk driver build files as they work with IOVA as VA. Updated cn9k and cn10k soc build configurations to disable the IOVA as PA build by default. Signed-off-by: Shijith Thotton Acked-by: Olivier Matz --- config/arm/meson.build

[PATCH v5 7/7] drivers: mark software PMDs work with IOVA as PA disabled

2022-10-07 Thread Shijith Thotton
Enabled software PMDs in IOVA as PA disabled build as they work with IOVA as VA. Signed-off-by: Shijith Thotton Acked-by: Olivier Matz --- drivers/crypto/armv8/meson.build| 1 + drivers/crypto/ipsec_mb/meson.build | 1 + drivers/crypto/null/meson.build | 1 + drivers/crypto/openssl

RE: [EXT] Re: [PATCH v5 5/7] lib: move mbuf next pointer to first cache line

2022-10-07 Thread Shijith Thotton
>> Swapped position of mbuf next pointer and second dynamic field (dynfield2) >> if the build is configured to disable IOVA as PA. This is to move the >> mbuf next pointer to first cache line. >> >> Signed-off-by: Shijith Thotton >> Acked-by: Olivier Matz

RE: [EXT] [PATCH v5 5/5] examples/l3fwd: use em vector path for event vector

2022-10-12 Thread Shijith Thotton
0; i != j; i += FWDSTEP) >+ processx4_step3(&vec->mbufs[i], &dst_ports[i]); >+ for (; i < vec->nb_elem; i++) >+ process_packet(vec->mbufs[i], &dst_ports[i]); >+ >+ process_event_vector(vec, dst_ports); > } > > #endif /* __L3FWD_EM_SEQUENTIAL_H__ */ >diff --git a/examples/l3fwd/l3fwd_event.h b/examples/l3fwd/l3fwd_event.h >index 3fe38aada0..e21817c36b 100644 >--- a/examples/l3fwd/l3fwd_event.h >+++ b/examples/l3fwd/l3fwd_event.h >@@ -103,27 +103,6 @@ process_dst_port(uint16_t *dst_ports, uint16_t >nb_elem) > } > #endif > >-static inline void >-event_vector_attr_validate(struct rte_event_vector *vec, struct rte_mbuf >*mbuf) >-{ >- /* l3fwd application only changes mbuf port while processing */ >- if (vec->attr_valid && (vec->port != mbuf->port)) >- vec->attr_valid = 0; >-} >- >-static inline void >-event_vector_txq_set(struct rte_event_vector *vec, uint16_t txq) >-{ >- if (vec->attr_valid) { >- vec->queue = txq; >- } else { >- int i; >- >- for (i = 0; i < vec->nb_elem; i++) >- rte_event_eth_tx_adapter_txq_set(vec->mbufs[i], txq); >- } >-} >- > static inline uint16_t > filter_bad_packets(struct rte_mbuf **mbufs, uint16_t *dst_port, > uint16_t nb_pkts) >-- >2.25.1 For the series: Acked-by: Shijith Thotton

[dpdk-dev] [PATCH 0/8] Crypto adapter support for Marvell CNXK driver

2021-08-30 Thread Shijith Thotton
First three patches are to support crypto adapter implementation and the remaining patches add event crypto adapter OP_FORWARD mode support on Marvell CN9K and CN10K SOC. Shijith Thotton (8): net/cnxk: add flag to show CPT can enqueue events event/cnxk: add macro to set eventdev ops common

[dpdk-dev] [PATCH 1/8] net/cnxk: add flag to show CPT can enqueue events

2021-08-30 Thread Shijith Thotton
CPT can be told to submit events to SSO upon completion. Crypto adapter uses this feature and the new flag can be used to optimize receive path in those cases. Signed-off-by: Shijith Thotton --- drivers/net/cnxk/cn10k_rx.h | 5 +++-- drivers/net/cnxk/cn9k_rx.h | 3 ++- 2 files changed, 5

[dpdk-dev] [PATCH 2/8] event/cnxk: add macro to set eventdev ops

2021-08-30 Thread Shijith Thotton
Added a common macro to set eventdev enqueue and dequeue operations to reduce code. Signed-off-by: Shijith Thotton Signed-off-by: Nithin Dabilpuram --- drivers/event/cnxk/cn10k_eventdev.c | 133 +- drivers/event/cnxk/cn9k_eventdev.c | 267 +++- 2 files

[dpdk-dev] [PATCH 3/8] common/cnxk: add API to check CPT IQ is full

2021-08-30 Thread Shijith Thotton
Added flow control based check to determine CPT IQ is full. Signed-off-by: Shijith Thotton --- drivers/common/cnxk/roc_cpt.c | 6 -- drivers/common/cnxk/roc_cpt.h | 11 +++ drivers/common/cnxk/roc_cpt_priv.h | 6 -- 3 files changed, 15 insertions(+), 8 deletions

[dpdk-dev] [PATCH 4/8] drivers: add cnxk crypto adapter eventdev ops

2021-08-30 Thread Shijith Thotton
Added eventdev ops required to initialize crypto adapter. Signed-off-by: Shijith Thotton --- drivers/crypto/cnxk/cnxk_cryptodev_ops.h | 9 +++ drivers/event/cnxk/cn10k_eventdev.c | 46 drivers/event/cnxk/cn9k_eventdev.c | 45 drivers/event/cnxk

[dpdk-dev] [PATCH 5/8] crypto/cnxk: add cn9k crypto adapter fast path ops

2021-08-30 Thread Shijith Thotton
Added crypto adapter enqueue and dequeue operations for CN9K. Signed-off-by: Shijith Thotton --- drivers/crypto/cnxk/cn9k_cryptodev_ops.c | 235 --- drivers/crypto/cnxk/cn9k_cryptodev_ops.h | 6 + drivers/crypto/cnxk/cnxk_cryptodev_ops.h | 28 +++ drivers/crypto/cnxk

[dpdk-dev] [PATCH 6/8] event/cnxk: add cn9k crypto adapter fast path ops

2021-08-30 Thread Shijith Thotton
Set crypto adapter enqueue and dequeue operations for CN9K. Signed-off-by: Shijith Thotton --- drivers/event/cnxk/cn9k_eventdev.c | 94 +++- drivers/event/cnxk/cn9k_worker.c | 22 + drivers/event/cnxk/cn9k_worker.h | 41 - drivers

[dpdk-dev] [PATCH 7/8] crypto/cnxk: add cn10k crypto adapter fast path ops

2021-08-30 Thread Shijith Thotton
Added crypto adapter enqueue and dequeue operations for CN10K. Signed-off-by: Shijith Thotton --- drivers/crypto/cnxk/cn10k_cryptodev_ops.c | 95 +++ drivers/crypto/cnxk/cn10k_cryptodev_ops.h | 6 ++ drivers/crypto/cnxk/version.map | 2 + 3 files changed, 103

[dpdk-dev] [PATCH 8/8] event/cnxk: add cn10k crypto adapter fast path ops

2021-08-30 Thread Shijith Thotton
Set crypto adapter enqueue and dequeue operations for CN10K. Signed-off-by: Shijith Thotton --- doc/guides/rel_notes/release_21_11.rst | 3 ++ drivers/event/cnxk/cn10k_eventdev.c | 45 +++- drivers/event/cnxk/cn10k_worker.c| 11 drivers/event/cnxk/cn10k_worker.h

[dpdk-dev] [PATCH v2] event/cnxk: fix SSO and TIM argument parsing

2021-08-30 Thread Shijith Thotton
Type of kvargs value and handler function argument should match to avoid spilling memory. Fixes: 7ffa7379965e ("event/cnxk: add option to configure getwork mode") Signed-off-by: Shijith Thotton --- v2: * Rebased. drivers/event/cnxk/cnxk_eventdev.c | 6 +++--- drivers/

[dpdk-dev] [PATCH v2] event/cnxk: reduce max timer chunk pool cache size

2021-08-30 Thread Shijith Thotton
Reduced max chunk pool cache size from RTE_MEMPOOL_CACHE_MAX_SIZE(512) to 128. If chunk pool cache is empty, it gets filled during arm. Filling 512 entries at a time will fail arm if timeout is shorter. Fixes: 0e792433d051 ("event/cnxk: create and free timer adapter") Signed-off-b

[dpdk-dev] [PATCH] examples/l3fwd: add changes to use event vector

2021-08-30 Thread Shijith Thotton
--mode=eventdev \ --eventq-sched=ordered --enable-vector --vector-size 16 Signed-off-by: Shijith Thotton --- Depends-on: series=18322 (eventdev: simplify Rx adapter event vector config) doc/guides/sample_app_ug/l3_forward.rst| 7 + examples/l3fwd/l3fwd.h | 26

[dpdk-dev] [PATCH] eventdev: update crypto adapter metadata structures

2021-08-30 Thread Shijith Thotton
the ordering issue mentioned before. This patch removes the reserve field from request info and makes event crypto metadata type to structure from union to make space for response info. App and drivers are updated as per metadata change. Signed-off-by: Shijith Thotton --- v1: * Rebased. app/test

[dpdk-dev] [PATCH v2] eventdev: update crypto adapter metadata structures

2021-08-30 Thread Shijith Thotton
the ordering issue mentioned before. This patch removes the reserve field from request info and makes event crypto metadata type to structure from union to make space for response info. App and drivers are updated as per metadata change. Signed-off-by: Shijith Thotton --- v2: * Updated deprecation

[dpdk-dev] [PATCH] app/test: fix event timer adapter create unit test

2021-08-30 Thread Shijith Thotton
Removed freeing of unallocated mempool in event timer adapter create unit test. Fixes: d1f3385d0076 ("test: add event timer adapter auto-test") Signed-off-by: Shijith Thotton --- app/test/test_event_timer_adapter.c | 2 -- 1 file changed, 2 deletions(-) diff --git

Re: [dpdk-dev] [PATCH v2] eventdev: update crypto adapter metadata structures

2021-08-30 Thread Shijith Thotton
ace out of request info. It >> also solves the ordering issue mentioned before. >> >> This patch removes the reserve field from request info and makes event >> crypto metadata type to structure from union to make space for response >> info. >> >> App and dri

[dpdk-dev] [PATCH v3] eventdev: update crypto adapter metadata structures

2021-08-31 Thread Shijith Thotton
the ordering issue mentioned before. This patch removes the reserve field from request info and makes event crypto metadata type to structure from union to make space for response info. App and drivers are updated as per metadata change. Signed-off-by: Shijith Thotton Acked-by: Anoob Joseph

[dpdk-dev] [PATCH v2 0/8] Crypto adapter support for Marvell CNXK driver

2021-09-02 Thread Shijith Thotton
First three patches are to support crypto adapter implementation and the remaining patches add event crypto adapter OP_FORWARD mode support on Marvell CN9K and CN10K SOC. v2: * Fixed checkpatch errors. Shijith Thotton (8): net/cnxk: add flag to show CPT can enqueue events event/cnxk: add

[dpdk-dev] [PATCH v2 1/8] net/cnxk: add flag to show CPT can enqueue events

2021-09-02 Thread Shijith Thotton
CPT can be told to submit events to SSO upon completion. Crypto adapter uses this feature and the new flag can be used to optimize receive path in those cases. Signed-off-by: Shijith Thotton --- drivers/net/cnxk/cn10k_rx.h | 5 +++-- drivers/net/cnxk/cn9k_rx.h | 3 ++- 2 files changed, 5

[dpdk-dev] [PATCH v2 2/8] event/cnxk: add macro to set eventdev ops

2021-09-02 Thread Shijith Thotton
Added a common macro to set eventdev enqueue and dequeue operations to reduce code. Signed-off-by: Shijith Thotton Signed-off-by: Nithin Dabilpuram --- drivers/event/cnxk/cn10k_eventdev.c | 139 +- drivers/event/cnxk/cn9k_eventdev.c | 273 +++- 2 files

[dpdk-dev] [PATCH v2 3/8] common/cnxk: add API to check CPT IQ is full

2021-09-02 Thread Shijith Thotton
Added flow control based check to determine CPT IQ is full. Signed-off-by: Shijith Thotton --- drivers/common/cnxk/roc_cpt.c | 6 -- drivers/common/cnxk/roc_cpt.h | 11 +++ drivers/common/cnxk/roc_cpt_priv.h | 6 -- 3 files changed, 15 insertions(+), 8 deletions

[dpdk-dev] [PATCH v2 4/8] drivers: add cnxk crypto adapter eventdev ops

2021-09-02 Thread Shijith Thotton
Added eventdev ops required to initialize crypto adapter. Signed-off-by: Shijith Thotton --- drivers/crypto/cnxk/cnxk_cryptodev_ops.h | 9 +++ drivers/event/cnxk/cn10k_eventdev.c | 46 drivers/event/cnxk/cn9k_eventdev.c | 45 drivers/event/cnxk

[dpdk-dev] [PATCH v2 5/8] crypto/cnxk: add cn9k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Added crypto adapter enqueue and dequeue operations for CN9K. Signed-off-by: Shijith Thotton Acked-by: Ray Kinsella Acked-by: Anoob Joseph --- drivers/crypto/cnxk/cn9k_cryptodev_ops.c | 235 --- drivers/crypto/cnxk/cn9k_cryptodev_ops.h | 6 + drivers/crypto/cnxk

[dpdk-dev] [PATCH v2 6/8] event/cnxk: add cn9k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Set crypto adapter enqueue and dequeue operations for CN9K. Signed-off-by: Shijith Thotton --- drivers/event/cnxk/cn9k_eventdev.c | 94 +++- drivers/event/cnxk/cn9k_worker.c | 22 + drivers/event/cnxk/cn9k_worker.h | 41 - drivers

[dpdk-dev] [PATCH v2 7/8] crypto/cnxk: add cn10k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Added crypto adapter enqueue and dequeue operations for CN10K. Signed-off-by: Shijith Thotton Acked-by: Ray Kinsella Acked-by: Anoob Joseph --- drivers/crypto/cnxk/cn10k_cryptodev_ops.c | 95 +++ drivers/crypto/cnxk/cn10k_cryptodev_ops.h | 6 ++ drivers/crypto/cnxk

[dpdk-dev] [PATCH v2 8/8] event/cnxk: add cn10k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Set crypto adapter enqueue and dequeue operations for CN10K. Signed-off-by: Shijith Thotton --- doc/guides/rel_notes/release_21_11.rst | 3 ++ drivers/event/cnxk/cn10k_eventdev.c | 45 +++- drivers/event/cnxk/cn10k_worker.c| 11 drivers/event/cnxk/cn10k_worker.h

[dpdk-dev] [PATCH v3 0/8] Crypto adapter support for Marvell CNXK driver

2021-09-02 Thread Shijith Thotton
First three patches are to support crypto adapter implementation and the remaining patches add event crypto adapter OP_FORWARD mode support on Marvell CN9K and CN10K SOC. v3: * Fixed checkpatch errors. v2: * Fixed checkpatch errors. Shijith Thotton (8): net/cnxk: add flag to show CPT can

[dpdk-dev] [PATCH v3 1/8] net/cnxk: add flag to show CPT can enqueue events

2021-09-02 Thread Shijith Thotton
CPT can be told to submit events to SSO upon completion. Crypto adapter uses this feature and the new flag can be used to optimize receive path in those cases. Signed-off-by: Shijith Thotton --- drivers/net/cnxk/cn10k_rx.h | 5 +++-- drivers/net/cnxk/cn9k_rx.h | 3 ++- 2 files changed, 5

[dpdk-dev] [PATCH v3 2/8] event/cnxk: add macro to set eventdev ops

2021-09-02 Thread Shijith Thotton
Added a common macro to set eventdev enqueue and dequeue operations to reduce code. Signed-off-by: Shijith Thotton Signed-off-by: Nithin Dabilpuram --- drivers/event/cnxk/cn10k_eventdev.c | 134 +- drivers/event/cnxk/cn9k_eventdev.c | 268 +++- 2 files

[dpdk-dev] [PATCH v3 3/8] common/cnxk: add API to check CPT IQ is full

2021-09-02 Thread Shijith Thotton
Added flow control based check to determine CPT IQ is full. Signed-off-by: Shijith Thotton --- drivers/common/cnxk/roc_cpt.c | 6 -- drivers/common/cnxk/roc_cpt.h | 11 +++ drivers/common/cnxk/roc_cpt_priv.h | 6 -- 3 files changed, 15 insertions(+), 8 deletions

[dpdk-dev] [PATCH v3 4/8] drivers: add cnxk crypto adapter eventdev ops

2021-09-02 Thread Shijith Thotton
Added eventdev ops required to initialize crypto adapter. Signed-off-by: Shijith Thotton --- drivers/crypto/cnxk/cnxk_cryptodev_ops.h | 9 +++ drivers/event/cnxk/cn10k_eventdev.c | 46 drivers/event/cnxk/cn9k_eventdev.c | 45 drivers/event/cnxk

[dpdk-dev] [PATCH v3 5/8] crypto/cnxk: add cn9k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Added crypto adapter enqueue and dequeue operations for CN9K. Signed-off-by: Shijith Thotton Acked-by: Ray Kinsella Acked-by: Anoob Joseph --- drivers/crypto/cnxk/cn9k_cryptodev_ops.c | 235 --- drivers/crypto/cnxk/cn9k_cryptodev_ops.h | 6 + drivers/crypto/cnxk

[dpdk-dev] [PATCH v3 6/8] event/cnxk: add cn9k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Set crypto adapter enqueue and dequeue operations for CN9K. Signed-off-by: Shijith Thotton --- drivers/event/cnxk/cn9k_eventdev.c | 94 +++- drivers/event/cnxk/cn9k_worker.c | 22 + drivers/event/cnxk/cn9k_worker.h | 41 - drivers

[dpdk-dev] [PATCH v3 7/8] crypto/cnxk: add cn10k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Added crypto adapter enqueue and dequeue operations for CN10K. Signed-off-by: Shijith Thotton Acked-by: Ray Kinsella Acked-by: Anoob Joseph --- drivers/crypto/cnxk/cn10k_cryptodev_ops.c | 95 +++ drivers/crypto/cnxk/cn10k_cryptodev_ops.h | 6 ++ drivers/crypto/cnxk

[dpdk-dev] [PATCH v3 8/8] event/cnxk: add cn10k crypto adapter fast path ops

2021-09-02 Thread Shijith Thotton
Set crypto adapter enqueue and dequeue operations for CN10K. Signed-off-by: Shijith Thotton --- doc/guides/rel_notes/release_21_11.rst | 3 ++ drivers/event/cnxk/cn10k_eventdev.c | 45 +++- drivers/event/cnxk/cn10k_worker.c| 11 drivers/event/cnxk/cn10k_worker.h

[dpdk-dev] [PATCH] eventdev: cryptodev start in crypto adapter spec

2021-09-02 Thread Shijith Thotton
Event crypto adapter spec does not mention about cryptodev start and stop. Cryptodev attached to the adapter should be started before calling crypto adapter start. Added the same in spec and test application. Signed-off-by: Shijith Thotton --- app/test/test_event_crypto_adapter.c| 8

[dpdk-dev] [PATCH] examples/l2fwd-event: changes to use event vector

2021-09-04 Thread Shijith Thotton
--mode=eventdev \ --eventq-sched=ordered --enable-vector --vector-size 16 Signed-off-by: Shijith Thotton --- Depends-on: series=18322 (eventdev: simplify Rx adapter event vector config) doc/guides/sample_app_ug/l2_forward_event.rst | 13 +- examples/l2fwd-event/l2fwd_common.h

[dpdk-dev] [PATCH v2] examples/l3fwd: add changes to use event vector

2021-09-04 Thread Shijith Thotton
--mode=eventdev \ --eventq-sched=ordered --enable-vector --vector-size 16 Signed-off-by: Shijith Thotton --- Depends-on: series=18322 (eventdev: simplify Rx adapter event vector config) v2: * Fixed setting event vector attribute. doc/guides/sample_app_ug/l3_forward.rst| 7 + examples

<    1   2   3   4   5   6   >