[ovs-dev] [IPv6 IV: A New Hope 5/6] nicira-ext: Support matching IPv6 traffic.

Ben Pfaff blp at nicira.com
Wed Feb 2 19:15:27 UTC 2011


On Wed, Feb 02, 2011 at 11:12:36AM -0800, Justin Pettit wrote:
> > In odp_flow_key_to_flow(), I don't really like the mixing of IP_TYPE_*
> > (from packets.h) with IPPROTO_* (from some system header).  I'd rather
> > add a IP_TYPE_ICMPV6 or use IPPROTO_* uniformly.  Ditto in
> > parse_protocol() in ofp-parse.c.
> 
> I agree and was planning to clean that up in a follow-on patch.  Do
> you know why we define IP_TYPE_* in packets.h, if they're defined in
> "netinet/in.h"?  I'd prefer to use the pre-defined ones and get rid of
> "IP_TYPE_*" from packets.h unless there's a compelling reason not to.

That's fine.  I imagine that I didn't realize that IPPROTO_* was
portable at the time I defined IP_TYPE_*.  The goal of packets.h was to
avoid dependencies on Linux kernel-specific macros and types.  I guess I
was overzealous in this case, since those macros are specified by POSIX.




More information about the dev mailing list