[PATCH] net/mlx5: do not poll CQEs when no available elts

2024-12-05 Thread Gavin Hu
replenishment succeeds during the polling process, the routine will resume its operation. Fixes: 1ded26239aa0 ("net/mlx5: refactor vectorized Rx") Cc: sta...@dpdk.org Reported-by: Changqi Dingluo Signed-off-by: Gavin Hu --- drivers/net/mlx5/mlx5_rxtx_vec.c | 3 +++ 1 file changed, 3 insertion

Re: [dpdk-dev] [PATCH v1 2/2] ring: use wfe to wait for ring tail update on aarch64

2020-04-26 Thread Gavin Hu
Hi Konstantin, > -Original Message- > From: Ananyev, Konstantin > Sent: Friday, April 24, 2020 9:24 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; jer...@marvell.com; Honnappa Nagarahalli > ; Ruifeng Wang >

[dpdk-dev] [PATCH v2 2/2] ring: use wfe to wait for ring tail update on aarch64

2020-04-26 Thread Gavin Hu
Instead of polling for tail to be updated, use wfe instruction. Signed-off-by: Gavin Hu Reviewed-by: Ruifeng Wang Reviewed-by: Steve Capper Reviewed-by: Ola Liljedahl Reviewed-by: Honnappa Nagarahalli --- lib/librte_ring/rte_ring_c11_mem.h | 4 ++-- lib/librte_ring/rte_ring_generic.h | 3

[dpdk-dev] [PATCH v2 1/2] spinlock: use wfe to reduce contention on aarch64

2020-04-26 Thread Gavin Hu
was measured. Signed-off-by: Gavin Hu Reviewed-by: Ruifeng Wang Reviewed-by: Phil Yang Reviewed-by: Steve Capper Reviewed-by: Ola Liljedahl Reviewed-by: Honnappa Nagarahalli Tested-by: Pavan Nikhilesh --- lib/librte_eal/include/generic/rte_spinlock.h | 4 ++-- 1 file changed, 2 insertions

[dpdk-dev] [PATCH v2 0/2] Use WFE for spinlock and ring

2020-04-26 Thread Gavin Hu
The rte_wait_until_equal_xxx APIs abstract the functionality of 'polling for a memory location to become equal to a given value'[1]. Use the API for the rte spinlock and ring implementations. [1] http://patches.dpdk.org/cover/62703/ Gavin Hu (2): spinlock: use wfe to reduce con

Re: [dpdk-dev] [PATCH v1 2/2] ring: use wfe to wait for ring tail update on aarch64

2020-04-26 Thread Gavin Hu
Hi Honnappa, > -Original Message- > From: Honnappa Nagarahalli > Sent: Friday, April 24, 2020 10:46 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; jer...@marvell.com; Ruifeng Wang > ; Phil Yang ; Joyce Kong > ; Hon

[dpdk-dev] [PATCH v1 1/2] spinlock: use wfe to reduce contention on aarch64

2020-04-24 Thread Gavin Hu
was measured. Signed-off-by: Gavin Hu Reviewed-by: Ruifeng Wang Reviewed-by: Phil Yang Reviewed-by: Steve Capper Reviewed-by: Ola Liljedahl Reviewed-by: Honnappa Nagarahalli Tested-by: Pavan Nikhilesh --- lib/librte_eal/include/generic/rte_spinlock.h | 5 ++--- 1 file changed, 2 insertions

[dpdk-dev] [PATCH v1 0/2] Use WFE for spinlock and ring

2020-04-24 Thread Gavin Hu
The rte_wait_until_equal_xxx APIs abstract the functionality of 'polling for a memory location to become equal to a given value'[1]. Use the API for the rte spinlock and ring implementations. [1] http://patches.dpdk.org/cover/62703/ Gavin Hu (2): spinlock: use wfe to reduce con

[dpdk-dev] [PATCH v1 2/2] ring: use wfe to wait for ring tail update on aarch64

2020-04-24 Thread Gavin Hu
Instead of polling for tail to be updated, use wfe instruction. Signed-off-by: Gavin Hu Reviewed-by: Ruifeng Wang Reviewed-by: Steve Capper Reviewed-by: Ola Liljedahl Reviewed-by: Honnappa Nagarahalli --- lib/librte_ring/rte_ring_c11_mem.h | 4 ++-- lib/librte_ring/rte_ring_generic.h | 3

Re: [dpdk-dev] [PATCH v2 2/2] net/i40e: restrict pointer aliasing for NEON vPMD

2020-04-21 Thread Gavin Hu
Hi Ferruh, > -Original Message- > From: Ferruh Yigit > Sent: Monday, April 20, 2020 10:51 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; jer...@marvell.com; xiaolong...@intel.com; > Honnappa Nagarahalli ; Ruifeng Wang

[dpdk-dev] [patch v1] net/i40e: fix the compile issue

