OK, great, thanks.

On Thu, Feb 02, 2017 at 11:01:19AM -0800, Yi-Hung Wei wrote:
> Hi Ben,
> 
> This patch is updated by [PATCH v2] ofp-actions: Fix variable length
> meta-flow field usage in flow actions, and it is applied to master
> yesterday.
> 
> Best,
> 
> -Yi-Hung
> 
> On Thu, Feb 2, 2017 at 10:45 AM, Ben Pfaff <b...@ovn.org> wrote:
> 
> > On Mon, Jan 09, 2017 at 06:09:33PM -0800, Yi-Hung Wei wrote:
> > > Previously, if a flow action that involves a tunnel metadata subfield is
> > > dumpped from vswitchd, the replied subfield length in the OXM header is
> > > filled with the maximum possible field length, instead of the length
> > > configured in the tunnel TLV mapping table.
> > >
> > > In order to derive the correct length of a tun_metadata subfield, this
> > patch
> > > passes the tunnel TLV mapping table (struct tun_table) to where we decode
> > > the flow actions. extract-ofp-actions is updated to pass the tunnel TLV
> > mapping
> > > table to the flow action decoding functions. Also, a new error code is
> > > introduced to identify a flow with an invalid tun_metadata ID. Moreover,
> > a
> > > testcase is added to prevent future regressions.
> > >
> > > Currently, this patch only addresses the usage of tun_metaata for
> > REG_MOVE action.
> > > Other actions that may use the tun_metadata will be addressed in
> > subsequent patches.
> > >
> > > VMWare-BZ: #1768370
> > > Reported-by: Harold Lim <haro...@vmware.com>
> > > Signed-off-by: Yi-Hung Wei <yihung....@gmail.com>
> >
> > I think I've seen some side discussion on this patch.  Is it still
> > relevant as is or is some revision already in progress?
> >
> > In any case, s/meatdata/metadata/ in the subject.
> >
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to