From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [RFC PATCH v3 3/3] acpi: apei: Warn when GHES marks correctable errors as "fatal" Date: Wed, 2 May 2018 21:10:30 +0200 Message-ID: <20180502191029.hcvf56xbdna7oi4k@devuan> References: <20180416215903.7318-1-mr.nuke.me@gmail.com> <20180425203957.18224-1-mr.nuke.me@gmail.com> <20180425203957.18224-4-mr.nuke.me@gmail.com> <20180426112057.GB15009@pd.tnic> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180426112057.GB15009@pd.tnic> Sender: linux-kernel-owner@vger.kernel.org To: Borislav Petkov Cc: Alexandru Gagniuc , linux-acpi@vger.kernel.org, linux-edac@vger.kernel.org, "Rafael J. Wysocki" , Len Brown , Tony Luck , Mauro Carvalho Chehab , Robert Moore , Erik Schmauss , Tyler Baicar , Will Deacon , James Morse , Shiju Jose , "Jonathan (Zhixiong) Zhang" , Dongjiu Geng , linux-kernel@vger.kernel.org, devel@acpica.org List-Id: linux-acpi@vger.kernel.org On Thu 2018-04-26 13:20:57, Borislav Petkov wrote: > On Wed, Apr 25, 2018 at 03:39:51PM -0500, Alexandru Gagniuc wrote: > > There seems to be a culture amongst BIOS teams to want to crash the > > OS when an error can't be handled in firmware. Marking GHES errors as > > "fatal" is a very common way to do this. > > > > However, a number of errors reported by GHES may be fatal in the sense > > a device or link is lost, but are not fatal to the system. When there > > is a disagreement with firmware about the handleability of an error, > > print a warning message. > > + > > + if ((sev >= GHES_SEV_PANIC) && (ghes_actual_severity(ghes) < sev)) { > > + pr_warn("FIRMWARE BUG: Firmware sent fatal error that we were able to correct"); > > + pr_warn("BROKEN FIRMWARE: Complain to your hardware vendor"); > > Pasting the same comment from last time since you missed it: > > "No, I don't want any of that crap issuing stuff in dmesg and then people > opening bugs and running around and trying to replace hardware. We want to see warnings. Maybe they can be toned done. We even have dedicated distros for firmware testing. > Good mailing practices for 400: avoid top-posting and trim the reply. Good mailing practices -- limit use of four letter words on public lists. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [RFC,v3,3/3] acpi: apei: Warn when GHES marks correctable errors as "fatal" From: Pavel Machek Message-Id: <20180502191029.hcvf56xbdna7oi4k@devuan> Date: Wed, 2 May 2018 21:10:30 +0200 To: Borislav Petkov Cc: Alexandru Gagniuc , linux-acpi@vger.kernel.org, linux-edac@vger.kernel.org, "Rafael J. Wysocki" , Len Brown , Tony Luck , Mauro Carvalho Chehab , Robert Moore , Erik Schmauss , Tyler Baicar , Will Deacon , James Morse , Shiju Jose , "Jonathan (Zhixiong) Zhang" , Dongjiu Geng , linux-kernel@vger.kernel.org, devel@acpica.org List-ID: T24gVGh1IDIwMTgtMDQtMjYgMTM6MjA6NTcsIEJvcmlzbGF2IFBldGtvdiB3cm90ZToKPiBPbiBX ZWQsIEFwciAyNSwgMjAxOCBhdCAwMzozOTo1MVBNIC0wNTAwLCBBbGV4YW5kcnUgR2Fnbml1YyB3 cm90ZToKPiA+IFRoZXJlIHNlZW1zIHRvIGJlIGEgY3VsdHVyZSBhbW9uZ3N0IEJJT1MgdGVhbXMg dG8gd2FudCB0byBjcmFzaCB0aGUKPiA+IE9TIHdoZW4gYW4gZXJyb3IgY2FuJ3QgYmUgaGFuZGxl ZCBpbiBmaXJtd2FyZS4gTWFya2luZyBHSEVTIGVycm9ycyBhcwo+ID4gImZhdGFsIiBpcyBhIHZl cnkgY29tbW9uIHdheSB0byBkbyB0aGlzLgo+ID4gCj4gPiBIb3dldmVyLCBhIG51bWJlciBvZiBl cnJvcnMgcmVwb3J0ZWQgYnkgR0hFUyBtYXkgYmUgZmF0YWwgaW4gdGhlIHNlbnNlCj4gPiBhIGRl dmljZSBvciBsaW5rIGlzIGxvc3QsIGJ1dCBhcmUgbm90IGZhdGFsIHRvIHRoZSBzeXN0ZW0uIFdo ZW4gdGhlcmUKPiA+IGlzIGEgZGlzYWdyZWVtZW50IHdpdGggZmlybXdhcmUgYWJvdXQgdGhlIGhh bmRsZWFiaWxpdHkgb2YgYW4gZXJyb3IsCj4gPiBwcmludCBhIHdhcm5pbmcgbWVzc2FnZS4KCgo+ ID4gKwo+ID4gKwlpZiAoKHNldiA+PSBHSEVTX1NFVl9QQU5JQykgJiYgKGdoZXNfYWN0dWFsX3Nl dmVyaXR5KGdoZXMpIDwgc2V2KSkgewo+ID4gKwkJcHJfd2FybigiRklSTVdBUkUgQlVHOiBGaXJt d2FyZSBzZW50IGZhdGFsIGVycm9yIHRoYXQgd2Ugd2VyZSBhYmxlIHRvIGNvcnJlY3QiKTsKPiA+ ICsJCXByX3dhcm4oIkJST0tFTiBGSVJNV0FSRTogQ29tcGxhaW4gdG8geW91ciBoYXJkd2FyZSB2 ZW5kb3IiKTsKPiAKPiBQYXN0aW5nIHRoZSBzYW1lIGNvbW1lbnQgZnJvbSBsYXN0IHRpbWUgc2lu Y2UgeW91IG1pc3NlZCBpdDoKPiAKPiAiTm8sIEkgZG9uJ3Qgd2FudCBhbnkgb2YgdGhhdCBjcmFw IGlzc3Vpbmcgc3R1ZmYgaW4gZG1lc2cgYW5kIHRoZW4gcGVvcGxlCj4gb3BlbmluZyBidWdzIGFu ZCBydW5uaW5nIGFyb3VuZCBhbmQgdHJ5aW5nIHRvIHJlcGxhY2UgaGFyZHdhcmUuCgpXZSB3YW50 IHRvIHNlZSB3YXJuaW5ncy4gTWF5YmUgdGhleSBjYW4gYmUgdG9uZWQgZG9uZS4gV2UgZXZlbiBo YXZlCmRlZGljYXRlZCBkaXN0cm9zIGZvciBmaXJtd2FyZSB0ZXN0aW5nLgoKPiBHb29kIG1haWxp bmcgcHJhY3RpY2VzIGZvciA0MDA6IGF2b2lkIHRvcC1wb3N0aW5nIGFuZCB0cmltIHRoZSByZXBs eS4KCkdvb2QgbWFpbGluZyBwcmFjdGljZXMgLS0gbGltaXQgdXNlIG9mIGZvdXIgbGV0dGVyIHdv cmRzIG9uIHB1YmxpYyBsaXN0cy4KCiAgICAgCSAgICAgCSAgICAgICAJICAJICAgICAgIAkgICAg CSAgIAkgICAgCSAgIFBhdmVsCg==