[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200522215145.GC11244@42.do-not-panic.com>
Date: Fri, 22 May 2020 21:51:45 +0000
From: Luis Chamberlain <mcgrof@...nel.org>
To: Johannes Berg <johannes@...solutions.net>
Cc: Jakub Kicinski <kuba@...nel.org>, derosier@...il.com,
greearb@...delatech.com, jeyu@...nel.org,
akpm@...ux-foundation.org, arnd@...db.de, rostedt@...dmis.org,
mingo@...hat.com, aquini@...hat.com, cai@....pw, dyoung@...hat.com,
bhe@...hat.com, peterz@...radead.org, tglx@...utronix.de,
gpiccoli@...onical.com, pmladek@...e.com, tiwai@...e.de,
schlad@...e.de, andriy.shevchenko@...ux.intel.com,
keescook@...omium.org, daniel.vetter@...ll.ch, will@...nel.org,
mchehab+samsung@...nel.org, kvalo@...eaurora.org,
davem@...emloft.net, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-wireless@...r.kernel.org,
ath10k@...ts.infradead.org, jiri@...nulli.us,
briannorris@...omium.org
Subject: Re: [RFC 1/2] devlink: add simple fw crash helpers
On Fri, May 22, 2020 at 10:46:07PM +0200, Johannes Berg wrote:
> FWIW, I still completely disagree on that taint. You (Luis) obviously
> have been running into a bug in that driver, I doubt the firmware
> actually managed to wedge the hardware.
This hasn't happened just once, its happed many times sporadically now,
once a week or two weeks I'd say. And the system isn't being moved
around.
> But even if it did, that's still not really a kernel taint. The kernel
> itself isn't in any way affected by this.
Of course it is, a full reboot is required.
> Yes, the system is in a weird state now. But that's *not* equivalent to
> "kernel tainted".
Requiring a full reboot is a dire situation to be in, and loosing
connectivity to the point this is not recoverable likewise.
You guys are making out a taint to be the end of the world. We have a
taint even for a kernel warning, and as others have mentioned mac80211
already produces these.
What exactly is the opposition to a taint to clarify that a device
firmware has crashed and your system requires a full reboot?
Luis
Powered by blists - more mailing lists