From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on0068.outbound.protection.outlook.com [104.47.0.68]) by dpdk.org (Postfix) with ESMTP id 5A1D9FEB; Mon, 29 Oct 2018 03:51:08 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=AasfNDqlVoB39p6v7rNcZ5WYWwgcXJlOifL2AZMpvuM=; b=XXbGwDDf/fvepxZwfNt9nZolVylJoPbvm3OgcAL+JE458LqBGMY2to4xkgx9DAn4HOX/hjCjXQJm9dfdScF+6xv74oXwnaFsCnLdtSWP6NAnWJB2jdKHBkQ4zlOmqPybhhcxdUijXAr/1FzVq0Z/F6J6z+wj+HA9hu5flDFP7q4= Received: from VI1PR08MB3167.eurprd08.prod.outlook.com (52.133.15.142) by VI1PR08MB3373.eurprd08.prod.outlook.com (20.177.58.215) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1273.25; Mon, 29 Oct 2018 02:51:06 +0000 Received: from VI1PR08MB3167.eurprd08.prod.outlook.com ([fe80::d46a:1de3:53cd:3e49]) by VI1PR08MB3167.eurprd08.prod.outlook.com ([fe80::d46a:1de3:53cd:3e49%4]) with mapi id 15.20.1273.027; Mon, 29 Oct 2018 02:51:06 +0000 From: "Gavin Hu (Arm Technology China)" To: Jerin Jacob , Thomas Monjalon CC: "dev@dpdk.org" , Honnappa Nagarahalli , "stable@dpdk.org" , Ola Liljedahl , "olivier.matz@6wind.com" , "chaozhu@linux.vnet.ibm.com" , "bruce.richardson@intel.com" , "konstantin.ananyev@intel.com" Thread-Topic: [dpdk-dev] [PATCH 1/2] ring: synchronize the load and store of the tail Thread-Index: AQHUZeLht2NjggOSDE2ZyYOtLzEKsqUi+lcwgBA/wQCAAAXIAIAAA34AgAAF24CAAjz78A== Date: Mon, 29 Oct 2018 02:51:06 +0000 Message-ID: References: <1537172244-64874-2-git-send-email-gavin.hu@arm.com> <1874944.OrACW1nkDZ@xps> <20181027150024.GA2294@jerin> <17713879.gC9jYcxDUo@xps> <20181027153355.GA9367@jerin> In-Reply-To: <20181027153355.GA9367@jerin> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [113.29.88.7] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1; VI1PR08MB3373; 6:mcPfcgmVEwuZxZ/IGHpvd9shYs6IYfogCFVBG8eXC1NDbpk8UKXwBHggI7Yjg5rY796V42xHfuVJPRbPISx/cl3W7hnGtVLAIcMxBWpfv5rxQCC0UZG/PslS0T+FENPJFefDk0+2E7z+aFioe5WM0nANZ63c6IYbGysZMY9TmeeKjpaVEoeTUZD44Rx94HrWntaVp1JEXmn3sE4Kol22ae8d5gWarDz2VzPLzG+9a9APdrCEDThcpMwh+UGr10dMKcW0BoNobasNzR6iYOxqBBUqqMJuGevy8qIVNOJpfqBlwJz+9DBQBInT9D8u0i9m27crjI3KjX/2BaDKTjo5ILTvOHODN3qJJ9xevwlK+QT6CdnT4xFOtJS2TtAGQ2jOPWNnpq/t6Ytm4ijoKPj2vk7yEBnRPmc2BPzv770UZFfb3X5EdvN4KMqZIs2egMAzcM71HixDaf+hIkqsFBHlKQ==; 5:sZb32rUlPwyOFfDvxBD1IFfr56IttEqBCSy3iijQIzv/j0bKGXd9Yfhf3RAUuIrTuMq1wNOKlyY+QKvL+aNp4esoKQq/mQb23Qg2OYOrC6NmxawpUWuDoy1Bpil2xmrG5uFHAVZsZ3hZoly95F5qVgtdlDhYo23/F7XL+Ma00iQ=; 7:cUdKfmkuJdxSBOLcH/S61iyTz9TM+JNO2bVzfw/i8WPS5aVEaDndUMLFpu3Y5M54ANhsTbxuPDpEhyeKxcM/1eNxv32M9/Yqf9qz/jD4ggmUh+f2TCpOHv9hAphm6yxkyoDQOhZOfQL0sk9loDhQdg== x-ms-exchange-antispam-srfa-diagnostics: SOS;SOR; x-ms-office365-filtering-correlation-id: 2a3f36c9-ecfa-46d8-77f1-08d63d495f74 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989299)(4534185)(7168020)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020); SRVR:VI1PR08MB3373; x-ms-traffictypediagnostic: VI1PR08MB3373: authentication-results: spf=none (sender IP is ) smtp.mailfrom=Gavin.Hu@arm.com; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(104084551191319)(228905959029699)(180628864354917); x-ms-exchange-senderadcheck: 1 x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231382)(944501410)(52105095)(93006095)(93001095)(10201501046)(3002001)(6055026)(148016)(149066)(150057)(6041310)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123558120)(20161123562045)(20161123564045)(20161123560045)(201708071742011)(7699051)(76991095); SRVR:VI1PR08MB3373; BCL:0; PCL:0; RULEID:; SRVR:VI1PR08MB3373; x-forefront-prvs: 084080FC15 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(376002)(346002)(366004)(39850400004)(136003)(396003)(13464003)(189003)(199004)(40434004)(72206003)(55236004)(68736007)(26005)(256004)(53546011)(102836004)(99286004)(6506007)(93886005)(478600001)(8936002)(4326008)(33656002)(7736002)(66066001)(5250100002)(2906002)(14454004)(97736004)(5024004)(14444005)(7696005)(76176011)(3846002)(6116002)(71200400001)(71190400001)(5660300001)(81166006)(81156014)(446003)(8676002)(74316002)(105586002)(6436002)(86362001)(186003)(106356001)(25786009)(55016002)(305945005)(11346002)(316002)(476003)(2900100001)(54906003)(6246003)(229853002)(110136005)(9686003)(486006)(53936002); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR08MB3373; H:VI1PR08MB3167.eurprd08.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: arm.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: /5RUaJTV9Ib+kd8fucmELkgMd/MnUnDjCm3qEeWTJdVLKmCOErO6Xbtuy6tlNCtiVe4XNjhRDvC3xHqs+v3dbDEULJYZjjQdcym/ZwokjyuvZ4BBH6KXQiw+2yrX2Mkf1xBhYhLN3g05ek/44H5+1QQAJx0wJaSFrErCfJTy5ipmwy7z86pbQCMALj2AO9xW/iVZ5DJFAem0HVJH2Yn4cib2Tz+HI8pukpN8vUIgNIMJMiXj5igRlXFLsNgwAJTVoSeyN0Ry3j0HoGBs1Z8h+XfcGHWQvP7aHyFzDLV8pY0AK7zEKoYGNm8qmAeQW0cmFp5UHdyRfB2MApbCS0NY62fNoC0e8KwT2zIvkpublUs= spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2a3f36c9-ecfa-46d8-77f1-08d63d495f74 X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Oct 2018 02:51:06.1472 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB3373 Subject: Re: [dpdk-dev] [PATCH 1/2] ring: synchronize the load and store of the tail 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, 29 Oct 2018 02:51:08 -0000 Hi Thomas and Jerin, The patches were extensively and heavily reviewed by Arm internally. As the= 1st patch was not concluded, so I create a new series(2 patches), > -----Original Message----- > From: Jerin Jacob > Sent: Saturday, October 27, 2018 11:34 PM > To: Thomas Monjalon > Cc: Gavin Hu (Arm Technology China) ; dev@dpdk.org; > Honnappa Nagarahalli ; stable@dpdk.org; > Ola Liljedahl ; olivier.matz@6wind.com; > chaozhu@linux.vnet.ibm.com; bruce.richardson@intel.com; > konstantin.ananyev@intel.com > Subject: Re: [dpdk-dev] [PATCH 1/2] ring: synchronize the load and store = of > the tail > > -----Original Message----- > > Date: Sat, 27 Oct 2018 17:13:10 +0200 > > From: Thomas Monjalon > > To: Jerin Jacob > > Cc: "Gavin Hu (Arm Technology China)" , > "dev@dpdk.org" > > , Honnappa Nagarahalli > , > > "stable@dpdk.org" , Ola Liljedahl > > , "olivier.matz@6wind.com" > > , "chaozhu@linux.vnet.ibm.com" > > , "bruce.richardson@intel.com" > > , "konstantin.ananyev@intel.com" > > > > Subject: Re: [dpdk-dev] [PATCH 1/2] ring: synchronize the load and > > store of the tail > > > > > > 27/10/2018 17:00, Jerin Jacob: > > > From: Thomas Monjalon > > > > 17/10/2018 08:35, Gavin Hu (Arm Technology China): > > > > > Hi Jerin > > > > > > > > > > As the 1st one of the 3-patch set was not concluded, I submit thi= s 2- > patch series to unblock the merge. > > > > > > > > The thread is totally messed up because: > > > > - there is no cover letter > > > > - some different series (testpmd, i40e and doc) are in the = same > thread > > > > - v4 replies to a different series > > > > - this version should be a v5 but has no number > > > > - this version replies to the v3 > > > > - patchwork still shows v3 and "v5" > > > > - replies from Ola are not quoting previous discussion > > > > > > > > Because of all of this, it is really difficult to follow. > > > > This is probably the reason of the lack of review outside of Arm. > > > > > > > > One more issue: you must Cc the relevant maintainers. > > > > Here: > > > > - Olivier for rte_ring > > > > - Chao for IBM platform > > > > - Bruce and Konstantin for x86 > > > > > > > > Guys, it is really cool to have more Arm developpers in DPDK. > > > > But please consider better formatting your discussions, it is > > > > really important in our contribution workflow. > > > > > > > > I don't know what to do. > > > > I suggest to wait for more feedbacks and integrate it in -rc2. > > > > > > This series has been acked and tested. Sure, if we are looking for > > > some more feedback we can push to -rc2 if not it a good candidate to > > > be selected for -rc1. > > > > It has been acked and tested only for Arm platforms. > > And Olivier, the ring maintainer, was not Cc. > > > > I feel it is not enough. > > Sure, More reviews is already better. But lets keep as -rc2 target. > > > > > > IMPORTANT NOTICE: The contents of this email and any attachments are confid= ential and may also be privileged. If you are not the intended recipient, p= lease notify the sender immediately and do not disclose the contents to any= other person, use it for any purpose, or store or copy the information in = any medium. Thank you.