Re: [PATCH 1/1] Update kernel headers

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

 



On Tue, May 25, 2021 at 9:18 PM Jason Gunthorpe <jgg@xxxxxxxxxx> wrote:
>
> On Tue, May 25, 2021 at 06:32:22PM -0400, Zhu Yanjun wrote:
> > From: Zhu Yanjun <zyjzyj2000@xxxxxxxxx>
> >
> > After the patches "RDMA/rxe: Implement memory windows", the kernel headers
> > are changed. This causes about 17 errors and 1 failure when running
> > "run_test.py" with rxe.
> > This commit will fix these errors and failures.
> >
> > Signed-off-by: Zhu Yanjun <zyjzyj2000@xxxxxxxxx>
> > ---
> >  kernel-headers/rdma/rdma_user_rxe.h | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
>
> Huh? Bob? You can't break the rxe uapi??
>
> > diff --git a/kernel-headers/rdma/rdma_user_rxe.h b/kernel-headers/rdma/rdma_user_rxe.h
> > index 068433e2..90ea477f 100644
> > --- a/kernel-headers/rdma/rdma_user_rxe.h
> > +++ b/kernel-headers/rdma/rdma_user_rxe.h
> > @@ -99,7 +99,17 @@ struct rxe_send_wr {
> >                       __u32   remote_qkey;
> >                       __u16   pkey_index;
> >               } ud;
> > +             struct {
> > +                     __aligned_u64   addr;
>
> > +                     __aligned_u64   length;
>
> > +                     __u32           mr_lkey;
> > +                     __u32           mw_rkey;
>
> > +                     __u32           rkey;
> > +                     __u32           access;
>
> > +                     __u32           flags;
> > +             } mw;
>
> There is room for 4 u64's in 'reg' and this has 5, so it is a no-go

What is 'reg'? Where is the definition of this 'reg'? What protocol
has the definition of this 'reg'?

Zhu Yanjun

>
> Jason



[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