All of lore.kernel.org
 help / color / mirror / Atom feed
From: huiwen.he@linux.dev
To: smfrench@gmail.com, linkinjeon@kernel.org, pc@manguebit.org,
	ronniesahlberg@gmail.com, sprasad@microsoft.com, tom@talpey.com,
	bharathsm@microsoft.com, senozhatsky@chromium.org,
	dhowells@redhat.com, chenxiaosong@kylinos.cn,
	chenxiaosong@chenxiaosong.com, tangyouling@kylinos.cn
Cc: linux-cifs@vger.kernel.org, Huiwen He <hehuiwen@kylinos.cn>
Subject: [PATCH v2 00/12] smb: improve search speed of SMB1 maperror
Date: Wed,  1 Apr 2026 07:29:00 +0000	[thread overview]
Message-ID: <20260401072912.355072-1-huiwen.he@linux.dev> (raw)

From: Huiwen He <hehuiwen@kylinos.cn>

v1->v2:
  - Sort arrays at build time
  - Merge nt_errs to ntstatus_to_dos_map
  - Check if arrays are sorted in smb1_init_maperror()

v1: https://lore.kernel.org/linux-cifs/20251208062100.3268777-1-chenxiaosong.chenxiaosong@linux.dev/

When searching for the last element, the comparison counts are shown in the table below:

+--------------------+--------+--------+
|                    |Before  |After   |
|                    |Patchset|Patchset|
+--------------------+--------+--------+
| ntstatus_to_dos_map|   533  |    9    |
+--------------------+--------+--------+
|mapping_table_ERRDOS|    34  |    5   |
+--------------------+--------+--------+
|mapping_table_ERRSRV|    37  |    5   |
+--------------------+--------+--------+

The following patches from v1 have already been merged into the mainline:
  - a71a4aab4834 smb/client: add parentheses to NT error code definitions containing bitwise OR operator
  - a9adafd40165 smb/client: add 4 NT error code definitions
  - 98def4eb0244 smb/server: remove unused nterr.h
  - 9f99caa8950a smb/client: fix NT_STATUS_UNABLE_TO_FREE_VM value
  - b2b50fca34da smb/client: fix NT_STATUS_DEVICE_DOOR_OPEN value
  - a1237c203f17 smb/client: fix NT_STATUS_NO_DATA_DETECTED value

Huiwen He (8):
  smb/client: annotate nterr.h with DOS error codes
  smb/client: autogenerate SMB1 NT status to DOS error mapping
  smb/client: replace nt_errs with ntstatus_to_dos_map
  smb/client: refactor ntstatus_to_dos() to return mapping entry
  smb/client: use binary search for NT status to DOS mapping
  smb/client: annotate smberr.h with POSIX error codes
  smb/client: autogenerate SMB1 DOS/SRV to POSIX error mapping
  smb/client: use binary search for SMB1 DOS/SRV error mapping

Youling Tang (4):
  smb/client: check if ntstatus_to_dos_map is sorted
  smb/client: introduce KUnit test to check ntstatus_to_dos_map search
  smb/client: check if SMB1 DOS/SRV error mapping arrays are sorted
  smb/client: introduce KUnit tests to check DOS/SRV err mapping search

 fs/smb/client/.gitignore          |    3 +
 fs/smb/client/Kconfig             |   11 +
 fs/smb/client/Makefile            |   25 +-
 fs/smb/client/cifsfs.c            |    6 +
 fs/smb/client/gen_smb1_mapping    |  109 +++
 fs/smb/client/nterr.c             |  704 -------------------
 fs/smb/client/nterr.h             | 1082 +++++++++++++++--------------
 fs/smb/client/smb1maperror.c      |  873 +++++------------------
 fs/smb/client/smb1maperror_test.c |   83 +++
 fs/smb/client/smb1proto.h         |    9 +
 fs/smb/client/smberr.h            |  415 +++++++----
 11 files changed, 1232 insertions(+), 2088 deletions(-)
 create mode 100644 fs/smb/client/gen_smb1_mapping
 delete mode 100644 fs/smb/client/nterr.c
 create mode 100644 fs/smb/client/smb1maperror_test.c

-- 
2.52.0


             reply	other threads:[~2026-04-01  7:30 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-01  7:29 huiwen.he [this message]
2026-04-01  7:29 ` [PATCH v2 01/12] smb/client: annotate nterr.h with DOS error codes huiwen.he
2026-04-01  7:29 ` [PATCH v2 02/12] smb/client: autogenerate SMB1 NT status to DOS error mapping huiwen.he
2026-04-01  7:29 ` [PATCH v2 03/12] smb/client: replace nt_errs with ntstatus_to_dos_map huiwen.he
2026-04-01  7:29 ` [PATCH v2 04/12] smb/client: refactor ntstatus_to_dos() to return mapping entry huiwen.he
2026-04-01  7:29 ` [PATCH v2 05/12] smb/client: use binary search for NT status to DOS mapping huiwen.he
2026-04-01  7:29 ` [PATCH v2 06/12] smb/client: check if ntstatus_to_dos_map is sorted huiwen.he
2026-04-01  7:29 ` [PATCH v2 07/12] smb/client: introduce KUnit test to check ntstatus_to_dos_map search huiwen.he
2026-04-01  7:29 ` [PATCH v2 08/12] smb/client: annotate smberr.h with POSIX error codes huiwen.he
2026-04-01  7:29 ` [PATCH v2 09/12] smb/client: autogenerate SMB1 DOS/SRV to POSIX error mapping huiwen.he
2026-04-01  7:29 ` [PATCH v2 10/12] smb/client: use binary search for SMB1 DOS/SRV " huiwen.he
2026-04-01  7:29 ` [PATCH v2 11/12] smb/client: check if SMB1 DOS/SRV error mapping arrays are sorted huiwen.he
2026-04-01  7:29 ` [PATCH v2 12/12] smb/client: introduce KUnit tests to check DOS/SRV err mapping search huiwen.he
2026-04-02  8:38 ` [PATCH v2 00/12] smb: improve search speed of SMB1 maperror ChenXiaoSong

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=20260401072912.355072-1-huiwen.he@linux.dev \
    --to=huiwen.he@linux.dev \
    --cc=bharathsm@microsoft.com \
    --cc=chenxiaosong@chenxiaosong.com \
    --cc=chenxiaosong@kylinos.cn \
    --cc=dhowells@redhat.com \
    --cc=hehuiwen@kylinos.cn \
    --cc=linkinjeon@kernel.org \
    --cc=linux-cifs@vger.kernel.org \
    --cc=pc@manguebit.org \
    --cc=ronniesahlberg@gmail.com \
    --cc=senozhatsky@chromium.org \
    --cc=smfrench@gmail.com \
    --cc=sprasad@microsoft.com \
    --cc=tangyouling@kylinos.cn \
    --cc=tom@talpey.com \
    /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.