On Tue, Nov 12, 2013 at 04:45:51PM +0900, Simon Horman wrote:
> Cc: Andy Zhou <az...@nicira.com>
> Signed-off-by: Simon Horman <ho...@verge.net.au>
> ---
> include/openflow/openflow-common.h | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/include/openflow/openflow-common.h
> b/include/openflow/openflow-common.h
> index 234bf96..17ba915 100644
> --- a/include/openflow/openflow-common.h
> +++ b/include/openflow/openflow-common.h
> @@ -475,4 +475,12 @@ enum ofp_table {
> flow stats and flow deletes. */
> };
>
> +enum ofp_table_config {
> + OFPTC_TABLE_MISS_CONTROLLER = 0, /* Send to controller. */
> + OFPTC_TABLE_MISS_CONTINUE = 1 << 0, /* Continue to the next table in the
> + pipeline (OpenFlow 1.0 behavior).
> */
> + OFPTC_TABLE_MISS_DROP = 1 << 1, /* Drop the packet. */
> + OFPTC_TABLE_MISS_MASK = 3
> +};
I know that at least some version of the standard writes it that way but
the constants don't make any sense in that form. 0 << 0, 1 << 0, 2 <<
0, 3 << 0 are a lot more sensible. So I changed the constants to read
that way and applied this.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
