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 <[email protected]>
> Signed-off-by: Yi-Hung Wei <[email protected]>

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
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to