[ovs-dev] [PATCH] vagrant: make sure to bootstrap before configure_ovs.

2015-01-09 Thread Motonori Shindo
The order of execution in Vagrantfile is "outside-in" meaning that all commands in outer scope are executed first and then the commands in inner scope. Because of this ordering, "configure_ovs" is executed before "bootstrap_fedora", resulting in "configure_ovs" to fail as it is not bootsrapped yet.

[ovs-dev] devis traduction de sites Internet - supports de communication

2015-01-09 Thread Aurelien SUPOT A.Text Work
Si vous ne visualisez pas correctement l’e-mail, cliquez ici Bonjour, Je me permets de vous écrire afin de vous proposer nos services de traduction. Disposez-vous déjà d'une agence partenaire? Notre

Re: [ovs-dev] [PATCH] vagrant: make sure to bootstrap before configure_ovs.

2015-01-09 Thread Thomas Graf
On 01/09/15 at 06:31pm, Motonori Shindo wrote: > The order of execution in Vagrantfile is "outside-in" meaning that > all commands in outer scope are executed first and then the commands > in inner scope. Because of this ordering, "configure_ovs" is executed > before "bootstrap_fedora", resulting i

[ovs-dev] [PATCH] pkg-config: Fix Cflags in package-config files

2015-01-09 Thread Thomas Graf
From: Amit Bose Cflags in pkg-config files sets the include path to $PREFIX/openflow, $PREFIX/openvswitch. This makes the including source files use the files like include instead of include Signed-off-by: Amit Bose --- lib/libopenvswitch.pc.in | 2 +- lib/libsflow.pc.in | 2 +- of

Re: [ovs-dev] Building OVS on Ubuntu 12.04 stuck in atomic operation unit test

2015-01-09 Thread Finucane, Stephen
> Jarno's idea of getting a disassembly is a good idea, but it might be > easier if you just pass along test-atomic.o from a broken machine. > Then we can just disassemble it and otherwise examine it for > ourselves. I applied Jarno's patch, but alas it had no effect. I've attached the '.o' file

Re: [ovs-dev] [PATCH] vagrant: make sure to bootstrap before configure_ovs.

2015-01-09 Thread Motonori Shindo
Thomas, Thank you for the review and merging. Next I will work on Vagrantfile for Ubuntu. Sorry for the inconvenience caused by unintended newlines. I will be more careful next time. Regards, --- Motonori Shindo 2015-01-09 20:11 GMT+09:00 Thomas Graf : > On 01/09/15 at 06:31pm, Motonori Shin

[ovs-dev] [PATCH 1/1] openvswitch: Remove unnecessary version.h inclusion

2015-01-09 Thread Syam Sidhardhan
version.h inclusion is not necessary as detected by versioncheck. Signed-off-by: Syam Sidhardhan --- net/openvswitch/vport-geneve.c |2 -- 1 file changed, 2 deletions(-) diff --git a/net/openvswitch/vport-geneve.c b/net/openvswitch/vport-geneve.c index 347fa23..70e2aae 100644 --- a/net/open

[ovs-dev] [PATCH] vagrant: Build kernel module RPM

2015-01-09 Thread Thomas Graf
Installs the kernel-devel package of the currently running kernel and builds the kmod RPM in the "install_rpm" phase. Signed-off-by: Thomas Graf --- Vagrantfile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Vagrantfile b/Vagrantfile index 82b157c..1cacd3b 100644 --- a/Va

Re: [ovs-dev] [PATCHv2] Update SECURITY.md

2015-01-09 Thread Ben Pfaff
On Fri, Jan 09, 2015 at 10:44:20AM +1300, Andrew Kampjes wrote: > +Reporters may ask for a GPG key while initiating contact with the > +security team to deliver more sensitive reports. > +If the reporter has used GPG while disclosing, further vulnerability > +details should also be discussed using

[ovs-dev] [PATCH] FAQ.md: Describe OpenFlow packet buffering.

2015-01-09 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- FAQ.md | 41 + 1 file changed, 41 insertions(+) diff --git a/FAQ.md b/FAQ.md index 458e07a..9e95d07 100644 --- a/FAQ.md +++ b/FAQ.md @@ -1622,6 +1622,47 @@ A: Reconfiguring your bridge can change your bridge's datapath-id beca

Re: [ovs-dev] [PATCH 2/6] vxlan: Group Policy extension

2015-01-09 Thread Alexei Starovoitov
On Thu, Jan 8, 2015 at 2:47 PM, Thomas Graf wrote: > + > +struct vxlan_gbp { > +} __packed; empty struct ? seems unused. looks good to me otherwise. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] FAQ.md: Describe OpenFlow packet buffering.

