Re: 2.6.17-rc4-mm1

Takashi Iwai
2006-05-15T17:30:26Z

At Mon, 15 May 2006 10:04:29 -0700,
Andrew Morton wrote:
>
> "Michal Piotrowski" <michal.k.k.piotrowski@gmail.com> wrote:
> >
> > Hi,
> >
> > On 15/05/06, Andrew Morton <akpm@osdl.org> wrote:
> > >
> > > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.17-rc4/2.6.17-rc4-mm1/
> > >
> > > - This tree contains a large number of new bugs^H^H^H^Hpatches.
> > [snip]
> > > git-alsa.patch
> >
> > BUG: sleeping function called from invalid context at
> > /usr/src/linux-mm/sound/core/info.c:117
> > in_atomic():1, irqs_disabled():0
> > <c1003ef9> show_trace+0xd/0xf <c100440c> dump_stack+0x17/0x19
> > <c10178ce> __might_sleep+0x93/0x9d <f988eeb5> snd_iprintf+0x1b/0x84 [snd]
> > <f988d808> snd_card_module_info_read+0x34/0x4e [snd] <f988f197>
> > snd_info_entry_open+0x20f/0x2cc [snd]
> > <c1067a17> __dentry_open+0x133/0x260 <c1067bb7> nameidata_to_filp+0x1c/0x2e
> > <c1067bf7> do_filp_open+0x2e/0x35 <c1068bf2> do_sys_open+0x54/0xd7
> > <c1068ca1> sys_open+0x16/0x18 <c11dab67> sysenter_past_esp+0x54/0x75
> > Non-volatile memory driver v1.2
> >
> > Here is dmesg http://www.stardust.webpages.pl/files/mm/2.6.17-rc4-mm1/mm-dmesg
> > Here is config http://www.stardust.webpages.pl/files/mm/2.6.17-rc4-mm1/mm-config
>
> heh, yes, Takashi baited his hook and pulled in a big one.
>
> snd_card_module_info_read() calls snd_iprintf() under read_lock()..

Ouch, I checked only spinlocks but forgot rwlocks. Will fix them.

Thanks.

Takashi