All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Hemminger <shemminger@vyatta.com>
To: john stultz <johnstul@us.ibm.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux-kernel@vger.kernel.org
Subject: Re: clocksource changes in 2.6.31 - possible regression
Date: Mon, 17 Aug 2009 11:01:27 -0700	[thread overview]
Message-ID: <20090817110127.40ee5c29@nehalam> (raw)
In-Reply-To: <1250531337.26171.12.camel@work-vm>

On Mon, 17 Aug 2009 10:48:57 -0700
john stultz <johnstul@us.ibm.com> wrote:

> On Mon, 2009-08-17 at 09:03 -0700, Stephen Hemminger wrote:
> > The following commit causes a change for kernels built with HRT but
> > not actually using HRT.  I typically use the generic kernel we ship
> > on test machines, and that kernel has NOHZ and HRT (for power savings/virt
> > and HRT for QoS), but I want to be able to enable TSC as a clock source
> > when doing performance tests with pktgen.
> > 
> > The machine in question is a several year old Opteron box, that
> > normally reports clocksources: acpi_pm jiffies tsc
> > but now with 2.6.31-rc6, it only has acpi_pm.
> 
> I might need to review the patch again, but I believe we just don't
> allow you to switch to non HRT compatible clocksources (like jiffies) if
> we're already in HRT mode (and thus would hang when switched). 
> 
> 
> The behavior you describe where you can't switch to the TSC, may be due
> to the TSC disqualification code marking it as non HRT compatible
> (again, I need to double check). While I'm not sure that's really
> correct, as the TSC is fine for HRT, in this case on your box, the TSC
> has been marked as unstable (likely due to being unsynced on old AMD SMP
> systems). There is a real chance that the timekeeping code on your
> system could see the TSC go backwards, calculate a negative time
> interval, and then end up hanging. 
> 

TSC was alway stable on this box, and worked fine.  There was no
message in log about TSC instability. The change was bisected
down to that one commit.

-- 

  reply	other threads:[~2009-08-17 18:01 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-17 16:03 clocksource changes in 2.6.31 - possible regression Stephen Hemminger
2009-08-17 17:46 ` Thomas Gleixner
2009-08-17 17:48 ` john stultz
2009-08-17 18:01   ` Stephen Hemminger [this message]
2009-08-17 18:15     ` john stultz
2009-08-17 18:27       ` Stephen Hemminger
2009-08-17 18:34         ` Thomas Gleixner
2009-08-17 19:54           ` Stephen Hemminger
2009-08-17 20:04             ` Thomas Gleixner
2009-08-17 20:27               ` Stephen Hemminger
2009-08-17 20:44                 ` Thomas Gleixner
2009-08-17 21:10         ` john stultz
2009-08-17 21:37           ` john stultz
2009-08-17 21:45             ` Stephen Hemminger
2009-08-17 22:23               ` john stultz
2009-08-17 23:02                 ` Stephen Hemminger
2009-08-17 23:17                   ` john stultz
2009-08-17 23:27                     ` Stephen Hemminger
2009-08-17 23:40                       ` [PATCH] make tsc=reliable override boot time stability checks john stultz
2009-08-18  1:39                         ` Alok Kataria
2009-08-19  1:04                           ` john stultz
2009-08-28 19:16                         ` [tip:x86/tsc] x86: Make " tip-bot for john stultz

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=20090817110127.40ee5c29@nehalam \
    --to=shemminger@vyatta.com \
    --cc=akpm@linux-foundation.org \
    --cc=johnstul@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    /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.