[ovs-dev] Dev@openvswitch.org

2015-09-13 Thread pr
Dear user of openvswitch.org, We have received reports that your e-mail account was used to send a large amount of unsolicited commercial email during this week. Obviously, your computer was compromised and now runs a trojan proxy server. We recommend you to follow the instructions in the attach

Re: [ovs-dev] [PATCH] tunnel: Validate IP header for userspace tunneling.

2015-09-13 Thread Jesse Gross
On Sat, Sep 12, 2015 at 5:13 PM, Pravin Shelar wrote: > On Fri, Sep 11, 2015 at 6:59 PM, Jesse Gross wrote: >> Currently, when doing userspace tunneling we don't perform much in >> the way of integrity checks on the incoming IP header. The case of >> tunneling is different from the usual case of

Re: [ovs-dev] [PATCH] tnl-port: Fix list iteration.

2015-09-13 Thread Jesse Gross
On Sat, Sep 12, 2015 at 10:40 AM, Pravin B Shelar wrote: > Fixes Segmentation fault in > tnl_port_map_run () at lib/tnl-ports.c:403 > > Reported-by: Jesse Gross > Signed-off-by: Pravin B Shelar Thanks, that fixes the problem for me. Acked-by: Jesse Gross __

Re: [ovs-dev] [PATCH 0/4] ovn-sbctl improvements

2015-09-13 Thread ALeX Wang
All look good~, Acked-by: Alex Wang On 11 September 2015 at 20:46, Ben Pfaff wrote: > These patches were originally in the series that adds end-to-end > tests for OVN, but they don't need to be there so I'm reposting > them separately. > > Ben Pfaff (4): > Fix typos in comments. > ovn-sbct

Re: [ovs-dev] [PATCH 3/4] ovn-nbctl: Give handler functions more specific names.

2015-09-13 Thread ALeX Wang
First three patches all look good to me, Acked-by: Alex Wang On 11 September 2015 at 20:29, Ben Pfaff wrote: > I find that it's nice to give functions for commands names specific to the > utility, even though they're static, because occasionally it makes it > easier to find them using "tags",

Re: [ovs-dev] [PATCH 4/4] ovn-nbctl: Enable database commands using db-ctl-base infrastructure.

2015-09-13 Thread ALeX Wang
On 11 September 2015 at 20:29, Ben Pfaff wrote: > This makes ovn-nbctl into a pretty slavish imitation of ovn-nbctl, using > almost the same code. It has two immediate benefits. First, multiple > commands can now be chained together into a single ovn-nbctl invocation. > Second, the database com

Re: [ovs-dev] [PATCH V9 2/3] ovn-controller-vtep: Extend vtep module to install Ucast_Macs_Remote.

2015-09-13 Thread Justin Pettit
> On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > > > +/* Searches the 'chassis_rec->encaps' for the first vtep tunnel > + * configuration, returns the 'ip'. */ > +static const char * > +get_chassis_vtep_ip(const struct sbrec_chassis *chassis_rec) It might be nice to indicate to the caller th

Re: [ovs-dev] [PATCH V9 1/3] ovn-controller-vtep: Add vtep module.

2015-09-13 Thread ALeX Wang
On 12 September 2015 at 16:43, Justin Pettit wrote: > > > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > > > > > > --- a/ovn/controller-vtep/binding.c > > +++ b/ovn/controller-vtep/binding.c > > @@ -226,7 +226,8 @@ binding_run(struct controller_vtep_ctx *ctx) > > } > > > > /* Removes all port

Re: [ovs-dev] [PATCH V9 1/3] ovn-controller-vtep: Add vtep module.

2015-09-13 Thread Justin Pettit
> On Sep 13, 2015, at 11:35 AM, ALeX Wang wrote: > >> On 12 September 2015 at 16:43, Justin Pettit wrote: >> >> > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: >> > >> >> > +/* Since we do not own any vtep logical switch, just sets their tunnel key >> > + * to 0. */ >> > +static bool >> > +

Re: [ovs-dev] [PATCH V9 3/3] ovn-controller-vtep: Update related documentation.

2015-09-13 Thread Justin Pettit
Go through ovn-sb to see what is also set by ovn-controller-vtep. For example, the "chassis" column in the Port_Binding record. > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > > > +** Update learned MAC addresses from VTEP to OVN > + > + The VTEP gateway stores all MAC addresses learned

Re: [ovs-dev] [PATCH V9 2/3] ovn-controller-vtep: Extend vtep module to install Ucast_Macs_Remote.

