Looks good.
Ethan
On Thu, Aug 4, 2011 at 16:29, Ben Pfaff <b...@nicira.com> wrote:
> The rest of the headers all follow the form "header(value)" or
> "header(key1=value1,key2=value2,...)" but VLAN headers left out the "="
> characters. This adds them in for consistency.
> ---
> lib/odp-util.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/lib/odp-util.c b/lib/odp-util.c
> index 93f8f8a..62cb504 100644
> --- a/lib/odp-util.c
> +++ b/lib/odp-util.c
> @@ -272,7 +272,7 @@ format_odp_key_attr(const struct nlattr *a, struct ds *ds)
> if (q_key->q_tpid != htons(ETH_TYPE_VLAN)) {
> ds_put_format(ds, "tpid=0x%04"PRIx16",", ntohs(q_key->q_tpid));
> }
> - ds_put_format(ds, "vid%"PRIu16",pcp%d)",
> + ds_put_format(ds, "vid=%"PRIu16",pcp=%d)",
> vlan_tci_to_vid(q_key->q_tci),
> vlan_tci_to_pcp(q_key->q_tci));
> break;
> --
> 1.7.4.4
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
