On Tue, Dec 08, 2015 at 10:27:18AM -0800, Tom Herbert wrote: > Add support for remote checksum offload in both the normal and GRO > paths. netlink command are used to enable sending of the Remote > Checksum Data, and allow processing of it on receive. The Remote
afsO&^(*&5^a+++NO CARRIER Did you mean to finish that sentence? ;-) > Signed-off-by: Tom Herbert <t...@herbertland.com> Jesse is going to have to comment on your (ab)use of the reserved fields. I presume that an RFC would be forthcoming? > diff --git a/include/net/geneve.h b/include/net/geneve.h > index 3106ed6..b59ad0a 100644 > --- a/include/net/geneve.h > +++ b/include/net/geneve.h > @@ -8,9 +8,9 @@ > > /* Geneve Header: > * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ > - * |Ver| Opt Len |O|C| Rsvd. | Protocol Type | > + * |Ver| Opt Len |O|C|X| Rsvd. | Protocol Type | > * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ > - * | Virtual Network Identifier (VNI) | Reserved | > + * | Virtual Network Identifier (VNI) |U| Csum start | > * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ > * | Variable Length Options | > * +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ Otherwise, LGTM -- I'm really happy to see Geneve get this attention! Acked-by: John W. Linville <linvi...@tuxdriver.com> -- John W. Linville Someday the world will need a hero, and you linvi...@tuxdriver.com might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
