linux-kernel - Re: [2.6.23-rt3] NMI watchdog trace of deadlock

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: <1193478216.7496.26.camel@Homer.simpson.net>
Date:	Sat, 27 Oct 2007 11:43:36 +0200
From:	Mike Galbraith <efault@....de>
To:	Nick Piggin <nickpiggin@...oo.com.au>
Cc:	Steven Rostedt <rostedt@...dmis.org>,
	LKML <linux-kernel@...r.kernel.org>,
	RT <linux-rt-users@...r.kernel.org>, Ingo Molnar <mingo@...e.hu>,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [2.6.23-rt3] NMI watchdog trace of deadlock

On Sat, 2007-10-27 at 19:15 +1000, Nick Piggin wrote:
> On Saturday 27 October 2007 15:21, Mike Galbraith wrote:
> > Greetings,
> >
> > For quite a while now, RT kernels have been locking up on me
> > occasionally while my back is turned.  Yesterday, the little bugger
> > finally pounced while my serial console box was up and waiting.
> >
> > [10138.162953] WARNING: at arch/i386/kernel/smp.c:581
> > native_smp_call_function_mask() [10138.170583]  [<c01051da>]
> > show_trace_log_lvl+0x1a/0x30
> > [10138.175796]  [<c0105de3>] show_trace+0x12/0x14
> > [10138.180291]  [<c0105dfb>] dump_stack+0x16/0x18
> > [10138.184769]  [<c011609f>] native_smp_call_function_mask+0x138/0x13d
> > [10138.191117]  [<c0117606>] smp_call_function+0x1e/0x24
> > [10138.196210]  [<c012f85c>] on_each_cpu+0x25/0x50
> > [10138.200807]  [<c0115c74>] flush_tlb_all+0x1e/0x20
> > [10138.205553]  [<c016caaf>] kmap_high+0x1b6/0x417
> > [10138.210118]  [<c011ec88>] kmap+0x4d/0x4f
> > [10138.214102]  [<c026a9d8>] ntfs_end_buffer_async_read+0x228/0x2f9
> > [10138.220163]  [<c01a0e9e>] end_bio_bh_io_sync+0x26/0x3f
> > [10138.225352]  [<c01a2b09>] bio_endio+0x42/0x6d
> > [10138.229769]  [<c02c2a08>] __end_that_request_first+0x115/0x4ac
> > [10138.235682]  [<c02c2da7>] end_that_request_chunk+0x8/0xa
> > [10138.241052]  [<c0365943>] ide_end_request+0x55/0x10a
> > [10138.246058]  [<c036dae3>] ide_dma_intr+0x6f/0xac
> > [10138.250727]  [<c0366d83>] ide_intr+0x93/0x1e0
> > [10138.255125]  [<c015afb4>] handle_IRQ_event+0x5c/0xc9
> 
> Looks like ntfs is kmap()ing from interrupt context. Should
> be using kmap_atomic instead, I think.
> 
> But the ntfs code I have seems to do just that...

The ntfs code does use kmap_atomic(), bit RT turns that into a wrapper
for kmap().  I fed it a local_irq_enable(), and am waiting for it to
tell me why that was stoopid ;-)

	thanks,

	-Mike

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