2020-04-21 Thread Gavin Hu
35:35: error: expected ';', ',' or ')' before 'rx_queue' i40e_recv_pkts_vec(void *restrict rx_queue, struct rte_mbuf **restrict rx_pkts, Fix this by using '__restrict' type qualifier, which compiles ok with old and new versions of gcc. Fixes:

Re: [dpdk-dev] [PATCH RFC v1 0/7] relax barriers for ENA PMD and small fixes

2020-04-21 Thread Gavin Hu
> -Original Message- > From: Chauskin, Igor > Sent: Thursday, April 16, 2020 9:38 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; m...@semihalf.com; Tzalik, Guy > ; Schmeilin, Evgeny ; > m...@semihalf.com; Honnappa Naga

Re: [dpdk-dev] [PATCH RFC v2 0/7] introduce new barrier class and use it for mlx5 PMD

2020-04-13 Thread Gavin Hu
Hi Andrew, > -Original Message- > From: Andrew Rybchenko > Sent: Monday, April 13, 2020 5:52 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; rasl...@mellanox.com; d...@linux.vnet.ibm.com; > bruce.richard...@intel

[dpdk-dev] [PATCH v3 1/2] net/i40e: relax barrier in Tx fastpath for NEON vPMD

2020-04-13 Thread Gavin Hu
case is the RFC2544 zero-loss test running testpmd. [1] http://inbox.dpdk.org/dev/CALBAE1M-ezVWCjqCZDBw+MMDEC4O9 qf0kpn89emdgdajepk...@mail.gmail.com Fixes: ae0eb310f253 ("net/i40e: implement vector PMD for ARM") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Acked-by: Jeri

[dpdk-dev] [PATCH v3 0/2] i40e NEON vPMD optimization on aarch64

2020-04-13 Thread Gavin Hu
command on Marvell ThunderX2 and Arm N1SDP and showed positive performance results. sudo ./build/app/testpmd -l 1,3 -w 0001:01:00.0 -w 0001:01:00.1 --master-lcore 1 -- -i --rxq=4 --txq=4 --nb-cores=1 --nb-ports=2 -a Gavin Hu (2): net/i40e: relax barrier in Tx fastpath for NEON vPMD net/i40e

[dpdk-dev] [PATCH v3 2/2] net/i40e: restrict pointer aliasing for NEON vPMD

2020-04-13 Thread Gavin Hu
Signed-off-by: Gavin Hu Reviewed-by: Steve Capper --- drivers/net/i40e/i40e_rxtx_vec_neon.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/net/i40e/i40e_rxtx_vec_neon.c b/drivers/net/i40e/i40e_rxtx_vec_neon.c index 4376d8911..405bd82df 100644 --- a

[dpdk-dev] [PATCH v2 1/2] net/i40e: relax barrier in Tx fastpath for NEON vPMD

2020-04-13 Thread Gavin Hu
case is the RFC2544 zero-loss test running testpmd. [1] http://inbox.dpdk.org/dev/CALBAE1M-ezVWCjqCZDBw+MMDEC4O9 qf0kpn89emdgdajepk...@mail.gmail.com Fixes: 4861cde46116 ("i40e: new poll mode driver") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu --- drivers/net/i40e/i40e_rxtx_vec_neo

[dpdk-dev] [PATCH v2 2/2] net/i40e: restrict pointer aliasing for NEON vPMD

2020-04-13 Thread Gavin Hu
Signed-off-by: Gavin Hu Reviewed-by: Steve Capper --- drivers/net/i40e/i40e_rxtx_vec_neon.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/net/i40e/i40e_rxtx_vec_neon.c b/drivers/net/i40e/i40e_rxtx_vec_neon.c index 4376d8911..405bd82df 100644 --- a

[dpdk-dev] [PATCH v2 0/2] i40e NEON vPMD optimization on aarch64

2020-04-13 Thread Gavin Hu
showed positive performance results. sudo ./build/app/testpmd -l 1,3 -w 0001:01:00.0 -w 0001:01:00.1 --master-lcore 1 -- -i --rxq=4 --txq=4 --nb-cores=1 --nb-ports=2 -a Gavin Hu (2): net/i40e: relax barrier in Tx fastpath for NEON vPMD net/i40e: restrict pointer aliasing for NEON vPMD

Re: [dpdk-dev] [PATCH RFC v2 0/7] introduce new barrier class and use it for mlx5 PMD

2020-04-10 Thread Gavin Hu
Hi Andrew, > -Original Message- > From: Andrew Rybchenko > Sent: Saturday, April 11, 2020 1:21 AM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; rasl...@mellanox.com; d...@linux.vnet.ibm.com; > bruce.richard...@intel

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-04-10 Thread Gavin Hu
> -Original Message- > From: Thomas Monjalon > Sent: Thursday, April 9, 2020 6:50 PM > To: Gavin Hu > Cc: David Marchand ; Kevin Traynor > ; Bruce Richardson ; > Morten Brørup ; Ferruh Yigit > ; dev@dpdk.org; nd ; > jer...@marvell.com; Honnappa Nagarahall

[dpdk-dev] [PATCH RFC v2 7/7] doc: clarify one configuration in mlx5 guide

2020-04-10 Thread Gavin Hu
s to these two memory regions. Interpreting it to 'non-cacheable' makes no distinction and is confusing. re-interprete it to 'non-combining' can make the distinction. Also explains why aarch64 default setting for this is different. Signed-off-by: Gavin Hu --- doc/guides/ni

[dpdk-dev] [PATCH RFC v2 6/7] net/mlx5: relax ordering for multi-packet RQ buffer refcnt

2020-04-10 Thread Gavin Hu
with one-way barrier to improve performance. Fixes: 7d6bf6b866b8 ("net/mlx5: add Multi-Packet Rx support") Cc: sta...@dpdk.org Suggested-by: Gavin Hu Signed-off-by: Phil Yang Reviewed-by: Gavin Hu --- drivers/net/mlx5/mlx5_rxq.c | 2 +- drivers/net/mlx5/mlx5_r

[dpdk-dev] [PATCH RFC v2 5/7] net/mlx5: add descriptive comment for a barrier

2020-04-10 Thread Gavin Hu
-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_rxtx.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/mlx5/mlx5_rxtx.h b/drivers/net/mlx5/mlx5_rxtx.h index 228e37de5..737d5716d 100644 --- a/drivers/net/mlx5/mlx5_rxtx.h +++ b/drivers/net/mlx5/mlx5_rxtx.h

[dpdk-dev] [PATCH RFC v2 3/7] net/mlx5: relax barrier to order UAR writes on aarch64

2020-04-10 Thread Gavin Hu
and ppc. Fixes: 6bf10ab69be0 ("net/mlx5: support 32-bit systems") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_rxtx.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.h b/drivers/net/mlx5/mlx5_

[dpdk-dev] [PATCH RFC v2 0/7] introduce new barrier class and use it for mlx5 PMD

2020-04-10 Thread Gavin Hu
between Arm and Ampere, 8%~13% performance boost was measured. As there is no functionality changes, it will not impact x86. Gavin Hu (6): eal: introduce new class of barriers for DMA use cases net/mlx5: dmb for immediate doorbell ring on aarch64 net/mlx5: relax barrier to order UAR writes on aar

[dpdk-dev] [PATCH RFC v2 4/7] net/mlx5: relax barrier for aarch64

2020-04-10 Thread Gavin Hu
ned-off-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_rxtx.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.h b/drivers/net/mlx5/mlx5_rxtx.h index da5d81350..228e37de5 100644 --- a/drivers/net/mlx5/mlx5_rxtx.h +++ b/driver

[dpdk-dev] [PATCH RFC v2 2/7] net/mlx5: dmb for immediate doorbell ring on aarch64

2020-04-10 Thread Gavin Hu
A 'DMB' is enough to evict the merge buffer on aarch64,when the doorbell register is mapped as 'Normal-NC', the counterpart of WC on x86. Otherwise, it is mapped as Device memory, no barriers required at all. Signed-off-by: Gavin Hu --- drivers/net/mlx5/mlx5_rxtx.h | 2 +

[dpdk-dev] [PATCH RFC v2 1/7] eal: introduce new class of barriers for DMA use cases

2020-04-10 Thread Gavin Hu
/kernel/git/torvalds/linux.git/ commit/?id=22ec71615d824f4f11d38d0e55a88d8956b7e45f Signed-off-by: Gavin Hu Reviewed-by: Steve Capper --- lib/librte_eal/arm/include/rte_atomic_32.h | 6 lib/librte_eal/arm/include/rte_atomic_64.h | 6 lib/librte_eal/include/gener

Re: [dpdk-dev] [dpdk-stable] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-04-09 Thread Gavin Hu
Hi David, > -Original Message- > From: David Marchand > Sent: Wednesday, April 8, 2020 11:22 PM > To: Gavin Hu > Cc: Kevin Traynor ; Bruce Richardson > ; Morten Brørup > ; Ferruh Yigit ; > dev@dpdk.org; nd ; tho...@monjalon.net; > jer...@marvell.com; Honna

Re: [dpdk-dev] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-04-08 Thread Gavin Hu
Hi Kevin, > -Original Message- > From: Kevin Traynor > Sent: Wednesday, April 8, 2020 1:14 AM > To: Gavin Hu ; Bruce Richardson > ; Morten Brørup > > Cc: Ferruh Yigit ; dev@dpdk.org; nd > ; david.march...@redhat.com; tho...@monjalon.net; > jer...@marvel

Re: [dpdk-dev] [PATCH v2 1/2] virtio: one way barrier for split vring used idx

2020-04-03 Thread Gavin Hu
Honnappa Nagarahalli > ; Gavin Hu ; nd > ; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 1/2] virtio: one way barrier for split vring > used idx > > On Thu, 2 Apr 2020 10:57:52 +0800 > Joyce Kong wrote: > > > - (

Re: [dpdk-dev] [PATCH v7 1/6] lib/eal: implement the family of PMD bit operation APIs

2020-04-02 Thread Gavin Hu
> -Original Message- > From: Jerin Jacob > Sent: Thursday, April 2, 2020 4:12 PM > To: tho...@monjalon.net > Cc: Joyce Kong ; Gavin Hu ; > step...@networkplumber.org; david.march...@redhat.com; > m...@smartsharesystems.com; jer...@marvell.com; > bruce.richar

Re: [dpdk-dev] [PATCH v7 1/6] lib/eal: implement the family of PMD bit operation APIs

2020-04-02 Thread Gavin Hu
Hi Thomas, > -Original Message- > From: Thomas Monjalon > Sent: Wednesday, April 1, 2020 5:45 PM > To: Joyce Kong ; Gavin Hu > Cc: step...@networkplumber.org; david.march...@redhat.com; > m...@smartsharesystems.com; jer...@marvell.com; > bruce.richard...@intel.com

Re: [dpdk-dev] [PATCH v2 2/2] vhost: cache gpa to hpa translation

2020-04-01 Thread Gavin Hu
Hi Marvin, > -Original Message- > From: Liu, Yong > Sent: Wednesday, April 1, 2020 9:01 PM > To: Gavin Hu ; maxime.coque...@redhat.com; Ye, > Xiaolong ; Wang, Zhihong > > Cc: dev@dpdk.org; nd > Subject: RE: [dpdk-dev] [PATCH v2 2/2] vhost: cach

Re: [dpdk-dev] [PATCH v2 1/2] vhost: utilize dpdk dynamic memory allocator

2020-04-01 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH v2 2/2] vhost: cache gpa to hpa translation

