RE: [EXT] Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-15 Thread Pavan Nikhilesh Bhagavatula
= > On Tue, 13 Jun 2023 14:55:48 +0530 > wrote: > > > static __rte_always_inline void > > cn10k_sso_txq_fc_wait(const struct cn10k_eth_txq *txq) > > { > > +#ifdef RTE_ARCH_ARM64 > > + uint64_t space; > > + > > + asm volatile(PLT_CPU_FEATURE_PREAMBLE > > +" ldxr %[s

Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-15 Thread Stephen Hemminger
On Tue, 13 Jun 2023 14:55:48 +0530 wrote: > static __rte_always_inline void > cn10k_sso_txq_fc_wait(const struct cn10k_eth_txq *txq) > { > +#ifdef RTE_ARCH_ARM64 > + uint64_t space; > + > + asm volatile(PLT_CPU_FEATURE_PREAMBLE > + " ldxr %[space], [%[addr]]

Re: [EXT] Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-14 Thread Jerin Jacob Kollanukkaran
Kottidi ; dev@dpdk.org Subject: [EXT] Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait External Email Hello Pavan and Jerin, The Community Lab's CI testing failed this patchseries on clang compile on ARM systems. That wasn't properly reported to Pat

RE: [EXT] Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-14 Thread Pavan Nikhilesh Bhagavatula
; Kiran Kumar Kokkilagadda ; Sunil Kumar Kori ; Satha Koteswara Rao Kottidi ; dev@dpdk.org Subject: [EXT] Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait External Email Hello Pavan and Jerin, The Community Lab's CI testing failed this patchseri

Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-14 Thread Patrick Robb
Hello Pavan and Jerin, The Community Lab's CI testing failed this patchseries on clang compile on ARM systems. That wasn't properly reported to Patchwork due to issues with our reporting process, which we are resolving currently. This updated report show the failed compile. Apologies for the incom

Re: [PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-14 Thread Jerin Jacob
On Tue, Jun 13, 2023 at 2:56 PM wrote: > > From: Pavan Nikhilesh > > Use WFE is Tx path when waiting for space in the Tx queue. > Depending upon the Tx queue contention and size, WFE will > reduce the cache pressure and power consumption. > In multi-core scenarios we have observed up to 8W power

[PATCH v2 3/3] event/cnxk: use WFE in Tx fc wait

2023-06-13 Thread pbhagavatula
From: Pavan Nikhilesh Use WFE is Tx path when waiting for space in the Tx queue. Depending upon the Tx queue contention and size, WFE will reduce the cache pressure and power consumption. In multi-core scenarios we have observed up to 8W power reduction. Signed-off-by: Pavan Nikhilesh --- driv