Re: [PATCH for-next v2 0/9] ICRC cleanup

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

 



On Mon, Jul 19, 2021 at 5:16 AM Zhu Yanjun <zyjzyj2000@xxxxxxxxx> wrote:
>
> On Sat, Jul 17, 2021 at 1:38 AM Jason Gunthorpe <jgg@xxxxxxxxxx> wrote:
> >
> > On Tue, Jul 06, 2021 at 11:00:32PM -0500, Bob Pearson wrote:
> > > This series of patches is a cleanup of the ICRC code in the rxe driver.
> > > The end result is to collect all the ICRC focused code into rxe_icrc.c
> > > with three APIs that are used by the rest of the driver. One to initialize
> > > the crypto engine used to compute crc32, and one each to generate and
> > > check the ICRC in a packet.
> > >
> > > Signed-off-by: Bob Pearson <rpearsonhpe@xxxxxxxxx>
> > > ---
> > > v2:
> > >   Split up the 5 patches in the first version into 9 patches which are
> > >   each focused on a single change.
> > >   Fixed sparse warnings in the first version.
> > >
> > > Bob Pearson (9):
> > >   RDMA/rxe: Move ICRC checking to a subroutine
> > >   RDMA/rxe: Move rxe_xmit_packet to a subroutine
> > >   RDMA/rxe: Fixup rxe_send and rxe_loopback
> > >   RDMA/rxe: Move ICRC generation to a subroutine
> > >   RDMA/rxe: Move rxe_crc32 to a subroutine
> > >   RDMA/rxe: Fixup rxe_icrc_hdr
> > >   RDMA/rxe: Move crc32 init code to rxe_icrc.c
> > >   RDMA/rxe: Add kernel-doc comments to rxe_icrc.c
> > >   RDMA/rxe: Fix types in rxe_icrc.c
> >
> > Applied to for-next, thanks
>
> Hi, Jason && Bob
>
> I confronted a problem.
> One hosts with this patch series, A
> other hosts, without this patch series, B
>
> I run rping between A <-------> B.
>
> I confronted the following errors, and rping can not succeed.
> "
> ...
> [ 1848.251273] rdma_rxe: bad ICRC from 172.16.1.61
> [ 1971.750691] rdma_rxe: bad ICRC from 172.16.1.61
> ...
> "
> It seems that this patch series breaks the Backward compatibility of RXE.
>
> Not sure if it is a problem. Please comment.
>
> Thanks a lot.
> Zhu Yanjun

I would like to second this post. I was about to submit a new post
asking about why rxe isn't working and if it's known. I'm trying to
figure out when/what patch broke things. From the stand point of
NFSoRDMA, it stopped working and I have discovered that simple rping
doesn't work as well. The last known working release was 5.11.



>
> >
> > 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