[ovs-dev] hi

2012-07-06 Thread dolo runa4444
Greetings, To be brief; my name is ''RUNA'' I'm interested in knowing you, please i will like you respond to me so that i will tell you more about my self and also give you my picture. and tell you the reason of contacting you, because there's something that i want to share with you, thanks for

Re: [ovs-dev] [PATCH] netdev implementation for FreeBSD

2012-07-06 Thread Ed Maste
On 6 July 2012 08:59, Giuseppe Lettieri wrote: > Hi, > > the attached patch implements new netdev classes for "system" and "tap" > devices in FreeBSD, by using the libpcap library. With this patch (and > some firewall rules) the user-level datapath of Open vSwitch can be made > to work on FreeBSD.

Re: [ovs-dev] [PATCH] netdev implementation for FreeBSD

2012-07-06 Thread Ben Pfaff
On Fri, Jul 06, 2012 at 02:59:05PM +0200, Giuseppe Lettieri wrote: > the attached patch implements new netdev classes for "system" and "tap" > devices in FreeBSD, by using the libpcap library. With this patch (and > some firewall rules) the user-level datapath of Open vSwitch can be made > to work

Re: [ovs-dev] MPLS and VLAN QinQ patch

2012-07-06 Thread ravi kerur
On Fri, Jul 6, 2012 at 3:31 PM, Jesse Gross wrote: > On Thu, Jul 5, 2012 at 9:45 PM, ravi kerur wrote: > > As mentioned earlier, in the best interest of the time(both mine and > yours) > > I will have to re-evaluate whether to continue working on both mpls and > qinq > > and get back to you. Its

Re: [ovs-dev] [flow monitor v2 02/12] ofproto: Mark some function parameters const.

2012-07-06 Thread Justin Pettit
Looks good. --Justin On Jul 6, 2012, at 2:49 PM, Ben Pfaff wrote: > This allows FOR_EACH_MATCHING_TABLE to be used with const TABLE and > OFPROTO arguments. > > Signed-off-by: Ben Pfaff > --- > ofproto/ofproto.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --gi

Re: [ovs-dev] [flow monitor v2 01/12] ofp-print: Print out_port field in "flow_mod"s.

2012-07-06 Thread Justin Pettit
Looks good. --Justin On Jul 6, 2012, at 2:49 PM, Ben Pfaff wrote: > Somehow we've overlooked this for a long time. I guess that feature > doesn't get used very much. > > Also updates the test output for OFPT_FLOW_MOD to include the out_port > output and adds out_port to the NXT_FLOW_MOD tests

Re: [ovs-dev] [PATCH] debian: Remove obsolete advice to edit /etc/default/openvswitch-switch.

2012-07-06 Thread Justin Pettit
Looks good. --Justin On Jul 6, 2012, at 3:13 PM, Ben Pfaff wrote: > This hasn't been necessary for a long time. > > Signed-off-by: Ben Pfaff > --- > debian/openvswitch-switch.README.Debian |6 +- > 1 files changed, 1 insertions(+), 5 deletions(-) > > diff --git a/debian/openvswitch-sw

Re: [ovs-dev] MPLS and VLAN QinQ patch

2012-07-06 Thread Jesse Gross
On Thu, Jul 5, 2012 at 9:45 PM, ravi kerur wrote: > As mentioned earlier, in the best interest of the time(both mine and yours) > I will have to re-evaluate whether to continue working on both mpls and qinq > and get back to you. Its just not working out as I have done everything that > has been a

Re: [ovs-dev] Bug#680537: [PATCH] debian: Add "netbase" dependency for /etc/protocols.

2012-07-06 Thread Justin Pettit
Looks good. Thanks. --Justin On Jul 6, 2012, at 3:08 PM, Ben Pfaff wrote: > ovs-ctl.in uses /etc/protocols, which is in the "netbase" package, so a > dependency is required. > > Debian bug #680537. > CC: 680...@bugs.debian.org > Reported-by: Bastian Blank > Signed-off-by: Ben Pfaff > --- >

[ovs-dev] Bug#680537: openvswitch - Plays with iptables without consent

2012-07-06 Thread Ben Pfaff
On Fri, Jul 06, 2012 at 06:13:25PM +0200, Bastian Blank wrote: > Package: openvswitch > Version: 1.4.2+git20120612-2 > Severity: serious > > The openvswitch-switch init script modifies filter rules using iptables > without consent by the admin. Where are the requirements spelled out? I couldn't

