linux-kernel - Re: [BUG] using smp_processor_id() in preemptible as suspending

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 27 Feb 2008 09:04:55 +0800
From:	"Dave Young" <hidave.darkstar@...il.com>
To:	paulmck@...ux.vnet.ibm.com
Cc:	"Andrew Morton" <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, "Ingo Molnar" <mingo@...e.hu>
Subject: Re: [BUG] using smp_processor_id() in preemptible as suspending

On Wed, Feb 27, 2008 at 3:45 AM, Paul E. McKenney
<paulmck@...ux.vnet.ibm.com> wrote:
> On Tue, Feb 26, 2008 at 08:33:54AM -0800, Andrew Morton wrote:
>  > On Tue, 26 Feb 2008 16:24:11 +0800 Dave Young <hidave.darkstar@...il.com> wrote:
>  >
>  > > I don't know whom I should mail to, could you cc the proper guy? Thanks.
>
>  Hello, Dave,
>
>  Would you be willing to try out the following (untested, might not even
>  compile) patch show at the very end of this message?  This assumes that
>  you were running with CONFIG_PREEMPT_RCU, which seems most likely based
>  on looking at your oops.

Built ok, but can't even resume now. (dark screen with cursor blinking
on top-left of minitor)

Forgot to tell, I saw this bug as enable PM_DEBUG, maybe it's related
to PM issue?

>
>                                                         Thanx, Paul
>
>
>
>  > > [  118.331674] acpi LNXSYSTM:00: suspend
>  > > [  118.331674] Disabling non-boot CPUs ...
>  > > [  118.331674] CPU0 attaching NULL sched-domain.
>  > > [  118.331674] CPU1 attaching NULL sched-domain.
>  > > [  118.438750] CPU 1 is now offline
>  > > [  118.438750] lockdep: fixing up alternatives.
>  > > [  118.438750] SMP alternatives: switching to UP code
>  > > [  118.438750] BUG: using smp_processor_id() in preemptible [00000000] code: s2ram/2818
>  > > [  118.438750] caller is rcu_offline_cpu+0x15a/0x1c0
>  > > [  118.438750] Pid: 2818, comm: s2ram Not tainted 2.6.25-rc3-test #2
>  > > [  118.438750]  [<c0129738>] ? printk+0x18/0x20
>  > > [  118.438750]  [<c025c551>] debug_smp_processor_id+0xb1/0xc0
>  > > [  118.438750]  [<c0161f8a>] rcu_offline_cpu+0x15a/0x1c0
>  > > [  118.438750]  [<c03d408f>] rcu_cpu_notify+0x3f/0x60
>  > > [  118.438750]  [<c0141b0d>] notifier_call_chain+0x3d/0x80
>  > > [  118.438750]  [<c0141db9>] __raw_notifier_call_chain+0x19/0x20
>  > > [  118.438750]  [<c0141dda>] raw_notifier_call_chain+0x1a/0x20
>  > > [  118.438750]  [<c015262b>] _cpu_down+0x13b/0x230
>  > > [  118.438750]  [<c01527c9>] disable_nonboot_cpus+0x49/0xd0
>  > > [  118.438750]  [<c0157492>] suspend_devices_and_enter+0x72/0x130
>  > > [  118.438750]  [<c0129738>] ? printk+0x18/0x20
>  > > [  118.438750]  [<c0157623>] enter_state+0xb3/0xe0
>  > > [  118.438750]  [<c015777d>] state_store+0x7d/0xc0
>  > > [  118.438750]  [<c0157700>] ? state_store+0x0/0xc0
>  > > [  118.438750]  [<c0249e4e>] kobj_attr_store+0x2e/0x40
>  > > [  118.438750]  [<c01ca957>] flush_write_buffer+0x47/0x70
>  > > [  118.438750]  [<c01ca9c9>] sysfs_write_file+0x49/0x70
>  > > [  118.438750]  [<c0188d41>] vfs_write+0x91/0x140
>  > > [  118.438750]  [<c0188e9d>] sys_write+0x3d/0x70
>  > > [  118.438750]  [<c0104fca>] syscall_call+0x7/0xb
>  > > [  118.438750]  =======================
>  > > [  118.438750] CPU0 attaching NULL sched-domain.
>  > > [  118.440335] CPU1 is down
>  >
>  > Paul & Ingo I guess
>  >
>  > > My .config
>  >
>  > Doesn't tell us whether you'r eusing CONFIG_CLASSIC_RCU or
>  > CONFIG_PREEMPT_RCU.  I assume CONFIG_CLASSIC_RCU, if you ran `make
>  > oldconfig'.
>  >
>  > Which kernel are you running here?
>
>  Signed-off-by: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>
>  ---
>
>   rcupreempt.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
>  diff -urpNa -X dontdiff linux-2.6.25-rc2/kernel/rcupreempt.c linux-2.6.25-rc2-rcuoffline/kernel/rcupreempt.c
>  --- linux-2.6.25-rc2/kernel/rcupreempt.c        2008-02-24 10:30:44.000000000 -0800
>  +++ linux-2.6.25-rc2-rcuoffline/kernel/rcupreempt.c     2008-02-26 11:39:33.000000000 -0800
>  @@ -702,8 +702,8 @@ void rcu_offline_cpu(int cpu)
>          * fix.
>          */
>
>  -       rdp = RCU_DATA_ME();
>         spin_lock_irqsave(&rdp->lock, flags);
>  +       rdp = RCU_DATA_ME();
>         *rdp->nexttail = list;
>         if (list)
>                 rdp->nexttail = tail;
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/