All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eduard Bachmakov <e.bachmakov@gmail.com>
To: linux-cifs@vger.kernel.org
Cc: Eduard Bachmakov <e.bachmakov@gmail.com>
Subject: [PATCH] cifs-utils: Fix documentation for character remappings.
Date: Thu, 14 Aug 2025 23:22:14 +0200	[thread overview]
Message-ID: <20250814212256.1653699-2-e.bachmakov@gmail.com> (raw)

Current documentation swapped the relevant specs. Additionally,
elaborate on SFM a bit.

Supported by fs/cifs/cifs_unicode.c:{convert_to_sfu_char,
convert_to_sfm_char}.

Previously raised in:

  https://lore.kernel.org/linux-cifs/CADCRUiMn_2Vk3HZzU0WKu3xPgo1P-1aqDy+NjEzOz03W-HFChw@mail.gmail.com/

Signed-off-by: Eduard Bachmakov <e.bachmakov@gmail.com>
---
 mount.cifs.rst | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/mount.cifs.rst b/mount.cifs.rst
index d489070..9eee7d5 100644
--- a/mount.cifs.rst
+++ b/mount.cifs.rst
@@ -307,10 +307,10 @@ rwpidforward
 
 mapchars
   Translate six of the seven reserved characters (not backslash, but
-  including the colon, question mark, pipe, asterik, greater than and
+  including the colon, question mark, pipe, asterisk, greater than and
   less than characters) to the remap range (above 0xF000), which also
   allows the CIFS client to recognize files created with such characters
-  by Windows's Services for Mac. This can also be useful when mounting to
+  by Windows's Services for Unix. This can also be useful when mounting to
   most versions of Samba (which also forbids creating and opening files
   whose names contain any of these seven characters). This has no effect
   if the server does not support Unicode on the wire. Please note that
@@ -322,7 +322,8 @@ nomapchars
 
 mapposix
   Translate reserved characters similarly to ``mapchars`` but use the
-  mapping from Microsoft "Services For Unix".
+  mapping from Microsoft "Services For Mac". This additionally remaps the
+  double quote and a trailing period or space.
 
 intr
   currently unimplemented.
-- 
2.50.1


             reply	other threads:[~2025-08-14 21:24 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-14 21:22 Eduard Bachmakov [this message]
2025-08-19 23:14 ` [PATCH] cifs-utils: Fix documentation for character remappings Steve French

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=20250814212256.1653699-2-e.bachmakov@gmail.com \
    --to=e.bachmakov@gmail.com \
    --cc=linux-cifs@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.