Re: [PATCH v7 1/2] smb: move CREATE_DURABLE_RECONN to common/smb2pdu.h

[Namjae Jeon]

On Thu, Nov 13, 2025 at 10:34 PM <chenxiaosong.chenxiaosong@xxxxxxxxx> wrote:
>
> From: ChenXiaoSong <chenxiaosong@xxxxxxxxxx>
>
> The fields in struct create_durable_reconn_req and struct create_durable
> are exactly the same, so remove create_durable_reconn_req from server,
> and use typedef to define both CREATE_DURABLE_REQ and CREATE_DURABLE_RECONN
> for a single struct.
>
> Rename the following places:
>
>   - struct create_durable -> CREATE_DURABLE_REQ
>   - struct create_durable_reconn_req -> CREATE_DURABLE_RECONN
>
> The documentation references are:
>
>   - SMB2_CREATE_DURABLE_HANDLE_REQUEST   in MS-SMB2 2.2.13.2.3
>   - SMB2_CREATE_DURABLE_HANDLE_RECONNECT in MS-SMB2 2.2.13.2.4
>   - SMB2_FILEID in MS-SMB2 2.2.14.1
>
> Descriptions of the struct fields:
>
>   - __u8  Reserved[16]: DurableRequest field of SMB2_CREATE_DURABLE_HANDLE_REQUEST.
>                         A 16-byte field that MUST be reserved.
>   - __u64 PersistentFileId: Persistent field of 2.2.14.1 SMB2_FILEID
>   - __u64 VolatileFileId: Volatile field of 2.2.14.1 SMB2_FILEID
>   - struct Fid: Data field of SMB2_CREATE_DURABLE_HANDLE_RECONNECT.
>                 An SMB2_FILEID structure, as specified in section 2.2.14.1.
>
> Suggested-by: Namjae Jeon <linkinjeon@xxxxxxxxxx>
> Signed-off-by: ChenXiaoSong <chenxiaosong@xxxxxxxxxx>
I have changed struct names like the following ones and applied it to
#ksmbd-for-next-next.
 - struct create_durable -> create_durable_req_t
 - struct create_durable_reconn_req -> create_durable_reconn_t
Thanks!