These cleanups lead the way to the unification of the path-manager
interfaces, and allow future extensions. The following patches are not
all linked to each others, but are all related to the path-managers.
- Patch 1: drop unneeded parameter in a function helper.
- Patch 2: clearer NL error message when an NL attribute is missing.
- Patch 3: more precise NL messages by avoiding 'this or that is NOK'.
- Patch 4: improve too vague or missing NL err messages.
- Patch 5: use GENL_REQ_ATTR_CHECK to look for mandatory NL attributes.
- Patch 6: avoid overriding the error message.
- Patch 7: check all mandatory NL attributes with GENL_REQ_ATTR_CHECK.
- Patch 8: use NL_SET_ERR_MSG_ATTR instead of GENL_SET_ERR_MSG
- Patch 9: move doit callbacks used for both PM to pm.c.
- Patch 10: drop another unneeded parameter in a function helper.
- Patch 11: share the ID parsing code for the 'get_addr' callback.
- Patch 12: share sending NL code for the 'get_addr' callback.
- Patch 13: drop yet another unneeded parameter in a function helper.
- Patch 14: pick the usual structure type for the remote address.
- Patch 15: share the local addr parsing code for the 'set_flags' cb.
The behaviour when there are no errors should then not be modified.
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
---
Geliang Tang (9):
mptcp: pm: drop info of userspace_pm_remove_id_zero_address
mptcp: pm: userspace: use GENL_REQ_ATTR_CHECK
mptcp: pm: make three pm wrappers static
mptcp: pm: drop skb parameter of get_addr
mptcp: pm: add id parameter for get_addr
mptcp: pm: reuse sending nlmsg code in get_addr
mptcp: pm: drop skb parameter of set_flags
mptcp: pm: change rem type of set_flags
mptcp: pm: add local parameter for set_flags
Matthieu Baerts (NGI0) (6):
mptcp: pm: userspace: flags: clearer msg if no remote addr
mptcp: pm: more precise error messages
mptcp: pm: improve error messages
mptcp: pm: remove duplicated error messages
mptcp: pm: mark missing address attributes
mptcp: pm: use NL_SET_ERR_MSG_ATTR when possible
net/mptcp/pm.c | 86 +++++++++++++++++--
net/mptcp/pm_netlink.c | 129 ++++++++++-------------------
net/mptcp/pm_userspace.c | 209 +++++++++++++++++++++--------------------------
net/mptcp/protocol.h | 14 ++--
4 files changed, 225 insertions(+), 213 deletions(-)
---
base-commit: b44e27b4df1a1cd3fd84cf26c82156ed0301575f
change-id: 20250116-net-next-mptcp-pm-misc-cleanup-2-b0f50eff8084
Best regards,
--
Matthieu Baerts (NGI0) <matttbe@kernel.org>