On Feb 14, 2012, at 11:05 AM, Ben Pfaff wrote:
> On Tue, Feb 14, 2012 at 08:01:58PM +0100, ravi.ke...@telekom.com wrote:
>> I started comparing the code base(my code base and latest), so far
>> the changes I suspect is in ofp_match structure. I am not sure
>> whether the fields I have added for mpls has anything to do with the
>> vlan match problem. I have looked into specific 'c' code
>> lib/classifier.c and lib/ofp-util.c related to dl_vlan and
>> dl_vlan_pcp, I don't see any difference.
>
> You must not change ofp_match, because its format is fixed by the
> OpenFlow 1.0 standard and there is no mechanism for extending it.
> Instead, you should define and use NXM ("Nicira Extensible Match")
> fields, which are extensible. OpenFlow 1.2 adopts an NXM variant
> called OXM that includes definitions for MPLS; you should adapt those
> to NXM.
Ravi, you may want to look at an example where we added a new match. For
example, commit a61680c (Support matching and modifying IP TTL.) adds an NXM
field and modifies the kernel.
--Justin
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
