On Mon, Mar 30, 2015 at 3:14 PM, Jesse Gross wrote:
> The GRE checksum is a 16 bit field stored in a 32 bit option (the
> rest is reserved). The current code treats the checksum as a 32-bit
> field and places it in the right place for little endian systems but
> not big endian. This fixes the prob
On Tue, Mar 31, 2015 at 05:26:19PM -0700, Jesse Gross wrote:
> On Tue, Mar 31, 2015 at 4:21 PM, Ben Pfaff wrote:
> > On Mon, Mar 30, 2015 at 03:14:46PM -0700, Jesse Gross wrote:
> >> The GRE checksum is a 16 bit field stored in a 32 bit option (the
> >> rest is reserved). The current code treats t
On Tue, Mar 31, 2015 at 4:21 PM, Ben Pfaff wrote:
> On Mon, Mar 30, 2015 at 03:14:46PM -0700, Jesse Gross wrote:
>> The GRE checksum is a 16 bit field stored in a 32 bit option (the
>> rest is reserved). The current code treats the checksum as a 32-bit
>> field and places it in the right place for
On Mon, Mar 30, 2015 at 03:14:46PM -0700, Jesse Gross wrote:
> The GRE checksum is a 16 bit field stored in a 32 bit option (the
> rest is reserved). The current code treats the checksum as a 32-bit
> field and places it in the right place for little endian systems but
> not big endian. This fixes
LGTM
Acked-by: Pritesh Kothari
> On Mar 30, 2015, at 3:14 PM, Jesse Gross wrote:
>
> The GRE checksum is a 16 bit field stored in a 32 bit option (the
> rest is reserved). The current code treats the checksum as a 32-bit
> field and places it in the right place for little endian systems but
>
The GRE checksum is a 16 bit field stored in a 32 bit option (the
rest is reserved). The current code treats the checksum as a 32-bit
field and places it in the right place for little endian systems but
not big endian. This fixes the problem by storing the 16 bit field
directly.
Signed-off-by: Jes
