public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: "Chen, Kenneth W" <kenneth.w.chen@intel.com>
To: linux-ia64@vger.kernel.org
Subject: RE: ia64 printk_clock()
Date: Mon, 06 Feb 2006 18:34:08 +0000	[thread overview]
Message-ID: <200602061834.k16IY9g22639@unix-os.sc.intel.com> (raw)
In-Reply-To: <20060202204422.GA27082@sgi.com>

Luck, Tony wrote on Monday, February 06, 2006 10:14 AM
> On Fri, Feb 03, 2006 at 09:43:29AM -0800, Chen, Kenneth W wrote:
> > Why don't we look at ar.k3, it contains per_cpu base address.
> > If it has a none zero value, then it is initialized by
> > per_cpu_init and is safe to call sched_clock.
> 
> We also need to check whether ar.itc is synchronized on different
> cpus ... if it isn't, then sched_clock() is useless.  Here's a
> simple extension of your patch that does this, falling back to using
> a jiffie based clock (which means you'd only have HZ resolution on
> systems where there is drift).
> 
> If that isn't good enough, then I'd be happy to see a patch that
> does a lockless interpolation.
> 
> diff --git a/arch/ia64/kernel/head.S b/arch/ia64/kernel/head.S
> index fbc7ea3..99cfef8 100644
> --- a/arch/ia64/kernel/head.S
> +++ b/arch/ia64/kernel/head.S
> @@ -352,6 +352,7 @@ start_ap:
>  	mov ar.rsc=0		// place RSE in enforced lazy mode
>  	;;
>  	loadrs			// clear the dirty partition
> +	mov ar.k3=r0		// clear physical per-CPU base
>  	;;
>  	mov ar.bspstore=r2	// establish the new RSE stack
>  	;;

Sorry, I'm the one who is spreading ... can we change this hunk
to use defined kr constant?

diff --git a/arch/ia64/kernel/head.S b/arch/ia64/kernel/head.S
index fbc7ea3..99cfef8 100644
--- a/arch/ia64/kernel/head.S
+++ b/arch/ia64/kernel/head.S
@@ -352,6 +352,7 @@ start_ap:
 	mov ar.rsc=0		// place RSE in enforced lazy mode
 	;;
 	loadrs			// clear the dirty partition
+	mov IA64_KR(PER_CPU_DATA)=r0
 	;;
 	mov ar.bspstore=r2	// establish the new RSE stack
 	;;



  parent reply	other threads:[~2006-02-06 18:34 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-02 20:44 ia64 printk_clock() Jack Steiner
2006-02-02 21:22 ` Dean Roe
2006-02-02 21:46 ` Luck, Tony
2006-02-02 21:50 ` Jack Steiner
2006-02-02 22:29 ` Luck, Tony
2006-02-02 22:52 ` Chen, Kenneth W
2006-02-03 17:43 ` Chen, Kenneth W
2006-02-03 18:04 ` Luck, Tony
2006-02-03 18:30 ` Chen, Kenneth W
2006-02-03 18:43 ` Luck, Tony
2006-02-03 18:55 ` Chen, Kenneth W
2006-02-04  0:16 ` Keith Owens
2006-02-04  0:28 ` Chen, Kenneth W
2006-02-04  0:32 ` Luck, Tony
2006-02-04  0:36 ` Keith Owens
2006-02-06 18:14 ` Luck, Tony
2006-02-06 18:34 ` Chen, Kenneth W [this message]
2006-02-06 20:40 ` Magenheimer, Dan (HP Labs Fort Collins)
2006-02-06 21:37 ` Chen, Kenneth W
2006-02-07 11:11 ` Jes Sorensen
2006-02-08  0:05 ` Luck, Tony
2006-02-08  6:29 ` Jes Sorensen
2006-02-08 18:18 ` Luck, Tony

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=200602061834.k16IY9g22639@unix-os.sc.intel.com \
    --to=kenneth.w.chen@intel.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