Skip to content

Add directory to place common code and move trivial functions into it

Leon Romanovsky linux-rdma 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 15 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5cmake: Update build instructions in CmakeListCMakeLists.txt+1 -00 replies
2/5utils: Create utils directory to put all common code and move min/max into itCMakeLists.txt ibacm/linux/osd.h libibverbs/examples/rc_pingpong.c +3 more+57 -273 replies
3/5utils: Remove container_of and offset local declarationsibacm/linux/osd.h libmlx5/src/list.h librdmacm/src/cma.h+0 -282 replies
4/5libocrdma: Move ocrdma's list implementation into common directorylibocrdma/src/ocrdma_list.h libocrdma/src/ocrdma_main.c libocrdma/src/ocrdma_main.h +2 more+122 -12210 replies
5/5libmlx5: Convert libmlx5 to use common list implementationlibmlx5/src/buf.c libmlx5/src/list.h libmlx5/src/mlx5.h +1 more+17 -3210 replies

Reviews trailers per patch

1 of 5 patches has a Reviewed-by or Acked-by.

PatchSean Hefty
1/5
2/5
3/5A-b
4/5
5/5

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH rdma-core 0/5] Add directory to place common code and move trivial functions into it

This is patch series which adds directory to place common code.

Highlights of this this series:
1. The min/max functions were moved into it.
2. OCRDMA list implementation was chosen as common code.
3. Libmlx5 was converted to use that list.h.
4. Remove container_of and offset local declarations.

View the commits on github:

https://github.com/linux-rdma/rdma-core/pull/4

Leon Romanovsky (5):
cmake: Update build instructions in CmakeList
utils: Create utils directory to put all common code and move min/max
into it
utils: Remove container_of and offset local declarations
libocrdma: Move ocrdma's list implementation into common directory
libmlx5: Convert libmlx5 to use common list implementation

 CMakeLists.txt                    |   7 +
 ibacm/linux/osd.h                 |   8 +-
 libibverbs/examples/rc_pingpong.c |   8 +-
 libmlx5/src/buf.c                 |  13 +-
 libmlx5/src/list.h                | 331 --------------------------------------
 libmlx5/src/mlx5.h                |  20 +--
 libocrdma/src/ocrdma_list.h       | 104 ------------
 libocrdma/src/ocrdma_main.c       |   8 +-
 libocrdma/src/ocrdma_main.h       |  12 +-
 libocrdma/src/ocrdma_verbs.c      |  16 +-
 librdmacm/src/cma.h               |  11 +-
 utils/list.h                      | 111 +++++++++++++
 utils/math.h                      |  44 +++++
 13 files changed, 196 insertions(+), 497 deletions(-)
 delete mode 100644 libmlx5/src/list.h
 delete mode 100644 libocrdma/src/ocrdma_list.h
 create mode 100644 utils/list.h
 create mode 100644 utils/math.h

--
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html