Re: [PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Thomas Gleixner
On Wed, 14 Sep 2016, Deepa Dinamani wrote: > >> + * current_time - Return FS time > >> + * @inode: inode. > >> + * > >> + * Return the current time truncated to the time granularity supported by > >> + * the fs. > >> + * > >> + * Note that inode and inode->sb cannot be NULL. > >> + * Otherwise,

Re: [PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Thomas Gleixner
On Wed, 14 Sep 2016, Deepa Dinamani wrote: > >> + * current_time - Return FS time > >> + * @inode: inode. > >> + * > >> + * Return the current time truncated to the time granularity supported by > >> + * the fs. > >> + * > >> + * Note that inode and inode->sb cannot be NULL. > >> + * Otherwise,

Re: [PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Deepa Dinamani
>> + * current_time - Return FS time >> + * @inode: inode. >> + * >> + * Return the current time truncated to the time granularity supported by >> + * the fs. >> + * >> + * Note that inode and inode->sb cannot be NULL. >> + * Otherwise, the function warns and returns time without truncation. >> +

Re: [PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Deepa Dinamani
>> + * current_time - Return FS time >> + * @inode: inode. >> + * >> + * Return the current time truncated to the time granularity supported by >> + * the fs. >> + * >> + * Note that inode and inode->sb cannot be NULL. >> + * Otherwise, the function warns and returns time without truncation. >> +

Re: [PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Thomas Gleixner
On Wed, 14 Sep 2016, Deepa Dinamani wrote: > +/** > + * current_time - Return FS time > + * @inode: inode. > + * > + * Return the current time truncated to the time granularity supported by > + * the fs. > + * > + * Note that inode and inode->sb cannot be NULL. > + * Otherwise, the function warns

Re: [PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Thomas Gleixner
On Wed, 14 Sep 2016, Deepa Dinamani wrote: > +/** > + * current_time - Return FS time > + * @inode: inode. > + * > + * Return the current time truncated to the time granularity supported by > + * the fs. > + * > + * Note that inode and inode->sb cannot be NULL. > + * Otherwise, the function warns

[PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Deepa Dinamani
current_fs_time() is used for inode timestamps. Change the signature of the function to take inode pointer instead of superblock as per Linus's suggestion. Also, move the api under vfs as per the discussion on the thread: https://lkml.org/lkml/2016/6/9/36 . As per Arnd's suggestion on the

[PATCH v5 1/5] vfs: Add current_time() api

2016-09-14 Thread Deepa Dinamani
current_fs_time() is used for inode timestamps. Change the signature of the function to take inode pointer instead of superblock as per Linus's suggestion. Also, move the api under vfs as per the discussion on the thread: https://lkml.org/lkml/2016/6/9/36 . As per Arnd's suggestion on the