xfs
[Top] [All Lists]

Re: [PATCH 1/5][TAKE3] fallocate() implementation on i86, x86_64 and pow

To: "Amit K. Arora" <aarora@xxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH 1/5][TAKE3] fallocate() implementation on i86, x86_64 and powerpc
From: Mingming Cao <cmm@xxxxxxxxxx>
Date: Tue, 15 May 2007 17:42:46 -0700
Cc: torvalds@xxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, suparna@xxxxxxxxxx
In-reply-to: <20070515200359.GA5834@xxxxxxxxxxxxxxxxxxxx>
Organization: IBM LTC
References: <20070329101010.7a2b8783.akpm@xxxxxxxxxxxxxxxxxxxx> <20070330071417.GI355@xxxxxxxxxxxxxxxxxxxxxxxx> <20070417125514.GA7574@xxxxxxxxxxxxxxxxxxxx> <20070418130600.GW5967@xxxxxxxxxxxxxxxxxxxx> <20070420135146.GA21352@xxxxxxxxxxxxxxxxxxxx> <20070420145918.GY355@xxxxxxxxxxxxxxxxxxxxxxxx> <20070424121632.GA10136@xxxxxxxxxxxxxxxxxxxx> <20070426175056.GA25321@xxxxxxxxxxxxxxxxxxxx> <20070515193722.GA3487@xxxxxxxxxxxxxxxxxxxx> <20070515195421.GA2948@xxxxxxxxxxxxxxxxxxxx> <20070515200359.GA5834@xxxxxxxxxxxxxxxxxxxx>
Reply-to: cmm@xxxxxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
On Wed, 2007-05-16 at 01:33 +0530, Amit K. Arora wrote:
> This patch implements sys_fallocate() and adds support on i386, x86_64
> and powerpc platforms.

> @@ -1137,6 +1148,8 @@ struct inode_operations {
>       ssize_t (*listxattr) (struct dentry *, char *, size_t);
>       int (*removexattr) (struct dentry *, const char *);
>       void (*truncate_range)(struct inode *, loff_t, loff_t);
> +     long (*fallocate)(struct inode *inode, int mode, loff_t offset,
> +                       loff_t len);
>  };

Does the return value from fallocate inode operation has to be *long*?
It's not consistent with the ext4_fallocate() define in patch 4/5, 

+int ext4_fallocate(struct inode *inode, int mode, loff_t offset, loff_t
len)

thus cause compile warnings.



Mingming


<Prev in Thread] Current Thread [Next in Thread>