<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 30, 2022 at 5:50 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
...<br>
> > +static struct tst_test test = {<br>
> > +   .test_all = run,<br>
> > +   .setup = setup,<br>
> > +   .needs_root = 1,<br>
> > +   .mount_device = 1,<br>
> > +   .mntpoint = MNT_POINT,<br>
> > +   .all_filesystems = 1,<br>
> > +   .skip_filesystems = (const char *const[]) {<br>
> > +           "vfat",<br>
> > +           "exfat",<br>
> I was looking what's wrong with vfat and exfat.<br>
> statvfs.f_namemax returns 1530, which is obviously too long, thus valid_fname<br>
> obviously returns ENAMETOOLONG (36). Tested on 6.1.0-rc6-1.g4c01546-default.<br>
> I wonder why, isn't that a bug?<br>
<br>
To reply myself, both glibc and musl defines:<br>
statvfs->f_namemax = statfs->f_namelen;<br>
<br>
TL;DR: 6 * 255 = 1530 due names being in UTF-8:<br>
<br>
Therefore looking into kernel sources for statfs->f_namelen:<br>
<br>
include/linux/nls.h<br>
#define NLS_MAX_CHARSET_SIZE 6 /* for UTF-8 */<br>
<br>
=== exfat ===<br>
exfat/exfat_raw.h<br>
#define EXFAT_MAX_FILE_LEN 255<br>
<br>
exfat/super.c<br>
static int exfat_statfs(struct dentry *dentry, struct kstatfs *buf)<br>
{<br>
        ...<br>
    /* Unicode utf16 255 characters */<br>
    buf->f_namelen = EXFAT_MAX_FILE_LEN * NLS_MAX_CHARSET_SIZE;<br>
<br>
=== vfat ===<br>
include/uapi/linux/msdos_fs.h<br>
#define FAT_LFN_LEN 255     /* maximum long name length */<br>
<br>
fat/inode.c<br>
static int fat_statfs(struct dentry *dentry, struct kstatfs *buf)<br>
{<br>
        ...<br>
    buf->f_namelen =<br>
        (sbi->options.isvfat ? FAT_LFN_LEN : 12) * NLS_MAX_CHARSET_SIZE;<br>
<br>
=> i.e. for vfat without long filename support it'd be 72.<br>
<br>
How about<br>
1) don't skip exfat and vfat but just skip creating file with valid name? or<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Sure, I think this method is better.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
2) Add #define NLS_MAX_CHARSET_SIZE 6 and for vfat and exfat calculate<br>
length as: buf.f_namemax / NLS_MAX_CHARSET_SIZE - 1 ?<br>
<br>
Kind regards,<br>
Petr<br>
<br>
-- <br>
Mailing list info: <a href="https://lists.linux.it/listinfo/ltp" rel="noreferrer" target="_blank">https://lists.linux.it/listinfo/ltp</a><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>