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