Looks good.
--Justin
On Aug 9, 2011, at 1:46 PM, Ben Pfaff wrote:
> The code and the specification say that nxm_length includes both value
> and mask, but this example showed nxm_length only including the value.
> This commit fixes it.
>
> Reported-by: Justin Pettit <jpet...@nicira.com>
> ---
> include/openflow/nicira-ext.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h
> index 357763c..8b06785 100644
> --- a/include/openflow/nicira-ext.h
> +++ b/include/openflow/nicira-ext.h
> @@ -1101,7 +1101,7 @@ enum nx_bd_algorithm {
> * numbers and byte indexes.
> *
> *
> - * 8-bit nxm_value, nxm_hasmask=1, nxm_length=1:
> + * 8-bit nxm_value, nxm_hasmask=1, nxm_length=2:
> *
> * 0 3 4 5
> * +------------+---+---+
> --
> 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
