On Wed, Jan 29, 2025 at 07:11:35PM +0100, Zhu Yanjun wrote: > 在 2025/1/27 23:38, Eric Biggers 写道: > > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > > > Since rxe_icrc_hdr() is always immediately followed by updating the CRC > > with the packet's payload, just rename it to rxe_icrc() and make it > > include the payload in the CRC, so it now handles the entire packet. > > > > This is a refactor with no change in behavior. > > In this commit, currently the entire packet are checked while the header is > checked in the original source code. > > Now it can work between RXE <----> RXE. > I am not sure whether RXE <---> MLX can work or not. > > If it can work well, I am fine with this patch. > > Reviewed-by: Zhu Yanjun <yanjun.zhu@xxxxxxxxx> > Both the header and payload are checksummed both before and after this patch. Can you elaborate on why you think this patch changed any behavior? - Eric
