ext4: make ext4_bio_write_page() use BH_Async_Write flags
authorJan Kara <jack@suse.cz>
Fri, 12 Apr 2013 03:48:32 +0000 (23:48 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Fri, 12 Apr 2013 03:48:32 +0000 (23:48 -0400)
commit0058f9658c94037173f7603fc8bae2007cc10253
tree8406b3704c0b7dc8f9057e9e8ae78187c528e594
parente1091b157c330a21bb0eaa881efe0489a1697ed7
ext4: make ext4_bio_write_page() use BH_Async_Write flags

So far ext4_bio_write_page() attached all the pages to ext4_io_end
structure.  This makes that structure pretty heavy (1 KB for pointers
+ 16 bytes per page attached to the bio).  Also later we would like to
share ext4_io_end structure among several bios in case IO to a single
extent needs to be split among several bios and pointing to pages from
ext4_io_end makes this complex.

We remove page pointers from ext4_io_end and use pointers from bio
itself instead.  This isn't as easy when blocksize < pagesize because
then we can have several bios in flight for a single page and we have
to be careful when to call end_page_writeback().  However this is a
known problem already solved by block_write_full_page() /
end_buffer_async_write() so we mimic its behavior here.  We mark
buffers going to disk with BH_Async_Write flag and in
ext4_bio_end_io() we check whether there are any buffers with
BH_Async_Write flag left.  If there are not, we can call
end_page_writeback().

Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Reviewed-by: Dmitry Monakhov <dmonakhov@openvz.org>
Reviewed-by: Zheng Liu <wenqing.lz@taobao.com>
fs/ext4/ext4.h
fs/ext4/page-io.c