[lkml]   [2001]   [Jan]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: Possible deadlock with ->writepaged version of flush_dirty_buffers() and 2.4.0

On Wednesday, January 10, 2001 05:56:09 PM -0200 Marcelo Tosatti
<> wrote:

> Hi Chris,
> It seems there is a possible deadlock condition with your patch which
> changes flush_dirty_buffers() to use ->writepage (something which we
> _definately_ want for 2.5). Take a look:
Yes, good catch.

> mark_buffer_dirty->balance_dirty->wakeup_bdflush->flush_dirty_buffers->
> writepage->block_write_full_page->__block_write_full_page->get_block->
> ext2_get_block->ext2_alloc_branch->
> ext2_alloc_block->ext2_new_block->lock_super
> or
> getblk()->lock_super
> I dont see any reason why this deadlock could'nt happen in practice now.
It won't happen until someone other than fs/buffer.c starts marking ext2
pages dirty. The normal file write path will make sure that any dirty
buffers are mapped, so the ext2_get_block code is never run.

> If I'm right, it will pretty nasty to fix this. One possible solution is
> to _never_ call mark_buffer_dirty() with the superblock lock held (ext2
> has a lot of places likes this right now)

This is probably the best solution, since it is a good idea regardless of
my patch.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:28    [W:0.042 / U:2.276 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site