linux-kernel - Re: 2.6.33.5 rt23: sleeping function called from invalid context

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]

Message-ID: <1278610624.3008.2.camel@localhost.localdomain>
Date:	Thu, 08 Jul 2010 10:37:04 -0700
From:	john stultz <johnstul@...ibm.com>
To:	Fernando Lopez-Lezcano <nando@...ma.Stanford.EDU>
Cc:	Thomas Gleixner <tglx@...utronix.de>,
	LKML <linux-kernel@...r.kernel.org>,
	rt-users <linux-rt-users@...r.kernel.org>,
	Steven Rostedt <rostedt@...dmis.org>,
	Nick Piggin <npiggin@...e.de>
Subject: Re: 2.6.33.5 rt23: sleeping function called from invalid context

On Wed, 2010-07-07 at 20:54 -0700, Fernando Lopez-Lezcano wrote:
> After a suspend/wake up cycle, just after upgrading to fc12 (I did not
> see this with the same basic kernel - that is, compiled from the same
> source + patches - under fc11). 
> 
> BUG: sleeping function called from invalid context at
> kernel/rtmutex.c:684
> pcnt: 0 0 in_atomic(): 0, irqs_disabled(): 1, pid: 10582, name:
> pm-suspend
> Pid: 10582, comm: pm-suspend Not tainted
> 2.6.33.5-120.rt23.1.fc11.ccrma.i686.rtPAE #1
> Call Trace:
> [<c042eced>] __might_sleep+0xcc/0xd4
> [<c0464f57>] rt_spin_lock_fastlock.clone.1+0x26/0x5f
> [<c0792862>] rt_spin_lock+0x8/0xa
> [<c040dddc>] read_persistent_clock+0x11/0x30
> [<c045d1de>] timekeeping_suspend+0xe/0x4e
> [<c0640c9e>] sysdev_suspend+0x15c/0x356
> [<c0792906>] ? _mutex_unlock+0x8/0xa
> [<c046afc1>] suspend_devices_and_enter+0xea/0x17f
> [<c046b11e>] enter_state+0xc8/0x114
> [<c046a9cf>] state_store+0x93/0xa7
> [<c046a93c>] ? state_store+0x0/0xa7
> [<c05a6505>] kobj_attr_store+0x16/0x22
> [<c0515afa>] sysfs_write_file+0xbf/0xea
> [<c0515a3b>] ? sysfs_write_file+0x0/0xea
> [<c04d26ba>] vfs_write+0x80/0xdf
> [<c0465134>] ? rt_up_read+0x13/0x15
> [<c04d27ad>] sys_write+0x3b/0x5d
> [<c040895f>] sysenter_do_call+0x12/0x28

Huh. Looks like the lock protecting the RTC/CMOS might need to be
converted to a raw spinlock, since suspend/resume is probably done with
irqs off.

A little baffled why the same kernel didn't see this with fc11.

Does this reproduce easily?

thanks
-john

--
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/