On Sun, Dec 9, 2018 at 12:10 PM Theodore Y. Ts'o <tytso@xxxxxxx> wrote:
>
> Gabriel added the Unicode tables for case folding to the fs/nls
> directory. If you'd prefer that we put them somewhere else, we
> can; do you have a preference?
I have a really hard time judging, since I haven't seen the code, just
a random diffstat and shortlog.
First off, there is no such thing as "one" unicode table for case
folding. There are lots and lots of tables, and I'm not clear what
table it is all about.
For example, both OS X and Windows do some form of case folding on
unicode. They don't do the *same* folding, though.
There are also various locale variations to case folding. This is
where I thought your nls choice came from, but then you tried to imply
that there are no locale issues and that directories can just have a
single flag to enable/disable the folding.
In some locales, "SS" and "ß" (perhaps "SZ" too) will compare the same
in case-insensitivity. Crazy in general, and afaik modern unicode even
has a real upper-case "ß" so it's arguably legacy, but...
And that's all entirely independent of the issues with all the
combining characters, modifier letters, white-space, overlong utf8
questions, etc etc.
It's also easy to generate overlong utf-8 that decodes to '/', for
example. Some broken systems might consider that identical to a real
'/' and it matters for path lookup.
So what's the actual code? What rules did you happen to pick? Did you
take the windows rules as-is (I _think_ they may be documented) since
the primary target apparently is just samba performance?
And even if the answer is "we follow NTFS rules", which *version* of
NTFS folding rules are you using if you're trying to speed up samba,
for example? Because afaik they have changed over time.
Is the *only* target samba? You are never interested for local loads
like "oh, people want to run Wine and might need it" or the
application testing parts?
All of these matter.
For example, if it's some "ext4 special case just for samba", then
perhaps the logical place to put all this is just in fs/ext4/ and not
bother anybody else about it.
But if it might be useful as some generic "NTFS hashing" library, then
make it that.
Linus
