lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <ECBC97E7-53C5-4B4C-BC4C-1FCDC4C371B9@dilger.ca> Date: Wed, 4 Sep 2019 12:39:22 -0600 From: Andreas Dilger <adilger@...ger.ca> To: Deepa Dinamani <deepa.kernel@...il.com> Cc: arnd@...db.de, linux-kernel@...r.kernel.org, tytso@....edu, adilger.kernel@...ger.ca, cai@....pw, jlayton@...nel.org, linux-ext4@...r.kernel.org, linux-fsdevel@...r.kernel.org, viro@...iv.linux.org.uk Subject: Re: [PATCH] ext4: Reduce ext4 timestamp warnings On Sep 4, 2019, at 09:02, Deepa Dinamani <deepa.kernel@...il.com> wrote: > > When ext4 file systems were created intentionally with 128 byte inodes, > the rate-limited warning of eventual possible timestamp overflow are > still emitted rather frequently. Remove the warning for now. > > Discussion for whether any warning is needed, > and where it should be emitted, can be found at > https://lore.kernel.org/lkml/1567523922.5576.57.camel@lca.pw/. > I can post a separate follow-up patch after the conclusion. > > Reported-by: Qian Cai <cai@....pw> > Signed-off-by: Deepa Dinamani <deepa.kernel@...il.com> I'd be in favor of a severely rare-limited warning in the actual case that Y2038 timestamps cannot be stored, but the current message is too verbose for now and I agree it is better to remove it while discussions on the best solution are underway. Reviewed-by: Andreas Dilger <adilger@...ger.ca> > --- > fs/ext4/ext4.h | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h > index 9e3ae3be3de9..24b14bd3feab 100644 > --- a/fs/ext4/ext4.h > +++ b/fs/ext4/ext4.h > @@ -833,10 +833,8 @@ do { \ > (raw_inode)->xtime ## _extra = \ > ext4_encode_extra_time(&(inode)->xtime); \ > } \ > - else {\ > + else \ > (raw_inode)->xtime = cpu_to_le32(clamp_t(int32_t, (inode)->xtime.tv_sec, S32_MIN, S32_MAX)); \ > - ext4_warning_inode(inode, "inode does not support timestamps beyond 2038"); \ > - } \ > } while (0) > > #define EXT4_EINODE_SET_XTIME(xtime, einode, raw_inode) \ > -- > 2.17.1 >
Powered by blists - more mailing lists