RE: [EXT] Re: [PATCH net-next] net: mvpp2: Use relaxed I/O in data path

From: Yan Markman
Date: Tue Mar 27 2018 - 13:18:37 EST


Thanks David
We are aware and it is working properly
Yan Markman


-----Original Message-----
From: David Miller [mailto:davem@xxxxxxxxxxxxx]
Sent: Tuesday, March 27, 2018 8:16 PM
To: maxime.chevallier@xxxxxxxxxxx
Cc: Yan Markman <ymarkman@xxxxxxxxxxx>; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; antoine.tenart@xxxxxxxxxxx; thomas.petazzoni@xxxxxxxxxxx; gregory.clement@xxxxxxxxxxx; miquel.raynal@xxxxxxxxxxx; Nadav Haklai <nadavh@xxxxxxxxxxx>; Stefan Chulski <stefanc@xxxxxxxxxxx>; mw@xxxxxxxxxxxx
Subject: [EXT] Re: [PATCH net-next] net: mvpp2: Use relaxed I/O in data path

External Email

----------------------------------------------------------------------
From: Maxime Chevallier <maxime.chevallier@xxxxxxxxxxx>
Date: Tue, 27 Mar 2018 16:49:05 +0200

> From: Yan Markman <ymarkman@xxxxxxxxxxx>
>
> Use relaxed I/O on the hot path. This achieves significant performance
> improvements. On a 10G link, this makes a basic iperf TCP test go from
> an average of 4.5 Gbits/sec to about 9.40 Gbits/sec.
>
> Signed-off-by: Yan Markman <ymarkman@xxxxxxxxxxx>
> [Maxime: Commit message, cosmetic changes]
> Signed-off-by: Maxime Chevallier <maxime.chevallier@xxxxxxxxxxx>
> ---
> David, this patch should not conflict with other the pending PPv2
> series I sent earlier ("[PATCH net-next 0/2] net: mvpp2: Remove
> unnecessary dynamic allocs")

Applied, but it is up to you to make sure the correct barriers are in place such that undesirable memory and IOMEM operation reordering does not occur.

Said another way, you get to keep the pieces if this breaks things :-)