[<prev] [next>] [day] [month] [year] [list]
Message-ID: <000001d8c816$b7ad4010$2707c030$@samsung.com>
Date: Wed, 14 Sep 2022 17:48:10 +0900
From: "Sungjong Seo" <sj1557.seo@...sung.com>
To: "'Rover'" <739817562@...com>,
"'linkinjeon'" <linkinjeon@...nel.org>
Cc: "'linux-kernel'" <linux-kernel@...r.kernel.org>,
"'linux-fsdevel'" <linux-fsdevel@...r.kernel.org>,
<sj1557.seo@...sung.com>
Subject: RE: [PATCH v1] exfat: remove the code that sets FileAttributes when
renaming
Hi, Rover,
This patch seems to violate the exFAT specification below.
Please refer to the description for ATTR_ARCHIVE in FAT32 Spec.
* Archive
This field is mandatory and conforms to the MS-DOS definition.
* ATTR_ARCHIVE
This attribute supports backup utilities. This bit is set by the FAT file
system driver when a file is created, renamed, or written to. Backup
utilities may use this attribute to indicate which files on the volume
have been modified since the last time that a backup was performed.
Thanks.
B.R.
Sungjong Seo
> When renaming, FileAttributes remain unchanged, do not need to be
> set, so the code that sets FileAttributes is unneeded, remove it.
>
> Signed-off-by: rover <739817562@...com>
> ---
> fs/exfat/namei.c | 12 ------------
> 1 file changed, 12 deletions(-)
>
> diff --git a/fs/exfat/namei.c b/fs/exfat/namei.c
> index b617bebc3d0f..5ffaf553155e 100644
> --- a/fs/exfat/namei.c
> +++ b/fs/exfat/namei.c
> @@ -1031,10 +1031,6 @@ static int exfat_rename_file(struct inode *inode,
> struct exfat_chain *p_dir,
> return -EIO;
>
> *epnew = *epold;
> - if (exfat_get_entry_type(epnew) == TYPE_FILE) {
> - epnew->dentry.file.attr |=
> cpu_to_le16(ATTR_ARCHIVE);
> - ei->attr |= ATTR_ARCHIVE;
> - }
> exfat_update_bh(new_bh, sync);
> brelse(old_bh);
> brelse(new_bh);
> @@ -1063,10 +1059,6 @@ static int exfat_rename_file(struct inode *inode,
> struct exfat_chain *p_dir,
> ei->dir = *p_dir;
> ei->entry = newentry;
> } else {
> - if (exfat_get_entry_type(epold) == TYPE_FILE) {
> - epold->dentry.file.attr |=
> cpu_to_le16(ATTR_ARCHIVE);
> - ei->attr |= ATTR_ARCHIVE;
> - }
> exfat_update_bh(old_bh, sync);
> brelse(old_bh);
> ret = exfat_init_ext_entry(inode, p_dir, oldentry,
> @@ -1112,10 +1104,6 @@ static int exfat_move_file(struct inode *inode,
> struct exfat_chain *p_olddir,
> return -EIO;
>
> *epnew = *epmov;
> - if (exfat_get_entry_type(epnew) == TYPE_FILE) {
> - epnew->dentry.file.attr |= cpu_to_le16(ATTR_ARCHIVE);
> - ei->attr |= ATTR_ARCHIVE;
> - }
> exfat_update_bh(new_bh, IS_DIRSYNC(inode));
> brelse(mov_bh);
> brelse(new_bh);
> --
> 2.25.1
Powered by blists - more mailing lists