From: Linus Torvalds <torvalds@linux-foundation.org>
To: Michal Piotrowski <michal.k.k.piotrowski@gmail.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Pavel Machek <pavel@ucw.cz>, "Rafael J. Wysocki" <rjw@sisk.pl>,
Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: Linux v2.6.21-rc3
Date: Wed, 7 Mar 2007 08:34:18 -0800 (PST) [thread overview]
Message-ID: <Pine.LNX.4.64.0703070827490.5963@woody.linux-foundation.org> (raw)
In-Reply-To: <45EEB66C.9090901@googlemail.com>
[ Ingo and Thomas added to Cc, because I think this is them.. ]
Ingo, I think this came in during commit 95492e4646, "x86: rewrite SMP TSC
sync code".
(Leaving the original message quoted in full for Ingo and Thomas, sorry
for the waste of bandwidth)
Linus
---
On Wed, 7 Mar 2007, Michal Piotrowski wrote:
>
> I get this while
> echo shutdown > /sys/power/disk; echo disk > /sys/power/state
>
> BUG: using smp_processor_id() in preemptible [00000001] code: swsusp_shutdown/3359
> caller is check_tsc_sync_source+0x1b/0xef
> [<c010503d>] show_trace_log_lvl+0x1a/0x2f
> [<c0105724>] show_trace+0x12/0x14
> [<c01057d6>] dump_stack+0x16/0x18
> [<c01f835e>] debug_smp_processor_id+0xa2/0xb4
> [<c0113cc5>] check_tsc_sync_source+0x1b/0xef
> [<c011367d>] __cpu_up+0x136/0x158
> [<c0141aec>] _cpu_up+0x74/0xbf
> [<c0141b5d>] cpu_up+0x26/0x38
> [<c0141bbc>] enable_nonboot_cpus+0x4d/0x9a
> [<c0146ae0>] pm_suspend_disk+0x11c/0x210
> [<c014597e>] enter_state+0x50/0x1d0
> [<c0145b84>] state_store+0x86/0x9c
> [<c01a53d0>] subsys_attr_store+0x20/0x25
> [<c01a54ea>] sysfs_write_file+0xc1/0xe9
> [<c017199b>] vfs_write+0xaf/0x138
> [<c0171f65>] sys_write+0x3d/0x61
> [<c0104064>] syscall_call+0x7/0xb
> =======================
>
> l *check_tsc_sync_source+0x1b/0xef
> 0xc0113caa is in check_tsc_sync_source (/mnt/md0/devel/linux-git/arch/i386/kernel/../../x86_64/kernel/tsc_sync.c:99).
> 94 /*
> 95 * Source CPU calls into this - it waits for the freshly booted
> 96 * target CPU to arrive and then starts the measurement:
> 97 */
> 98 void __cpuinit check_tsc_sync_source(int cpu)
> 99 {
> 100 int cpus = 2;
> 101
> 102 /*
> 103 * No need to check if we already know that the TSC is not
>
> echo platform > /sys/power/disk; echo disk > /sys/power/state
> doesn't work (as always).
>
> http://www.stardust.webpages.pl/files/tbf/bitis-gabonica/2.6.21-rc3/boot.log
> http://www.stardust.webpages.pl/files/tbf/bitis-gabonica/2.6.21-rc3/git-config
>
> Regards,
> Michal
>
> --
> Michal K. K. Piotrowski
> LTG - Linux Testers Group (PL)
> (http://www.stardust.webpages.pl/ltg/)
> LTG - Linux Testers Group (EN)
> (http://www.stardust.webpages.pl/linux_testers_group_en/)
>
next prev parent reply other threads:[~2007-03-07 16:34 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-07 4:59 Linux v2.6.21-rc3 Linus Torvalds
2007-03-07 10:25 ` Benjamin Herrenschmidt
2007-03-07 13:26 ` Greg KH
2007-03-07 14:15 ` Mark Lord
2007-03-07 14:22 ` Greg KH
2007-03-07 15:39 ` Linus Torvalds
2007-03-07 20:52 ` Arnd Bergmann
2007-03-08 8:10 ` Benjamin Herrenschmidt
2007-03-08 8:08 ` Benjamin Herrenschmidt
2007-03-07 12:56 ` Michal Piotrowski
2007-03-07 16:34 ` Linus Torvalds [this message]
2007-03-07 17:12 ` [patch] CPU hotplug: call check_tsc_sync_source() with irqs off Ingo Molnar
2007-03-07 17:45 ` Michal Piotrowski
2007-03-07 19:12 ` Linux-2.6.21-rc3 : Dynticks and High resolution Timer hanging the system Stephane Casset
2007-03-07 19:52 ` Thomas Gleixner
2007-03-07 21:16 ` Stephane Casset
2007-03-07 22:09 ` Thomas Gleixner
2007-03-07 13:09 ` Linux v2.6.21-rc3 Michal Piotrowski
2007-03-07 16:25 ` Linus Torvalds
2007-03-07 17:14 ` [Linux-parport] " Stephen Mollett
2007-03-07 17:35 ` Russell King
2007-03-07 14:22 ` Thomas Gleixner
2007-03-07 17:14 ` Thomas Gleixner
2007-03-07 17:42 ` Soeren Sonnenburg
2007-03-08 17:28 ` Alistair John Strachan
2007-03-13 12:49 ` [1/6] 2.6.21-rc3: known regressions Adrian Bunk
2007-03-13 13:08 ` Pierre Ossman
2007-03-13 13:36 ` Oliver Neukum
2007-03-13 18:11 ` Pavel Machek
2007-03-13 19:07 ` Pierre Ossman
2007-03-13 19:12 ` Mws
2007-03-13 19:15 ` Adrian Bunk
2007-03-13 20:05 ` Pavel Machek
2007-03-13 20:31 ` Pierre Ossman
2007-03-13 13:40 ` Takashi Iwai
2007-03-13 12:50 ` [2/6] " Adrian Bunk
2007-03-13 13:30 ` Cornelia Huck
2007-03-13 13:35 ` Mark Lord
2007-03-13 18:13 ` Pavel Machek
2007-03-13 12:50 ` [3/6] " Adrian Bunk
2007-03-13 14:03 ` Alan Cox
2007-03-13 20:12 ` Fabio Comolli
2007-03-13 15:13 ` Andi Kleen
2007-03-13 12:50 ` [4/6] " Adrian Bunk
2007-03-13 12:50 ` [5/6] " Adrian Bunk
2007-03-13 13:29 ` Lukas Hejtmanek
2007-03-13 18:14 ` Pavel Machek
2007-03-13 21:46 ` Arkadiusz Miskiewicz
2007-03-13 12:50 ` [6/6] " Adrian Bunk
2007-03-13 20:05 ` Thomas Gleixner
2007-03-14 11:31 ` Adrian Bunk
2007-03-13 20:46 ` Thomas Gleixner
2007-03-14 11:44 ` Adrian Bunk
2007-03-14 12:16 ` Jiri Slaby
2007-03-14 17:31 ` Adrian Bunk
2007-03-14 18:02 ` Florian Lohoff
2007-03-14 18:28 ` Thomas Gleixner
2007-03-13 19:26 ` Linux v2.6.21-rc3 Eric W. Biederman
2007-03-13 19:40 ` Greg KH
2007-03-13 19:48 ` Linus Torvalds
2007-03-13 20:04 ` Eric W. Biederman
2007-03-14 18:11 ` 2.6.21-rc3: known regressions with patches Adrian Bunk
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=Pine.LNX.4.64.0703070827490.5963@woody.linux-foundation.org \
--to=torvalds@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=michal.k.k.piotrowski@gmail.com \
--cc=mingo@elte.hu \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
--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 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).