Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1519943 > unrolled thread

[RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()

Started byDeepa Dinamani <deepa.kernel@gmail.com>
First post2016-11-11 19:10 +0100
Last post2016-11-11 20:00 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time() Deepa Dinamani <deepa.kernel@gmail.com> - 2016-11-11 19:10 +0100
    Re: [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by  current_time() Dave Kleikamp <dave.kleikamp@oracle.com> - 2016-11-11 19:50 +0100
      Re: [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time() Deepa Dinamani <deepa.kernel@gmail.com> - 2016-11-11 20:00 +0100

#1519943 — [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()

FromDeepa Dinamani <deepa.kernel@gmail.com>
Date2016-11-11 19:10 +0100
Subject[RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()
Message-ID<sCoqZ-7IG-13@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1519965 — Re: [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()

FromDave Kleikamp <dave.kleikamp@oracle.com>
Date2016-11-11 19:50 +0100
SubjectRe: [RESEND PATCH] fs: jfs: Replace CURRENT_TIME_SEC by current_time()
Message-ID<sCp3H-7YK-9@gated-at.bofh.it>
In reply to#1519943
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);
> 

[toc] | [prev] | [next] | [standalone]


#1519966

FromDeepa Dinamani <deepa.kernel@gmail.com>
Date2016-11-11 20:00 +0100
Message-ID<sCpdn-82Q-1@gated-at.bofh.it>
In reply to#1519965
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web