Re: [PATCH for-next v7 00/10] RDMA/rxe: Implement memory windows

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 25, 2021 at 12:04 AM Pearson, Robert B
<rpearsonhpe@xxxxxxxxx> wrote:
>
> On 5/23/2021 10:14 PM, Zhu Yanjun wrote:
> > On Sat, May 22, 2021 at 4:19 AM Bob Pearson <rpearsonhpe@xxxxxxxxx> wrote:
> >> This series of patches implement memory windows for the rdma_rxe
> >> driver. This is a shorter reimplementation of an earlier patch set.
> >> They apply to and depend on the current for-next linux rdma tree.
> >>
> >> Signed-off-by: Bob Pearson <rpearsonhpe@xxxxxxxxx>
> >> ---
> >> v7:
> >>    Fixed a duplicate INIT_RDMA_OBJ_SIZE(ib_mw, ...) in rxe_verbs.c.
> > With this patch series, there are about 17 errors and 1 failure in rdma-core.
>
> Zhu,
>
> You have to sync the kernel-header file with the kernel.

>From the link https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/kbuild/headers_install.rst?h=v5.13-rc3
you mean "make headers_install"?

In fact, after "make headers_install", these patches still cause
errors and failures in rdma-core.

I will delve into these errors of rdma-core. Too many errors.

Zhu Yanjun

>
> Bob
>
> > "
> > ----------------------------------------------------------------------
> > Ran 183 tests in 2.130s
> >
> > FAILED (failures=1, errors=17, skipped=124)
> > "
> >
> > After these patches, not sure if rxe can communicate with the physical
> > NICs correctly because of the
> > above errors and failure.
> >
> > Zhu Yanjun
> >
> >> v6:
> >>    Added rxe_ prefix to subroutine names in lines that changed
> >>    from Zhu's review of v5.
> >> v5:
> >>    Fixed a typo in 10th patch.
> >> v4:
> >>    Added a 10th patch to check when MRs have bound MWs
> >>    and disallow dereg and invalidate operations.
> >> v3:
> >>    cleaned up void return and lower case enums from
> >>    Zhu's review.
> >> v2:
> >>    cleaned up an issue in rdma_user_rxe.h
> >>    cleaned up a collision in rxe_resp.c
> >>
> >> Bob Pearson (9):
> >>    RDMA/rxe: Add bind MW fields to rxe_send_wr
> >>    RDMA/rxe: Return errors for add index and key
> >>    RDMA/rxe: Enable MW object pool
> >>    RDMA/rxe: Add ib_alloc_mw and ib_dealloc_mw verbs
> >>    RDMA/rxe: Replace WR_REG_MASK by WR_LOCAL_OP_MASK
> >>    RDMA/rxe: Move local ops to subroutine
> >>    RDMA/rxe: Add support for bind MW work requests
> >>    RDMA/rxe: Implement invalidate MW operations
> >>    RDMA/rxe: Implement memory access through MWs
> >>
> >>   drivers/infiniband/sw/rxe/Makefile     |   1 +
> >>   drivers/infiniband/sw/rxe/rxe.c        |   1 +
> >>   drivers/infiniband/sw/rxe/rxe_comp.c   |   1 +
> >>   drivers/infiniband/sw/rxe/rxe_loc.h    |  29 +-
> >>   drivers/infiniband/sw/rxe/rxe_mr.c     |  79 ++++--
> >>   drivers/infiniband/sw/rxe/rxe_mw.c     | 356 +++++++++++++++++++++++++
> >>   drivers/infiniband/sw/rxe/rxe_opcode.c |  11 +-
> >>   drivers/infiniband/sw/rxe/rxe_opcode.h |   3 +-
> >>   drivers/infiniband/sw/rxe/rxe_param.h  |  19 +-
> >>   drivers/infiniband/sw/rxe/rxe_pool.c   |  45 ++--
> >>   drivers/infiniband/sw/rxe/rxe_pool.h   |   8 +-
> >>   drivers/infiniband/sw/rxe/rxe_req.c    | 102 ++++---
> >>   drivers/infiniband/sw/rxe/rxe_resp.c   | 110 +++++---
> >>   drivers/infiniband/sw/rxe/rxe_verbs.c  |   5 +-
> >>   drivers/infiniband/sw/rxe/rxe_verbs.h  |  38 ++-
> >>   include/uapi/rdma/rdma_user_rxe.h      |  34 ++-
> >>   16 files changed, 691 insertions(+), 151 deletions(-)
> >>   create mode 100644 drivers/infiniband/sw/rxe/rxe_mw.c
> >> --
> >> 2.27.0
> >>



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux