[ovs-dev] [PATCH v2 2/7] flow: Add support for IGMP protocol

[Ben Pfaff]

On Wed, May 28, 2014 at 10:10:29PM -0300, Flavio Leitner wrote:
> Add IGMP support to struct flow, flow compose and flow extract.
> 
> Signed-off-by: Flavio Leitner <fbl at redhat.com>

The OVS_LIKELY on this line in miniflow_extract() looks weird to me.
I realize that you were just copying what was already there, but that
looked weird to me too:
> +        } else if (OVS_LIKELY(nw_proto == IPPROTO_IGMP)) {
I sent out a message to ask about the existing style:
        http://openvswitch.org/pipermail/dev/2014-May/040810.html

I think that this patch should add IGMP support to match_format().
Otherwise ofp_packet_to_string() won't be able to print IGMP fields,
which could be important for debugging.

Otherwise it looks good, thank you!