From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail138.messagelabs.com (mail138.messagelabs.com [216.82.249.35]) by kanga.kvack.org (Postfix) with ESMTP id D43916B00CA for ; Wed, 23 Nov 2011 05:38:32 -0500 (EST) Date: Wed, 23 Nov 2011 11:38:29 +0100 From: Christoph Hellwig Subject: Re: [PATCH 2/2] fs: wire up .truncate_range and .fallocate Message-ID: <20111123103829.GA23168@lst.de> References: <1322038412-29013-1-git-send-email-amwang@redhat.com> <1322038412-29013-2-git-send-email-amwang@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1322038412-29013-2-git-send-email-amwang@redhat.com> Sender: owner-linux-mm@kvack.org List-ID: To: Cong Wang Cc: linux-kernel@vger.kernel.org, akpm@linux-foundation.org, Hugh Dickins , Al Viro , Christoph Hellwig , Matthew Wilcox , Andrea Arcangeli , Rik van Riel , Mel Gorman , Minchan Kim , Johannes Weiner , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org On Wed, Nov 23, 2011 at 04:53:31PM +0800, Cong Wang wrote: > +int vmtruncate_file_range(struct file *file, struct inode *inode, > + loff_t lstart, loff_t lend) We can always get the inode from file->f_path.dentry->d_inode, thus passing both of them doesn't make much sense. > + if (!file->f_op->fallocate) > return -ENOSYS; > > mutex_lock(&inode->i_mutex); > inode_dio_wait(inode); > unmap_mapping_range(mapping, holebegin, holelen, 1); > - inode->i_op->truncate_range(inode, lstart, lend); > + mutex_unlock(&inode->i_mutex); > + > + err = do_fallocate(file, FALLOC_FL_KEEP_SIZE|FALLOC_FL_PUNCH_HOLE, > + holebegin, holelen); > + if (err) > + return err; > + > + mutex_lock(&inode->i_mutex); > /* unmap again to remove racily COWed private pages */ > unmap_mapping_range(mapping, holebegin, holelen, 1); > mutex_unlock(&inode->i_mutex); I suspect this should be turned inside out, just we do for normal truncate. That is instead of keeping vmtruncate(_file)_range we should have a truncate_pagecache_range do to the actual zapping, closely mirroring what we do for truncate. In the best case we'd even implement the regular truncate ones on top of the range version. It also seems like all fallocate implementaions for far got away without the unmap_mapping_range, so either people didn't test them hard enough, or tmpfs doesn't need it either. I fear the former is true. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org