2020-04-01 Thread Gavin Hu
Hi Marvin, > -Original Message- > From: dev On Behalf Of Marvin Liu > Sent: Wednesday, April 1, 2020 10:50 PM > To: maxime.coque...@redhat.com; xiaolong...@intel.com; > zhihong.w...@intel.com > Cc: dev@dpdk.org; Marvin Liu > Subject: [dpdk-dev] [PATCH v2 2/2] vhost: cache gpa to hpa tran

Re: [dpdk-dev] [PATCH v7 1/6] lib/eal: implement the family of PMD bit operation APIs

2020-04-01 Thread Gavin Hu
vi1.ku...@amd.com; rm...@marvell.com; > shsha...@marvell.com; xuanziya...@huawei.com; > cloud.wangxiao...@huawei.com; zhouguoy...@huawei.com; Honnappa > Nagarahalli ; Phil Yang > ; Gavin Hu ; nd > ; dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v7 1/6] lib/eal: implement the family of PMD &

Re: [dpdk-dev] [PATCH v2 4/8] eal: move arch-specific header files

2020-03-26 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH RFC v1 0/7] relax barriers for ENA PMD and small fixes

2020-03-17 Thread Gavin Hu
Hi Igor, > -Original Message- > From: Chauskin, Igor > Sent: Monday, March 16, 2020 5:35 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; tho...@monjalon.net; > m...@semihalf.com; Tzalik, Guy ; Schmeilin, Evgeny > ; m...@semihalf.com;

