All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephane Eranian <eranian@hpl.hp.com>
To: linux-ia64@vger.kernel.org
Subject: Re: HP_SIM with performance monitoring
Date: Tue, 26 Aug 2003 18:16:49 +0000	[thread overview]
Message-ID: <marc-linux-ia64-106192195729229@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-106186435806640@msgid-missing>

Ian,

On Tue, Aug 26, 2003 at 12:18:46PM +1000, Ian Wienand wrote:
> 
> I noticed that when trying to boot 2.6.0-test4 in the simulator with
> perfmon turned on it ended up oopsing with a "General Exception: IA-64
> Reserved Register/Field fault" in pfm_init_percpu() when it tried to
> zero out the PMD registers.
> 

That does not surprise me. The simulator does have some PMD and PMC registers
but they do not count anything.

> I have no idea if the simulator implements perfmon support, but
> noticed that there are PMC and PMD registers in the register window.
> Am I right in saying the simulator has only 8, skipping odd registers,
> where mckinley has 15 PMC and 17 PMD registers?

Ski seems to have 16 PMCS and 16 PMDS implemented.

> 
> I can think of a number of things to fix it but am not sure which is
> correct -- the GENERIC definitions look kind of correct; maybe
> CONFIG_PERFMON should be turned off for HP_SIM; or create a dummy
> PMC/PMD register definition (attached).  Any enlightenment
> appreciated.

Yes, something along the lines of your patch would be a clean way of fixing
the problem. However, I would do something similar to what is in 
perfmon_generic.h. Simply define the PMDS as counters (PFM_REG_COUNTING). 
It will not hurt because they actually do not count anything. And create 
the PMC table as counter (PFM_REG_COUNTING).

Thanks for spotting this problem.

--
-Stephane

  reply	other threads:[~2003-08-26 18:16 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-08-26  2:18 HP_SIM with performance monitoring Ian Wienand
2003-08-26 18:16 ` Stephane Eranian [this message]
2003-08-27  1:17 ` Ian Wienand

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=marc-linux-ia64-106192195729229@msgid-missing \
    --to=eranian@hpl.hp.com \
    --cc=linux-ia64@vger.kernel.org \
    /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.