On 06/01/2010 02:36 PM, Christoph Hellwig wrote:
> On Tue, Jun 01, 2010 at 02:31:30PM +0300, Boaz Harrosh wrote:
>> +static int _do_truncate(struct inode *inode, loff_t newsize)
>> {
>> struct exofs_i_info *oi = exofs_i(inode);
>> int ret;
>>
>> inode->i_mtime = inode->i_ctime = CURRENT_TIME;
>>
>> + ret = exofs_oi_truncate(oi, (u64)newsize);
>> + EXOFS_DBGMSG("(0x%lx) size=0x%llx\n", inode->i_ino, newsize);
>> return ret;
>
> Maybe I'm beeing picky, but I really don't see the need for this
> function either, especially given that is has a single caller.
>
It's fine.
It must stay for a very hidden reason. In the pnfs tree, this inside
chunk is preformed from a callback pointer within the pnfs_recall mechanism.
The pnfs export layer has a recall facility that receives a pointer to
a doer. It will block all layouts, recall the layout, preform the doer
let in layouts, and return. So this is split this way to minimize maintenance
efforts.
See an old patch here:
http://git.open-osd.org/gitweb.cgi?p=linux-open-osd.git;a=commitdiff;h=ec3c599368550b0cf792738ae6448c932244df51
specifically the small hunk at inode.c
Thanks
Boaz
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