Re: [dpdk-dev] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-03-13 Thread Gavin Hu
Hi Bruce, > -Original Message- > From: Bruce Richardson > Sent: Wednesday, March 11, 2020 8:08 PM > To: Morten Brørup > Cc: Gavin Hu ; Ferruh Yigit ; > dev@dpdk.org; nd ; david.march...@redhat.com; > tho...@monjalon.net; ktray...@redhat.com; jer...@marvell.com; &

[dpdk-dev] [PATCH RFC v1 7/7] net/ena: remove duplicate memset

2020-03-13 Thread Gavin Hu
There is a memset operation for the whole structure so no need to memset the sub fields in it. Fixes: 1173fca25af9 ("ena: add polling-mode driver") Cc: sta...@dpdk.org Suggested-by: Honnappa Nagarahalli Signed-off-by: Gavin Hu Reviewed-by: Honnappa Nagarahalli --- drive

[dpdk-dev] [PATCH RFC v1 1/7] net/ena: remove duplicate barrier

2020-03-13 Thread Gavin Hu
As there is necessary barriers before calling the function everywhere, then using the relaxed write function ENA_REG_WRITE32_RELAXED is ok. Fixes: 99ecfbf845b3 ("ena: import communication layer") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Honnappa Nagarahalli --- drive

[dpdk-dev] [PATCH RFC v1 6/7] net/ena: use c11 atomic for statistics

2020-03-13 Thread Gavin Hu
The statistics logging are in the data path, as rte_atomic APIs enforce unnecessary barriers on aarch64, this patch uses c11 atomic APIs with the __ATOMIC_RELAXED memory ordering to give CPU more freedom to optimize by making use of out-of-order execution. Signed-off-by: Gavin Hu Reviewed-by

[dpdk-dev] [PATCH RFC v1 3/7] net/ena: relax the rmb for DMA

2020-03-13 Thread Gavin Hu
#n48 [4] http://code.dpdk.org/dpdk/v20.02/source/drivers/net/ena/ ena_ethdev.c#L2021 Fixes: b68309be44c0 ("net/ena/base: update communication layer for the ENAv2") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Honnappa Nagarahalli --- drivers/net/ena/base/ena_plat_dpdk.h | 2

[dpdk-dev] [PATCH RFC v1 2/7] net/ena: relax the barrier for doorbell ring

2020-03-13 Thread Gavin Hu
t/ena: fix unneeded doorbell submission") Cc: sta...@dpdk.org [1] http://code.dpdk.org/dpdk/v20.02/source/lib/librte_eal/common/ include/generic/rte_atomic.h#L137 Signed-off-by: Gavin Hu Reviewed-by: Honnappa Nagarahalli --- drivers/net/ena/ena_ethdev.c | 6 +++--- 1 file changed, 3 insert

[dpdk-dev] [PATCH RFC v1 5/7] net/ena: relax the barrier for bounce buffer

2020-03-13 Thread Gavin Hu
To gaurantee the update observed by NIC HW, a cio barrier is sufficient, a io barrier, which translates to dsb on aarch64, is overkill. Suggested-by: Honnappa Nagarahalli Signed-off-by: Gavin Hu Reviewed-by: Honnappa Nagarahalli --- drivers/net/ena/base/ena_eth_com.c | 2 +- 1 file changed, 1

[dpdk-dev] [PATCH RFC v1 0/7] relax barriers for ENA PMD and small fixes

