public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: Keith Owens <kaos@sgi.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [RFC Patch]Use ar.kr2 for smp_processor_id
Date: Thu, 08 Feb 2007 06:55:04 +0000	[thread overview]
Message-ID: <7809.1170917704@kao2.melbourne.sgi.com> (raw)
In-Reply-To: <1170905324.3230.7.camel@linux-znh>

Keith Owens (on Thu, 08 Feb 2007 17:37:54 +1100) wrote:
>Zou Nan hai (on 08 Feb 2007 12:27:31 +0800) wrote:
>>On Thu, 2007-02-08 at 14:04, Keith Owens wrote:
>>> Zou Nan hai (on 08 Feb 2007 11:28:44 +0800) wrote:
>>> >Pin ar.kr2 of each CPU, so that smp_processor_id can use it.
>>> 
>>> Historically ar.k2 has been reserved for debugging purposes, for
>>> example in ivt.S.  Debuggers often need a location that can be used to
>>> track progress, it has to be somewhere that does not rely on TLB
>>> entries and is guaranteed to appear in MCA/INIT records - ar.k2 is
>>> perfect for this.
>>> 
>>  Ok, seems that current kr3 is only used by ia64_itc_printk_clock?
>>> Use Tony's suggestion of testing for a change in ar.k3 (guaranteed to
>>> be unique on every cpu) and caching the corresponding cpu number when
>>> it changes.
>>> 
>>  But why do we even need to cache it? 
>>
>>  It is already in a register if we put it to kr3. 
>>  so smp_processor_id() could be very fast. and later sys_getcpu can
>>also be very fast.
>
>ar.k3 is currently used for the address of the per-cpu data area, which
>speeds up access to all the per-cpu data.  Changing ar.k3 to hold the
>cpu number means an extra array calculation and lookup for every
>per-cpu variable, slowing down the rest of the system.

Correction: ar.k3 contains the physical address of the per-cpu data
area, virtual access to per-cpu data goes via the cpu local TLB and
does not rely on an ar.k<n> variable.  ar.k3 is used in the MCA
assembler handler, see GET_THIS_PADDR in include/asm-ia64/mca_asm.h and
arch/ia64/kernel/mca_asm.S.


  parent reply	other threads:[~2007-02-08  6:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-08  3:28 [RFC Patch]Use ar.kr2 for smp_processor_id Zou Nan hai
2007-02-08  4:27 ` Zou Nan hai
2007-02-08  4:59 ` Zou Nan hai
2007-02-08  5:11 ` Zou Nan hai
2007-02-08  6:04 ` Keith Owens
2007-02-08  6:37 ` Keith Owens
2007-02-08  6:55 ` Keith Owens [this message]
2007-02-08  7:14 ` Zou Nan hai
2007-02-08  7:38 ` Zou Nan hai
2007-02-08  8:28 ` peterc
2007-02-08  8:40 ` Keith Owens
2007-02-08 18:03 ` Luck, Tony
2007-02-08 23:59 ` Keith Owens

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=7809.1170917704@kao2.melbourne.sgi.com \
    --to=kaos@sgi.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox