From: Chuck Lever <chuck.lever@oracle.com>
To: Cedric Blancher <cedric.blancher@gmail.com>
Cc: Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [patch] mount.nfs: Add support for nfs://-URLs ...
Date: Tue, 10 Dec 2024 09:15:36 -0500 [thread overview]
Message-ID: <71e31b5b-1318-4d5f-a96c-abd0b2fc0018@oracle.com> (raw)
In-Reply-To: <CALXu0Udmn=AqRBAcBPf8=VAP3KN4f-vnEJFLg3OnkyRdT3b9rw@mail.gmail.com>
On 12/10/24 7:13 AM, Cedric Blancher wrote:
> On Sun, 8 Dec 2024 at 23:12, NeilBrown <neilb@suse.de> wrote:
>>> +
>>> + /*
>>> + * |pnu.uctx->path| is in UTF-8, but we need the data
>>> + * in the current local locale's encoding, as mount(2)
>>> + * does not have something like a |MS_UTF8_SPEC| flag
>>> + * to indicate that the input path is in UTF-8,
>>> + * independently of the current locale
>>> + */
>>
>> I don't understand this comment at all.
>> mount(2) doesn't care about locale (as far as I know). The "source" is
>> simply a string of bytes that it is up to the filesystem to interpret.
>> NFS will always interpret it as utf8. So no conversion is needed.
>
> Not all versions of NFS use UTF-8. For example if you have NFSv3 and
> the server runs ISO8859-16 (French), then the filenames are encoded
> using ISO8859-16, and the NFS client is assumed to use ISO8859-16 too.
I'm still trying to understand the usefulness of supporting NFSv3 as
well. I understand that your usage requirement is only NFSv4.
However, does your comment mean that there might be some cases where
a client administrator might want to mount with NFSv3 using non-ASCII
characters (of any flavor) in the export path?
I'm also wondering about non-ASCII characters in the server hostname.
That does not depend on which NFS version is in play.
> mount(2) has options to do filename encoding conversion
> NFSv4 is an improvement compared to NFSv3 because it uses UTF-8 on the
> wire, but if you use the (ANSSI/Clip-OS) nls=/iocharset= mount option
> you can enable filename encoding conversion there.
>
> Traditionally it is assumed that the admin runs its shell in the
> locale (implies filesystem encoding) used to do the mount(2), hence
> the iconv() to make sure there is no encoding mismatch.
>
> Finally: Not everyone and everything will use English (we frog eaters
> are proud of our language!) or UTF8, and it will remain that way for
> the foreseeable future.
I'll repeat: none of us are denigrating languages or usages here. We're
attempting to clarify the problem statement and refine the use cases.
--
Chuck Lever
next prev parent reply other threads:[~2024-12-10 14:15 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-06 10:54 [patch] mount.nfs: Add support for nfs://-URLs Roland Mainz
2024-12-06 15:54 ` Chuck Lever
2024-12-06 16:15 ` nfs-utils library dependency littering - fork nfs-utils for Debian? " Mark Liam Brown
2024-12-06 16:58 ` Chuck Lever
2024-12-06 17:13 ` Mark Liam Brown
2024-12-06 19:40 ` Chuck Lever
2024-12-06 23:33 ` Roland Mainz
2024-12-07 15:20 ` Chuck Lever
2024-12-07 20:29 ` Chuck Lever
2024-12-19 14:47 ` Cedric Blancher
2024-12-07 20:53 ` NeilBrown
2024-12-07 22:54 ` Chuck Lever
2024-12-08 3:29 ` NeilBrown
2024-12-08 15:58 ` Chuck Lever
2024-12-09 15:15 ` Takeshi Nishimura
2024-12-09 15:46 ` Chuck Lever
2024-12-08 7:31 ` patchwork instance for linux-nfs patches? Cedric Blancher
2024-12-08 16:01 ` Chuck Lever
2024-12-08 7:41 ` [patch] mount.nfs: Add support for nfs://-URLs Cedric Blancher
2024-12-08 16:43 ` Chuck Lever
2024-12-10 11:46 ` Benjamin Coddington
2024-12-10 12:09 ` Steve Dickson
2024-12-07 17:05 ` Takeshi Nishimura
2024-12-08 22:12 ` NeilBrown
2024-12-10 12:13 ` Cedric Blancher
2024-12-10 14:15 ` Chuck Lever [this message]
2024-12-11 4:26 ` NeilBrown
2024-12-19 14:03 ` Cedric Blancher
2024-12-24 22:59 ` NeilBrown
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=71e31b5b-1318-4d5f-a96c-abd0b2fc0018@oracle.com \
--to=chuck.lever@oracle.com \
--cc=cedric.blancher@gmail.com \
--cc=linux-nfs@vger.kernel.org \
/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.