udf: Avoid IO in udf_clear_inode
authorJan Kara <jack@suse.cz>
Thu, 3 Dec 2009 12:39:28 +0000 (13:39 +0100)
committerJan Kara <jack@suse.cz>
Mon, 14 Dec 2009 20:40:04 +0000 (21:40 +0100)
commit2c948b3f86e5f0327e2e57858600af6e6f0ae29a
treea40d5fd7a5757d26fc41829e5620b4685bac44e2
parente971b0b9e0dd50d9ceecb67a6a6ab80a80906033
udf: Avoid IO in udf_clear_inode

It is not very good to do IO in udf_clear_inode. First, VFS does not really
expect inode to become dirty there and thus we have to write it ourselves,
second, memory reclaim gets blocked waiting for IO when it does not really
expect it, third, the IO pattern (e.g. on umount) resulting from writes in
udf_clear_inode is bad and it slows down writing a lot.

The reason why UDF needed to do IO in udf_clear_inode is that UDF standard
mandates extent length to exactly match inode size. But when we allocate
extents to a file or directory, we don't really know what exactly the final
file size will be and thus temporarily set it to block boundary and later
truncate it to exact length in udf_clear_inode. Now, this is changed to
truncate to final file size in udf_release_file for regular files. For
directories and symlinks, we do the truncation at the moment when learn
what the final file size will be.

Signed-off-by: Jan Kara <jack@suse.cz>
fs/udf/file.c
fs/udf/inode.c
fs/udf/namei.c