On Sun, Jul 11, 2021 at 7:34 PM Ruifeng Wang <[email protected]> wrote:
> > -----Original Message----- > > From: Lance Richardson <[email protected]> > > Sent: Saturday, July 10, 2021 12:39 AM > > To: Ajit Khaparde ([email protected]) > > <[email protected]>; Somnath Kotur > > <[email protected]>; Bruce Richardson > > <[email protected]>; Konstantin Ananyev > > <[email protected]>; [email protected]; Ruifeng Wang > > <[email protected]>; Stephen Hurd <[email protected]>; > > David Christensen <[email protected]> > > Cc: [email protected]; [email protected] > > Subject: [PATCH v2] net/bnxt: fix missing barriers in completion handling > > > > Ensure that Rx/Tx/Async completion entry fields are accessed > > only after the completion's valid flag has been loaded and > > verified. This is needed for correct operation on systems that > > use relaxed memory consistency models. > > > > Fixes: 2eb53b134aae ("net/bnxt: add initial Rx code") > > Fixes: 6eb3cc2294fd ("net/bnxt: add initial Tx code") > > Cc: [email protected] > > Signed-off-by: Lance Richardson <[email protected]> > > Reviewed-by: Ajit Khaparde <[email protected]> > > --- > > v2: > > * Corrected name of the first parameter to bnxt_cpr_cmp_valid() in > > comments ('cmp' to 'cmpl'). > > > > drivers/net/bnxt/bnxt_cpr.h | 36 ++++++++++++++++++++++++--- > > drivers/net/bnxt/bnxt_ethdev.c | 16 ++++++------ > > drivers/net/bnxt/bnxt_irq.c | 7 +++--- > > drivers/net/bnxt/bnxt_rxr.c | 9 ++++--- > > drivers/net/bnxt/bnxt_rxtx_vec_avx2.c | 2 +- > > drivers/net/bnxt/bnxt_rxtx_vec_neon.c | 2 +- > > drivers/net/bnxt/bnxt_rxtx_vec_sse.c | 2 +- > > drivers/net/bnxt/bnxt_txr.c | 2 +- > > 8 files changed, 54 insertions(+), 22 deletions(-) > > > Reviewed-by: Ruifeng Wang <[email protected]> > Patch applied to dpdk-next-net-brcm. Thanks
