Re: [ovs-dev] [PATCH 2/2] [RFC] Add OXM data to mf_fields

2012-04-25 Thread Simon Horman
On Wed, Apr 25, 2012 at 09:00:44AM -0700, Ben Pfaff wrote: > On Wed, Apr 25, 2012 at 09:20:10AM +0900, Simon Horman wrote: > > On Tue, Apr 24, 2012 at 09:14:40AM -0700, Ben Pfaff wrote: > > > I think that there are also some semantic differences between the VLAN > > > VID field that we implement cu

Re: [ovs-dev] [PATCH 2/2] [RFC] Add OXM data to mf_fields

2012-04-25 Thread Ben Pfaff
On Wed, Apr 25, 2012 at 09:20:10AM +0900, Simon Horman wrote: > On Tue, Apr 24, 2012 at 09:14:40AM -0700, Ben Pfaff wrote: > > 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

Re: [ovs-dev] [PATCH 2/2] [RFC] Add OXM data to mf_fields

2012-04-24 Thread Simon Horman
On Tue, Apr 24, 2012 at 09:14:40AM -0700, Ben Pfaff wrote: > 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 im

Re: [ovs-dev] [PATCH 2/2] [RFC] Add OXM data to mf_fields

2012-04-24 Thread Ben Pfaff
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

Re: [ovs-dev] [PATCH 2/2] [RFC] Add OXM data to mf_fields

2012-04-23 Thread Simon Horman
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

[ovs-dev] [PATCH 2/2] [RFC] Add OXM data to mf_fields

2012-04-23 Thread Simon Horman
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 di