Re: [ovs-dev] [PATCH v2] vswitch.xml: Improve description of forward-bpdu.

2015-03-31 Thread Ben Pfaff
On Tue, Mar 31, 2015 at 06:17:42PM +, Field, Brian wrote: > I?m L2 challenged, so please forgive my lack of clue in this space. Me too ;-) > First, is CDP really a BPDU? It?s not performing any sort of spanning > tree functionality so I question why OVS?s forwarding of CDP frames is > tied t

Re: [ovs-dev] [PATCH v2] vswitch.xml: Improve description of forward-bpdu.

2015-03-31 Thread Ben Pfaff
On Tue, Mar 31, 2015 at 10:14:55AM -0700, Justin Pettit wrote: > > > On Mar 31, 2015, at 8:31 AM, Ben Pfaff wrote: > > > > + If STP is enabled on the bridge (see the > + table="Open_vSwitch"/> column in the table), > > I think that should be "Bridge" instead of "Open_vSwitch". >

Re: [ovs-dev] [PATCH v2] vswitch.xml: Improve description of forward-bpdu.

2015-03-31 Thread Field, Brian
I¹m L2 challenged, so please forgive my lack of clue in this space. First, is CDP really a BPDU? It¹s not performing any sort of spanning tree functionality so I question why OVS¹s forwarding of CDP frames is tied to the bpdu-forwarding value. Is there a reason why CDP is included in the list p

Re: [ovs-dev] [PATCH v2] vswitch.xml: Improve description of forward-bpdu.

2015-03-31 Thread Justin Pettit
> On Mar 31, 2015, at 8:31 AM, Ben Pfaff wrote: > > + If STP is enabled on the bridge (see the + table="Open_vSwitch"/> column in the table), I think that should be "Bridge" instead of "Open_vSwitch". > + Set this option to true if the Open vSwitch bridge > + conn

[ovs-dev] [PATCH v2] vswitch.xml: Improve description of forward-bpdu.

2015-03-31 Thread Ben Pfaff
The description was at best incomplete and difficult to understand. Reported-by: Brian Field Signed-off-by: Ben Pfaff --- v1->v2: Fix reporter. AUTHORS | 1 + vswitchd/vswitch.xml | 52 2 files changed, 41 insertions(+), 12 del