On Tue, Apr 24, 2012 at 10:32:55AM +0900, Simon Horman wrote: > Add oxm_name and oxm_header elements to struct mf_field > and populate those entries for fields that are present > in both NXM and the OXM basic class. > > This implementation was suggested by Ben Pfaff. > > This does not address any possible differences in the NXM and > OXM basic class fields, for instance different maskability. > That may be addressed later as needed. > > Signed-off-by: Simon Horman <[email protected]>
Seems reasonable. I think that there are also some semantic differences between the VLAN VID field that we implement currently, which is based on OF1.0, and the VLAN VID field in OF1.2. I'm not sure how we should deal with those. I think I'm OK with committing (the second version of) this once you've revised the first patch to make sure the field widths are correct. Thanks, Ben. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
