On Fri, Feb 12, 2016 at 5:57 AM, Arnd Bergmann arnd@arndb.de wrote:
On Friday 12 February 2016 01:45:47 Deepa Dinamani wrote:
ts = vfs_time_to_timespec(inode->i_mtime);
if (!timespec_equal(&ts, &now))
inode->i_mtime = timespec_to_vfs_time(now);
ts = vfs_time_to_timespec(inode->i_mtime);
if (!timespec_equal(&ts, &now))
inode->i_ctime = timespec_to_vfs_time(now);
The second one needs to be fs_time_to_timespec(inode->i_ctime), not i_mtime.
Yes, you are correct. I will wait for some consensus on the proposal to figure out which version to post again.
Thanks, -Deepa