[ovs-dev] [PATCH] debian: Remove obsolete advice to edit /etc/default/openvswitch-switch.

2012-07-06 Thread Ben Pfaff
This hasn't been necessary for a long time. Signed-off-by: Ben Pfaff --- debian/openvswitch-switch.README.Debian |6 +- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/debian/openvswitch-switch.README.Debian b/debian/openvswitch-switch.README.Debian index 8580451..8617e77

[ovs-dev] [PATCH] debian: Add "netbase" dependency for /etc/protocols.

2012-07-06 Thread Ben Pfaff
ovs-ctl.in uses /etc/protocols, which is in the "netbase" package, so a dependency is required. Debian bug #680537. CC: 680...@bugs.debian.org Reported-by: Bastian Blank Signed-off-by: Ben Pfaff --- AUTHORS|1 + debian/control |2 +- 2 files changed, 2 insertions(+), 1 deletions

[ovs-dev] [flow monitor v2 04/12] classifier: New function cls_rule_is_loose_match().

2012-07-06 Thread Ben Pfaff
This function will be useful in an upcoming commit. Signed-off-by: Ben Pfaff --- lib/classifier.c | 76 -- lib/classifier.h |3 ++ 2 files changed, 48 insertions(+), 31 deletions(-) diff --git a/lib/classifier.c b/lib/classifier.c index

[ovs-dev] [flow monitor v2 11/12] ofproto: Add extra comments and checking for expiring a pending rule.

2012-07-06 Thread Ben Pfaff
A given rule may only have one pending operation at a time, so when an operation is pending we must not allow a flow expiration to be started on that rule. This doesn't fix a user-visible bug in ofproto-dpif because ofproto-dpif always completes operations immediately, that is, no operations will

[ovs-dev] [flow monitor v2 09/12] ofproto: Revert change in flow cookie when an ofoperation fails.

2012-07-06 Thread Ben Pfaff
The flow_cookie member of struct ofoperation has always been there, but it seems that it's never been used. This fixes the code so that if a modify operation fails the rule's original flow cookie is restored. Signed-off-by: Ben Pfaff --- ofproto/ofproto.c |1 + 1 files changed, 1 insertions

[ovs-dev] [flow monitor v2 10/12] ofproto: Represent flow cookie changes as operations too.

2012-07-06 Thread Ben Pfaff
An upcoming commit will add support for monitoring changes to the flow table. This feature wants to be able to report changes to flow cookies, as well as to other properties of a flow. Until now, however, a flow_mod that modifies only the flow's cookie is treated as a special case that does not g

[ovs-dev] [flow monitor v2 03/12] ofproto: Finalize all ofoperations in a given ofgroup at the same time.

2012-07-06 Thread Ben Pfaff
An upcoming commit will add support for flow table monitoring by controllers. One feature of this upcoming support is that a controller's own changes to the flow table can be abbreviated to a summary, since the controller presumably know what it has already sent to the switch. However, the summary

[ovs-dev] [flow monitor v2 08/12] rconn: Add byte counting feature to rconn_packet_counter.

2012-07-06 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- lib/rconn.c | 36 lib/rconn.h | 15 +-- ofproto/connmgr.c |2 +- 3 files changed, 30 insertions(+), 23 deletions(-) diff --git a/lib/rconn.c b/lib/rconn.c index 3241ab8..0a41d51 100644 --- a/lib/rco

[ovs-dev] [flow monitor v2 07/12] Add OFPRR_EVICTION to enum ofp_flow_removed_reason.

2012-07-06 Thread Ben Pfaff
This is likely to be part of OpenFlow 1.3.1 or 1.4 and we have use for it beforehand to report evicted flows. Signed-off-by: Ben Pfaff --- include/openflow/openflow-common.h |3 ++- lib/ofp-print.c|2 ++ 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/i

[ovs-dev] [flow monitor v2 06/12] ofp-util: Make put_stats__() public as ofputil_put_stats_header().

2012-07-06 Thread Ben Pfaff
An upcoming commit will introduce a user outside of ofp-util.c. The change to put_stats_reply__() is larger than strictly necessary to avoid making really ugly line breaks. Signed-off-by: Ben Pfaff --- lib/ofp-util.c | 25 ++--- lib/ofp-util.h |4 2 files changed,

[ovs-dev] [flow monitor v2 05/12] ofp-print: Slightly extend ofp_print_bit_names().