2020-03-13 Thread Gavin Hu
and the other patch removing duplicate memset. Note this set is submitted for RFC as we don't have physical ENA NICs in the lab and the patch set was not verified nor benchmarked. Gavin Hu (7): net/ena: remove duplicate barrier net/ena: relax the barrier for doorbell ring net/ena: relax

[dpdk-dev] [PATCH RFC v1 4/7] net/ena: relax barrier for completion queue update

2020-03-13 Thread Gavin Hu
To gaurantee the update observed by NIC HW, a cio barrier is sufficient, an io barrier, which translates to dsb on aarch64, is overkill. Suggested-by: Honnappa Nagarahalli Signed-off-by: Gavin Hu Reviewed-by: Honnappa Nagarahalli --- drivers/net/ena/base/ena_eth_com.h | 4 +++- 1 file changed

Re: [dpdk-dev] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-03-13 Thread Gavin Hu
Hi Bruce, > -Original Message- > From: Bruce Richardson > Sent: Wednesday, March 11, 2020 8:08 PM > To: Morten Brørup > Cc: Gavin Hu ; Ferruh Yigit ; > dev@dpdk.org; nd ; david.march...@redhat.com; > tho...@monjalon.net; ktray...@redhat.com; jer...@marvell.com; &

Re: [dpdk-dev] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-03-11 Thread Gavin Hu
Hi Morten, > -Original Message- > From: Morten Brørup > Sent: Monday, March 9, 2020 9:31 PM > To: Ferruh Yigit ; Gavin Hu ; > dev@dpdk.org > Cc: nd ; david.march...@redhat.com; > tho...@monjalon.net; ktray...@redhat.com; jer...@marvell.com; > Honnappa Nagarahalli

Re: [dpdk-dev] [PATCH v2] cycles: add isb before read cntvct_el0

2020-03-10 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH] cycles: add isb before read cntvct_el0

2020-03-10 Thread Gavin Hu
Hi Haifeng, > -Original Message- > From: dev On Behalf Of Linhaifeng > Sent: Monday, March 9, 2020 5:23 PM > To: dev@dpdk.org; tho...@monjalon.net > Cc: chenchanghu ; xudingke > ; Lilijun (Jerry) > Subject: [dpdk-dev] [PATCH] cycles: add isb before read cntvct_el0 > > We should use isb

Re: [dpdk-dev] [PATCH] eal/arm64: remove internal macros

2020-03-09 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-03-09 Thread Gavin Hu
Hi Ferruh, > -Original Message- > From: Ferruh Yigit > Sent: Monday, March 9, 2020 4:55 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; david.march...@redhat.com; tho...@monjalon.net; > ktray...@redhat.com; jer...@marvell.com; Honnappa Nagarahalli > ; Ruifeng Wang >

Re: [dpdk-dev] [PATCH v1 3/3] net/i40e: auto-vectorization to speed up Tx free

2020-03-09 Thread Gavin Hu
Hi Jerin, > -Original Message- > From: Jerin Jacob > Sent: Monday, March 9, 2020 3:36 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; jer...@marvell.com; > Ye, Xiaolong ; Honnappa Nagarahalli > ; Ruifeng Wang > ; Phil Yang ;

[dpdk-dev] [PATCH v2] mbuf: replace zero-length marker with unnamed union

2020-03-07 Thread Gavin Hu
://bugs.dpdk.org/show_bug.cgi?id=396 Bugzilla ID: 396 [1] https://gcc.gnu.org/onlinedocs/gcc/Zero-Length.html Fixes: 3e6181b07038 ("mbuf: use structure marker from EAL") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu --- v2: * change 'uint64_t rearm_data' to 'uint_64_t rearm_

Re: [dpdk-dev] [PATCH v1 3/3] net/i40e: auto-vectorization to speed up Tx free

2020-03-07 Thread Gavin Hu
Hi Jerin, > -Original Message- > From: Jerin Jacob > Sent: Friday, March 6, 2020 3:45 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; > jer...@marvell.com; Ye, Xiaolong ; Honnappa > Nagarahalli ; Ruifeng Wang > ; Phil Yang ;

Re: [dpdk-dev] [PATCH v1] mbuf: replace zero-length marker with unnamed union

2020-03-07 Thread Gavin Hu
Hi Kevin, > -Original Message- > From: Kevin Traynor > Sent: Wednesday, March 4, 2020 8:33 PM > To: Gavin Hu ; dev@dpdk.org > Cc: nd ; tho...@monjalon.net; > david.march...@redhat.com; jer...@marvell.com; Honnappa Nagarahalli > ; Ruifeng Wang > ; Phil Yang ; >

[dpdk-dev] [PATCH v1 2/3] net/i40e: restrict pointer aliasing for neon vec

2020-03-05 Thread Gavin Hu
Signed-off-by: Gavin Hu Reviewed-by: Steve Capper --- drivers/net/i40e/i40e_rxtx_vec_neon.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/net/i40e/i40e_rxtx_vec_neon.c b/drivers/net/i40e/i40e_rxtx_vec_neon.c index 4376d8911..405bd82df 100644 --- a

[dpdk-dev] [PATCH v1 3/3] net/i40e: auto-vectorization to speed up Tx free

2020-03-05 Thread Gavin Hu
: Gavin Hu Reviewed-by: Steve Capper --- drivers/net/i40e/i40e_rxtx_vec_common.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/i40e/i40e_rxtx_vec_common.h b/drivers/net/i40e/i40e_rxtx_vec_common.h index 0e6ffa007..fc0fa45d4 100644 --- a/drivers/net/i40e/i40e_rxtx_vec_common.h

