On Tue, Jul 30, 2024 at 09:25:05AM +0800, Zhengchao Shao wrote: > When the SMC client perform CLC handshake, it will check whether > the clc header type is correct in receiving SMC_CLC_ACCEPT packet. > The specific invoking path is as follows: > __smc_connect > smc_connect_clc > smc_clc_wait_msg > smc_clc_msg_hdr_valid > smc_clc_msg_acc_conf_valid > Therefore, the smc_connect_check_aclc interface invoked by > __smc_connect does not need to check type again. > > Signed-off-by: Zhengchao Shao <shaozhengchao@xxxxxxxxxx> Thanks, I agree that in the case of a SMC_CLC_ACCEPT packet, which is the case here, this check is unnecessary as it has already been performed by smc_clc_msg_acc_conf_valid(). Reviewed-by: Simon Horman <horms@xxxxxxxxxx>
