From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-CO1-obe.outbound.protection.outlook.com (mail-co1nam03on0052.outbound.protection.outlook.com [104.47.40.52]) by dpdk.org (Postfix) with ESMTP id 3AE974C9C; Mon, 5 Nov 2018 14:41:35 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=50bPR4n8K8TTKRttTrgmmGKrXCnrdcVG+HRg7ZTEX60=; b=JVzFWD3EZUr0caDcYIENAXjRfBCvORfE/39ZVM044ZA77wdC5lzbyeAam+tGeo8dRAlaLEJGoOuuTpDqiwjUn6AiU6FXqhYc9toTewbX2qdYgs/sApCf6MiBakNK+G8O9Up6UWN0jRxwVjhseVToIVVNsaQ0YU1a7efZh78TXnU= Received: from BYAPR07MB4997.namprd07.prod.outlook.com (52.135.238.214) by BYAPR07MB5160.namprd07.prod.outlook.com (20.176.254.213) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1294.31; Mon, 5 Nov 2018 13:41:33 +0000 Received: from BYAPR07MB4997.namprd07.prod.outlook.com ([fe80::2d56:eab:242f:fdfc]) by BYAPR07MB4997.namprd07.prod.outlook.com ([fe80::2d56:eab:242f:fdfc%3]) with mapi id 15.20.1294.028; Mon, 5 Nov 2018 13:41:33 +0000 From: Jerin Jacob To: Thomas Monjalon CC: Honnappa Nagarahalli , "stable@dpdk.org" , "Gavin Hu (Arm Technology China)" , Bruce Richardson , "dev@dpdk.org" , "stephen@networkplumber.org" , "olivier.matz@6wind.com" , "chaozhu@linux.vnet.ibm.com" , "konstantin.ananyev@intel.com" , nd , "hemant.agrawal@nxp.com" , "shreyansh.jain@nxp.com" Thread-Topic: [dpdk-stable] [PATCH v5 2/2] ring: move the atomic load of head above the loop Thread-Index: AQHUcp49h/bTcwT3A0WOdl7oFN9AOqU8XWEAgADj64CAAIpVAIADYu2AgAAGp4A= Date: Mon, 5 Nov 2018 13:41:33 +0000 Message-ID: <20181105134114.GA4270@jerin> References: <1541066031-29125-1-git-send-email-gavin.hu@arm.com> <1825633.NNxUzH26Cz@xps> In-Reply-To: <1825633.NNxUzH26Cz@xps> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [115.113.156.3] x-clientproxiedby: BM1PR0101CA0009.INDPRD01.PROD.OUTLOOK.COM (2603:1096:b00:18::19) To BYAPR07MB4997.namprd07.prod.outlook.com (2603:10b6:a03:5b::22) authentication-results: spf=none (sender IP is ) smtp.mailfrom=Jerin.JacobKollanukkaran@cavium.com; x-ms-exchange-messagesentrepresentingtype: 1 x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; BYAPR07MB5160; 6:WUnXjJJ6AQ6z9nLuJ/bmaG/Ad3V2dS4PPPjRtVb+nFTDrr7PFu9eqlhwoo2rplwEhqwxHQnyrjPgzjeLX1lPV+7mysL+rkxyhmMLz0LFGMJByAfcun10dr0OE1bQdfaYhgE+VjnZ9oeYTX1sFqztAJezXkzUBLqMVNJ0aZiU9FOChnfzyu5g8cRxdF6QUY+u+lqzO5TMP8cL7JYlL8RkYpHYbYR/V2zjlMurVXlrBzeV/GptwHbysGIt15/zH6wWuWj6ADQbQuZ8Zjh5tWN5QvD0AkOUFL41K+UkNnRw4GWmMzZ77aDaGTzxZBCVcjwAC1eWASyDlntQNOT/SGi16aRrBXo1PERkleHuJUuCBQwY3b88dMfzDQe78NaIYe67WA/lHv4I3mGRAwhVqkudhiexv/aBGygWiEFF9oSN60ZpVbSilWBj/1iHZKzcly2WQu1Jyn0X6zBhixzetHRQeQ==; 5:31QpUOHJlvkxzCXZfwmGxMPumDvlg0fdCON9MB5sDyfxaZd3VevLyRNCJbYzzVeja3H4R4ka4gL4+FgkSgr+J7PZJedc597+zcyoWYNQGpV6ik0tC6xdsHE9owJLen1lyCbVTT0gHxi13CfaHT2tT45Q5VgG/47DEZw0+sJzL/M=; 7:9NFvGPof4E6GAw/Ms7VG4L8IyCnmtm7tjY5DaRbLILyZA6RwlKIGocbVXEAmv/6W8m7t4AUF1h1BIiUjFqBnfSOngn/D1okSKhG4KepmXQaqhRx1BgZGk/07zSJzt9FI5byu0fds0b0oTJ7KgBJdnw== x-ms-office365-filtering-correlation-id: 61e67b57-0965-4f1e-7853-08d6432465a8 x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(5600074)(711020)(4534185)(7168020)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7153060)(7193020); SRVR:BYAPR07MB5160; x-ms-traffictypediagnostic: BYAPR07MB5160: x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(104084551191319)(228905959029699)(185117386973197)(163750095850)(180628864354917); x-ms-exchange-senderadcheck: 1 x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(3231382)(944501410)(52105095)(3002001)(10201501046)(93006095)(148016)(149066)(150057)(6041310)(20161123558120)(20161123562045)(20161123564045)(20161123560045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(201708071742011)(7699051)(76991095); SRVR:BYAPR07MB5160; BCL:0; PCL:0; RULEID:; SRVR:BYAPR07MB5160; x-forefront-prvs: 08476BC6EF x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(7916004)(346002)(136003)(396003)(376002)(39850400004)(366004)(13464003)(199004)(189003)(186003)(26005)(2900100001)(6116002)(3846002)(1076002)(72206003)(55236004)(102836004)(68736007)(6916009)(6512007)(9686003)(76176011)(53936002)(5660300001)(229853002)(99286004)(52116002)(33656002)(14454004)(6436002)(6246003)(97736004)(71190400001)(81166006)(81156014)(2906002)(8676002)(93886005)(25786009)(4326008)(486006)(71200400001)(11346002)(446003)(476003)(42882007)(316002)(54906003)(8936002)(6486002)(7416002)(106356001)(105586002)(386003)(6506007)(66066001)(33716001)(478600001)(14444005)(33896004)(256004)(7736002)(305945005); DIR:OUT; SFP:1101; SCL:1; SRVR:BYAPR07MB5160; H:BYAPR07MB4997.namprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; received-spf: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: c0Wl7zS5Hi09VkiGlTAZt/WgPgu6QhtN8VisYlXTVedaGJF9tiB3flFNBkh/NMQCWIDt0yQ56bzgGcmPe8Z3rQKPOcsPbUdeDuGCd7t9V3VK4oiQa46dRAZ8lVuHUdoFV6JjLfJ09BgzJ0h0blV0ooN6d0/Jw6Q+X7Yy4Mor9dNCCvbfcOrIUTJck+ubhPSke5q6EezhdhEGa/rqYciLlBH9tryy59P0fbQUpPJ+0l9wr4Od6Bx/MZQCXQ+Dm6+s9vkwM3qHYgF/eseylpMLR4YxrIHFMlrE6Lam1iIBpxfrq0AxfYHu5ymiGF7XMoxBJEi6G6flKFUJ2LyAiL4YB8YPWoYbagZMPCQ+UZJ99VM= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-ID: <291E6DAB55DB6C4FB6257F97EAE85910@namprd07.prod.outlook.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-Network-Message-Id: 61e67b57-0965-4f1e-7853-08d6432465a8 X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Nov 2018 13:41:33.0432 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR07MB5160 Subject: Re: [dpdk-dev] [dpdk-stable] [PATCH v5 2/2] ring: move the atomic load of head above the loop X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 05 Nov 2018 13:41:35 -0000 -----Original Message----- > Date: Mon, 05 Nov 2018 14:17:27 +0100 > From: Thomas Monjalon > To: Honnappa Nagarahalli > Cc: stable@dpdk.org, "Gavin Hu (Arm Technology China)" = , > Bruce Richardson , "dev@dpdk.org" > , "stephen@networkplumber.org" , > "olivier.matz@6wind.com" , > "chaozhu@linux.vnet.ibm.com" , > "konstantin.ananyev@intel.com" , > "jerin.jacob@caviumnetworks.com" , nd > , hemant.agrawal@nxp.com, shreyansh.jain@nxp.com > Subject: Re: [dpdk-stable] [PATCH v5 2/2] ring: move the atomic load of > head above the loop >=20 > External Email >=20 > 03/11/2018 10:34, Honnappa Nagarahalli: > > > > > --- > > > > > doc/guides/rel_notes/release_18_11.rst | 7 +++++++ > > > > > lib/librte_ring/rte_ring_c11_mem.h | 10 ++++------ > > > > > 2 files changed, 11 insertions(+), 6 deletions(-) > > > > > > > > > > diff --git a/doc/guides/rel_notes/release_18_11.rst > > > > > b/doc/guides/rel_notes/release_18_11.rst > > > > > index 376128f..b68afab 100644 > > > > > --- a/doc/guides/rel_notes/release_18_11.rst > > > > > +++ b/doc/guides/rel_notes/release_18_11.rst > > > > > @@ -69,6 +69,13 @@ New Features > > > > > checked out against that dma mask and rejected if out of range= . > > > > > If more > > > > than > > > > > one device has addressing limitations, the dma mask is the mor= e > > > > restricted one. > > > > > > > > > > +* **Updated the ring library with C11 memory model.** > > > > > + > > > > > + Updated the ring library with C11 memory model, in our tests t= he > > > > > + changes decreased latency by 27~29% and 3~15% for MPMC and SPS= C > > > > cases respectively. > > > > > + The real improvements may vary with the number of contending > > > > > + lcores and the size of ring. > > > > > + > > > > Is this a little misleading, and will users expect massive performa= nce > > > > improvements generally? The C11 model seems to be used only on some= , > > > > but not all, arm platforms, and then only with "make" builds. > > > > > > > > config/arm/meson.build: ['RTE_USE_C11_MEM_MODEL', false]] > > This is an error. There is already an agreement that on Arm based platf= orms, C11 memory model would be used by default. Specific platforms can ove= rride it if required. > > Would this be ab acceptable change for RC2 or RC3? >=20 > If NXP and Cavium agrees, I think it can go in RC2. Yes. meson and make config should be same. i.e on Arm based platforms, C11 memory model would be used by default. Specific platforms can override it if required. I think, meson config needs to be updated to be inline with make config. > For RC3, not sure. >=20 >=20 >=20