From: Chuck Lever <cel@kernel.org>
To: Al Viro <viro@zeniv.linux.org.uk>,
Christian Brauner <brauner@kernel.org>, Jan Kara <jack@suse.cz>
Cc: linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org,
linux-xfs@vger.kernel.org, linux-cifs@vger.kernel.org,
linux-nfs@vger.kernel.org, linux-api@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net,
hirofumi@mail.parknet.co.jp, linkinjeon@kernel.org,
sj1557.seo@samsung.com, yuezhang.mo@sony.com,
almaz.alexandrovich@paragon-software.com, slava@dubeyko.com,
glaubitz@physik.fu-berlin.de, frank.li@vivo.com, tytso@mit.edu,
adilger.kernel@dilger.ca, cem@kernel.org, sfrench@samba.org,
pc@manguebit.org, ronniesahlberg@gmail.com,
sprasad@microsoft.com, trondmy@kernel.org, anna@kernel.org,
jaegeuk@kernel.org, chao@kernel.org, hansg@kernel.org,
senozhatsky@chromium.org, Chuck Lever <chuck.lever@oracle.com>,
Roland Mainz <roland.mainz@nrubsig.org>
Subject: [PATCH v14 15/15] ksmbd: Report filesystem case sensitivity via FS_ATTRIBUTE_INFORMATION
Date: Thu, 07 May 2026 04:53:08 -0400 [thread overview]
Message-ID: <20260507-case-sensitivity-v14-15-e62cc8200435@oracle.com> (raw)
In-Reply-To: <20260507-case-sensitivity-v14-0-e62cc8200435@oracle.com>
From: Chuck Lever <chuck.lever@oracle.com>
FS_ATTRIBUTE_INFORMATION responses have always reported
FILE_CASE_SENSITIVE_SEARCH and FILE_CASE_PRESERVED_NAMES
unconditionally. Case-insensitive filesystems like exFAT, and
casefolded directories on ext4 or f2fs, have no way to signal
their actual semantics to SMB clients.
Now that filesystems expose case behavior through ->fileattr_get,
query it via vfs_fileattr_get() and translate the FS_XFLAG_CASEFOLD
and FS_XFLAG_CASENONPRESERVING flags into the corresponding SMB
attributes. Filesystems without ->fileattr_get continue reporting
default POSIX behavior (case-sensitive, case-preserving).
SMB's FS_ATTRIBUTE_INFORMATION reports per-share attributes from
the share root, not per-file. Shares mixing casefold and
non-casefold directories report the root directory's behavior.
Acked-by: Namjae Jeon <linkinjeon@kernel.org>
Reviewed-by: Roland Mainz <roland.mainz@nrubsig.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
---
fs/smb/server/smb2pdu.c | 30 ++++++++++++++++++++++++------
1 file changed, 24 insertions(+), 6 deletions(-)
diff --git a/fs/smb/server/smb2pdu.c b/fs/smb/server/smb2pdu.c
index ee32e61b6d3c..cf0bc453a036 100644
--- a/fs/smb/server/smb2pdu.c
+++ b/fs/smb/server/smb2pdu.c
@@ -14,6 +14,7 @@
#include <linux/falloc.h>
#include <linux/mount.h>
#include <linux/filelock.h>
+#include <linux/fileattr.h>
#include "glob.h"
#include "smbfsctl.h"
@@ -5541,16 +5542,33 @@ static int smb2_get_info_filesystem(struct ksmbd_work *work,
case FS_ATTRIBUTE_INFORMATION:
{
FILE_SYSTEM_ATTRIBUTE_INFO *info;
+ struct file_kattr fa = {};
size_t sz;
+ u32 attrs;
+ int err;
info = (FILE_SYSTEM_ATTRIBUTE_INFO *)rsp->Buffer;
- info->Attributes = cpu_to_le32(FILE_SUPPORTS_OBJECT_IDS |
- FILE_PERSISTENT_ACLS |
- FILE_UNICODE_ON_DISK |
- FILE_CASE_PRESERVED_NAMES |
- FILE_CASE_SENSITIVE_SEARCH |
- FILE_SUPPORTS_BLOCK_REFCOUNTING);
+ attrs = FILE_SUPPORTS_OBJECT_IDS |
+ FILE_PERSISTENT_ACLS |
+ FILE_UNICODE_ON_DISK |
+ FILE_SUPPORTS_BLOCK_REFCOUNTING;
+ err = vfs_fileattr_get(path.dentry, &fa);
+ /*
+ * -EINVAL, -EOPNOTSUPP: ntfs-3g and other FUSE
+ * filesystems that lack FS_IOC_FSGETXATTR support.
+ */
+ if (err && err != -ENOIOCTLCMD && err != -ENOTTY &&
+ err != -EINVAL && err != -EOPNOTSUPP) {
+ path_put(&path);
+ return err;
+ }
+ if (!(fa.fsx_xflags & FS_XFLAG_CASEFOLD))
+ attrs |= FILE_CASE_SENSITIVE_SEARCH;
+ if (!(fa.fsx_xflags & FS_XFLAG_CASENONPRESERVING))
+ attrs |= FILE_CASE_PRESERVED_NAMES;
+
+ info->Attributes = cpu_to_le32(attrs);
info->Attributes |= cpu_to_le32(server_conf.share_fake_fscaps);
if (test_share_config_flag(work->tcon->share_conf,
--
2.53.0
WARNING: multiple messages have this Message-ID (diff)
From: Chuck Lever via Linux-f2fs-devel <linux-f2fs-devel@lists.sourceforge.net>
To: Al Viro <viro@zeniv.linux.org.uk>,
Christian Brauner <brauner@kernel.org>, Jan Kara <jack@suse.cz>
Cc: pc@manguebit.org, yuezhang.mo@sony.com, cem@kernel.org,
Roland Mainz <roland.mainz@nrubsig.org>,
almaz.alexandrovich@paragon-software.com,
adilger.kernel@dilger.ca, linux-cifs@vger.kernel.org,
sfrench@samba.org, slava@dubeyko.com, linux-ext4@vger.kernel.org,
linkinjeon@kernel.org, sprasad@microsoft.com, frank.li@vivo.com,
ronniesahlberg@gmail.com, glaubitz@physik.fu-berlin.de,
jaegeuk@kernel.org, hirofumi@mail.parknet.co.jp,
linux-nfs@vger.kernel.org, tytso@mit.edu,
linux-api@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net,
linux-xfs@vger.kernel.org, senozhatsky@chromium.org,
Chuck Lever <chuck.lever@oracle.com>,
hansg@kernel.org, anna@kernel.org, linux-fsdevel@vger.kernel.org,
sj1557.seo@samsung.com, trondmy@kernel.org
Subject: [f2fs-dev] [PATCH v14 15/15] ksmbd: Report filesystem case sensitivity via FS_ATTRIBUTE_INFORMATION
Date: Thu, 07 May 2026 04:53:08 -0400 [thread overview]
Message-ID: <20260507-case-sensitivity-v14-15-e62cc8200435@oracle.com> (raw)
In-Reply-To: <20260507-case-sensitivity-v14-0-e62cc8200435@oracle.com>
From: Chuck Lever <chuck.lever@oracle.com>
FS_ATTRIBUTE_INFORMATION responses have always reported
FILE_CASE_SENSITIVE_SEARCH and FILE_CASE_PRESERVED_NAMES
unconditionally. Case-insensitive filesystems like exFAT, and
casefolded directories on ext4 or f2fs, have no way to signal
their actual semantics to SMB clients.
Now that filesystems expose case behavior through ->fileattr_get,
query it via vfs_fileattr_get() and translate the FS_XFLAG_CASEFOLD
and FS_XFLAG_CASENONPRESERVING flags into the corresponding SMB
attributes. Filesystems without ->fileattr_get continue reporting
default POSIX behavior (case-sensitive, case-preserving).
SMB's FS_ATTRIBUTE_INFORMATION reports per-share attributes from
the share root, not per-file. Shares mixing casefold and
non-casefold directories report the root directory's behavior.
Acked-by: Namjae Jeon <linkinjeon@kernel.org>
Reviewed-by: Roland Mainz <roland.mainz@nrubsig.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
---
fs/smb/server/smb2pdu.c | 30 ++++++++++++++++++++++++------
1 file changed, 24 insertions(+), 6 deletions(-)
diff --git a/fs/smb/server/smb2pdu.c b/fs/smb/server/smb2pdu.c
index ee32e61b6d3c..cf0bc453a036 100644
--- a/fs/smb/server/smb2pdu.c
+++ b/fs/smb/server/smb2pdu.c
@@ -14,6 +14,7 @@
#include <linux/falloc.h>
#include <linux/mount.h>
#include <linux/filelock.h>
+#include <linux/fileattr.h>
#include "glob.h"
#include "smbfsctl.h"
@@ -5541,16 +5542,33 @@ static int smb2_get_info_filesystem(struct ksmbd_work *work,
case FS_ATTRIBUTE_INFORMATION:
{
FILE_SYSTEM_ATTRIBUTE_INFO *info;
+ struct file_kattr fa = {};
size_t sz;
+ u32 attrs;
+ int err;
info = (FILE_SYSTEM_ATTRIBUTE_INFO *)rsp->Buffer;
- info->Attributes = cpu_to_le32(FILE_SUPPORTS_OBJECT_IDS |
- FILE_PERSISTENT_ACLS |
- FILE_UNICODE_ON_DISK |
- FILE_CASE_PRESERVED_NAMES |
- FILE_CASE_SENSITIVE_SEARCH |
- FILE_SUPPORTS_BLOCK_REFCOUNTING);
+ attrs = FILE_SUPPORTS_OBJECT_IDS |
+ FILE_PERSISTENT_ACLS |
+ FILE_UNICODE_ON_DISK |
+ FILE_SUPPORTS_BLOCK_REFCOUNTING;
+ err = vfs_fileattr_get(path.dentry, &fa);
+ /*
+ * -EINVAL, -EOPNOTSUPP: ntfs-3g and other FUSE
+ * filesystems that lack FS_IOC_FSGETXATTR support.
+ */
+ if (err && err != -ENOIOCTLCMD && err != -ENOTTY &&
+ err != -EINVAL && err != -EOPNOTSUPP) {
+ path_put(&path);
+ return err;
+ }
+ if (!(fa.fsx_xflags & FS_XFLAG_CASEFOLD))
+ attrs |= FILE_CASE_SENSITIVE_SEARCH;
+ if (!(fa.fsx_xflags & FS_XFLAG_CASENONPRESERVING))
+ attrs |= FILE_CASE_PRESERVED_NAMES;
+
+ info->Attributes = cpu_to_le32(attrs);
info->Attributes |= cpu_to_le32(server_conf.share_fake_fscaps);
if (test_share_config_flag(work->tcon->share_conf,
--
2.53.0
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
next prev parent reply other threads:[~2026-05-07 8:55 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-07 8:52 [PATCH v14 00/15] Exposing case folding behavior Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:52 ` [PATCH v14 01/15] fs: Move file_kattr initialization to callers Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:52 ` [PATCH v14 02/15] fs: Add case sensitivity flags to file_kattr Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:52 ` [PATCH v14 03/15] fat: Implement fileattr_get for case sensitivity Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-20 14:31 ` Mark Brown
2026-05-20 14:39 ` Chuck Lever
2026-05-20 14:39 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-20 14:54 ` Mark Brown
2026-05-20 15:12 ` Chuck Lever
2026-05-20 15:12 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-20 15:19 ` Mark Brown
2026-05-20 16:58 ` Chuck Lever
2026-05-20 16:58 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-20 17:11 ` Mark Brown
2026-05-20 17:30 ` Chuck Lever
2026-05-20 17:30 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:52 ` [PATCH v14 04/15] exfat: " Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:52 ` [PATCH v14 05/15] ntfs3: " Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:52 ` [PATCH v14 06/15] hfs: " Chuck Lever
2026-05-07 8:52 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 07/15] hfsplus: Report case sensitivity in fileattr_get Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 08/15] xfs: " Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 09/15] cifs: Implement fileattr_get for case sensitivity Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 10/15] nfs: " Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 11/15] vboxsf: " Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 12/15] isofs: " Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 13/15] nfsd: Report export case-folding via NFSv3 PATHCONF Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` [PATCH v14 14/15] nfsd: Implement NFSv4 FATTR4_CASE_INSENSITIVE and FATTR4_CASE_PRESERVING Chuck Lever
2026-05-07 8:53 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-07 8:53 ` Chuck Lever [this message]
2026-05-07 8:53 ` [f2fs-dev] [PATCH v14 15/15] ksmbd: Report filesystem case sensitivity via FS_ATTRIBUTE_INFORMATION Chuck Lever via Linux-f2fs-devel
2026-05-11 14:02 ` [PATCH v14 00/15] Exposing case folding behavior Christian Brauner
2026-05-11 14:02 ` [f2fs-dev] " Christian Brauner via Linux-f2fs-devel
2026-05-11 14:07 ` Chuck Lever
2026-05-11 14:07 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-11 14:55 ` Christian Brauner
2026-05-11 14:55 ` [f2fs-dev] " Christian Brauner via Linux-f2fs-devel
2026-05-16 6:43 ` Cedric Blancher
2026-05-16 6:43 ` [f2fs-dev] " Cedric Blancher
2026-05-16 14:48 ` Chuck Lever
2026-05-16 14:48 ` [f2fs-dev] " Chuck Lever via Linux-f2fs-devel
2026-05-11 14:02 ` Christian Brauner
2026-05-11 14:02 ` [f2fs-dev] " Christian Brauner via Linux-f2fs-devel
2026-05-11 14:55 ` Christian Brauner
2026-05-11 14:55 ` [f2fs-dev] " Christian Brauner via Linux-f2fs-devel
2026-06-24 8:59 ` patchwork-bot+f2fs
2026-06-24 8:59 ` patchwork-bot+f2fs--- via Linux-f2fs-devel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260507-case-sensitivity-v14-15-e62cc8200435@oracle.com \
--to=cel@kernel.org \
--cc=adilger.kernel@dilger.ca \
--cc=almaz.alexandrovich@paragon-software.com \
--cc=anna@kernel.org \
--cc=brauner@kernel.org \
--cc=cem@kernel.org \
--cc=chao@kernel.org \
--cc=chuck.lever@oracle.com \
--cc=frank.li@vivo.com \
--cc=glaubitz@physik.fu-berlin.de \
--cc=hansg@kernel.org \
--cc=hirofumi@mail.parknet.co.jp \
--cc=jack@suse.cz \
--cc=jaegeuk@kernel.org \
--cc=linkinjeon@kernel.org \
--cc=linux-api@vger.kernel.org \
--cc=linux-cifs@vger.kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=linux-xfs@vger.kernel.org \
--cc=pc@manguebit.org \
--cc=roland.mainz@nrubsig.org \
--cc=ronniesahlberg@gmail.com \
--cc=senozhatsky@chromium.org \
--cc=sfrench@samba.org \
--cc=sj1557.seo@samsung.com \
--cc=slava@dubeyko.com \
--cc=sprasad@microsoft.com \
--cc=trondmy@kernel.org \
--cc=tytso@mit.edu \
--cc=viro@zeniv.linux.org.uk \
--cc=yuezhang.mo@sony.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.