On Wed, Feb 06, 2013 at 09:12:10AM -0800, Justin Pettit wrote: > On Feb 6, 2013, at 8:43 AM, Ben Pfaff <b...@nicira.com> wrote: > > > + In some situations, you can use "normal switching" as a component > > + of your OpenFlow actions, e.g. by putting "normal" into the lists > > + of actions on ovs-ofctl or by outputting to OFPP_NORMAL from an > > + OpenFlow controller. In situations where this is not suitable, you > > + can implement VLAN handling yourself, e.g.: > > + > > + - If a packet comes in on an access port, and the flow table > > + needs to send it out on a trunk port, then the flow can add > > + the appropriate VLAN tag with the "mod_vlan_vid" action. > > + > > + - If a packet comes in on a trunk port, and the flow table > > + needs to send it out on an access port, then the flow can > > + strip the VLAN tag with the "strip_vlan" action. > > I think it would be clearer if you dropped "In some situations" from > the beginning of the sentence. Otherwise, I think it looks great.
I made that change and applied this to master. Thanks. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
