On Oct 26, 2011, at 10:09 AM, Ben Pfaff wrote:

> +static const char *
> +ovs_key_attr_to_string(enum ovs_key_attr attr)
> +{
> +    switch (attr) {

I think it was added after this patch went out, but you may want to add 
OVS_KEY_ATTR_PRIORITY.

> +     case OVS_KEY_ATTR_UNSPEC: return "unspec";
> +     case OVS_KEY_ATTR_TUN_ID: return "tun_id";

Not sure if you care about the ordering, but "TUN_ID" is currently after "ND" 
in the ovs_key_attr enum.  Otherwise, the list is in order.

--Justin


_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to