* blk: bd_block_size_semaphore related lockdep warning
@ 2012-12-01 20:42 Sasha Levin
2012-12-07 2:58 ` Mikulas Patocka
0 siblings, 1 reply; 2+ messages in thread
From: Sasha Levin @ 2012-12-01 20:42 UTC (permalink / raw)
To: axboe
Cc: mpatocka, linux-kernel@vger.kernel.org, linux-fsdevel, Al Viro,
Dave Jones
Hi all,
While fuzzing with trinity inside a KVM tools guest, running latest -next, I've
stumbled on:
[ 3130.099477] ======================================================
[ 3130.104862] [ INFO: possible circular locking dependency detected ]
[ 3130.104862] 3.7.0-rc7-next-20121130-sasha-00015-g06fcc7a-dirty #2 Tainted: G W
[ 3130.104862] -------------------------------------------------------
[ 3130.104862] trinity-child77/12730 is trying to acquire lock:
[ 3130.104862] (&sb->s_type->i_mutex_key#17/1){+.+.+.}, at: [<ffffffff8128a605>] pipe_lock_nested.isra.2+0x15/0x20
[ 3130.104862]
[ 3130.104862] but task is already holding lock:
[ 3130.104862] (sb_writers#16){.+.+..}, at: [<ffffffff812b439f>] generic_file_splice_write+0x6f/0x170
[ 3130.104862]
[ 3130.104862] which lock already depends on the new lock.
[ 3130.104862]
[ 3130.104862]
[ 3130.104862] the existing dependency chain (in reverse order) is:
[ 3130.104862]
-> #3 (sb_writers#16){.+.+..}:
[ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
[ 3130.104862] [<ffffffff81285dd6>] __sb_start_write+0x146/0x1b0
[ 3130.104862] [<ffffffff812b439f>] generic_file_splice_write+0x6f/0x170
[ 3130.104862] [<ffffffff812be0ca>] blkdev_splice_write+0x5a/0x80
[ 3130.104862] [<ffffffff812b2983>] do_splice_from+0x83/0xb0
[ 3130.104862] [<ffffffff812b4de2>] sys_splice+0x492/0x690
[ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
[ 3130.104862]
-> #2 (&ei->bdev.bd_block_size_semaphore){++++.+}:
[ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
[ 3130.104862] [<ffffffff81a0ec15>] percpu_down_read+0x55/0x90
[ 3130.104862] [<ffffffff812be123>] blkdev_mmap+0x33/0x60
[ 3130.104862] [<ffffffff8123ec9b>] mmap_region+0x31b/0x600
[ 3130.104862] [<ffffffff8123f237>] do_mmap_pgoff+0x2b7/0x330
[ 3130.104862] [<ffffffff81228b9a>] vm_mmap_pgoff+0x7a/0xa0
[ 3130.104862] [<ffffffff8123d70e>] sys_mmap_pgoff+0x16e/0x1b0
[ 3130.104862] [<ffffffff8107492d>] sys_mmap+0x1d/0x20
[ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
[ 3130.104862]
-> #1 (&mm->mmap_sem){++++++}:
[ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
[ 3130.104862] [<ffffffff81239d6b>] might_fault+0x7b/0xa0
[ 3130.104862] [<ffffffff812b4805>] sys_vmsplice+0xd5/0x220
[ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
[ 3130.104862]
-> #0 (&sb->s_type->i_mutex_key#17/1){+.+.+.}:
[ 3130.104862] [<ffffffff8118759f>] __lock_acquire+0x147f/0x1c30
[ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
[ 3130.104862] [<ffffffff83cac6d9>] __mutex_lock_common+0x59/0x5a0
[ 3130.104862] [<ffffffff83cacc5f>] mutex_lock_nested+0x3f/0x50
[ 3130.104862] [<ffffffff8128a605>] pipe_lock_nested.isra.2+0x15/0x20
[ 3130.104862] [<ffffffff8128a6f5>] pipe_lock+0x15/0x20
[ 3130.104862] [<ffffffff812b43a7>] generic_file_splice_write+0x77/0x170
[ 3130.104862] [<ffffffff812be0ca>] blkdev_splice_write+0x5a/0x80
[ 3130.104862] [<ffffffff812b2983>] do_splice_from+0x83/0xb0
[ 3130.104862] [<ffffffff812b4de2>] sys_splice+0x492/0x690
[ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
[ 3130.104862]
[ 3130.104862] other info that might help us debug this:
[ 3130.104862]
[ 3130.104862] Chain exists of:
&sb->s_type->i_mutex_key#17/1 --> &ei->bdev.bd_block_size_semaphore --> sb_writers#16
[ 3130.104862] Possible unsafe locking scenario:
[ 3130.104862]
[ 3130.104862] CPU0 CPU1
[ 3130.104862] ---- ----
[ 3130.104862] lock(sb_writers#16);
[ 3130.104862] lock(&ei->bdev.bd_block_size_semaphore);
[ 3130.104862] lock(sb_writers#16);
[ 3130.104862] lock(&sb->s_type->i_mutex_key#17/1);
[ 3130.104862]
[ 3130.104862] *** DEADLOCK ***
[ 3130.104862]
[ 3130.104862] 2 locks held by trinity-child77/12730:
[ 3130.104862] #0: (&ei->bdev.bd_block_size_semaphore){++++.+}, at: [<ffffffff812be0b5>] blkdev_splice_write+0x45/0x80
[ 3130.104862] #1: (sb_writers#16){.+.+..}, at: [<ffffffff812b439f>] generic_file_splice_write+0x6f/0x170
[ 3130.104862]
[ 3130.104862] stack backtrace:
[ 3130.104862] Pid: 12730, comm: trinity-child77 Tainted: G W 3.7.0-rc7-next-20121130-sasha-00015-g06fcc7a-dirty #2
[ 3130.104862] Call Trace:
[ 3130.104862] [<ffffffff83c54d8a>] print_circular_bug+0x1fb/0x20c
[ 3130.104862] [<ffffffff8118759f>] __lock_acquire+0x147f/0x1c30
[ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
[ 3130.104862] [<ffffffff8128a605>] ? pipe_lock_nested.isra.2+0x15/0x20
[ 3130.104862] [<ffffffff83cac6d9>] __mutex_lock_common+0x59/0x5a0
[ 3130.104862] [<ffffffff8128a605>] ? pipe_lock_nested.isra.2+0x15/0x20
[ 3130.104862] [<ffffffff81185d8a>] ? __lock_is_held+0x5a/0x80
[ 3130.104862] [<ffffffff8128a605>] ? pipe_lock_nested.isra.2+0x15/0x20
[ 3130.104862] [<ffffffff83cacc5f>] mutex_lock_nested+0x3f/0x50
[ 3130.104862] [<ffffffff8128a605>] pipe_lock_nested.isra.2+0x15/0x20
[ 3130.104862] [<ffffffff8128a6f5>] pipe_lock+0x15/0x20
[ 3130.104862] [<ffffffff812b43a7>] generic_file_splice_write+0x77/0x170
[ 3130.104862] [<ffffffff812be0ca>] blkdev_splice_write+0x5a/0x80
[ 3130.104862] [<ffffffff812b2983>] do_splice_from+0x83/0xb0
[ 3130.104862] [<ffffffff812b4de2>] sys_splice+0x492/0x690
[ 3130.104862] [<ffffffff8107eaf0>] ? syscall_trace_enter+0x20/0x2e0
[ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
Thanks,
sasha
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: blk: bd_block_size_semaphore related lockdep warning
2012-12-01 20:42 blk: bd_block_size_semaphore related lockdep warning Sasha Levin
@ 2012-12-07 2:58 ` Mikulas Patocka
0 siblings, 0 replies; 2+ messages in thread
From: Mikulas Patocka @ 2012-12-07 2:58 UTC (permalink / raw)
To: Sasha Levin
Cc: axboe, linux-kernel@vger.kernel.org, linux-fsdevel, Al Viro,
Dave Jones
Hi
It should be fixed in 3.7-rc8
Mikulas
On Sat, 1 Dec 2012, Sasha Levin wrote:
> Hi all,
>
> While fuzzing with trinity inside a KVM tools guest, running latest -next, I've
> stumbled on:
>
> [ 3130.099477] ======================================================
> [ 3130.104862] [ INFO: possible circular locking dependency detected ]
> [ 3130.104862] 3.7.0-rc7-next-20121130-sasha-00015-g06fcc7a-dirty #2 Tainted: G W
> [ 3130.104862] -------------------------------------------------------
> [ 3130.104862] trinity-child77/12730 is trying to acquire lock:
> [ 3130.104862] (&sb->s_type->i_mutex_key#17/1){+.+.+.}, at: [<ffffffff8128a605>] pipe_lock_nested.isra.2+0x15/0x20
> [ 3130.104862]
> [ 3130.104862] but task is already holding lock:
> [ 3130.104862] (sb_writers#16){.+.+..}, at: [<ffffffff812b439f>] generic_file_splice_write+0x6f/0x170
> [ 3130.104862]
> [ 3130.104862] which lock already depends on the new lock.
> [ 3130.104862]
> [ 3130.104862]
> [ 3130.104862] the existing dependency chain (in reverse order) is:
> [ 3130.104862]
> -> #3 (sb_writers#16){.+.+..}:
> [ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
> [ 3130.104862] [<ffffffff81285dd6>] __sb_start_write+0x146/0x1b0
> [ 3130.104862] [<ffffffff812b439f>] generic_file_splice_write+0x6f/0x170
> [ 3130.104862] [<ffffffff812be0ca>] blkdev_splice_write+0x5a/0x80
> [ 3130.104862] [<ffffffff812b2983>] do_splice_from+0x83/0xb0
> [ 3130.104862] [<ffffffff812b4de2>] sys_splice+0x492/0x690
> [ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
> [ 3130.104862]
> -> #2 (&ei->bdev.bd_block_size_semaphore){++++.+}:
> [ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
> [ 3130.104862] [<ffffffff81a0ec15>] percpu_down_read+0x55/0x90
> [ 3130.104862] [<ffffffff812be123>] blkdev_mmap+0x33/0x60
> [ 3130.104862] [<ffffffff8123ec9b>] mmap_region+0x31b/0x600
> [ 3130.104862] [<ffffffff8123f237>] do_mmap_pgoff+0x2b7/0x330
> [ 3130.104862] [<ffffffff81228b9a>] vm_mmap_pgoff+0x7a/0xa0
> [ 3130.104862] [<ffffffff8123d70e>] sys_mmap_pgoff+0x16e/0x1b0
> [ 3130.104862] [<ffffffff8107492d>] sys_mmap+0x1d/0x20
> [ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
> [ 3130.104862]
> -> #1 (&mm->mmap_sem){++++++}:
> [ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
> [ 3130.104862] [<ffffffff81239d6b>] might_fault+0x7b/0xa0
> [ 3130.104862] [<ffffffff812b4805>] sys_vmsplice+0xd5/0x220
> [ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
> [ 3130.104862]
> -> #0 (&sb->s_type->i_mutex_key#17/1){+.+.+.}:
> [ 3130.104862] [<ffffffff8118759f>] __lock_acquire+0x147f/0x1c30
> [ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
> [ 3130.104862] [<ffffffff83cac6d9>] __mutex_lock_common+0x59/0x5a0
> [ 3130.104862] [<ffffffff83cacc5f>] mutex_lock_nested+0x3f/0x50
> [ 3130.104862] [<ffffffff8128a605>] pipe_lock_nested.isra.2+0x15/0x20
> [ 3130.104862] [<ffffffff8128a6f5>] pipe_lock+0x15/0x20
> [ 3130.104862] [<ffffffff812b43a7>] generic_file_splice_write+0x77/0x170
> [ 3130.104862] [<ffffffff812be0ca>] blkdev_splice_write+0x5a/0x80
> [ 3130.104862] [<ffffffff812b2983>] do_splice_from+0x83/0xb0
> [ 3130.104862] [<ffffffff812b4de2>] sys_splice+0x492/0x690
> [ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
> [ 3130.104862]
> [ 3130.104862] other info that might help us debug this:
> [ 3130.104862]
> [ 3130.104862] Chain exists of:
> &sb->s_type->i_mutex_key#17/1 --> &ei->bdev.bd_block_size_semaphore --> sb_writers#16
> [ 3130.104862] Possible unsafe locking scenario:
> [ 3130.104862]
> [ 3130.104862] CPU0 CPU1
> [ 3130.104862] ---- ----
> [ 3130.104862] lock(sb_writers#16);
> [ 3130.104862] lock(&ei->bdev.bd_block_size_semaphore);
> [ 3130.104862] lock(sb_writers#16);
> [ 3130.104862] lock(&sb->s_type->i_mutex_key#17/1);
> [ 3130.104862]
> [ 3130.104862] *** DEADLOCK ***
> [ 3130.104862]
> [ 3130.104862] 2 locks held by trinity-child77/12730:
> [ 3130.104862] #0: (&ei->bdev.bd_block_size_semaphore){++++.+}, at: [<ffffffff812be0b5>] blkdev_splice_write+0x45/0x80
> [ 3130.104862] #1: (sb_writers#16){.+.+..}, at: [<ffffffff812b439f>] generic_file_splice_write+0x6f/0x170
> [ 3130.104862]
> [ 3130.104862] stack backtrace:
> [ 3130.104862] Pid: 12730, comm: trinity-child77 Tainted: G W 3.7.0-rc7-next-20121130-sasha-00015-g06fcc7a-dirty #2
> [ 3130.104862] Call Trace:
> [ 3130.104862] [<ffffffff83c54d8a>] print_circular_bug+0x1fb/0x20c
> [ 3130.104862] [<ffffffff8118759f>] __lock_acquire+0x147f/0x1c30
> [ 3130.104862] [<ffffffff8118a34a>] lock_acquire+0x1aa/0x240
> [ 3130.104862] [<ffffffff8128a605>] ? pipe_lock_nested.isra.2+0x15/0x20
> [ 3130.104862] [<ffffffff83cac6d9>] __mutex_lock_common+0x59/0x5a0
> [ 3130.104862] [<ffffffff8128a605>] ? pipe_lock_nested.isra.2+0x15/0x20
> [ 3130.104862] [<ffffffff81185d8a>] ? __lock_is_held+0x5a/0x80
> [ 3130.104862] [<ffffffff8128a605>] ? pipe_lock_nested.isra.2+0x15/0x20
> [ 3130.104862] [<ffffffff83cacc5f>] mutex_lock_nested+0x3f/0x50
> [ 3130.104862] [<ffffffff8128a605>] pipe_lock_nested.isra.2+0x15/0x20
> [ 3130.104862] [<ffffffff8128a6f5>] pipe_lock+0x15/0x20
> [ 3130.104862] [<ffffffff812b43a7>] generic_file_splice_write+0x77/0x170
> [ 3130.104862] [<ffffffff812be0ca>] blkdev_splice_write+0x5a/0x80
> [ 3130.104862] [<ffffffff812b2983>] do_splice_from+0x83/0xb0
> [ 3130.104862] [<ffffffff812b4de2>] sys_splice+0x492/0x690
> [ 3130.104862] [<ffffffff8107eaf0>] ? syscall_trace_enter+0x20/0x2e0
> [ 3130.104862] [<ffffffff83cb1358>] tracesys+0xe1/0xe6
>
>
> Thanks,
> sasha
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-12-07 2:58 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-01 20:42 blk: bd_block_size_semaphore related lockdep warning Sasha Levin
2012-12-07 2:58 ` Mikulas Patocka
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).