dcbnl: Major simplifications
Status from this message: Applied, but could you send a follow-on patch to use BUG_ON() instead
Get series with b4
b4 am [email protected]Versions newest first
Patches in v1 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | dcbnl: Prepare framework to shorten handling functionsnet/ | +71 | 0 replies | |
| 2/8 | dcbnl: Shorten all command handling functionsnet/ | +172 | 1 reply | |
| 3/8 | dcbnl: Remove now unused dcbnl_reply()net/ | +0 | 0 replies | |
| 4/8 | dcbnl: Use dcbnl_newmsg() where possiblenet/ | +2 | 0 replies | |
| 5/8 | dcbnl: Return consistent error codesnet/ | +138 | 0 replies | |
| 6/8 | dcbnl: Move dcb app lookup code into dcb_app_lookup()net/ | +35 | 0 replies | |
| 7/8 | dcbnl: Move dcb app allocation into dcb_app_add()net/ | +22 | 0 replies | |
| 8/8 | dcbnl: Use type safe nlmsg_data()net/ | +1 | 0 replies |
Reviews trailers per patch
None of the 8 patches has a Reviewed-by or Acked-by.
| Patch | [email protected] |
|---|---|
| Series | T-b |
| 1/8 | |
| 2/8 | |
| 3/8 | |
| 4/8 | |
| 5/8 | |
| 6/8 | |
| 7/8 | |
| 8/8 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/8] dcbnl: Major simplifications
The dcbnl code is overly complicated when constructing the reply message. This series of patches simplifies the process and introduces consistent error codes allowing user space to figure out what went wrong.
Patches have been tested with ixgbe in a FCoE setup.
Thomas Graf (8):
dcbnl: Prepare framework to shorten handling functions
dcbnl: Shorten all command handling functions
dcbnl: Remove now unused dcbnl_reply()
dcbnl: Use dcbnl_newmsg() where possible
dcbnl: Return consistent error codes
dcbnl: Move dcb app lookup code into dcb_app_lookup()
dcbnl: Move dcb app allocation into dcb_app_add()
dcbnl: Use type safe nlmsg_data()
net/dcb/dcbnl.c | 1169 +++++++++++++++++++------------------------------------ 1 files changed, 401 insertions(+), 768 deletions(-)
Signature · 1 line
1.7.7.6