Thanks, I pushed this.
On Tue, Aug 09, 2011 at 01:56:34PM -0700, Justin Pettit wrote:
> 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
