linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: john stultz <johnstul@us.ibm.com>
To: Mattia Dongili <malattia@linux.it>
Cc: Andrew Morton <akpm@osdl.org>, linux-kernel@vger.kernel.org
Subject: Re: first bisection results in -mm3 [was: Re: 2.6.15-mm2: reiser3 oops on suspend and more (bonus oops shot!)]
Date: Mon, 16 Jan 2006 12:16:21 -0800	[thread overview]
Message-ID: <1137442582.27699.12.camel@cog.beaverton.ibm.com> (raw)
In-Reply-To: <20060114120816.GA3554@inferi.kami.home>

On Sat, 2006-01-14 at 13:08 +0100, Mattia Dongili wrote:
> On Thu, Jan 12, 2006 at 03:26:01PM -0800, john stultz wrote:
> > On Thu, 2006-01-12 at 23:08 +0100, Mattia Dongili wrote:
> > > [cleaned up some Cc as this is not interesting to all MLs]
> > > first bisection spotted the cause of the stalls at boot (happening while
> > > starting portmap and after usb-storage scan):
> > > 
> > > time-clocksource-infrastructure.patch
> > > time-generic-timekeeping-infrastructure.patch
> > > time-i386-conversion-part-1-move-timer_pitc-to-i8253c.patch
> > > time-i386-conversion-part-2-rework-tsc-support.patch
> > > time-i386-conversion-part-3-enable-generic-timekeeping.patch
> > > time-i386-conversion-part-4-remove-old-timer_opts-code.patch
> > > time-i386-conversion-part-5-acpi-pm-variable-renaming-and-config-change.patch
> > > time-i386-clocksource-drivers.patch
> > > time-fix-cpu-frequency-detection.patch
> > > 
> > > Cc-ed john stultz
> > > 
> > > actually git bisect[1] pointed to time-fix-cpu-frequency-detection.patch
> > > but it's clearly wrong. Reverting all the above patches (I suppose they
> > > are somewhat related) fixes the stalls I experience. I can test
> > > corrections if necessary.
[snip]
> > Looking at the log here:
> > http://oioio.altervista.org/linux/boot-2.6.15-mm2.3
> > 
> > I'm curious if you're getting cpufreq effects during interval while the
> > TSC is being used as a clocksource before we switch to using the acpi_pm
> > clocksource.
> 
> What should I expect? I didn't notice anything in particular and
> actually the box stays alive for just a few minutes, then reiserfs
> explodes so I have no chance to notice anything in the long run.
> 
> > After the system boots up, does it keep accurate time? Time doesn't
> > obviously move too fast or to slow compared to a watch?
> 
> yes, during the short time it stays alive there's no difference between
> my watch and my laptop.

Ok, that's good. It means the ACPI PM clocksource is doing the right
thing.


> > Few things to try (independently):
> > 1. Does booting w/ idle=poll change the behavior?
> 
> yes, no more stalls

Ok, this points to the TSC is changing frequency (likely due to C3
halting). 

> > 2. Does booting w/ clocksource=jiffies change the behavior?
> 
> yes, same as above

Ok, good, interrupts are getting there at the right frequency.

> > 3. After booting up, run: 
> >    echo tsc > /sys/devices/system/clocksource/clocksource0/current_clocksource
> >    And check that the system keeps accurate time.
> 
> didn't try as there seems to be no problem in timekeeping

Well, it would have re-inforced the TSC being the issue, but I'm fairly
confident that that is the case.


My theory: The stalls are due to the TSC frequency not being consistent
for the small window at boot between when it is installed and when the
ACPI PM clocksource is installed.

I'll try to narrow that window down a bit and see if that doesn't
resolve the issue.

thanks
-john



  reply	other threads:[~2006-01-16 20:16 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-10 23:55 2.6.15-mm2: reiser3 oops on suspend and more (bonus oops shot!) Mattia Dongili
2006-01-11  1:00 ` Andrew Morton
2006-01-11 10:00   ` Pavel Machek
2006-01-11 11:24     ` Mattia Dongili
2006-01-11 18:40     ` Mattia Dongili
2006-01-12 22:08       ` first bisection results in -mm3 [was: Re: 2.6.15-mm2: reiser3 oops on suspend and more (bonus oops shot!)] Mattia Dongili
2006-01-12 23:26         ` john stultz
2006-01-14 12:08           ` Mattia Dongili
2006-01-16 20:16             ` john stultz [this message]
2006-01-16 20:40               ` Mattia Dongili
2006-01-16 21:42                 ` john stultz
2006-01-16 22:19                   ` Andi Kleen
2006-01-16 22:41                     ` john stultz
2006-01-17 16:43                   ` Mattia Dongili
2006-01-17 16:45                   ` Mattia Dongili
2006-01-17  0:49                 ` john stultz
2006-01-17 17:49                   ` Mattia Dongili
2006-01-17 19:11                     ` john stultz
2006-01-17 22:49                       ` Mattia Dongili
2006-01-18 10:47                         ` Mattia Dongili
2006-01-24 22:27                           ` john stultz
2006-01-24 23:04                             ` Mattia Dongili
2006-01-24 23:48                               ` john stultz
2006-01-25  0:12                                 ` john stultz
2006-01-25 18:12                                   ` Mattia Dongili
2006-01-11 10:02   ` 2.6.15-mm2: reiser3 oops on suspend and more (bonus oops shot!) Pavel Machek
2006-01-11 14:27   ` Mattia Dongili
2006-01-15 22:14   ` 2.6.15-mm3 bisection: git-xfs.patch makes reiserfs oops Mattia Dongili
2006-01-15 22:48     ` Nathan Scott
2006-01-15 23:22       ` Mattia Dongili
2006-01-16  9:29       ` Hans Reiser
2006-01-16 11:48         ` Christoph Hellwig
2006-01-16 15:50           ` Mattia Dongili
2006-01-17  3:27           ` Hans Reiser
2006-01-16 11:51         ` Mattia Dongili

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=1137442582.27699.12.camel@cog.beaverton.ibm.com \
    --to=johnstul@us.ibm.com \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=malattia@linux.it \
    /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;
as well as URLs for NNTP newsgroup(s).