From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 23190A2E1B for ; Thu, 5 Sep 2019 14:12:10 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D54191EF6D; Thu, 5 Sep 2019 14:12:09 +0200 (CEST) Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-eopbgr130044.outbound.protection.outlook.com [40.107.13.44]) by dpdk.org (Postfix) with ESMTP id 22E731EF6D; Thu, 5 Sep 2019 14:12:08 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZbKFYj4BJo6g+nc+wV967FuWU0YcpUeyHJko3+kp6TRZ13pKjFa5wdrV9QOe2YM6vu3hsAgh1A5PSe4m69LhBc0Md+BIBMHVOhO3fTqlCRj3DKVIUAbh7pJ+PNDcuCW4n06cQcR8e1BHgot6C9OxqITjm1I/PcaUVPEDnkhUJZfoUVeMnRljUy4DFAn6PEYIPSsNcggKasvSY6aaYxDaX0pZnDrrEExZ813H2uUibx8EaKczy03t/CYlaYVlV6Cq+d9zJIob2MQ69ARSVs0YOcRz4eeqGODjtv8YhV1SDjSqUVHMRfHEqCUOmsfAstgONP+3tqjNt65X+OzKn5YdNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=yG6nhGxRVNFGNaZAWMksnm0bMDhvSE2J8bD4FqkepWk=; b=b1R5W14ANA9qebMmdEsQ9m0EB8zkzBaqLJ1Uzm7TkTREZo9WpGYIZO5pKXNNaakV+GnMzkHiCGyp+0jnJY3+0gn5mK/Ah8K0lLSScUl+QOZgm5gLHW8jPfD7WJzlQUtzkUOYCQRHFec5ZNsNou0UgcMr0P5lNDFJvS1chKZFywlXCiMoUnEXQ9qMUEdnAf+gJG/gMLUKxjLy+F6RBnLaSF1/mWyuvv8X8rGdrdv8PtEe8cn2cO45hCt0k0tczli5U6mik71kv2ZtRkS64+7v1ZoPtGJNfvlobAbriSgwOLnLBVuBoupHpahGJoYk0fSW1vc4tqJZdr7/lk/PGcOjkQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=mellanox.com; dmarc=pass action=none header.from=mellanox.com; dkim=pass header.d=mellanox.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=yG6nhGxRVNFGNaZAWMksnm0bMDhvSE2J8bD4FqkepWk=; b=aGM7EHDsCOIeo9jfAJDpQm6+75JdeyfhngS8+28AZbLPbqkgubV0BkyVxAWS8oqou9wMt3dO0otH4zlSFbobm6VZw6ypUzlphUdAfHGRmcWMIvX9VKK1X0ZLEehGtilaJ0/FCuK8nGewkjeFldR484dQcDpg3xJaCavGF6HL6fc= Received: from AM4PR05MB3265.eurprd05.prod.outlook.com (10.171.188.154) by AM4PR05MB3460.eurprd05.prod.outlook.com (10.171.187.153) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2220.21; Thu, 5 Sep 2019 12:12:03 +0000 Received: from AM4PR05MB3265.eurprd05.prod.outlook.com ([fe80::da9:65ba:1323:a39b]) by AM4PR05MB3265.eurprd05.prod.outlook.com ([fe80::da9:65ba:1323:a39b%7]) with mapi id 15.20.2220.022; Thu, 5 Sep 2019 12:12:03 +0000 From: Slava Ovsiienko To: Phil Yang , Yongseok Koh , Matan Azrad , =?iso-8859-1?Q?N=E9lio_Laranjeiro?= , "dev@dpdk.org" CC: Thomas Monjalon , "jerinj@marvell.com" , "Honnappa.Nagarahalli@arm.com" , "gavin.hu@arm.com" , "nd@arm.com" , "stable@dpdk.org" Thread-Topic: [PATCH 2/2] net/mlx5: fix Tx CQ doorbell synchronization on aarch64 Thread-Index: AQHVY9h4bf4CU/U3oE66T8cjKZO3GKcc66Lw Date: Thu, 5 Sep 2019 12:12:02 +0000 Message-ID: References: <1567680908-31210-1-git-send-email-phil.yang@arm.com> <1567680908-31210-2-git-send-email-phil.yang@arm.com> In-Reply-To: <1567680908-31210-2-git-send-email-phil.yang@arm.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=viacheslavo@mellanox.com; x-originating-ip: [95.67.35.250] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 5ff934ef-ac86-46fd-1f5d-08d731fa42e5 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600166)(711020)(4605104)(1401327)(4618075)(2017052603328)(7193020); SRVR:AM4PR05MB3460; x-ms-traffictypediagnostic: AM4PR05MB3460:|AM4PR05MB3460: x-ld-processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:10000; x-forefront-prvs: 015114592F x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(376002)(366004)(39860400002)(346002)(396003)(136003)(13464003)(199004)(189003)(186003)(54906003)(316002)(76176011)(110136005)(53546011)(71190400001)(71200400001)(7696005)(26005)(256004)(14454004)(14444005)(2501003)(9686003)(55016002)(4326008)(25786009)(6436002)(6246003)(53936002)(6506007)(102836004)(76116006)(66946007)(66476007)(66446008)(64756008)(66556008)(446003)(11346002)(66066001)(81156014)(305945005)(74316002)(33656002)(6116002)(3846002)(7736002)(476003)(486006)(81166006)(8936002)(2906002)(52536014)(229853002)(5660300002)(66574012)(99286004)(478600001)(8676002)(86362001); DIR:OUT; SFP:1101; SCL:1; SRVR:AM4PR05MB3460; H:AM4PR05MB3265.eurprd05.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: Gxtqov65wbTJxapcfoQfKteUX+wsNLcu1Db0a+CqxwF6k/v/bzPL4LURK7mv5kUdOUKKSsT5OalM6q3HbBNKyXdMjMtgNqwzEPt01oIrIdY+ry5bEdAQTevj1lZKbAYMyY1cpLz/UAU/4YArAVK6B9AsdfKx/lq7DoIRUXDflmqtbf2yLsyPTYAQro55d/IMm+FKuDpvoeIS6hBGIV/CxLnoVEpj2xUgeLf7Kcfr69KaxNnzidKuF6fi0K0NF91h9VYWJBuxkZHK6/7U26b9xkTHANLuv4YAtuUNnL0fnNNMnaKGrkRXh0ng1jC0YbNGaXDLcjMh+5h+XWiMSa96klZe4+2LB4DAWAooTQOiMfy8p/nnZldNqT6JKD6ta0fagR+nJCmqiXt4xHxRSVEKupLteYHmWWI7TAZgTc9DE8I= Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5ff934ef-ac86-46fd-1f5d-08d731fa42e5 X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Sep 2019 12:12:02.8492 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: h8Lp3xcaaAlCTeJJhMzm3xIABJ8UiDa+LPUk3Io78U1TIz/HA06mJS9U8CeM7d1hwtpKX/pzhtBqJMUj2/ddK7HSsC+MQ6QlWYUQlxA+nY4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR05MB3460 Subject: Re: [dpdk-stable] [PATCH 2/2] net/mlx5: fix Tx CQ doorbell synchronization on aarch64 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, Phil This point is in datapath and performance is very critical. The rte_cio_wmb() may take a lot of CPU cycles, waiting till all previous w= rites become visible for all external (relating to core) agents. The Tx CQE doorbelling = does not need any writes to other locations to be completed, the only concern is not to r= eorder/merge the writes to the same doorbell register of the same sending queue in the t= x_burst() internal sending loop/subsequent calls.=20 As far as I know - the writes to the same location should not be reordered = by any arch (may be merged if memory settings allow this, it is not critical for CQE do= orbell), could you, please, explain why we need explicit hardware fence before CQE d= oorbell update? Do you think doorbell write might be rearranged with previously rea= ds from the ring buffer? WBR, Slava > -----Original Message----- > From: Phil Yang > Sent: Thursday, September 5, 2019 13:55 > To: Yongseok Koh ; Slava Ovsiienko > ; Matan Azrad ; N=E9lio > Laranjeiro ; dev@dpdk.org > Cc: Thomas Monjalon ; jerinj@marvell.com; > Honnappa.Nagarahalli@arm.com; gavin.hu@arm.com; nd@arm.com; > stable@dpdk.org > Subject: [PATCH 2/2] net/mlx5: fix Tx CQ doorbell synchronization on > aarch64 >=20 > For the weaker memory model processors, the compiler barrier is not > sufficient to guarantee the coherent memory update be observed by I/O > device. It needs the coherent I/O memory barrier to enforce the ordering = of > Tx completion queue doorbell operation. >=20 > Fixes: da1df1ccabad ("net/mlx5: fix completion queue drain loop") > Cc: stable@dpdk.org >=20 > Suggested-by: Gavin Hu > Signed-off-by: Phil Yang > Reviewed-by: Gavin Hu > --- > drivers/net/mlx5/mlx5_rxtx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >=20 > diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c > index 4c01187..c11148b 100644 > --- a/drivers/net/mlx5/mlx5_rxtx.c > +++ b/drivers/net/mlx5/mlx5_rxtx.c > @@ -2042,7 +2042,7 @@ mlx5_tx_comp_flush(struct mlx5_txq_data > *restrict txq, > } else { > return; > } > - rte_compiler_barrier(); > + rte_cio_wmb(); > *txq->cq_db =3D rte_cpu_to_be_32(txq->cq_ci); > if (likely(tail !=3D txq->elts_tail)) { > mlx5_tx_free_elts(txq, tail, olx); > -- > 2.7.4