From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Zheng, Lv" Subject: RE: [RFC PATCH 5/5] GHES: Make NMI handler have a single reader Date: Tue, 28 Apr 2015 00:44:12 +0000 Message-ID: <1AE640813FDE7649BE1B193DEA596E880270F55F@SHSMSX101.ccr.corp.intel.com> References: <1427448178-20689-1-git-send-email-bp@alien8.de> <1427448178-20689-6-git-send-email-bp@alien8.de> <1AE640813FDE7649BE1B193DEA596E880270F2B2@SHSMSX101.ccr.corp.intel.com> <20150427084631.GA6774@pd.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com ([134.134.136.65]:29586 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752597AbbD1AoS (ORCPT ); Mon, 27 Apr 2015 20:44:18 -0400 In-Reply-To: <20150427084631.GA6774@pd.tnic> Content-Language: en-US Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Borislav Petkov Cc: linux-edac , Jiri Kosina , Borislav Petkov , "Rafael J. Wysocki" , Len Brown , "Luck, Tony" , Tomasz Nowicki , "Chen, Gong" , Wolfram Sang , Naoya Horiguchi , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" SGksDQoNCj4gRnJvbTogQm9yaXNsYXYgUGV0a292IFttYWlsdG86YnBAYWxpZW44LmRlXQ0KPiBT ZW50OiBNb25kYXksIEFwcmlsIDI3LCAyMDE1IDQ6NDcgUE0NCj4gDQo+IE9uIE1vbiwgQXByIDI3 LCAyMDE1IGF0IDAzOjE2OjAwQU0gKzAwMDAsIFpoZW5nLCBMdiB3cm90ZToNCj4gPiA+IEBAIC04 NDAsNyArODQwLDkgQEAgc3RhdGljIGludCBnaGVzX25vdGlmeV9ubWkodW5zaWduZWQgaW50IGNt ZCwgc3RydWN0IHB0X3JlZ3MgKnJlZ3MpDQo+ID4gPiAgCXN0cnVjdCBnaGVzICpnaGVzOw0KPiA+ ID4gIAlpbnQgc2V2LCByZXQgPSBOTUlfRE9ORTsNCj4gPiA+DQo+ID4gPiAtCXJhd19zcGluX2xv Y2soJmdoZXNfbm1pX2xvY2spOw0KPiA+ID4gKwlpZiAoIWF0b21pY19hZGRfdW5sZXNzKCZnaGVz X2luX25taSwgMSwgMSkpDQo+ID4gPiArCQlyZXR1cm4gcmV0Ow0KPiA+ID4gKw0KPiA+DQo+ID4g SnVzdCBhIHNpbXBsZSBxdWVzdGlvbi4NCj4gPiBXaHkgbm90IGp1c3QgdXNpbmcgY21weGNoZyBo ZXJlIGluc3RlYWQgb2YgYXRvbWljX2FkZF91bmxlc3Mgc28gdGhhdCBubyBhdG9taWNfZGVjIHdp bGwgYmUgbmVlZGVkLg0KPiANCj4gV2hhdCBkbyB5b3UgdGhpbmsgYXRvbWljX2FkZF91bmxlc3Mg ZW5kcyB1cCBkb2luZzoNCj4gDQo+ICNBUFANCj4gIyAxNzcgIi4vYXJjaC94ODYvaW5jbHVkZS9h c20vYXRvbWljLmgiIDENCj4gCS5wdXNoc2VjdGlvbiAuc21wX2xvY2tzLCJhIg0KPiAuYmFsaWdu IDQNCj4gLmxvbmcgNjcxZiAtIC4NCj4gLnBvcHNlY3Rpb24NCj4gNjcxOg0KPiAJbG9jazsgY21w eGNoZ2wgJWVkeCxnaGVzX2luX25taSglcmlwKQkjIEQuMzcwNTYsIE1FTVsodm9sYXRpbGUgdTMy ICopJmdoZXNfaW5fbm1pXQ0KPiAjIDAgIiIgMg0KPiAjTk9fQVBQDQo+IA0KPiBBbmQgeW91IG5l ZWQgdG8gYXRvbWljX2RlYygpIHNvIHRoYXQgYW5vdGhlciByZWFkZXIgY2FuIGVudGVyLCBpLmUu IGhvdw0KPiB0aGUgZXhjbHVzaW9uIHByaW1pdGl2ZSB3b3Jrcy4NCj4gDQo+IE9yIGRpZCB5b3Ug aGF2ZSBzb21ldGhpbmcgZWxzZSBpbiBtaW5kPw0KDQpNeSBtaXN0YWtlLg0KSSBtZWFuIGNtcHhj aGcoKSBhbmQgeGNoZygpIChvciBhdG9taWNfY21weGNoZygpIGFuZCBhdG9taWNfeGNoZygpKSBw YWlyIGhlcmUsIHNvIG5vdGhpbmcgY2FuIGJlIHJlZHVjZWQuDQoNCkJ1dCBJTU8sIGF0b21pY19h ZGRfdW5sZXNzKCkgaXMgaW1wbGVtZW50ZWQgdmlhIGNtcHhjaGcgb24gbWFueSBhcmNoaXRlY3R1 cmVzLg0KQW5kIGl0IG1pZ2h0IGJlIGJldHRlciB0byB1c2UgaXQgZGlyZWN0bHkgaGVyZSB3aGlj aCBpcyBhIGJpdCBmYXN0ZXIgYXMgeW91IGFjdHVhbGx5IG9ubHkgbmVlZCBvbmUgdmFsdWUgc3dp dGNoIGhlcmUuDQoNClRoYW5rcyBhbmQgYmVzdCByZWdhcmRzDQotTHYNCg0KDQo+IA0KPiAtLQ0K PiBSZWdhcmRzL0dydXNzLA0KPiAgICAgQm9yaXMuDQo+IA0KPiBFQ08gdGlwICMxMDE6IFRyaW0g eW91ciBtYWlscyB3aGVuIHlvdSByZXBseS4NCj4gLS0NCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932238AbbD1AoV (ORCPT ); Mon, 27 Apr 2015 20:44:21 -0400 Received: from mga03.intel.com ([134.134.136.65]:29586 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752597AbbD1AoS (ORCPT ); Mon, 27 Apr 2015 20:44:18 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,660,1422950400"; d="scan'208";a="716770616" From: "Zheng, Lv" To: Borislav Petkov CC: linux-edac , Jiri Kosina , Borislav Petkov , "Rafael J. Wysocki" , "Len Brown" , "Luck, Tony" , Tomasz Nowicki , "Chen, Gong" , Wolfram Sang , Naoya Horiguchi , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [RFC PATCH 5/5] GHES: Make NMI handler have a single reader Thread-Topic: [RFC PATCH 5/5] GHES: Make NMI handler have a single reader Thread-Index: AQHQaG/ro4UZP9zjdEyK4aA7ppX1rp1gX79Q///WlYCAAY9TgA== Date: Tue, 28 Apr 2015 00:44:12 +0000 Message-ID: <1AE640813FDE7649BE1B193DEA596E880270F55F@SHSMSX101.ccr.corp.intel.com> References: <1427448178-20689-1-git-send-email-bp@alien8.de> <1427448178-20689-6-git-send-email-bp@alien8.de> <1AE640813FDE7649BE1B193DEA596E880270F2B2@SHSMSX101.ccr.corp.intel.com> <20150427084631.GA6774@pd.tnic> In-Reply-To: <20150427084631.GA6774@pd.tnic> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id t3S0iQ9M016337 Hi, > From: Borislav Petkov [mailto:bp@alien8.de] > Sent: Monday, April 27, 2015 4:47 PM > > On Mon, Apr 27, 2015 at 03:16:00AM +0000, Zheng, Lv wrote: > > > @@ -840,7 +840,9 @@ static int ghes_notify_nmi(unsigned int cmd, struct pt_regs *regs) > > > struct ghes *ghes; > > > int sev, ret = NMI_DONE; > > > > > > - raw_spin_lock(&ghes_nmi_lock); > > > + if (!atomic_add_unless(&ghes_in_nmi, 1, 1)) > > > + return ret; > > > + > > > > Just a simple question. > > Why not just using cmpxchg here instead of atomic_add_unless so that no atomic_dec will be needed. > > What do you think atomic_add_unless ends up doing: > > #APP > # 177 "./arch/x86/include/asm/atomic.h" 1 > .pushsection .smp_locks,"a" > .balign 4 > .long 671f - . > .popsection > 671: > lock; cmpxchgl %edx,ghes_in_nmi(%rip) # D.37056, MEM[(volatile u32 *)&ghes_in_nmi] > # 0 "" 2 > #NO_APP > > And you need to atomic_dec() so that another reader can enter, i.e. how > the exclusion primitive works. > > Or did you have something else in mind? My mistake. I mean cmpxchg() and xchg() (or atomic_cmpxchg() and atomic_xchg()) pair here, so nothing can be reduced. But IMO, atomic_add_unless() is implemented via cmpxchg on many architectures. And it might be better to use it directly here which is a bit faster as you actually only need one value switch here. Thanks and best regards -Lv > > -- > Regards/Gruss, > Boris. > > ECO tip #101: Trim your mails when you reply. > -- {.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I