On Mon, Apr 23, 2018 at 11:19:25PM -0500, Alex G. wrote: > That tells you what FFS said about the error. I betcha those status and command values have a human-readable counterparts. Btw, what do you abbreviate with "FFS"? > It's immediately obvious if there's a glaring FFS bug and if we get bogus > data. If you distrust firmware as much as I do, then you will find great > value in having such info in the logs. It's probably not too useful to a > casual user, but then neither is a majority of the system log. No no, you're missing the point - I *want* all data in the error log which helps debug a hardware issue. I just want it humanly readable so that I don't have to jot down the values and go scour the manuals to map what it actually means. > You're missing the timing and assuming you will get the hotplug interrupt. > In this example, you have 22ms between the link down and presence detect > state change. This is a fairly fast removal. > > Hotplug dependencies aside (you can have the kernel run without PCIe hotplug > support), I don't think you want to just linger in NMI for dozens of > milliseconds waiting for presence detect confirmation. No, I don't mean that. I mean something like deferred processing: you get an error, you notice it is a device which supports physical removal so you exit the NMI handler and process the error in normal, process context which allows you to query the device and say, "Hey device, are you still there?" If it is not, you drop all the hw I/O errors reported for it. Hmmm? -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html