Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-23 Thread Ferruh Yigit
On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: > Cache align Rx and Tx queue struct to avoid false sharing. > > RX struct happens to be 64 bytes on x86_64 already, so cache alignment > makes no change there, but it does on 32-bit ISAs. > > TX struct is 56 bytes on x86_64. > Hi Mattias, No obje

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-23 Thread Mattias Rönnblom
On 2024-04-23 13:15, Ferruh Yigit wrote: On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: Cache align Rx and Tx queue struct to avoid false sharing. RX struct happens to be 64 bytes on x86_64 already, so cache alignment makes no change there, but it does on 32-bit ISAs. TX struct is 56 bytes on

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-23 Thread Honnappa Nagarahalli
> On Apr 23, 2024, at 3:56 PM, Mattias Rönnblom wrote: > > On 2024-04-23 13:15, Ferruh Yigit wrote: >> On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: >>> Cache align Rx and Tx queue struct to avoid false sharing. >>> >>> RX struct happens to be 64 bytes on x86_64 already, so cache alignment >>

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-23 Thread Mattias Rönnblom
On 2024-04-24 02:27, Honnappa Nagarahalli wrote: On Apr 23, 2024, at 3:56 PM, Mattias Rönnblom wrote: On 2024-04-23 13:15, Ferruh Yigit wrote: On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: Cache align Rx and Tx queue struct to avoid false sharing. RX struct happens to be 64 bytes on x86_

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Ferruh Yigit
On 4/23/2024 9:56 PM, Mattias Rönnblom wrote: > On 2024-04-23 13:15, Ferruh Yigit wrote: >> On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: >>> Cache align Rx and Tx queue struct to avoid false sharing. >>> >>> RX struct happens to be 64 bytes on x86_64 already, so cache alignment >>> makes no chang

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Bruce Richardson
On Wed, Apr 24, 2024 at 11:21:52AM +0100, Ferruh Yigit wrote: > On 4/23/2024 9:56 PM, Mattias Rönnblom wrote: > > On 2024-04-23 13:15, Ferruh Yigit wrote: > >> On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: > >>> Cache align Rx and Tx queue struct to avoid false sharing. > >>> > >>> RX struct happe

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Mattias Rönnblom
On 2024-04-24 12:21, Ferruh Yigit wrote: On 4/23/2024 9:56 PM, Mattias Rönnblom wrote: On 2024-04-23 13:15, Ferruh Yigit wrote: On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: Cache align Rx and Tx queue struct to avoid false sharing. RX struct happens to be 64 bytes on x86_64 already, so cach

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Ferruh Yigit
On 4/24/2024 12:57 PM, Mattias Rönnblom wrote: > On 2024-04-24 12:21, Ferruh Yigit wrote: >> On 4/23/2024 9:56 PM, Mattias Rönnblom wrote: >>> On 2024-04-23 13:15, Ferruh Yigit wrote: On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: > Cache align Rx and Tx queue struct to avoid false sharing

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Ferruh Yigit
On 4/24/2024 11:28 AM, Bruce Richardson wrote: > On Wed, Apr 24, 2024 at 11:21:52AM +0100, Ferruh Yigit wrote: >> On 4/23/2024 9:56 PM, Mattias Rönnblom wrote: >>> On 2024-04-23 13:15, Ferruh Yigit wrote: On 4/23/2024 10:08 AM, Mattias Rönnblom wrote: > Cache align Rx and Tx queue struct t

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Stephen Hemminger
On Wed, 24 Apr 2024 18:50:50 +0100 Ferruh Yigit wrote: > > I don't know how slow af_packet is, but if you care about performance, > > you don't want to use atomic add for statistics. > > > > There are a few soft drivers already using atomics adds for updating stats. > If we document expectati

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Mattias Rönnblom
On 2024-04-24 21:13, Stephen Hemminger wrote: On Wed, 24 Apr 2024 18:50:50 +0100 Ferruh Yigit wrote: I don't know how slow af_packet is, but if you care about performance, you don't want to use atomic add for statistics. There are a few soft drivers already using atomics adds for updatin

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-24 Thread Stephen Hemminger
On Thu, 25 Apr 2024 00:27:36 +0200 Mattias Rönnblom wrote: > On 2024-04-24 21:13, Stephen Hemminger wrote: > > On Wed, 24 Apr 2024 18:50:50 +0100 > > Ferruh Yigit wrote: > > > >>> I don't know how slow af_packet is, but if you care about performance, > >>> you don't want to use atomic add for

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Mattias Rönnblom
On 2024-04-25 01:55, Stephen Hemminger wrote: On Thu, 25 Apr 2024 00:27:36 +0200 Mattias Rönnblom wrote: On 2024-04-24 21:13, Stephen Hemminger wrote: On Wed, 24 Apr 2024 18:50:50 +0100 Ferruh Yigit wrote: I don't know how slow af_packet is, but if you care about performance, you don't

