> On Nov 17, 2015, at 1:07 AM, Arnd Bergmann <a...@arndb.de> wrote:
> 
>> On Tuesday 17 November 2015 17:04:42 deng.ch...@zte.com.cn wrote:
>> Many time_t issues in filesystems is involved with VFS i_mtime/i_ctime.
>> for example:
>> 
>>  static void hpfs_update_directory_times(struct inode *dir)
>>  {
>> -       time_t t = get_seconds();
>> +       time64_t t = ktime_get_real_seconds();
>> -       if (t == dir->i_mtime.tv_sec &&
>> -           t == dir->i_ctime.tv_sec)
>> +       if ((time_t)t == dir->i_mtime.tv_sec &&
>> +           (time_t)t == dir->i_ctime.tv_sec)
>>                 return;
>> Can I replace get_seconds with ktime_get_real_seconds first, and do a 
>> 64_to_32 cast so as to be compatible with VFS i_mtime/i_ctime like above?
>> Or just leave all the stuff which is involved with VFS i_mtime/i_ctime until 
>> then change of generic VFS code.
> 
> Better just leave it alone for now. Deepa is looking into the
> VFS changes, and I think it will make her work easier that way.
> 
Yes, I'm looking at vfs.
I haven't decided what to do with this yet. Let's leave it as is for now.

-Deepa
_______________________________________________
Y2038 mailing list
Y2038@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/y2038

Reply via email to