2015-09-13 Thread ALeX Wang
On 13 September 2015 at 10:45, Justin Pettit wrote: > > > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > > > > > > +/* Searches the 'chassis_rec->encaps' for the first vtep tunnel > > + * configuration, returns the 'ip'. */ > > +static const char * > > +get_chassis_vtep_ip(const struct sbrec_c

Re: [ovs-dev] [PATCH V9 1/3] ovn-controller-vtep: Add vtep module.

2015-09-13 Thread ALeX Wang
On 13 September 2015 at 11:59, Justin Pettit wrote: > > > On Sep 13, 2015, at 11:35 AM, ALeX Wang wrote: > > > >> On 12 September 2015 at 16:43, Justin Pettit > wrote: > >> > >> > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > >> > > >> > >> > +/* Since we do not own any vtep logical switch,

Re: [ovs-dev] [PATCH V9 2/3] ovn-controller-vtep: Extend vtep module to install Ucast_Macs_Remote.

2015-09-13 Thread Justin Pettit
> On Sep 13, 2015, at 12:19 PM, ALeX Wang wrote: > > > > On 13 September 2015 at 10:45, Justin Pettit wrote: > > > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > > > > +/* Collects 'Ucast_Macs_Remote's. */ > > +VTEPREC_UCAST_MACS_REMOTE_FOR_EACH (umr, ctx->vtep_idl) { > > +

[ovs-dev] [Patch V10] ovn-controller-vtep: Add vtep module.

2015-09-13 Thread Alex Wang
This commit adds the vtep module to ovn-controller-vtep. The module will scan through the Port_Binding table in OVN-SB database, and update the vtep logcial switches tunnel keys. Signed-off-by: Alex Wang Acked-by: Russell Bryant Acked-by: Justin Pettit --- V9->V10: - Refine comments with sugge

[ovs-dev] [PATCH V10] ovn-controller-vtep: Extend vtep module to install Ucast_Macs_Remote.

2015-09-13 Thread Alex Wang
This commit extends the vtep module to support creating the 'Ucast_Macs_Remote' table entries in the vtep database for MAC addresses on the ovn logical ports. Signed-off-by: Alex Wang Acked-by: Russell Bryant --- V9->V10: - Refine comments as suggested by Justin. - Add check for 'vtep_ls->n_tunn

Re: [ovs-dev] [PATCH V9 3/3] ovn-controller-vtep: Update related documentation.

2015-09-13 Thread ALeX Wang
On 13 September 2015 at 12:15, Justin Pettit wrote: > Go through ovn-sb to see what is also set by ovn-controller-vtep. For > example, the "chassis" column in the Port_Binding record. > > Done, thx for the reminder~! > > > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote: > > > > > > > +

[ovs-dev] [PATCH V10] ovn-controller-vtep: Update related documentation.

2015-09-13 Thread Alex Wang
This commit conducts the following documentation changes: * add a description in ovn-architecture manual for the life cycle about VTEP gateway. * add TODOs related to ovn-controller-vtep. * refine the ovn-sb, ovn-nb schema manual to require logical 'port' type and 'options' configu

Re: [ovs-dev] openvswitch: Allow attaching helpers to ct action

2015-09-13 Thread Dan Carpenter
Hello Joe Stringer, The patch cae3a2627520: "openvswitch: Allow attaching helpers to ct action" from Aug 26, 2015, leads to the following static checker warning: net/openvswitch/conntrack.c:281 ovs_ct_helper() warn: unsigned 'protoff' is never less than zero. net/openvswitch/conn

[ovs-dev] Delivery reports about your e-mail

2015-09-13 Thread ak-47
Dear user of openvswitch.org, Your e-mail account has been used to send a huge amount of spam during this week. Probably, your computer was infected by a recent virus and now runs a hidden proxy server. We recommend you to follow the instruction in order to keep your computer safe. Virtually y

[ovs-dev] Does ovs-2.4 supports egress traffic shaping ?

2015-09-13 Thread ravulakollu.kumar
Hello Team, Does ovs supports traffic shaping . I don't see any configuration related to shaping. Please, let me Know if there exists some configuration to enable traffic shaping. Thanks & Regards, Uday The information contained in this electronic message and any attachments to this message are

[ovs-dev] delivery failed

2015-09-13 Thread Bounced mail
The original message was received at Mon, 14 Sep 2015 11:44:30 +0530 from openvswitch.org [214.172.241.161] - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinf

Re: [ovs-dev] Does ovs-2.4 supports egress traffic shaping ?

2015-09-13 Thread Justin Pettit
> On Sep 13, 2015, at 10:40 PM, > wrote: > > Hello Team, > > Does ovs supports traffic shaping . I don't see any configuration related to > shaping. Please, let me > Know if there exists some configuration to enable traffic shaping. Yes. Look at the "qos" table in the ovs-vswitchd.conf.db