Re: checkpatch, a patch checking script.

Alan Cox
2007-04-28T10:11:40Z

> > The warning is bogus imho. How do you write recovery code for internal
> > broken code logic?
>
> Yes, it is marginal. But people do very often reach for BUG_ON() where
> they could have at least partly recovered in some fashion - enough for the
> info to hit the logs so we have a better chance of fixing it.

At least one way to handle BUG_ON() type situations more cleanly (for
some anyway) is to fake a hot-unplug/plug event. Thats something that
would be nice to get into the PCI code as it'll let you recover from many
things (even in some cases hardware fails if you go via D3). Does need
some supporting logic (eg for networking to get back with the same config
and online)