All of lore.kernel.org
 help / color / mirror / Atom feed
From: Don Zickus <dzickus@redhat.com>
To: Ingo Molnar <mingo@kernel.org>
Cc: akpm@linux-foundation.org, mm-commits@vger.kernel.org,
	joe@perches.com, mingo@elte.hu, nzimmer@sgi.com,
	linux-kernel@vger.kernel.org,
	Peter Zijlstra <a.p.zijlstra@chello.nl>
Subject: Re: + nmi-watchdog-quiet-down-the-boot-messages.patch added to -mm tree
Date: Mon, 11 Jun 2012 09:18:04 -0400	[thread overview]
Message-ID: <20120611131804.GX32472@redhat.com> (raw)
In-Reply-To: <20120611084744.GH31556@gmail.com>

On Mon, Jun 11, 2012 at 10:47:44AM +0200, Ingo Molnar wrote:
> >     NMI watchdog enabled, takes one hw-pmu counter.
> >     Booting Node   0, Processors  #1
> >     NMI watchdog enabled, takes one hw-pmu counter.
> >      #2
> >     NMI watchdog enabled, takes one hw-pmu counter.
> >      #3 Ok.
> >     NMI watchdog enabled, takes one hw-pmu counter.
> >     Brought up 4 CPUs
> >     Total of 4 processors activated (22607.24 BogoMIPS).
> 
> the problem really was that it didn't print which CPU it 
> referred to - it just repeated the same message over and over 
> again.
> 
> If it said:
> 
>      NMI watchdog enabled on CPU#0, takes one hw-pmu counter.
> 
> it would be far less confusing.

Sure.  The problem SGI has with 4096 cpus is the volume.  Specifying which
cpu the message belongs to, I do not think helps with this problem.

> 
> This:
> 
> > After the change, it is simplified to:
> > 
> >     ..TIMER: vector=0x30 apic1=0 pin1=2 apic2=-1 pin2=-1
> >     CPU0: Intel(R) Core(TM)2 Quad CPU    Q9550  @ 2.83GHz stepping 0a
> >     Performance Events: PEBS fmt0+, Core2 events, Intel PMU driver.
> >     ... version:                2
> >     ... bit width:              40
> >     ... generic registers:      2
> >     ... value mask:             000000ffffffffff
> >     ... max period:             000000007fffffff
> >     ... fixed-purpose events:   3
> >     ... event mask:             0000000700000003
> >     NMI watchdog enabled, takes one hw-pmu counter.
> >     Booting Node   0, Processors  #1 #2 #3 Ok.
> >     Brought up 4 CPUs
> 
> Is fine too but should be clarified to something like:
> 
>      NMI watchdog enabled on all CPUs, permanently consumes one hw-PMU counter
> 
> (Please note the capitalization of 'PMU' as well.)

Ok.  I can modify that.

Cheers,
Don

      reply	other threads:[~2012-06-11 13:18 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-08 21:37 + nmi-watchdog-quiet-down-the-boot-messages.patch added to -mm tree akpm
2012-06-11  8:47 ` Ingo Molnar
2012-06-11 13:18   ` Don Zickus [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120611131804.GX32472@redhat.com \
    --to=dzickus@redhat.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mingo@kernel.org \
    --cc=mm-commits@vger.kernel.org \
    --cc=nzimmer@sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.