From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from out-174.mta0.migadu.com (out-174.mta0.migadu.com [91.218.175.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 38A7E2D73B5 for ; Thu, 13 Nov 2025 09:27:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=91.218.175.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763026037; cv=none; b=mRo+AVLlUWFgpbaPH8yM1FqgsqPRdlZFk2COPLtR/OSPMttECWFVpAoodeFD6hXktI1CCKa3YnPvVhOhB7bf9n2MiVNClciV7z2u94M61MXmK6grOjRo5capDXZAEqiX2DaEQqWt0y1lTm5cASCFQnwZbCH8BVC5Ocz8jCaV5Ew= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763026037; c=relaxed/simple; bh=lFyIlcliweTDArjOud38dDnDHE7GQErnUCwb4MO0ybo=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=tziIW5MM10cZIb58arHF8R/b1nSst/3YSqRWCnhmYz2if5fnDVSM/zXBRTs8cHsl/sC6LU9pXokfWAHl8xt51qgLe46nlLomk3MyKih4DIt1Bmd4XQlkTLnPvBPk9QepVwcqheA3AHRMW9zYMOissZHTxFneXDyZ85zHlyokz3I= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=ngBDeKim; arc=none smtp.client-ip=91.218.175.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="ngBDeKim" X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1763026031; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=f9Gs7sU3C6pl28t3n8A03C3rIVhAKjaPqrv88ZWeXpE=; b=ngBDeKimW5D2u6XFUVjbX0IOfqnXBVIgx4DbaXBAjtGN3yh9i/RU7gIw1X5pePtksSX+Cb A27R/Kf2marevi9aDe98L9+K9iPRHACcOCXX11Fi2cAP3ghFz9I03TYiOlTCndzF4xn+38 oClU9JygROCdn1S5KHQSEACIraBTxVg= From: chenxiaosong.chenxiaosong@linux.dev To: sfrench@samba.org, smfrench@gmail.com, linkinjeon@kernel.org, linkinjeon@samba.org, christophe.jaillet@wanadoo.fr Cc: linux-cifs@vger.kernel.org, linux-kernel@vger.kernel.org, chenxiaosong@chenxiaosong.com, ChenXiaoSong Subject: [PATCH v6 0/2] smb: move duplicate definitions to common header file Date: Thu, 13 Nov 2025 17:26:00 +0800 Message-ID: <20251113092602.24733-1-chenxiaosong.chenxiaosong@linux.dev> Precedence: bulk X-Mailing-List: linux-cifs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT From: ChenXiaoSong In order to maintain the code more easily, move some duplicate definitions to common header file. Add some MS documentation references for macro and struct definitions. I have tested all patches using xfstests and smbtorture, and no additional test failures were observed in the results. The detailed test results can be found in https://chenxiaosong.com/en/smb-test/20251113.html v3: https://lore.kernel.org/all/20251014071917.3004573-1-chenxiaosong.chenxiaosong@linux.dev/ The following patches from v3 have already been merged into the mainline: - d877470b5991 smb: move some duplicate definitions to common/cifsglob.h - 379510a815cb smb/server: fix possible refcount leak in smb2_sess_setup() - 6fced056d2cc smb/server: fix possible memory leak in smb2_read() v4: https://lore.kernel.org/all/20251027071316.3468472-1-chenxiaosong.chenxiaosong@linux.dev/ v5: https://lore.kernel.org/all/20251102073059.3681026-1-chenxiaosong.chenxiaosong@linux.dev/ The following patches from v4 and v5 have been applied to the ksmbd-for-next-next branch: https://git.samba.org/?p=ksmbd.git;a=shortlog;h=refs/heads/ksmbd-for-next-next - smb: fix some warnings reported by scripts/checkpatch.pl - smb: do some cleanups - smb: move FILE_SYSTEM_SIZE_INFO to common/fscc.h - smb: move some duplicate struct definitions to common/fscc.h - smb: move list of FileSystemAttributes to common/fscc.h - smb: move SMB_NEGOTIATE_REQ to common/smb2pdu.h - smb: move some duplicate definitions to common/smb2pdu.h - smb: move create_durable_rsp_v2 to common/smb2pdu.h - smb: move create_durable_handle_reconnect_v2 to common/smb2pdu.h - smb: move create_durable_req_v2 to common/smb2pdu.h - smb: move MAX_CIFS_SMALL_BUFFER_SIZE to common/smbglob.h - smb/client: fix CAP_BULK_TRANSFER value - smb: move resume_key_ioctl_rsp to common/smb2pdu.h - smb: move copychunk definitions to common/smb2pdu.h - smb: move smb_sockaddr_in and smb_sockaddr_in6 to common/smb2pdu.h - smb: move SMB1_PROTO_NUMBER to common/smbglob.h - smb: move get_rfc1002_len() to common/smbglob.h - smb: move smb_version_values to common/smbglob.h - smb: rename common/cifsglob.h to common/smbglob.h v5->v6: - Patch #0001: use typedef to define both CREATE_DURABLE and CREATE_DURABLE_RECONN_REQ for a single struct. - Patch #0002: make FILE_SYSTEM_ATTRIBUTE_INFO.FileSystemName a flexible array member. ChenXiaoSong (2): smb: move CREATE_DURABLE_RECONN_REQ to common/smb2pdu.h smb: move FILE_SYSTEM_ATTRIBUTE_INFO to common/fscc.h fs/smb/client/cifspdu.h | 10 ---------- fs/smb/client/smb2pdu.c | 25 ++++++++++++------------- fs/smb/common/fscc.h | 8 ++++++++ fs/smb/common/smb2pdu.h | 4 ++-- fs/smb/server/smb2pdu.c | 12 ++++++------ fs/smb/server/smb2pdu.h | 12 ------------ fs/smb/server/smb_common.h | 7 ------- 7 files changed, 28 insertions(+), 50 deletions(-) -- 2.43.0