2015-01-09 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme On Jan 9, 2015, at 8:28 AM, Ben Pfaff wrote: > Signed-off-by: Ben Pfaff > --- > FAQ.md | 41 + > 1 file changed, 41 insertions(+) > > diff --git a/FAQ.md b/FAQ.md > index 458e07a..9e95d07 100644 > --- a/FAQ.md > +++ b/FAQ.md > @

Re: [ovs-dev] [PATCH 1/1] openvswitch: Remove unnecessary version.h inclusion

2015-01-09 Thread Pravin Shelar
On Fri, Jan 9, 2015 at 6:56 AM, Syam Sidhardhan wrote: > version.h inclusion is not necessary as detected by versioncheck. > > Signed-off-by: Syam Sidhardhan You should use net-next prefix flag for net-next tree patches. Otherwise looks good. Acked-by: Pravin B Shelar > --- > net/openvswitch/

Re: [ovs-dev] [PATCH] vagrant: Build kernel module RPM

2015-01-09 Thread Andy Zhou
Thomas, Thanks for fixing this. Acked-by: Andy Zhou On Fri, Jan 9, 2015 at 7:25 AM, Thomas Graf wrote: > Installs the kernel-devel package of the currently running kernel and > builds the kmod RPM in the "install_rpm" phase. > > Signed-off-by: Thomas Graf > --- > Vagrantfile | 4 +++- > 1 fil

[ovs-dev] [PATCH v3 1/2] classifier: Make classifier_lookup() 'flow' parameter non-const.

2015-01-09 Thread Ben Pfaff
An upcoming commit will make classifier_lookup() sometimes modify its 'flow' argument temporarily during the lookup. Signed-off-by: Ben Pfaff Acked-by: Jarno Rajahalme --- v2: New patch. v2.1: Rebase. v3: Rebase. --- lib/classifier.c |7 +-- lib/classifier.h |2 +- lib/t

[ovs-dev] [PATCH v3 2/2] [RFC] classifier: Add support for conjunctive matches.

2015-01-09 Thread Ben Pfaff
A "conjunctive match" allows higher-level matches in the flow table, such as set membership matches, without causing a cross-product explosion for multidimensional matches. Please refer to the documentation that this commit adds to ovs-ofctl(8) for a better explanation, including an example. Sign

Re: [ovs-dev] [PATCH v2.1 2/2] [RFC] classifier: Add support for conjunctive matches.

2015-01-09 Thread Ben Pfaff
Thanks. I posted v3: http://openvswitch.org/pipermail/dev/2015-January/050297.html http://openvswitch.org/pipermail/dev/2015-January/050296.html I posted it marked as [RFC], but that's a mistake: I consider this now ready for review. On Tue, Jan 06, 2015 at 02:45:34PM -0800, Jarn

Re: [ovs-dev] [PATCH 2/6] vxlan: Group Policy extension

2015-01-09 Thread Thomas Graf
On 01/09/15 at 09:37am, Alexei Starovoitov wrote: > On Thu, Jan 8, 2015 at 2:47 PM, Thomas Graf wrote: > > + > > +struct vxlan_gbp { > > +} __packed; > > empty struct ? seems unused. > looks good to me otherwise. Poor leftover, must feel all lonely there. Thanks for the reviews. Will wait a litt

Re: [ovs-dev] [PATCHv2] Update SECURITY.md

2015-01-09 Thread Andrew Kampjes
So the way that I would see this working, is the security team would have upto maybe 4 people on it. If a researcher just sends the report in the clear to the list, all good, just keep discussing on the list in plaintext. If a researcher requests GPG encryption, then someone from the list would se

Re: [ovs-dev] [PATCH] vagrant: Build kernel module RPM

2015-01-09 Thread Thomas Graf
On 01/09/15 at 11:10am, Andy Zhou wrote: > Thomas, Thanks for fixing this. > > Acked-by: Andy Zhou Thanks, pushed to master. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v3 2/2] [RFC] classifier: Add support for conjunctive matches.

2015-01-09 Thread Jarno Rajahalme
With the small nits below: Acked-by: Jarno Rajahalme On Jan 9, 2015, at 11:42 AM, Ben Pfaff wrote: > A "conjunctive match" allows higher-level matches in the flow table, such > as set membership matches, without causing a cross-product explosion for > multidimensional matches. Please refer

Re: [ovs-dev] [PATCH v2.1 2/2] [RFC] classifier: Add support for conjunctive matches.

2015-01-09 Thread Jarno Rajahalme
On Jan 9, 2015, at 11:44 AM, Ben Pfaff wrote: > Thanks. > > I posted v3: >http://openvswitch.org/pipermail/dev/2015-January/050297.html >http://openvswitch.org/pipermail/dev/2015-January/050296.html > > I posted it marked as [RFC], but that's a mistake: I consider this now > re