[dpdk-dev] [PATCH v1 0/3] i40e vPMD optimization on aarch64

2020-03-05 Thread Gavin Hu
=4 --txq=4 --nb-cores=1 --nb-ports=2 -a Gavin Hu (3): net/i40e: relax barrier in the Tx fastpath of vPMD net/i40e: restrict pointer aliasing for neon vec net/i40e: auto-vectorization to speed up Tx free drivers/net/i40e/i40e_rxtx_vec_common.h | 5 + drivers/net/i40e/i40e_rxtx_vec_neon.c

[dpdk-dev] [PATCH v1 1/3] net/i40e: relax barrier in the Tx fastpath of vPMD

2020-03-05 Thread Gavin Hu
the RFC2544 zero-loss test running testpmd. [1] http://inbox.dpdk.org/dev/CALBAE1M-ezVWCjqCZDBw+MMDEC4O9 qf0kpn89emdgdajepk...@mail.gmail.com Fixes: 4861cde46116 ("i40e: new poll mode driver") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu --- drivers/net/i40e/i40e_rxtx_vec_neon.c | 6

[dpdk-dev] [PATCH v1] mbuf: replace zero-length marker with unnamed union

2020-03-03 Thread Gavin Hu
96 [1] https://gcc.gnu.org/onlinedocs/gcc/Zero-Length.html Fixes: 3e6181b07038 ("mbuf: use structure marker from EAL") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu --- lib/librte_mbuf/rte_mbuf_core.h | 54 +++-- 1 file changed, 32 insertions(+), 22 delet

Re: [dpdk-dev] [PATCH] net/mlx5: add BlueField-2 device ID

2020-02-16 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH] event/octeontx2: remove WFE from dualslot dequeue

2020-02-15 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH] event/octeontx2: remove WFE from dualslot dequeue

2020-02-14 Thread Gavin Hu
Hi Pavan, > -Original Message- > From: pbhagavat...@marvell.com > Sent: Friday, February 14, 2020 2:45 PM > To: jer...@marvell.com; Pavan Nikhilesh > Cc: Gavin Hu ; dev@dpdk.org > Subject: [dpdk-dev] [PATCH] event/octeontx2: remove WFE from dualslot > dequeue >

Re: [dpdk-dev] [PATCH v4] net/i40e: relaxed barrier in the tx fastpath

2020-02-14 Thread Gavin Hu
Hi Jerin, Could you help review this patch? We discussed a lot on this topic and I think we agreed already to relax the barrier only on the fast path. Best Regards, Gavin > -Original Message- > From: Gavin Hu > Sent: Friday, February 14, 2020 3:58 PM > To: Gavin Hu &

[dpdk-dev] [PATCH RFC v1 5/6] net/mlx5: non-cacheable mapping defaulted for aarch64

2020-02-13 Thread Gavin Hu
fix Tx doorbell write memory barrier") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_txq.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c index bc13abfe6..144bab4a6 100644 --

[dpdk-dev] [PATCH RFC v1 6/6] net/mlx5: relaxed ordering for multi-packet RQ buffer refcnt

2020-02-13 Thread Gavin Hu
with one-way barrier to improve performance. Fixes: 7d6bf6b866b8 ("net/mlx5: add Multi-Packet Rx support") Cc: sta...@dpdk.org Suggested-by: Gavin Hu Signed-off-by: Phil Yang Reviewed-by: Gavin Hu --- drivers/net/mlx5/mlx5_rxq.c | 2 +- drivers/net/mlx5/mlx5_r

[dpdk-dev] [PATCH RFC v1 3/6] net/mlx5: add missing barrier

2020-02-13 Thread Gavin Hu
quot;net/mlx5: support 32-bit systems") Cc: sta...@dpdk.org Suggested-by: Phil Yang Signed-off-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_rxq.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx

[dpdk-dev] [PATCH RFC v1 4/6] net/mlx5: add descriptive comment for a barrier

2020-02-13 Thread Gavin Hu
-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_rxtx.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/mlx5/mlx5_rxtx.h b/drivers/net/mlx5/mlx5_rxtx.h index c672af4c4..d32c4e430 100644 --- a/drivers/net/mlx5/mlx5_rxtx.h +++ b/drivers/net/mlx5/mlx5_rxtx.h

[dpdk-dev] [PATCH RFC v1 2/6] net/mlx5: use cio barrier before the BF WQE

2020-02-13 Thread Gavin Hu
To ensure the WQE and doorbell record, which reside in the host memory, are visible to HW before the blue frame, a CIO barrier is sufficient, a rte_wmb is overkill. Fixes: 6cb559d67b83 ("net/mlx5: add vectorized Rx/Tx burst for x86") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewe

[dpdk-dev] [PATCH RFC v1 1/6] net/mlx5: relax the barrier for UAR write

2020-02-13 Thread Gavin Hu
mapped as nGnRE device memory, not cachable nor write-combine. 2. io accesses to a single device is total ordered. Fixes: 6bf10ab69be0 ("net/mlx5: support 32-bit systems") Cc: sta...@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Phil Yang --- drivers/net/mlx5/mlx5_rxtx.h | 2 +- 1 fi

