On Sat, Sep 25, 2010 at 07:42:48PM -0400, Christoph Hellwig wrote: > As mentioned before I think xfs_trans_ichgtime should go into > xfs_trans_inode.c/xfs_trans.h. Instead of keeping xfs_ichgtime around > for one caller it might be easier to just opencode it similar to > what other filesystems do, e.g. just: > > inode->i_mtime = inode->i_ctime = current_fs_time(inode->i_sb); > mark_inode_dirty_sync(inode); > > there's really no point in doing the check for an actual time change > first as it's a relatively rare operation. Ok. I'll respin it to do this. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
