Re: [PATCH 18/20] fs: omfs: Initialize filesystem timestamp ranges

[Bob Copeland]

On Mon, Jul 29, 2019 at 06:49:22PM -0700, Deepa Dinamani wrote:
> Fill in the appropriate limits to avoid inconsistencies
> in the vfs cached inode times when timestamps are
> outside the permitted range.
> 
> Signed-off-by: Deepa Dinamani 
> Cc: m...@bobcopeland.com
> Cc: linux-karma-de...@lists.sourceforge.net
> ---
>  fs/omfs/inode.c | 4 
>  1 file changed, 4 insertions(+)
> 
> diff --git a/fs/omfs/inode.c b/fs/omfs/inode.c
> index 08226a835ec3..b76ec6b88ded 100644
> --- a/fs/omfs/inode.c
> +++ b/fs/omfs/inode.c
> @@ -478,6 +478,10 @@ static int omfs_fill_super(struct super_block *sb, void 
> *data, int silent)
>  
>   sb->s_maxbytes = 0x;
>  
> + sb->s_time_gran = NSEC_PER_MSEC;
> + sb->s_time_min = 0;
> + sb->s_time_max = U64_MAX / MSEC_PER_SEC;
> +

I honestly don't know if it should be s64 rather than u64, but considering
that none of the devices with this filesystem ever exposed dates to users in
the negative era, it should be fine.

Acked-by: Bob Copeland 

-- 
Bob Copeland %% https://bobcopeland.com/

[PATCH 18/20] fs: omfs: Initialize filesystem timestamp ranges

[Deepa Dinamani]

Fill in the appropriate limits to avoid inconsistencies
in the vfs cached inode times when timestamps are
outside the permitted range.

Signed-off-by: Deepa Dinamani 
Cc: m...@bobcopeland.com
Cc: linux-karma-de...@lists.sourceforge.net
---
 fs/omfs/inode.c | 4 
 1 file changed, 4 insertions(+)

diff --git a/fs/omfs/inode.c b/fs/omfs/inode.c
index 08226a835ec3..b76ec6b88ded 100644
--- a/fs/omfs/inode.c
+++ b/fs/omfs/inode.c
@@ -478,6 +478,10 @@ static int omfs_fill_super(struct super_block *sb, void 
*data, int silent)
 
sb->s_maxbytes = 0x;
 
+   sb->s_time_gran = NSEC_PER_MSEC;
+   sb->s_time_min = 0;
+   sb->s_time_max = U64_MAX / MSEC_PER_SEC;
+
sb_set_blocksize(sb, 0x200);
 
bh = sb_bread(sb, 0);
-- 
2.17.1