[dpdk-dev] [PATCH RFC v1 0/6] barrier fix and optimization for mlx5 on aarch64

2020-02-13 Thread Gavin Hu
Using just sufficient barriers really matters to performance. Insufficient barriers will cause issues while barriers stronger than required, especially in the fast path is a performance killer. In the joint preliminary testing between Arm and Ampere, 8%~13% performance was measured. Gavin Hu (5

Re: [dpdk-dev] [PATCH v3] net/i40e: relaxed barrier in the tx fastpath

2020-02-11 Thread Gavin Hu
Hi Xiaolong, Just sent v4 for this, thanks for reminding! Best Regards, Gavin > -Original Message- > From: Ye Xiaolong > Sent: Tuesday, February 11, 2020 10:11 AM > To: Gavin Hu > Cc: dev@dpdk.org; nd ; david.march...@redhat.com; > tho...@monjalon.net; ra

[dpdk-dev] [PATCH v4] net/i40e: relaxed barrier in the tx fastpath

2020-02-11 Thread Gavin Hu
.@dpdk.org Signed-off-by: Gavin Hu --- V4: - add the Fixes tag and CC stable V3: - optimize the barriers in the fast path only, leave as it is for the barriers in the slow path and control path - drop the virtio patches from the list as they are in the control path - it makes more sense to relax t

[dpdk-dev] [PATCH v3] net/i40e: relaxed barrier in the tx fastpath

2020-02-08 Thread Gavin Hu
To keep ordering of mixed accesses, rte_cio is sufficient. The rte_io barrier is overkill.[1] [1] http://inbox.dpdk.org/dev/CALBAE1M-ezVWCjqCZDBw+MMDEC4O9 qf0kpn89emdgdajepk...@mail.gmail.com Signed-off-by: Gavin Hu --- V3: - optimize the barriers in the fast path only, leave as it is for the

Re: [dpdk-dev] [PATCH v13 2/5] eal: add the APIs to wait until equal

2020-01-16 Thread Gavin Hu
> -Original Message- > From: David Marchand > Sent: Friday, January 17, 2020 5:38 AM > To: Honnappa Nagarahalli > Cc: Gavin Hu ; Jerin Jacob ; > Ananyev, Konstantin ; > tho...@monjalon.net; dev@dpdk.org; Mcnamara, John > ; Kovacevic, Marko > ; jer...@marve

Re: [dpdk-dev] [PATCH v4 0/6] OCTEON TX2 End Point Driver

2020-01-08 Thread Gavin Hu
Hi Mahipal, Please cc me in your future optimization patches. Series-reviewed-by: Gavin Hu > -Original Message- > From: Mahipal Challa > Sent: Wednesday, January 8, 2020 3:32 PM > To: dev@dpdk.org; tho...@monjalon.net; jer...@marvell.com; Gavin Hu > > Cc:

Re: [dpdk-dev] [PATCH v3 2/6] raw/octeontx2_ep: add device configuration

2020-01-06 Thread Gavin Hu
Hi Mahipal, > -Original Message- > From: Mahipal Challa > Sent: Monday, January 6, 2020 8:07 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com; Gavin Hu > Subject: [dpdk-dev] [PATCH v3 2/6] raw/oct

Re: [dpdk-dev] [PATCH v3 4/6] raw/octeontx2_ep: add enqueue operation

2020-01-06 Thread Gavin Hu
Hi Mahipal, > -Original Message- > From: Mahipal Challa > Sent: Monday, January 6, 2020 8:07 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com; Gavin Hu > Subject: [dpdk-dev] [PATCH v3 4/6] raw/octe

Re: [dpdk-dev] [PATCH v3 5/6] raw/octeontx2_ep: add dequeue operation

2020-01-06 Thread Gavin Hu
Hi Mahipal, Jerin, > -Original Message- > From: Mahipal Challa > Sent: Monday, January 6, 2020 8:07 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com; Gavin Hu > Subject: [dpdk-dev] [PATCH v3 5/6]

Re: [dpdk-dev] [PATCH v3 6/6] raw/octeontx2_ep: add driver self test

2020-01-06 Thread Gavin Hu
Hi Mahipal, > -Original Message- > From: Mahipal Challa > Sent: Monday, January 6, 2020 8:07 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com; Gavin Hu > Subject: [dpdk-dev] [PATCH v3 6/6] raw/octeontx

Re: [dpdk-dev] [PATCH v2 1/3] eal/arm64: relax the io barrier for aarch64

2020-01-03 Thread Gavin Hu
Hi Jerin, > -Original Message- > From: Jerin Jacob > Sent: Friday, January 3, 2020 3:35 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; > rasl...@mellanox.com; maxime.coque...@redhat.com; tiwei@intel.com; > hemant.agra...@nxp

Re: [dpdk-dev] [PATCH v2 1/3] eal/arm64: relax the io barrier for aarch64

2020-01-02 Thread Gavin Hu
Hi Jerin, > -Original Message- > From: Jerin Jacob > Sent: Thursday, January 2, 2020 5:52 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; > rasl...@mellanox.com; maxime.coque...@redhat.com; tiwei@intel.com; > hemant.agra...@nxp

Re: [dpdk-dev] [dpdk-dev PATCH v2 4/6] raw/octeontx2_ep: add enqueue operation

2020-01-02 Thread Gavin Hu
Hi Mahipal, > -Original Message- > From: dev On Behalf Of Mahipal Challa > Sent: Friday, December 27, 2019 8:40 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com > Subject: [dpdk-dev] [dpdk-dev PATCH v2 4/6] raw/octeontx2_ep: a

Re: [dpdk-dev] [dpdk-dev PATCH v2 6/6] raw/octeontx2_ep: add driver self test

2020-01-02 Thread Gavin Hu
Hi Mahipal, > -Original Message- > From: dev On Behalf Of Mahipal Challa > Sent: Friday, December 27, 2019 8:40 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com > Subject: [dpdk-dev] [dpdk-dev PATCH v2 6/6] raw/octeontx2_ep: a

Re: [dpdk-dev] [dpdk-dev PATCH v2 2/6] raw/octeontx2_ep: add device configuration

2020-01-02 Thread Gavin Hu
Hi Mahipal, 3 comments inline. > -Original Message- > From: dev On Behalf Of Mahipal Challa > Sent: Friday, December 27, 2019 8:40 PM > To: dev@dpdk.org > Cc: jer...@marvell.com; pathr...@marvell.com; sni...@marvell.com; > venk...@marvell.com > Subject: [dpdk-dev] [dpdk-dev PATCH v2 2/6]

Re: [dpdk-dev] [PATCH v2 0/6] OCTEON TX2 End Point Driver

2020-01-02 Thread Gavin Hu
Hi Mahipal, Could you make old versions 'Superseded' when you have new ones? /Gavin

Re: [dpdk-dev] [PATCH v6 1/6] lib/eal: implement the family of rte bit operation APIs

2019-12-29 Thread Gavin Hu
.richard...@intel.com; ravi1.ku...@amd.com; > rm...@marvell.com; shsha...@marvell.com; xuanziya...@huawei.com; > cloud.wangxiao...@huawei.com; zhouguoy...@huawei.com; Phil Yang > ; Gavin Hu ; nd ; > dev@dpdk.org > Subject: Re: [PATCH v6 1/6] lib/eal: implement the family of rte bit

Re: [dpdk-dev] [PATCH] raw/ntb: fix write memory barrier issue

2019-12-24 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH v2] raw/ntb: fix write memory barrier issue

