Re: [PATCH] Add the word 'Warning' in check_nmi_watchdog() output

On Wed, 21 Nov 2007, Don Zickus wrote:

> On Wed, Nov 21, 2007 at 10:34:59AM -0800, Arjan van de Ven wrote:
> > On Wed, 21 Nov 2007 12:14:20 -0500
> > Don Zickus <[email protected]> wrote:
> > 
> > > Our automated test suite looks for keywords like error, fail, warning
> > > in the boot log.  In the case when the nmi watchdog is determined to
> > > be stuck in check_nmi_watchdog(), none of those keywords are
> > > displayed.
> > 
> > 
> > please make it at least consistent with the other kernel level bad
> > things, and use "WARNING:"....
> 
> Good point.  I'll repost.

Before you repost, please add an appropriate KERN_WARN prefix to the
printk and run your patch through scripts/checkpatch.pl.

	tglx
-
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/

---

• References:
  • [PATCH] Add the word 'Warning' in check_nmi_watchdog() output
    • From: Don Zickus <[email protected]>
  • Re: [PATCH] Add the word 'Warning' in check_nmi_watchdog() output
    • From: Arjan van de Ven <[email protected]>
  • Re: [PATCH] Add the word 'Warning' in check_nmi_watchdog() output
    • From: Don Zickus <[email protected]>

• Prev by Date: Re: spreading the heat?
• Next by Date: Re: [PATCH]loop cleanup in fs/namespace.c - repost
• Previous by thread: Re: [PATCH] Add the word 'Warning' in check_nmi_watchdog() output
• Next by thread: [PATCH]loop cleanup in fs/namespace.c - repost
• 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]