Skip to content

RDMA resource tracking

Leon Romanovsky linux-rdma netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest7 of 10 patches 3 repliesIn review

Changes in v1 from the cover letter

* Fixed subject title in patch #1: rdam -> rdma.
* Added newline between variable declaration and the code.
* Add check to failure in strdup() call in rd_check_is_string_filtered().
* Rewrote res_qp_parse_cb() to avoid long lines and extra indentation.

7 of 10 patches are in this archive. The cover letter announces 10.

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/10rdma: Add option to provide "-" sign for the port numberrdma/link.c rdma/rdma.h rdma/utils.c+44 -110 replies
2/10rdma: Make visible the number of argumentsrdma/rdma.h rdma/utils.c+2 -10 replies
4/10rdma: Set pointer to device name positionrdma/rdma.h rdma/utils.c+20 -00 replies
5/10rdma: Allow external usage of compare string routinerdma/rdma.h rdma/utils.c+3 -10 replies
6/10rdma: Update kernel header fileinclude/uapi/rdma/rdma_netlink.h+57 -100 replies
9/10rdma: Document resource trackingman/man8/rdma-resource.8+86 -00 replies
10/10rdma: Check return value of strdup callrdma/utils.c+4 -00 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH iproute2-next 00/10] RDMA resource tracking

Changelog:
v2 -> v3:
* Rebased to commit: 1e24e773f144 ("Merge branch 'iproute2-master' into iproute2-next")
* Refreshed include/uapi/rdma/rdma_netlink.h file
* Fixed bug, where cxgb4 was printed twice.
v1 -> v2;
* Added checks for all occurrences of strdup failures and added patch with fix of already merged code.
* Sync with latest kernel code.
* Rewrote table representation to be similar to "ip route" output.
* Implemented string filters.
* Removed curr/max representation from the summary output.
v0 -> v1:
* Fixed subject title in patch #1: rdam -> rdma.
* Added newline between variable declaration and the code.
* Add check to failure in strdup() call in rd_check_is_string_filtered().
* Rewrote res_qp_parse_cb() to avoid long lines and extra indentation.

------------------------------------------------------------------------

David, Stephen,

The kernel code is accepted to the RDMA and will be sent to Linus in this merge window, and this is refreshed version of user space part.

Because, I found bug in handling cxgb4 devices and we cleaned header file a little bit more, it was more wise to resend the series instead of applying v2.

https://patchwork.ozlabs.org/project/netdev/list/?series=RDMA+resource+tracking&submitter=68852&state=8&q=&archive=&delegate=

Thanks

Cc RDMA mailing list <[email protected]>
Cc Steve Wise <[email protected]>

[1] https://www.spinics.net/lists/linux-rdma/msg59535.html

Leon Romanovsky (10):
rdma: Add option to provide "-" sign for the port number
rdma: Make visible the number of arguments
rdma: Add filtering infrastructure
rdma: Set pointer to device name position
rdma: Allow external usage of compare string routine
rdma: Update kernel header file
rdma: Add resource tracking summary
rdma: Add QP resource tracking information
rdma: Document resource tracking
rdma: Check return value of strdup call

 include/uapi/rdma/rdma_netlink.h |  67 +++++-
 man/man8/rdma-resource.8         |  86 +++++++
 rdma/Makefile                    |   2 +-
 rdma/link.c                      |   2 +-
 rdma/rdma.c                      |   4 +-
 rdma/rdma.h                      |  28 ++-
 rdma/res.c                       | 486 +++++++++++++++++++++++++++++++++++++++
 rdma/utils.c                     | 321 +++++++++++++++++++++++++-
 8 files changed, 971 insertions(+), 25 deletions(-)
 create mode 100644 man/man8/rdma-resource.8
 create mode 100644 rdma/res.c