[ovs-dev] [nxm 21/42] Sync Nicira error number assignments with "wdp" branch.

Justin Pettit jpettit at nicira.com
Fri Nov 5 00:49:23 UTC 2010


Looks fine.  (I'm not going to say "good", because it's not actually adding anything useful to this branch.  I don't want you to think I give my "Looks good." away lightly.)

--Justin


On Oct 28, 2010, at 10:27 AM, Ben Pfaff wrote:

> This adds the error numbers that the "wdp" branch added, without adding
> any uses of them (because they are not needed on "master" yet).
> ---
> include/openflow/nicira-ext.h |   18 ++++++++++++++++++
> 1 files changed, 18 insertions(+), 0 deletions(-)
> 
> diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h
> index 55b3548..9b4d9d2 100644
> --- a/include/openflow/nicira-ext.h
> +++ b/include/openflow/nicira-ext.h
> @@ -69,6 +69,24 @@ struct nx_vendor_error {
>     /* Followed by at least the first 64 bytes of the failed request. */
> };
> 
> +/* Specific Nicira extension error numbers.
> + *
> + * These are the "code" values used in nx_vendor_error.  So far, the "type"
> + * values in nx_vendor_error are the same as those in ofp_error_msg.  That is,
> + * at Nicira so far we've only needed additional vendor-specific 'code' values,
> + * so we're using the existing 'type' values to avoid having to invent new ones
> + * that duplicate the current ones' meanings. */
> +
> +/* Additional "code" values for OFPET_FLOW_MOD_FAILED. */
> +enum {
> +    /* Generic hardware error. */
> +    NXFMFC_HARDWARE = 0x100,
> +
> +    /* A nonexistent table ID was specified in the "command" field of struct
> +     * ofp_flow_mod, when the nxt_flow_mod_table_id extension is enabled. */
> +    NXFMFC_BAD_TABLE_ID
> +};
> +
> /* Nicira vendor requests and replies. */
> 
> enum nicira_type {
> -- 
> 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