[ovs-dev] [#8024v2 2/5] Don't overload IP TOS with the frag matching bits.

[Ben Pfaff]

On Tue, Nov 08, 2011 at 08:13:31PM -0800, Jesse Gross wrote:
> On Tue, Nov 8, 2011 at 4:24 PM, Ben Pfaff <blp at nicira.com> wrote:
> > On Tue, Nov 08, 2011 at 03:57:29PM -0800, Justin Pettit wrote:
> > Did you mean to update the comment on 'frag' below?
> >
> >> ??struct sw_flow_key {
> >> ?? ?? ?? struct {
> >> ?? ?? ?? ?? ?? ?? ?? __be64 ??tun_id; ?? ?? ?? ?? /* Encapsulating tunnel ID. */
> >> @@ -48,8 +44,8 @@ struct sw_flow_key {
> >> ?? ?? ?? } eth;
> >> ?? ?? ?? struct {
> >> ?? ?? ?? ?? ?? ?? ?? u8 ?? ?? proto; ?? ?? ?? ?? ?? /* IP protocol or lower 8 bits of ARP opcode. */
> >> - ?? ?? ?? ?? ?? ?? u8 ?? ?? tos_frag; ?? ?? ?? ??/* IP ToS DSCP in high 6 bits,
> >> - ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ?? ??* OVS_FRAG_TYPE_* in low 2 bits. */
> >> + ?? ?? ?? ?? ?? ?? u8 ?? ?? tos; ?? ?? ?? ?? /* IP ToS DSCP in high 6 bits. */
> >> + ?? ?? ?? ?? ?? ?? u8 ?? ?? frag; ?? ?? ?? ??/* OVS_FRAG_TYPE_* in low 2 bits. */
> >> ?? ?? ?? } ip;
> >> ?? ?? ?? union {
> >> ?? ?? ?? ?? ?? ?? ?? struct {
> 
> I think the comments here are still accurate.

I just meant that "in low 2 bits" seemed odd now, since it's just a
OVS_FRAG_TYPE_*, but of course it's still accurate.