2019-12-24 Thread Gavin Hu
Hi Xiaoyun, > -Original Message- > From: Li, Xiaoyun > Sent: Monday, December 23, 2019 5:36 PM > To: Gavin Hu ; Wu, Jingjing > Cc: dev@dpdk.org; Maslekar, Omkar ; > sta...@dpdk.org; nd ; jer...@marvell.com; Honnappa > Nagarahalli ; Richardson, Bruce > ; nd

Re: [dpdk-dev] [PATCH v2 1/3] eal/arm64: relax the io barrier for aarch64

2019-12-23 Thread Gavin Hu
Hi Jerin, > -Original Message- > From: Jerin Jacob > Sent: Monday, December 23, 2019 5:20 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; > rasl...@mellanox.com; maxime.coque...@redhat.com; > tiwei@intel.com; hemant.agra...@nxp

Re: [dpdk-dev] [PATCH v2 1/3] eal/arm64: relax the io barrier for aarch64

2019-12-23 Thread Gavin Hu
2019 2:56 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; > rasl...@mellanox.com; maxime.coque...@redhat.com; > tiwei@intel.com; hemant.agra...@nxp.com; jer...@marvell.com; > Pavan Nikhilesh ; Honnappa Nagarahalli > ; Ruifeng Wang > ;

Re: [dpdk-dev] [PATCH v2] raw/ntb: fix write memory barrier issue

2019-12-23 Thread Gavin Hu
Hi Xiaoyun, > -Original Message- > From: Li, Xiaoyun > Sent: Monday, December 23, 2019 3:52 PM > To: Gavin Hu ; Wu, Jingjing > Cc: dev@dpdk.org; Maslekar, Omkar ; > sta...@dpdk.org; nd > Subject: RE: [dpdk-dev] [PATCH v2] raw/ntb: fix write memory barrier issue >

Re: [dpdk-dev] [PATCH v3] net/virtio: packed ring notification data feature support

2019-12-22 Thread Gavin Hu
Reviewed-by: Gavin Hu

Re: [dpdk-dev] [PATCH v2 2/3] net/virtio: virtual PCI requires smp barriers

2019-12-20 Thread Gavin Hu
Hi Tiwei, Thanks for review. > -Original Message- > From: Tiwei Bie > Sent: Friday, December 20, 2019 4:18 PM > To: Gavin Hu > Cc: dev@dpdk.org; nd ; david.march...@redhat.com; > tho...@monjalon.net; rasl...@mellanox.com; > maxime.coque...@redhat.com; hemant.a

Re: [dpdk-dev] [PATCH v2 1/3] eal/arm64: relax the io barrier for aarch64

2019-12-19 Thread Gavin Hu
Hi Jerin, > -Original Message- > From: Jerin Jacob > Sent: Friday, December 20, 2019 12:34 PM > To: Gavin Hu > Cc: dpdk-dev ; nd ; David Marchand > ; tho...@monjalon.net; > rasl...@mellanox.com; maxime.coque...@redhat.com; > tiwei@intel.com; hem

Re: [dpdk-dev] [PATCH v2 1/3] eal: introduce structure marker typedefs

2019-12-19 Thread Gavin Hu
Series-reviewed-by: Gavin Hu

  1   2   3   4   5   6   7   >