linux-kernel - 回复: [PATCH v2 2/3] hfs: correct superblock flags

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <SEZPR06MB5269D12DE8D4F48AF96E7409E867A@SEZPR06MB5269.apcprd06.prod.outlook.com>
Date: Wed, 28 May 2025 16:37:57 +0000
From: 李扬韬 <frank.li@...o.com>
To: Viacheslav Dubeyko <slava@...eyko.com>, "glaubitz@...sik.fu-berlin.de"
	<glaubitz@...sik.fu-berlin.de>
CC: "linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Slava.Dubeyko@....com" <Slava.Dubeyko@....com>
Subject:
 回复: [PATCH v2 2/3] hfs: correct superblock flags

Hi Slava,

> I am slightly confused by comment. Does it mean that the fix introduces more errors? It looks like we need to have more clear explanation of the fix here.

I'll update commit msg.

> s->s_flags |= SB_NODIRATIME | SB_NOATIME;

IIUC, SB_NOATIME > SB_NODIRATIME.

So we should correct flags in smb, ceph.

2091 bool atime_needs_update(const struct path *path, struct inode *inode)
2092 {
2093         struct vfsmount *mnt = path->mnt;
2094         struct timespec64 now, atime;
2095
2096         if (inode->i_flags & S_NOATIME)
2097                 return false;
2098
2099         /* Atime updates will likely cause i_uid and i_gid to be written
2100         ¦* back improprely if their true value is unknown to the vfs.
2101         ¦*/
2102         if (HAS_UNMAPPED_ID(mnt_idmap(mnt), inode))
2103                 return false;
2104
2105         if (IS_NOATIME(inode))
2106                 return false;
2107         if ((inode->i_sb->s_flags & SB_NODIRATIME) && S_ISDIR(inode->i_mode))
2108                 return false;

Thx,
Yangtao