From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [patch 10/11] udf: convert to use the new truncate convention. Date: Fri, 21 Aug 2009 16:22:31 +0200 Message-ID: <20090821142231.GB3007@duck.novell.com> References: <20090820163504.131529718@suse.de> <20090820164051.879013051@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andrew Morton , linux-fsdevel@vger.kernel.org, Christoph Hellwig , Jan Kara To: npiggin@suse.de Return-path: Received: from cantor.suse.de ([195.135.220.2]:44295 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755671AbZHUOWf (ORCPT ); Fri, 21 Aug 2009 10:22:35 -0400 Content-Disposition: inline In-Reply-To: <20090820164051.879013051@suse.de> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri 21-08-09 02:35:14, npiggin@suse.de wrote: > Cc: Jan Kara > Cc: Christoph Hellwig > Signed-off-by: Nick Piggin The patch is fine. Acked-by: Jan Kara I won't pull the patch to my UDF tree for now since it depends on the generic patches so I guess it's better if Andrew carries all the patches in one bunch. Once we decide it's time to merge with Linus, feel free to either merge it directly with my Ack or via my tree. Honza > --- > fs/udf/file.c | 3 +- > fs/udf/inode.c | 73 +++++++++++++++++++++++++++++++++++++++++++++++++++---- > fs/udf/udfdecl.h | 2 - > 3 files changed, 71 insertions(+), 7 deletions(-) > > Index: linux-2.6/fs/udf/inode.c > =================================================================== > --- linux-2.6.orig/fs/udf/inode.c > +++ linux-2.6/fs/udf/inode.c > @@ -67,6 +67,7 @@ static void udf_update_extents(struct in > struct extent_position *); > static int udf_get_block(struct inode *, sector_t, struct buffer_head *, int); > > +static void udf_truncate_blocks(struct inode *inode); > > void udf_delete_inode(struct inode *inode) > { > @@ -76,7 +77,7 @@ void udf_delete_inode(struct inode *inod > goto no_delete; > > inode->i_size = 0; > - udf_truncate(inode); > + udf_truncate_blocks(inode); > lock_kernel(); > > udf_update_inode(inode, IS_SYNC(inode)); > @@ -127,9 +128,34 @@ static int udf_write_begin(struct file * > loff_t pos, unsigned len, unsigned flags, > struct page **pagep, void **fsdata) > { > + int ret; > + > *pagep = NULL; > - return block_write_begin(file, mapping, pos, len, flags, pagep, fsdata, > + ret = block_write_begin(file, mapping, pos, len, flags, pagep, fsdata, > udf_get_block); > + if (ret < 0) { > + struct inode *inode = mapping->host; > + loff_t isize = inode->i_size; > + if (pos + len > isize) > + udf_truncate_blocks(inode); > + } > + return ret; > +} > + > +static int udf_write_end(struct file *file, struct address_space *mapping, > + loff_t pos, unsigned len, unsigned copied, > + struct page *page, void *fsdata) > +{ > + int ret; > + > + ret = generic_write_end(file, mapping, pos, len, copied, page, fsdata); > + if (ret < len) { > + struct inode *inode = mapping->host; > + loff_t isize = inode->i_size; > + if (pos + len > isize) > + udf_truncate_blocks(inode); > + } > + return ret; > } > > static sector_t udf_bmap(struct address_space *mapping, sector_t block) > @@ -141,8 +167,8 @@ const struct address_space_operations ud > .readpage = udf_readpage, > .writepage = udf_writepage, > .sync_page = block_sync_page, > - .write_begin = udf_write_begin, > - .write_end = generic_write_end, > + .write_begin = udf_write_begin, > + .write_end = udf_write_end, > .bmap = udf_bmap, > }; > > @@ -1011,7 +1037,7 @@ struct buffer_head *udf_bread(struct ino > return NULL; > } > > -void udf_truncate(struct inode *inode) > +static void udf_truncate_blocks(struct inode *inode) > { > int offset; > int err; > @@ -1057,6 +1083,43 @@ void udf_truncate(struct inode *inode) > unlock_kernel(); > } > > +static int udf_setsize(struct inode *inode, loff_t newsize) > +{ > + loff_t oldsize; > + int error; > + > + error = inode_newsize_ok(inode, newsize); > + if (error) > + return error; > + > + oldsize = inode->i_size; > + i_size_write(inode, newsize); > + truncate_pagecache(inode, oldsize, newsize); > + > + udf_truncate_blocks(inode); > + > + return error; > +} > + > +int udf_setattr(struct dentry *dentry, struct iattr *attr) > +{ > + struct inode *inode = dentry->d_inode; > + int error; > + > + if (attr->ia_valid & ATTR_SIZE) { > + error = udf_setsize(inode, attr->ia_size); > + if (error) > + return error; > + } > + > + error = simple_setattr(dentry, attr); > + if (error) > + return error; > + > + mark_inode_dirty(inode); > + return error; > +} > + > static void __udf_read_inode(struct inode *inode) > { > struct buffer_head *bh = NULL; > Index: linux-2.6/fs/udf/udfdecl.h > =================================================================== > --- linux-2.6.orig/fs/udf/udfdecl.h > +++ linux-2.6/fs/udf/udfdecl.h > @@ -138,7 +138,7 @@ extern int udf_sync_inode(struct inode * > extern void udf_expand_file_adinicb(struct inode *, int, int *); > extern struct buffer_head *udf_expand_dir_adinicb(struct inode *, int *, int *); > extern struct buffer_head *udf_bread(struct inode *, int, int, int *); > -extern void udf_truncate(struct inode *); > +extern int udf_setattr(struct dentry *, struct iattr *); > extern void udf_read_inode(struct inode *); > extern void udf_delete_inode(struct inode *); > extern void udf_clear_inode(struct inode *); > Index: linux-2.6/fs/udf/file.c > =================================================================== > --- linux-2.6.orig/fs/udf/file.c > +++ linux-2.6/fs/udf/file.c > @@ -215,5 +215,6 @@ const struct file_operations udf_file_op > }; > > const struct inode_operations udf_file_inode_operations = { > - .truncate = udf_truncate, > + .new_truncate = 1, > + .setattr = udf_setattr, > }; > > -- Jan Kara SUSE Labs, CR