From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D53353375C5; Sat, 2 May 2026 14:22:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777731738; cv=none; b=QICeMcLU+9uasy2VXA96DofITM4eFd2dfhGh0JjZUvz9YPBfJrKiiA/iXpCHLxji7hVO+7i80WtZ9PQ/4kx6fYWLbgSGf7WusoodZuvyzemynrQu+X250kCPmJPnxugaLsc5pGCeARWNEwVqpNTzxm0w7t7zbBa3jOSyoA7eeIU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777731738; c=relaxed/simple; bh=4PnVhA68/lwOR6kFOGBj/fhhmjpnw+nIMjeX3NeO8qI=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=OFyugg+XlLwqfXBzpe5IKr3A8kRQVlQGaHgzLrR0L3RIEvv6p+C/gfXhUjffgsqnkwEa2Mv3gKKz6lkG87m+lQIko/PBCfBotmQ7LBDLp6OkPaYEsJuNZU/bSszNGmitcC45UmZUPRfwrg1HwXcXHipuL5WSE2ozhXOUHpeS8BU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=b6+Jz5+t; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="b6+Jz5+t" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C7B40C2BCB3; Sat, 2 May 2026 14:22:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1777731738; bh=4PnVhA68/lwOR6kFOGBj/fhhmjpnw+nIMjeX3NeO8qI=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=b6+Jz5+tDXgXOlXvuRBJ0OOfPIeZX48o0SQPtL71eCwkWOt7kzjQ6Z8IpnM4wHhms inYzX7xxRer2/qgUi6naA23KdN/ahhDbCzUQpUnDNBxcUP4rPv9q4TC+kbCzuzN186 5l90C9yKDFdiIabfzQVPKa/7W2W6Tq3nuLuosOGaB9QncdMsXf6zDAVPcxILgqnoSr /FiD1YZAWkdbo1XQ1y/aYZkyzv7s36FIYVc6QEaRI0VgL4Zrs/7qDVI+sTrFEZB+vV zctzdYtGg7MdM6vpSsakFnVnVoqImUVLweM0qvXlHXRfDjRppoumAuwixKvrNIJR28 GsImvCWGQnOzQ== From: Chuck Lever Date: Sat, 02 May 2026 10:20:56 -0400 Subject: [PATCH v13 11/15] vboxsf: Implement fileattr_get for case sensitivity Precedence: bulk X-Mailing-List: linux-ext4@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260502-case-sensitivity-v13-11-aa853140311f@oracle.com> References: <20260502-case-sensitivity-v13-0-aa853140311f@oracle.com> In-Reply-To: <20260502-case-sensitivity-v13-0-aa853140311f@oracle.com> To: Al Viro , Christian Brauner , Jan Kara 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 , Roland Mainz X-Mailer: b4 0.16-dev X-Developer-Signature: v=1; a=openpgp-sha256; l=4902; i=chuck.lever@oracle.com; h=from:subject:message-id; bh=IS91PG/gteNIi+mr5kWAFzEwamMgenFgJebFqDZEBwA=; b=owEBbQKS/ZANAwAKATNqszNvZn+XAcsmYgBp9ghRGaq22rBmq5sMiRGQNGLEfSeErS/5j5Fw7 Yaeu3ir7z2JAjMEAAEKAB0WIQQosuWwEobfJDzyPv4zarMzb2Z/lwUCafYIUQAKCRAzarMzb2Z/ lwdgEACWTM+qGq6HJw1gIEOAfHMOm58Q5i9dlmUeo7OkrQii33AMabBCM0uJS22Qbl0GT5JAJv9 jG3a4c6DG8h1KzAqEgM4PneninsOxc2j5/AHZNNUM8QV0rmEcApr6BH8fZ0MgdZuTYtgkzCqWD7 iQgOTPXIXkdNj5iHw6GBSjuvXxQaJMCpj2tif9l7vWqBX39ut7Y1IQz4G7ncRQ55v3AeDhWFf9F 04xz21qcF+hIKFaYT9tAOPalMPm6ZW6CmHaVoApR2duac0+c46K5yL/ogXmQM/Eazh1xhz4JVnE FS3UsD+S00Aab8039V3Vd8WxIUDoPwk3mMiyw+Rhgjd3oWmoN4D1rpb4WM+OiwCRYuT+Dv4dS+H B7Wfgl2prIVE/2IPHLb6Wve0kbbWSGxlIB2HtYtqWxqeuZX3MxsbhPyUx5mlwdrZ21l4iDgfIW2 UiYHpuJL7tNH6K3kjbela82VbwcPCKYHOQPuO5QxwjmwQO6TX1GDiRqKgLoQRuQQw7qlQB3FrWM w4i0GgL8lKgy3L11CRJ0Dn/vWF4qUqkeMywmDYNoZyu5HDqt+blYHnwFB9tglOdrcR9LhD6uWHa 1aauVyHhkQI87BwUDShg6tkp58hKJJYpqCFFbjRADpSDDiY8B4BbY76to+7LCXkIBff09NKnISD ay4CH9Le2lUwuzQ== X-Developer-Key: i=chuck.lever@oracle.com; a=openpgp; fpr=28B2E5B01286DF243CF23EFE336AB3336F667F97 From: Chuck Lever Upper layers such as NFSD need a way to query whether a filesystem handles filenames in a case-sensitive manner. Report VirtualBox shared folder case handling behavior via the FS_XFLAG_CASEFOLD flag. The case sensitivity property is queried from the VirtualBox host service at mount time and cached in struct vboxsf_sbi. The host determines case sensitivity based on the underlying host filesystem (for example, Windows NTFS is case-insensitive while Linux ext4 is case-sensitive). VirtualBox shared folders always preserve filename case exactly as provided by the guest. The host interface does not expose a separate case-preserving property; leaving FS_XFLAG_CASENONPRESERVING unset reports the POSIX-default case-preserving behavior, which matches vboxsf semantics. The callback is registered in all three inode_operations structures (directory, file, and symlink) to ensure consistent reporting across all inode types. Reviewed-by: Roland Mainz Signed-off-by: Chuck Lever --- fs/vboxsf/dir.c | 1 + fs/vboxsf/file.c | 6 ++++-- fs/vboxsf/super.c | 7 +++++++ fs/vboxsf/utils.c | 30 ++++++++++++++++++++++++++++++ fs/vboxsf/vfsmod.h | 6 ++++++ 5 files changed, 48 insertions(+), 2 deletions(-) diff --git a/fs/vboxsf/dir.c b/fs/vboxsf/dir.c index 42bedc4ec7af..c5bd3271aa96 100644 --- a/fs/vboxsf/dir.c +++ b/fs/vboxsf/dir.c @@ -477,4 +477,5 @@ const struct inode_operations vboxsf_dir_iops = { .symlink = vboxsf_dir_symlink, .getattr = vboxsf_getattr, .setattr = vboxsf_setattr, + .fileattr_get = vboxsf_fileattr_get, }; diff --git a/fs/vboxsf/file.c b/fs/vboxsf/file.c index 7a7a3fbb2651..943953867e18 100644 --- a/fs/vboxsf/file.c +++ b/fs/vboxsf/file.c @@ -222,7 +222,8 @@ const struct file_operations vboxsf_reg_fops = { const struct inode_operations vboxsf_reg_iops = { .getattr = vboxsf_getattr, - .setattr = vboxsf_setattr + .setattr = vboxsf_setattr, + .fileattr_get = vboxsf_fileattr_get, }; static int vboxsf_read_folio(struct file *file, struct folio *folio) @@ -389,5 +390,6 @@ static const char *vboxsf_get_link(struct dentry *dentry, struct inode *inode, } const struct inode_operations vboxsf_lnk_iops = { - .get_link = vboxsf_get_link + .get_link = vboxsf_get_link, + .fileattr_get = vboxsf_fileattr_get, }; diff --git a/fs/vboxsf/super.c b/fs/vboxsf/super.c index a618cb093e00..a61fbab51d37 100644 --- a/fs/vboxsf/super.c +++ b/fs/vboxsf/super.c @@ -185,6 +185,13 @@ static int vboxsf_fill_super(struct super_block *sb, struct fs_context *fc) if (err) goto fail_unmap; + /* + * A failed query leaves sbi->case_insensitive false, so the + * mount defaults to reporting case-sensitive behavior. Do not + * fail the mount over an advisory attribute. + */ + vboxsf_query_case_sensitive(sbi); + sb->s_magic = VBOXSF_SUPER_MAGIC; sb->s_blocksize = 1024; sb->s_maxbytes = MAX_LFS_FILESIZE; diff --git a/fs/vboxsf/utils.c b/fs/vboxsf/utils.c index 440e8c50629d..298bfc93255c 100644 --- a/fs/vboxsf/utils.c +++ b/fs/vboxsf/utils.c @@ -11,6 +11,7 @@ #include #include #include +#include #include "vfsmod.h" struct inode *vboxsf_new_inode(struct super_block *sb) @@ -567,3 +568,32 @@ int vboxsf_dir_read_all(struct vboxsf_sbi *sbi, struct vboxsf_dir_info *sf_d, return err; } + +int vboxsf_query_case_sensitive(struct vboxsf_sbi *sbi) +{ + struct shfl_volinfo volinfo = {}; + u32 buf_len; + int err; + + buf_len = sizeof(volinfo); + err = vboxsf_fsinfo(sbi->root, 0, SHFL_INFO_GET | SHFL_INFO_VOLUME, + &buf_len, &volinfo); + if (err) + return err; + if (buf_len < sizeof(volinfo)) + return 0; + + sbi->case_insensitive = !volinfo.properties.case_sensitive; + return 0; +} + +int vboxsf_fileattr_get(struct dentry *dentry, struct file_kattr *fa) +{ + struct vboxsf_sbi *sbi = VBOXSF_SBI(dentry->d_sb); + + if (sbi->case_insensitive) { + fa->fsx_xflags |= FS_XFLAG_CASEFOLD; + fa->flags |= FS_CASEFOLD_FL; + } + return 0; +} diff --git a/fs/vboxsf/vfsmod.h b/fs/vboxsf/vfsmod.h index 05973eb89d52..b61afd0ce842 100644 --- a/fs/vboxsf/vfsmod.h +++ b/fs/vboxsf/vfsmod.h @@ -47,6 +47,7 @@ struct vboxsf_sbi { u32 next_generation; u32 root; int bdi_id; + bool case_insensitive; }; /* per-inode information */ @@ -111,6 +112,11 @@ void vboxsf_dir_info_free(struct vboxsf_dir_info *p); int vboxsf_dir_read_all(struct vboxsf_sbi *sbi, struct vboxsf_dir_info *sf_d, u64 handle); +int vboxsf_query_case_sensitive(struct vboxsf_sbi *sbi); + +struct file_kattr; +int vboxsf_fileattr_get(struct dentry *dentry, struct file_kattr *fa); + /* from vboxsf_wrappers.c */ int vboxsf_connect(void); void vboxsf_disconnect(void); -- 2.53.0