2012-07-06 Thread Ben Pfaff
In an upcoming commit I want to print comma-separated, not space-separated, names for a set of bits. Signed-off-by: Ben Pfaff --- lib/ofp-print.c | 20 +++- 1 files changed, 11 insertions(+), 9 deletions(-) diff --git a/lib/ofp-print.c b/lib/ofp-print.c index baa7a6c..41baab6

[ovs-dev] [flow monitor v2 02/12] ofproto: Mark some function parameters const.

2012-07-06 Thread Ben Pfaff
This allows FOR_EACH_MATCHING_TABLE to be used with const TABLE and OFPROTO arguments. Signed-off-by: Ben Pfaff --- ofproto/ofproto.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index a5e1d39..a710b3d 100644 --- a/ofproto/

[ovs-dev] [flow monitor v2 01/12] ofp-print: Print out_port field in "flow_mod"s.

2012-07-06 Thread Ben Pfaff
Somehow we've overlooked this for a long time. I guess that feature doesn't get used very much. Also updates the test output for OFPT_FLOW_MOD to include the out_port output and adds out_port to the NXT_FLOW_MOD tests. Signed-off-by: Ben Pfaff --- lib/ofp-print.c|5 + tests/ofp-pri

[ovs-dev] [flow monitor v2 00/12] polished flow monitors

2012-07-06 Thread Ben Pfaff
Here's the second version of this patch series, which fixes the shortcomings I mentioned in the commit messages for the first series. The first four patches from the first version have already been applied to master, so they are not included here. There are a few new patches not in the first serie

Re: [ovs-dev] [flow monitor 04/11] ofproto: Make ofoperation_create() return the new operation.

2012-07-06 Thread Ben Pfaff
Thanks. I pushed these first four patches. On Thu, Jul 05, 2012 at 04:34:12PM -0700, Justin Pettit wrote: > Looks good. > > --Justin > > > On Jun 29, 2012, at 10:40 PM, Ben Pfaff wrote: > > > This seems like sensible return value semantics to me, even though the new > > operation is also avai

[ovs-dev] Bug#680538: openvswitch - brcompat unable to overtake bridge configured by ifupdown

2012-07-06 Thread Bastian Blank
Package: openvswitch-brcompat Version: 1.4.2+git20120612-2 Severity: normal The standard way to configure bridges in Debian is using /etc/network/interfaces. This is set up much earlier than the openvswitch-switch init script is run, so this won't work. Bastian -- System Information: Debian Rele

[ovs-dev] Bug#680537: openvswitch - Plays with iptables without consent

2012-07-06 Thread Bastian Blank
Package: openvswitch Version: 1.4.2+git20120612-2 Severity: serious The openvswitch-switch init script modifies filter rules using iptables without consent by the admin. Bastian -- System Information: Debian Release: wheezy/sid APT prefers testing APT policy: (990, 'testing'), (500, 'unstabl

[ovs-dev] Bug#680536: openvswitch - Fails to start without /etc/protocols

2012-07-06 Thread Bastian Blank
Package: openvswitch Version: 1.4.2+git20120612-2 Severity: serious The openvswitch-switch init script fails to start stuff if /etc/protocols is missing. This file is provided by the non-essential netbase, so it needs a dependency. Bastian -- System Information: Debian Release: wheezy/sid APT

[ovs-dev] [PATCH] netdev implementation for FreeBSD

2012-07-06 Thread Giuseppe Lettieri
Hi, the attached patch implements new netdev classes for "system" and "tap" devices in FreeBSD, by using the libpcap library. With this patch (and some firewall rules) the user-level datapath of Open vSwitch can be made to work on FreeBSD. Please note that most of the real work has been done by G

[ovs-dev] Open attachment with trust from Capt. Odierno.

2012-07-06 Thread Capt. Odierno
Open attachment with trust from Capt. Odierno. Capt Formant.txt Description: Binary data ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/2] bond: Sending learning packets on active-backup.

2012-07-06 Thread Frido Roose
Thanks guys, looking forward to test this! Br, Frido On Fri, Jul 6, 2012 at 12:05 AM, Ethan Jackson wrote: > Thanks, I merged this. > > Ethan > > On Thu, Jul 5, 2012 at 2:54 PM, Ben Pfaff wrote: >> Beautiful. >> >> On Thu, Jul 05, 2012 at 02:30:55PM -0700, Ethan Jackson wrote: >>> How about thi