RE: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Morten Brørup
> From: Mattias Rönnblom [mailto:hof...@lysator.liu.se] > Sent: Thursday, 25 April 2024 11.26 > > On 2024-04-25 01:55, Stephen Hemminger wrote: > > On Thu, 25 Apr 2024 00:27:36 +0200 > > Mattias Rönnblom wrote: > > > >> On 2024-04-24 21:13, Stephen Hemminger wrote: > >>> On Wed, 24 Apr 2024 18:50

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Ferruh Yigit
On 4/25/2024 10:26 AM, Mattias Rönnblom wrote: > On 2024-04-25 01:55, Stephen Hemminger wrote: >> On Thu, 25 Apr 2024 00:27:36 +0200 >> Mattias Rönnblom wrote: >> >>> On 2024-04-24 21:13, Stephen Hemminger wrote: On Wed, 24 Apr 2024 18:50:50 +0100 Ferruh Yigit wrote:    >> I do

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Ferruh Yigit
On 4/23/2024 12:15 PM, Ferruh Yigit wrote: >> Signed-off-by: Mattias Rönnblom >> --- >> drivers/net/af_packet/rte_eth_af_packet.c | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/af_packet/rte_eth_af_packet.c >> b/drivers/net/af_packet/rte_eth_af_packe

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Mattias Rönnblom
On 2024-04-25 16:04, Ferruh Yigit wrote: On 4/25/2024 10:26 AM, Mattias Rönnblom wrote: On 2024-04-25 01:55, Stephen Hemminger wrote: On Thu, 25 Apr 2024 00:27:36 +0200 Mattias Rönnblom wrote: On 2024-04-24 21:13, Stephen Hemminger wrote: On Wed, 24 Apr 2024 18:50:50 +0100 Ferruh Yigit wro

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Mattias Rönnblom
On 2024-04-25 16:08, Ferruh Yigit wrote: On 4/23/2024 12:15 PM, Ferruh Yigit wrote: Signed-off-by: Mattias Rönnblom --- drivers/net/af_packet/rte_eth_af_packet.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/net/af_packet/rte_eth_af_packet.c b/drivers/net/

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Ferruh Yigit
On 4/25/2024 4:08 PM, Mattias Rönnblom wrote: > On 2024-04-25 16:08, Ferruh Yigit wrote: >> On 4/23/2024 12:15 PM, Ferruh Yigit wrote: Signed-off-by: Mattias Rönnblom ---   drivers/net/af_packet/rte_eth_af_packet.c | 5 +++--   1 file changed, 3 insertions(+), 2 deletions(-) >>>

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-25 Thread Ferruh Yigit
On 4/25/2024 4:06 PM, Mattias Rönnblom wrote: > On 2024-04-25 16:04, Ferruh Yigit wrote: >> On 4/25/2024 10:26 AM, Mattias Rönnblom wrote: >>> On 2024-04-25 01:55, Stephen Hemminger wrote: On Thu, 25 Apr 2024 00:27:36 +0200 Mattias Rönnblom wrote: > On 2024-04-24 21:13, Stephen

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-26 Thread Mattias Rönnblom
On 2024-04-25 17:35, Ferruh Yigit wrote: On 4/25/2024 4:08 PM, Mattias Rönnblom wrote: On 2024-04-25 16:08, Ferruh Yigit wrote: On 4/23/2024 12:15 PM, Ferruh Yigit wrote: Signed-off-by: Mattias Rönnblom ---   drivers/net/af_packet/rte_eth_af_packet.c | 5 +++--   1 file changed, 3 insertions

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-26 Thread Ferruh Yigit
On 4/26/2024 8:38 AM, Mattias Rönnblom wrote: > Cache align Rx and Tx queue struct to avoid false sharing. > > RX struct happens to be 64 bytes on x86_64 already, so cache alignment > has no effect there, but it does on 32-bit ISAs. > > TX struct is 56 bytes on x86_64. > > Both structs keep coun

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-26 Thread Mattias Rönnblom
On 2024-04-26 10:27, Ferruh Yigit wrote: On 4/26/2024 8:38 AM, Mattias Rönnblom wrote: Cache align Rx and Tx queue struct to avoid false sharing. RX struct happens to be 64 bytes on x86_64 already, so cache alignment has no effect there, but it does on 32-bit ISAs. TX struct is 56 bytes on x86

Re: [PATCH] net/af_packet: cache align Rx/Tx structs

2024-04-26 Thread Patrick Robb
Recheck-request: iol-compile-amd64-testing The DPDK Community Lab updated to the latest Alpine image yesterday, which resulted in all Alpine builds failing. The failure is unrelated to your patch, and this recheck should remove the fail on Patchwork, as we have disabled Alpine testing for now.