* [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()
@ 2016-11-11 18:00 Deepa Dinamani
2016-11-11 18:42 ` Dave Kleikamp
0 siblings, 1 reply; 3+ messages in thread
From: Deepa Dinamani @ 2016-11-11 18:00 UTC (permalink / raw)
To: linux-kernel, dave.kleikamp, jfs-discussion; +Cc: arnd, y2038
jfs uses nanosecond granularity for filesystem timestamps.
Only this assignment is not using nanosecond granularity.
Use current_time() to get the right granularity.
Signed-off-by: Deepa Dinamani <deepa.kernel@gmail.com>
Acked-by: Dave Kleikamp <dave.kleikamp@oracle.com>
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
---
fs/jfs/ioctl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/jfs/ioctl.c b/fs/jfs/ioctl.c
index 8653cac..b6fd1ff 100644
--- a/fs/jfs/ioctl.c
+++ b/fs/jfs/ioctl.c
@@ -121,7 +121,7 @@ long jfs_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
jfs_set_inode_flags(inode);
inode_unlock(inode);
- inode->i_ctime = CURRENT_TIME_SEC;
+ inode->i_ctime = current_time(inode);
mark_inode_dirty(inode);
setflags_out:
mnt_drop_write_file(filp);
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()
2016-11-11 18:00 [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time() Deepa Dinamani
@ 2016-11-11 18:42 ` Dave Kleikamp
2016-11-11 18:50 ` Deepa Dinamani
0 siblings, 1 reply; 3+ messages in thread
From: Dave Kleikamp @ 2016-11-11 18:42 UTC (permalink / raw)
To: Deepa Dinamani, linux-kernel, jfs-discussion; +Cc: arnd, y2038
On 11/11/2016 12:00 PM, Deepa Dinamani wrote:
> jfs uses nanosecond granularity for filesystem timestamps.
> Only this assignment is not using nanosecond granularity.
> Use current_time() to get the right granularity.
I had thought these were being handled as a group. I'll push this one
through the jfs tree.
Thanks,
Shaggy
>
> Signed-off-by: Deepa Dinamani <deepa.kernel@gmail.com>
> Acked-by: Dave Kleikamp <dave.kleikamp@oracle.com>
> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
> ---
> fs/jfs/ioctl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/jfs/ioctl.c b/fs/jfs/ioctl.c
> index 8653cac..b6fd1ff 100644
> --- a/fs/jfs/ioctl.c
> +++ b/fs/jfs/ioctl.c
> @@ -121,7 +121,7 @@ long jfs_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
>
> jfs_set_inode_flags(inode);
> inode_unlock(inode);
> - inode->i_ctime = CURRENT_TIME_SEC;
> + inode->i_ctime = current_time(inode);
> mark_inode_dirty(inode);
> setflags_out:
> mnt_drop_write_file(filp);
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()
2016-11-11 18:42 ` Dave Kleikamp
@ 2016-11-11 18:50 ` Deepa Dinamani
0 siblings, 0 replies; 3+ messages in thread
From: Deepa Dinamani @ 2016-11-11 18:50 UTC (permalink / raw)
To: Dave Kleikamp
Cc: Linux Kernel Mailing List, jfs-discussion, Arnd Bergmann,
y2038 Mailman List
On Fri, Nov 11, 2016 at 10:42 AM, Dave Kleikamp
<dave.kleikamp@oracle.com> wrote:
> On 11/11/2016 12:00 PM, Deepa Dinamani wrote:
>> jfs uses nanosecond granularity for filesystem timestamps.
>> Only this assignment is not using nanosecond granularity.
>> Use current_time() to get the right granularity.
>
> I had thought these were being handled as a group. I'll push this one
> through the jfs tree.
Yes, that was the original plan.
Then, Greg suggested pushing only the big patches as a group and
sending individual filesystem patches separately.
Big patches introducing current_time() have been merged in 4.9 rc1.
-Deepa
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-11-11 18:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-11 18:00 [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time() Deepa Dinamani
2016-11-11 18:42 ` Dave Kleikamp
2016-11-11 18:50 ` Deepa Dinamani
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox