fix udf reservation discarding
Subsystems (2)
PATCH 1/1 fix udf reservation discarding
UDF discards file preallocations on every ->put_inode which is totally bogus. It already discards them in ->release which makes sense for normal writes, so the only additional discard is in ->clear_inode so we make sure we don't leak any reservations for shared writeable mappings.
This follows similar changes to ext2 and ext3.
(This patch needs to be applied after the iget patch)
fs/udf/inode.c +6 -24
@@ -70,30 +70,6 @@static int udf_get_block (struct inode *,sector_t ,struct buffer_head *,int );/* -* udf_put_inode -* -* PURPOSE -* -* DESCRIPTION -* This routine is called whenever the kernel no longer needs the inode. -* -* HISTORY -* July 1, 1997 - Andrew E. Mileski -* Written, tested, and released. -* -* Called at each iput() -*/ -void udf_put_inode (struct inode * inode) -{ -if (!(inode->i_sb->s_flags &MS_RDONLY )) - { -lock_kernel (); -udf_discard_prealloc (inode); -unlock_kernel (); - } -} - -/* * udf_delete_inode * * PURPOSE
@@ -129,6 +105,12 @@void udf_clear_inode (struct inode *inode) { +if (!(inode->i_sb->s_flags &MS_RDONLY )) { +lock_kernel (); +udf_discard_prealloc (inode); +unlock_kernel (); + } +kfree (UDF_I_DATA (inode));UDF_I_DATA (inode) =NULL ; }
fs/udf/super.c +0 -1
@@ -163,7 +163,6 @@ .alloc_inode = udf_alloc_inode, .destroy_inode = udf_destroy_inode, .write_inode = udf_write_inode, - .put_inode = udf_put_inode, .delete_inode = udf_delete_inode, .clear_inode = udf_clear_inode, .put_super = udf_put_super,
fs/udf/udfdecl.h +0 -1
@@ -96,7 +96,6 @@extern struct buffer_head *udf_bread (struct inode *,int ,int ,int *);extern void udf_truncate (struct inode *);extern void udf_read_inode (struct inode *); -extern void udf_put_inode (struct inode *);extern void udf_delete_inode (struct inode *);extern void udf_clear_inode (struct inode *);extern int udf_write_inode (struct inode *,int );