On Sun, 15 Nov 2009, Matt Zimmerman wrote: > On Sun, Nov 15, 2009 at 05:13:53PM -0000, Henrique de Moraes Holschuh wrote: > > Oh yes, I am going to change kernel code because your tool scared your > > users senseless for a LOG_WARN message. Forget it. > > I don't know what you're referring to here. Did someone ask you to change > kernel code?
Yes. A snide comment promoting a kernel change to get rid of "WARNING" in the printks in the thinkpad-acpi driver, to avoid triggering the tool, instead of fixing the tool (and whatever was causing it to be triggered). Which is, obviously, the exactly wrong way to go about it, and the way it was delivered pissed me off. I don't have a web interface handy right now, or I'd tell you the comment number exactly. > In my view, the way to fix this is to enable apport or kerneloops to > differentiate between serious and non-serious events, and only trigger an > apport problem report for a serious one. > > Is there anything about that which you find problematic? No, I don't think what you propose problematic: that's the correct way to fix the issue. Obviously KERN_WARN and higher-urgency messages need to be shown to the user, it is just a matter of using the right language for each severity level (and also of handling WARN_ON and BUG_ON output blocks). -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh -- Kernel oops dialog is inconsistent with kerneloops semantics https://bugs.launchpad.net/bugs/346303 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
