[ovs-dev] [ofp-print 08/18] ofp-print: Print OFPUTIL_NXT_ROLE_REQUEST and OFPUTIL_NXT_ROLE_REPLY.

Justin Pettit jpettit at nicira.com
Thu Dec 9 01:32:18 UTC 2010


Looks good.

--Justin


On Dec 8, 2010, at 4:27 PM, Ben Pfaff wrote:

> ---
> lib/ofp-print.c |   20 +++++++++++++++++++-
> 1 files changed, 19 insertions(+), 1 deletions(-)
> 
> diff --git a/lib/ofp-print.c b/lib/ofp-print.c
> index 4dd8924..1cbcb82 100644
> --- a/lib/ofp-print.c
> +++ b/lib/ofp-print.c
> @@ -1422,6 +1422,24 @@ ofp_print_nxt_tun_id_from_cookie(struct ds *string,
> }
> 
> static void
> +ofp_print_nxt_role_message(struct ds *string,
> +                           const struct nx_role_request *nrr)
> +{
> +    unsigned int role = ntohl(nrr->role);
> +
> +    ds_put_cstr(string, " role=");
> +    if (role == NX_ROLE_OTHER) {
> +        ds_put_cstr(string, "other");
> +    } else if (role == NX_ROLE_MASTER) {
> +        ds_put_cstr(string, "master");
> +    } else if (role == NX_ROLE_SLAVE) {
> +        ds_put_cstr(string, "slave");
> +    } else {
> +        ds_put_format(string, "%u", role);
> +    }
> +}
> +
> +static void
> ofp_print_nxt_set_flow_format(struct ds *string,
>                               const struct nxt_set_flow_format *nsff)
> {
> @@ -1580,7 +1598,7 @@ ofp_to_string__(const struct ofp_header *oh,
> 
>     case OFPUTIL_NXT_ROLE_REQUEST:
>     case OFPUTIL_NXT_ROLE_REPLY:
> -        /* XXX */
> +        ofp_print_nxt_role_message(string, msg);
>         break;
> 
>     case OFPUTIL_NXT_SET_FLOW_FORMAT:
> -- 
> 1.7.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org





More information about the dev mailing list