| Message ID | 20210901031932.7734-1-wanjiabing@vivo.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 780aa1209f88fd96d40572b62df922662f2b896d |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [v3] mptcp: Fix duplicated argument in protocol.h | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Guessed tree name to be net-next |
| netdev/subject_prefix | warning | Target tree name not specified in the subject |
| netdev/cc_maintainers | success | CCed 7 of 7 maintainers |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 6 this patch: 6 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 6 this patch: 6 |
| netdev/header_inline | success | Link |
Hello, On Wed, 2021-09-01 at 11:19 +0800, Wan Jiabing wrote: > Fix the following coccicheck warning: > ./net/mptcp/protocol.h:36:50-73: duplicated argument to & or | > > The OPTION_MPTCP_MPJ_SYNACK here is duplicate. > Here should be OPTION_MPTCP_MPJ_ACK. > > Fixes: 74c7dfbee3e18 ("mptcp: consolidate in_opt sub-options fields in a bitmask") > Signed-off-by: Wan Jiabing <wanjiabing@vivo.com> Not sure what happen to my previous reply, most probably PEBKAC, sorry. WRT this patch, note that the dup is harmless, as in the input path we always use the mask and not the individual bit - vice versa in the output path. Still the cleanup is worthy and patch LGTM. Acked-by: Paolo Abeni <pabeni@redhat.com>
Hi Wan, On 01/09/2021 05:19, Wan Jiabing wrote: > Fix the following coccicheck warning: > ./net/mptcp/protocol.h:36:50-73: duplicated argument to & or | > > The OPTION_MPTCP_MPJ_SYNACK here is duplicate. > Here should be OPTION_MPTCP_MPJ_ACK. > > Fixes: 74c7dfbee3e18 ("mptcp: consolidate in_opt sub-options fields in a bitmask") > Signed-off-by: Wan Jiabing <wanjiabing@vivo.com> Thank you for the patch! It looks good to me too and MPTCP selftests are still happy with this patch. Reviewed-by: Matthieu Baerts <matthieu.baerts@tessares.net> Just one thing: please next time mention for which "net" tree this patch is for. That's why you got one warning on Patchwork [1]. See [2] for more details. Here this patch is for -net: [PATCH net]. [1] https://patchwork.kernel.org/project/netdevbpf/patch/20210901031932.7734-1-wanjiabing@vivo.com/ [2] https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html Cheers, Matt
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Wed, 1 Sep 2021 11:19:32 +0800 you wrote: > Fix the following coccicheck warning: > ./net/mptcp/protocol.h:36:50-73: duplicated argument to & or | > > The OPTION_MPTCP_MPJ_SYNACK here is duplicate. > Here should be OPTION_MPTCP_MPJ_ACK. > > Fixes: 74c7dfbee3e18 ("mptcp: consolidate in_opt sub-options fields in a bitmask") > Signed-off-by: Wan Jiabing <wanjiabing@vivo.com> > > [...] Here is the summary with links: - [v3] mptcp: Fix duplicated argument in protocol.h https://git.kernel.org/netdev/net/c/780aa1209f88 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/net/mptcp/protocol.h b/net/mptcp/protocol.h index d7aba1c4dc48..64c9a30e0871 100644 --- a/net/mptcp/protocol.h +++ b/net/mptcp/protocol.h @@ -34,7 +34,7 @@ #define OPTIONS_MPTCP_MPC (OPTION_MPTCP_MPC_SYN | OPTION_MPTCP_MPC_SYNACK | \ OPTION_MPTCP_MPC_ACK) #define OPTIONS_MPTCP_MPJ (OPTION_MPTCP_MPJ_SYN | OPTION_MPTCP_MPJ_SYNACK | \ - OPTION_MPTCP_MPJ_SYNACK) + OPTION_MPTCP_MPJ_ACK) /* MPTCP option subtypes */ #define MPTCPOPT_MP_CAPABLE 0
Fix the following coccicheck warning: ./net/mptcp/protocol.h:36:50-73: duplicated argument to & or | The OPTION_MPTCP_MPJ_SYNACK here is duplicate. Here should be OPTION_MPTCP_MPJ_ACK. Fixes: 74c7dfbee3e18 ("mptcp: consolidate in_opt sub-options fields in a bitmask") Signed-off-by: Wan Jiabing <wanjiabing@vivo.com> --- Changelog: v2: - Add a Fixes-tag. v3: - Make Fixes-tag to be a single line. --- net/mptcp/protocol.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)