On Fri, May 4, 2018 at 8:30 PM, Alexander Duyck <alexander.du...@gmail.com> wrote: > From: Alexander Duyck <alexander.h.du...@intel.com> > > This patch is meant to allow us to avoid having to recompute the checksum > from scratch and have it passed as a parameter. > > Instead of taking that approach we can take advantage of the fact that the > length that was used to compute the existing checksum is included in the > UDP header. If we cancel that out by adding the value XOR with 0xFFFF we > can then just add the new length in and fold that into the new result. > > I think this may be fixing a checksum bug in the original code as well > since the checksum that was passed included the UDP header in the checksum > computation, but then excluded it for the adjustment on the last frame. I > believe this may have an effect on things in the cases where the two differ > by bits that would result in things crossing the byte boundaries.
The replacement code, below, subtracts original payload size then adds the new payload size. mss here excludes the udp header size. > /* last packet can be partial gso_size */ > - if (!seg->next) > - csum_replace2(&uh->check, htons(mss), > - htons(seg->len - hdrlen - sizeof(*uh)));
