Re: checkpatch, a patch checking script.

On Sat, Apr 28, 2007 at 11:15:04AM +0100, Alan Cox wrote:
> > > 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

That would have a high risk of deadlock on some lost lock.

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: checkpatch, a patch checking script.
    • From: Alan Cox <[email protected]>

• References:
  • Re: [PATCH 0/9] Kconfig: cleanup s390 v2.
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH 0/9] Kconfig: cleanup s390 v2.
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH 0/9] Kconfig: cleanup s390 v2.
    • From: Dave Jones <[email protected]>
  • Re: [PATCH 0/9] Kconfig: cleanup s390 v2.
    • From: Randy Dunlap <[email protected]>
  • Re: [PATCH 0/9] Kconfig: cleanup s390 v2.
    • From: Andrew Morton <[email protected]>
  • checkpatch, a patch checking script.
    • From: Dave Jones <[email protected]>
  • Re: checkpatch, a patch checking script.
    • From: Andrew Morton <[email protected]>
  • Re: checkpatch, a patch checking script.
    • From: Andi Kleen <[email protected]>
  • Re: checkpatch, a patch checking script.
    • From: Andrew Morton <[email protected]>
  • Re: checkpatch, a patch checking script.
    • From: Alan Cox <[email protected]>

• Prev by Date: entropy of /dev/random vs. openssl rand
• Next by Date: Re: [Bugme-new] [Bug 8378] New: Averatec 3156X laptop doesn't reboot with kernels > 2.6.13.5 (responsible commit found)
• Previous by thread: Re: checkpatch, a patch checking script.
• Next by thread: Re: checkpatch, a patch checking script.
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]