[ovs-dev] [PATCH V8] windows: Added lockf function and lock PID file

2016-07-14 Thread Paul Boca
If the PID file isn't locked then appctl.py detects it as stale and bails out without doing anything. Because of this lots of Python tests fail. Also this protects the PID file from being overwritten. I used only shared lock, in order to be compatible with Python tests, which try to acquire the lo

[ovs-dev] Returned mail: see transcript for details

2016-07-14 Thread connie
The message could not be delivered ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Paul Boca
Hi Guru! I will re-spin the patch with LOCKFILE_FAIL_IMMEDIATELY flag set to avoid the hang of other daemons. Thanks, Paul > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Guru Shetty > Sent: Friday, July 15, 2016 6:58 AM > To: Alin Serdean > Cc: dev@ope

[ovs-dev] Returned mail: Data format error

2016-07-14 Thread The Post Office
Dear user of openvswitch.org, mail system administrator of openvswitch.org would like to let you know that: We have received reports that your e-mail account has been used to send a large amount of junk e-mail messages during this week. We suspect that your computer had been compromised and now

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

2016-07-14 Thread MAILER-DAEMON
a.D‹¨ÀÞ.áÔÎçKàJã ýcª—šY~œwíZ0H±"â*ÜsƒpyÁŒS¦õ®\©j%°×1ƒhÐ{ÖY‹é_;þ}&oµÑ¦¯G®tìM 5Ԝ %ÓXX˜ÆT4PG«úWM´Â‡eLºÎôôgÞ¶‰ö£|²ePØ\Gsܶ_Í[,|”<ݾôú ÜcúÛæà©a^²ù‚UH—òðai!5ùފòçjž<ô&;Ìþù4‘W–hœIÅw`)Ù3‹‰°tqÅàÓé,U†U,l`¦üIù&Üö¬Ã#¯×Õ¡çåkÎ_œa*]“î¼¶¾¼a"â-éäºÚ¯Hæ^7ÛiÜ¥ñ *vC!v× ´G¨`Ð!º¤éoIê¤N·¾z½¸]3Šaˆ¿ähªæØHš™¿w»üPmú

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Guru Shetty
On 14 July 2016 at 20:03, Alin Serdean wrote: > > > De la: Guru Shetty [mailto:g...@ovn.org] > Trimis: Friday, July 15, 2016 5:49 AM > Către: Alin Serdean > Cc: Paul Boca ; dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock > PID file > > > > On 14 Jul

Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation

2016-07-14 Thread Guru Shetty
On 14 July 2016 at 20:42, Guru Shetty wrote: > > > On 14 July 2016 at 19:57, Alin Serdean > wrote: > >> On 12 July 2016 at 20:32, Alin Serdean >> wrote: >> >> Currently in the case of command line arguments punix/unix, on Windows >> we create a file, write a TCP port number to connect. This is

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Alin Serdean
Another option would be to ditch fopen(which opens a file with FILE_SHARE_WRITE | FILE_SHARE_WRITE) and use CreateFile(https://msdn.microsoft.com/en-us/library/windows/desktop/aa363858(v=vs.85).aspx ) without FILE_SHARE_WRITE and from python check if we can write to it (since only the owner can

Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation

2016-07-14 Thread Guru Shetty
On 14 July 2016 at 19:57, Alin Serdean wrote: > On 12 July 2016 at 20:32, Alin Serdean > wrote: > > Currently in the case of command line arguments punix/unix, on Windows > we create a file, write a TCP port number to connect. This is a security > concern. > > This patch adds support for the com

[ovs-dev] [RFC] datapath-windows: Allow validation ports processing

2016-07-14 Thread Alin Serdean
Currently we do not validate ports prior a VM being started. This patch allows ports to be validated during creation. Validation ports have the same lifecycle as an operational port so allow them to have the same treatment as an operational port. Beside the above this also allows us to add OVS po

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Alin Serdean
De la: Guru Shetty [mailto:g...@ovn.org] Trimis: Friday, July 15, 2016 5:49 AM Către: Alin Serdean Cc: Paul Boca ; dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file On 14 July 2016 at 19:25, Alin Serdean mailto:aserd...@cloudbasesolutions.

Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation

2016-07-14 Thread Alin Serdean
On 12 July 2016 at 20:32, Alin Serdean mailto:aserd...@cloudbasesolutions.com>> wrote: Currently in the case of command line arguments punix/unix, on Windows we create a file, write a TCP port number to connect. This is a security concern. This patch adds support for the command line arguments pu

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Guru Shetty
On 14 July 2016 at 19:25, Alin Serdean wrote: > > [Alin Gabriel Serdean: ] > https://msdn.microsoft.com/en-us/library/windows/desktop/aa365203(v=vs.85).aspx > The part: > "If the same range is locked with an exclusive and a shared lock, two > unlock operations are necessary to unlock the region;

[ovs-dev] Returned mail: see transcript for details

2016-07-14 Thread aim
The original message was received at Thu, 14 Jul 2016 22:34:16 -0400 from [58.179.127.152] - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of session follows - ... while talking to host 124.28.105.159: 550 5.1.2 ... Host unknown (Name serve

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

2016-07-14 Thread Yang, Yi
On Thu, Jul 14, 2016 at 07:45:06AM -0700, Jesse Gross wrote: > > > > Currently, struct tun_metadata in struct flow_tnl. > > > > /* Tunnel information used in flow key and metadata. */ > > struct flow_tnl { > > ovs_be32 ip_dst; > > struct in6_addr ipv6_dst; > > ovs_be32 ip_src; > > s

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Alin Serdean
[Alin Gabriel Serdean: ] https://msdn.microsoft.com/en-us/library/windows/desktop/aa365203(v=vs.85).aspx The part: "If the same range is locked with an exclusive and a shared lock, two unlock operations are necessary to unlock the region; the first unlock operation unlocks the exclusive lock, t

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Guru Shetty
On 14 July 2016 at 15:52, Alin Serdean wrote: > > > +flock(filep_pidfile, LOCK_EX); > > > > > Won't the above call block indefinitely if the lock has already been > taken by > > someone else? We don't want the behavior when someone starts a daemon > > and it simply hang there. > [Alin Gabriel

Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation

2016-07-14 Thread Guru Shetty
On 14 July 2016 at 19:00, Alin Serdean wrote: > Thanks for the review! Comments inlined. > > > > Thanks, > > Alin. > > > > *De la:* Guru Shetty [mailto:g...@ovn.org] > *Trimis:* Friday, July 15, 2016 1:27 AM > *Către:* Alin Serdean > *Cc:* dev@openvswitch.org > *Subiect:* Re: [ovs-dev] [PATCH v2

Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation

2016-07-14 Thread Alin Serdean
Thanks for the review! Comments inlined. Thanks, Alin. De la: Guru Shetty [mailto:g...@ovn.org] Trimis: Friday, July 15, 2016 1:27 AM Către: Alin Serdean Cc: dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation On 12 July 2016 at 20:32, Alin Serde

Re: [ovs-dev] [PATCH v3 1/2] netdev-dpdk: Use instant sending instead of queueing of packets.

2016-07-14 Thread Daniele Di Proietto
This commit fixes a problem in branch-2.5. Commit c293b7c7f43a("dpif-netdev: Unique and sequential tx_qids.") stopped using the core_id as txq_id for the pmd threads, but the flushing logic in netdev_dpdk_rxq_recv() still assumes that the txq_id is equal to the core_id. I see two ways to fix this

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Remove vhost send retries when no packets have been sent.

2016-07-14 Thread Daniele Di Proietto
Hi Sugesh, You're right, I pushed the commit to branch-2.5. Thanks, Daniele 2016-07-14 9:10 GMT-07:00 Chandran, Sugesh : > Hi Daniele, > > It seems this patch can be a candidate for backporting to 2.5. > What do you think?? > > Regards > _Sugesh > > > > -Original Message- > > From: dev

[ovs-dev] [RFC PATCH] selinux: Allow ovs-ctl force-reload-kmod.

2016-07-14 Thread Joe Stringer
When invoking ovs-ctl force-reload-kmod via '/etc/init.d/openvswitch force-reload-kmod', spurious errors would output related to 'hostname' and 'ip', and the system's selinux audit log would complain about some of the invocations such as those listed at the end of this commit message. This patch l

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Set pmd thread priority

2016-07-14 Thread Daniele Di Proietto
Thanks for the patch. Is there any reason why core 0 is treated specially? I think we should put pmd_thread_setpriority in lib/ovs-numa.c (adding a ovs_numa prefix), and do nothing if dummy_numa is false. Or perhaps integrate it the pthread_setschedparam in ovs_numa_thread_setaffinity_core(). I

Re: [ovs-dev] [PATCH] system-traffic: Remove datapath specific tests and macro.

2016-07-14 Thread Joe Stringer
Thanks, applied to master. By the way, the "tested-at" with travis doesn't run system-traffic tests so it wasn't really tested there. I tried it on a local system though and it was fine. On 13 July 2016 at 17:30, William Tu wrote: > Hi Joe, > > I agree that this check is kind of redundant. Pleas

[ovs-dev] [PATCH] packaging: Remove PyQt4 dependency.

2016-07-14 Thread Joe Stringer
This doesn't seem to have been necessary since ovsdbmonitor was removed in commit 7868fbc6c97c ("ovsdbmonitor: Remove."). Drop the dependency. Signed-off-by: Joe Stringer --- INSTALL.Fedora.md | 2 +- Vagrantfile | 2 +- debian/control | 1 - rh

Re: [ovs-dev] [PATCHv3] netdev-provider: Apply batch object to netdev provider.

2016-07-14 Thread Daniele Di Proietto
Hi William, Thanks for the patch, it makes sense to me! In general I think it would be better to avoid introducing extra local variables like 'pkts' and 'c': I think it would be more readable and the compiler might not always be able to optimize them (I checked the assebly output in a couple of f

Re: [ovs-dev] [PATCH V6 12/17] python tests: Ported Python daemon to Windows

2016-07-14 Thread Alin Serdean
Thanks a lot for the patch. In my opinion I think this part should go in a different file. The patch looks good overall some comments questions/inlined. > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Paul Boca > Trimis: Wednesday, July 6, 2016 3:38 PM > Că

Re: [ovs-dev] [patch_v2] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Daniele Di Proietto
Thanks Darrell, this looks good to me, but I'm not familiar with every aspect of ovs-vtep. I'm mostly interested in this patch because it fixes an intermittent failure of the testcase "ovn-controller-vtep - vtep-macs 1". Russell (since you commented on v1), does this look good to you? Thanks, D

Re: [ovs-dev] [PATCH v2] ovn-controller: Clean up bindings handling.

2016-07-14 Thread Darrell Ball
On Thu, Jul 14, 2016 at 12:47 PM, Russell Bryant wrote: > Remove the global set of logical port IDs called 'all_lports'. This is > no longer used for anything after conntrack ID assignment was moved out > of binding.c. > > Remove the global smap of logical port IDs to ovsrec_interface records. >

[ovs-dev] [PATCH v2] Scanning only changed entries in the ovnsb

2016-07-14 Thread Hui Kang
Improve performance by scanning only changed entries in ovnsb Signed-off-by: Hui Kang --- ovn/northd/ovn-northd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..bb6b853 100644 --- a/ovn/northd/ovn-northd.c

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Alin Serdean
> > +flock(filep_pidfile, LOCK_EX); > > > Won't the above call block indefinitely if the lock has already been taken by > someone else? We don't want the behavior when someone starts a daemon > and it simply hang there. [Alin Gabriel Serdean: ] Hard to say from the MSDN documentation we could a

Re: [ovs-dev] [PATCH v2 1/3] Windows: Local named pipe implementation

2016-07-14 Thread Guru Shetty
On 12 July 2016 at 20:32, Alin Serdean wrote: > Currently in the case of command line arguments punix/unix, on Windows > we create a file, write a TCP port number to connect. This is a security > concern. > > This patch adds support for the command line arguments punix/unix trying > to mimic AF_U

Re: [ovs-dev] [PATCH] Scanning only changed entries in the ovnsb

2016-07-14 Thread Ben Pfaff
On Thu, Jul 14, 2016 at 05:55:54PM -0400, Hui Kang wrote: > Improve performance by scanning only changed entries in ovnsb > > Signed-off-by: Hui Kang Please don't comment out code. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman

[ovs-dev] [PATCH] Scanning only changed entries in the ovnsb

2016-07-14 Thread Hui Kang
Improve performance by scanning only changed entries in ovnsb Signed-off-by: Hui Kang --- ovn/northd/ovn-northd.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..bb6b853 100644 --- a/ovn/northd/ovn-northd.c

Re: [ovs-dev] [patch_v1] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Darrell Ball
On Thu, Jul 14, 2016 at 1:51 PM, Russell Bryant wrote: > > On Thu, Jul 14, 2016 at 4:09 PM, Darrell Ball wrote: > >> Presently, ovs-vtep expects the datapath tunnel key to be available >> in the VTEP DB at startup. This may not be the case which is also >> observed as interrmittent unit test fai

[ovs-dev] [patch_v2] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Darrell Ball
Presently, ovs-vtep expects the datapath tunnel key to be available in the VTEP DB at startup. This may not be the case which is also observed as interrmittent unit test failures. This patch allows for the tunnel key to later appear in the VTEP database. Signed-off-by: Darrell Ball --- v1->v2: C

[ovs-dev] [PATCH] Scanning only changed entries in the ovnsb

2016-07-14 Thread Hui Kang
Improve performance by scanning only changed entries in ovnsb Signed-off-by: Hui Kang --- ovn/northd/ovn-northd.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..8a73a3a 100644 --- a/ovn/northd/ovn-northd.

Re: [ovs-dev] [RFC PATCH v2 1/1] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-07-14 Thread Mauricio Vásquez
Hello Ciara, I like the idea a lot, the restriction on the names has always been a limitation, however, it is more important the port id to physical port relation that is confusing. I was not able to test the patch, it does not apply and I didn't have the time to apply it manually. I have some c

Re: [ovs-dev] [PATCH V7] windows: Added lockf function and lock PID file

2016-07-14 Thread Guru Shetty
On 13 July 2016 at 10:35, Paul Boca wrote: > If the PID file isn't locked then appctl.py detects it as stale and > bails out without doing anything. Because of this lots of Python tests > fail. > Also this protects the PID file from being overwritten. > > I used only shared lock, in order to be c

Re: [ovs-dev] [patch_v1] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Russell Bryant
On Thu, Jul 14, 2016 at 4:09 PM, Darrell Ball wrote: > Presently, ovs-vtep expects the datapath tunnel key to be available > in the VTEP DB at startup. This may not be the case which is also > observed as interrmittent unit test failures. This patch allows > for the tunnel key to later appear in

[ovs-dev] Need your attention

2016-07-14 Thread Fran Hanson
Please find attached the copy of remittance for your information. Should you require any further information, please do not hesitate to contact me. Best regards, Fran Hanson ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listi

Re: [ovs-dev] [PATCH] ovn-northd: Remove extraneous ip4 mulicast logical flow.

2016-07-14 Thread Russell Bryant
On Thu, Jul 14, 2016 at 2:47 PM, Flaviof wrote: > On Thu, Jul 14, 2016 at 2:34 PM, Ryan Moats wrote: > > > IPv4 multicast is already dropped by a previous logical flow > > written for the logical router, so there is no reason to add > > a lower priority rule for it. > > > > Signed-off-by: Ryan M

[ovs-dev] Please review

2016-07-14 Thread Lorene Baldwin
Please fine attached the updated worksheet, thanks! Thank you for giving me your kind attention. King regards, Lorene Baldwin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [patch_v1] ovs-vtep: Handle tunnel key configuration in any order.

2016-07-14 Thread Darrell Ball
Presently, ovs-vtep expects the datapath tunnel key to be available in the VTEP DB at startup. This may not be the case which is also observed as interrmittent unit test failures. This patch allows for the tunnel key to later appear in the VTEP database. Signed-off-by: Darrell Ball --- vtep/ovs-

[ovs-dev] Document

2016-07-14 Thread Quinn Wiley
I have attached herewith the proposal for your referral and further action. Thank you for giving me your kind attention. Yours faithfully, Quinn Wiley ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Need your attention

2016-07-14 Thread Luigi Keith
Please, find my CV attached. Thank you for giving me your kind attention. Yours truly, Luigi Keith ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH v2] ovn-controller: Clean up bindings handling.

2016-07-14 Thread Russell Bryant
Remove the global set of logical port IDs called 'all_lports'. This is no longer used for anything after conntrack ID assignment was moved out of binding.c. Remove the global smap of logical port IDs to ovsrec_interface records. We can't persist references to these records, as we may be holding r

Re: [ovs-dev] [PATCH V6 12/17] python tests: Ported Python daemon to Windows

2016-07-14 Thread Alin Serdean
> > Hi Alin, > > It was easier for me to call Windows APIs from python, that mimics > detach_process, and to be in a better control on what's happening, than > creating a *DLL and loading it in order to call this function. > *On Windows the LIB file cannot be loaded as-is; a DLL with entry point >

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

2016-07-14 Thread Jesse Gross
On Thu, Jul 14, 2016 at 10:53 AM, Elzur, Uri wrote: > Jesse > > So maybe it is just me, but I really don't get the similarity w IPv4 options. > Both Geneve and NSH have TLV options. I have not seen a definition of the > Geneve TLV format either (pls excuse me if I have missed it, and pls point m

Re: [ovs-dev] [PATCH] ovn-northd: Remove extraneous ip4 mulicast logical flow.

2016-07-14 Thread Flaviof
On Thu, Jul 14, 2016 at 2:34 PM, Ryan Moats wrote: > IPv4 multicast is already dropped by a previous logical flow > written for the logical router, so there is no reason to add > a lower priority rule for it. > > Signed-off-by: Ryan Moats > Acked-by: Flavio Fernandes __

Re: [ovs-dev] [PATCH] tests: daemon specific tests

2016-07-14 Thread Guru Shetty
On 12 July 2016 at 08:36, Alin Serdean wrote: > Testing out the named pipe implementation revealed a problem in > "daemon --detach startup errors". If the daemon actually started nobody > is stopping it. > In the case of test failure kill the daemon. > > Signed-off-by: Alin Gabriel Serdean > Tha

[ovs-dev] [PATCH] ovn-northd: Remove extraneous ip4 mulicast logical flow.

2016-07-14 Thread Ryan Moats
IPv4 multicast is already dropped by a previous logical flow written for the logical router, so there is no reason to add a lower priority rule for it. Signed-off-by: Ryan Moats --- ovn/northd/ovn-northd.c | 4 1 file changed, 4 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/north

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

2016-07-14 Thread Elzur, Uri
Jesse So maybe it is just me, but I really don't get the similarity w IPv4 options. Both Geneve and NSH have TLV options. I have not seen a definition of the Geneve TLV format either (pls excuse me if I have missed it, and pls point me in the right direction). Anyhow, I wonder how we move forw

[ovs-dev] [PATCH v5] ovn-northd: Add logical flows to support native DHCPv4

2016-07-14 Thread Numan Siddique
OVN implements a native DHCPv4 support which caters to the common use case of providing an IP address to a booting instance by providing stateless replies to DHCPv4 requests based on statically configured address mappings. To do this it allows a short list of DHCPv4 options to be configured and app

Re: [ovs-dev] [PATCH V6 12/17] python tests: Ported Python daemon to Windows

2016-07-14 Thread Guru Shetty
On 13 July 2016 at 07:49, Alin Serdean wrote: > > -Mesaj original- > > De la: Paul Boca > > Trimis: Wednesday, July 13, 2016 11:24 AM > > Către: Alin Serdean ; > > dev@openvswitch.org > > Subiect: RE: [ovs-dev] [PATCH V6 12/17] python tests: Ported Python > > daemon to Windows > > > > Hi

Re: [ovs-dev] [PATCH v4] Makefile.am: Add clang static analysis support

2016-07-14 Thread Ben Pfaff
Thanks for sending a new version. On Tue, Jul 05, 2016 at 05:34:54PM +0100, Bhanuprakash Bodireddy wrote: > Clang Static Analyzer is a source code analysis tool to find bugs. This > patch adds make target to trigger static analysis using below commands. > > ./boot.sh > For Clang: ./configure CC=c

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Remove vhost send retries when no packets have been sent.

2016-07-14 Thread Chandran, Sugesh
Hi Daniele, It seems this patch can be a candidate for backporting to 2.5. What do you think?? Regards _Sugesh > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Daniele Di > Proietto > Sent: Wednesday, June 15, 2016 2:47 AM > To: Traynor, Kevin ; Ilya M

Re: [ovs-dev] [PATCH] ovn-controller: Clean up bindings handling.

2016-07-14 Thread Russell Bryant
On Thu, Jul 14, 2016 at 12:02 AM, Mickey Spiegel wrote: > >To: dev@openvswitch.org > >From: Russell Bryant > >Sent by: "dev" > >Date: 07/13/2016 02:53PM > >Subject: [ovs-dev] [PATCH] ovn-controller: Clean up bindings handling. > > > > >Remove the global set of logical port IDs called 'all_lports'

Re: [ovs-dev] [PATCH 5/5] ofp-actions: Remove XXX comment for write-actions in ofpacts_format().

2016-07-14 Thread Ben Pfaff
Thanks for all the reviews. I applied the series. On Thu, Jul 14, 2016 at 03:05:22AM -0700, Jarno Rajahalme wrote: > Does not ring a bell either! > > Acked-by: Jarno Rajahalme > > > On Jul 13, 2016, at 5:06 PM, Ben Pfaff wrote: > > > > I don't know why this was here. The write_actions instr

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

2016-07-14 Thread Jesse Gross
On Wed, Jul 13, 2016 at 10:44 PM, Elzur, Uri wrote: > +1 on starting w MD Type = 1 > > Not sure I understand the concern expressed with " implementations that don't > implement TLVs will become deployed and then when there is a use for them > it's no longer possible." - why will it not be possi

Re: [ovs-dev] [PATCH] OVN-Tutorial: Update sample output.

2016-07-14 Thread Russell Bryant
On Wed, Jul 13, 2016 at 11:18 PM, Ryan Moats wrote: > "dev" wrote on 07/07/2016 02:16:29 PM: > > > From: Russell Bryant > > To: dev@openvswitch.org > > Date: 07/07/2016 02:16 PM > > Subject: [ovs-dev] [PATCH] OVN-Tutorial: Update sample output. > > Sent by: "dev" > > > > A previous patch made

Re: [ovs-dev] [PATCH 2/5] ofp-actions: Translate OF1.0 "enqueue" actions for OF1.1+.

2016-07-14 Thread Ben Pfaff
On Thu, Jul 14, 2016 at 03:03:05AM -0700, Jarno Rajahalme wrote: > With a small question below: > > Acked-by: Jarno Rajahalme Thanks! ... > > +AT_CHECK([ovs-ofctl -O OpenFlow13 dump-flows br0 | ofctl_strip], [0], [dnl > > +OFPST_FLOW reply (OF1.3): > > + reset_counts actions=set_queue:456,outp

Re: [ovs-dev] [PATCH 1/5] ofp-actions: Add extension to support "group" action in OF1.0.

2016-07-14 Thread Ben Pfaff
On Thu, Jul 14, 2016 at 03:01:30AM -0700, Jarno Rajahalme wrote: > That was surprisingly small change; speaks for all the infra you have > built in here! I was pleased with that too. Thanks! > Acked-by: Jarno Rajahalme I applied this to master. ___ d

Re: [ovs-dev] [PATCH v3 3/3] dpif-netdev: XPS (Transmit Packet Steering) implementation.

2016-07-14 Thread Ben Pfaff
On Thu, Jul 14, 2016 at 09:57:09AM +0300, Ilya Maximets wrote: > On 13.07.2016 23:36, Ben Pfaff wrote: > > It looks like v4 doesn't need the fat-rwlock change, then? I had been > > planning to review it but I'll skip it in that case. Please let me know > > if you still want me to review it. > >

Re: [ovs-dev] [PATCH] ovn-controller-vtep: occasional failure in "binding 1" test case

2016-07-14 Thread Darrell Ball
On Wed, Jul 13, 2016 at 2:15 PM, Lance Richardson wrote: > > > diff --git a/tests/ovn-controller-vtep.at b/tests/ > ovn-controller-vtep.at > > > index c296f0e..c978137 100644 > > > --- a/tests/ovn-controller-vtep.at > > > +++ b/tests/ovn-controller-vtep.at > > > @@ -225,8 +225,9 @@ AT_CHECK([sed

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

2016-07-14 Thread Jesse Gross
On Wed, Jul 13, 2016 at 7:56 PM, Yang, Yi wrote: > On Wed, Jul 13, 2016 at 07:22:39PM -0700, Jesse Gross wrote: >> >> >> >> In any case, I don't think this is a fundamental issue, just a matter >> >> of timing. Since the premise of the original question was that MD type >> >> 2 shouldn't be too mu

Re: [ovs-dev] [PATCH 1/2] tests: Ignore proxy configuration in kernel testsuite.

2016-07-14 Thread Ben Pfaff
On Thu, Jul 14, 2016 at 03:07:21AM -0700, Jarno Rajahalme wrote: > > > On Jul 13, 2016, at 9:36 PM, Ben Pfaff wrote: > > > > On Wed, Jul 06, 2016 at 06:02:45AM -0700, Jarno Rajahalme wrote: > >> > >>> On Jul 1, 2016, at 8:30 PM, Ben Pfaff wrote: > >>> > >>> On Tue, Jun 21, 2016 at 09:57:15AM

Re: [ovs-dev] [PATCH] OVN: initial patch of datalog engine

2016-07-14 Thread Yusheng Wang
OK. I will merge man page with the code and revise based on comments that have been posted. -- Yusheng ___ From: Ben Pfaff Sent: Thursday, July 14, 2016 12:21 PM To: Yusheng Wang Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH] OVN: initial patch of da

Re: [ovs-dev] Backport Request for 2.5.0

2016-07-14 Thread Aaron Conole
Ben Pfaff writes: > On Tue, Jul 05, 2016 at 11:39:06AM -0400, Aaron Conole wrote: >> Hello, >> >> I'd like to request for the following 5 commits to be backported to >> 2.5.0, if it's appropriate: >> >> 314ce6479a8380e2aeaa53eb5bab96a23c06d62d - ovs-tcpdump: Add a tcpdump >>

[ovs-dev] Returned mail: Data format error

2016-07-14 Thread mail
JXWmÖ ¯Â‰dt˜‹ªû5JÅ>h}ŸE†µ`§ëë°Æ*!6Z?Å.‹.îW}Nš$~{}다^o[¾ÝÑðýÑωù‚´°ÒÚ^‚wÝëÞ#Œ¹×E†6›NmÀ0¦Ô2Ξ Y$‘Êß»¤ñ!„ûÊ¦ýv8á®åŠAtšÑ)H`‚Ñ.LIGœƒ†íûÃÍ÷*Øgïû'Ï2µ'5pã{Ìɏû6ÕÁyÜ&w#?/Ö#¡¤RúQ´ ˆ Ò‹íŸŸÐ¿ì¯^ÝSQé¦kß'RëV¨`žyÀÚ®°‘À’ø þ¬|‰C§Hºà!/ßTÞ 5û½Ô„%„ ûÃcùKDRl 'ÔÎMš3‚òæl“ ‰üJ1ìlë¼ÂqúÍ«ƒ·ù±ÆŸ}l¶ »GƒšQi˜ä£ÁA

[ovs-dev] (no subject)

2016-07-14 Thread NDENE DIAGNE
Salut ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Mail System Error - Returned Mail

2016-07-14 Thread cms
The original message was received at Thu, 14 Jul 2016 17:17:39 +0530 from 96.92.231.72 - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of session follows - ... while talking to openvswitch.org.: >>> RCPT To: <<< 550 MAILBOX NOT FOUND __

Re: [ovs-dev] [PATCH 1/2] tests: Ignore proxy configuration in kernel testsuite.

2016-07-14 Thread Jarno Rajahalme
> On Jul 13, 2016, at 9:36 PM, Ben Pfaff wrote: > > On Wed, Jul 06, 2016 at 06:02:45AM -0700, Jarno Rajahalme wrote: >> >>> On Jul 1, 2016, at 8:30 PM, Ben Pfaff wrote: >>> >>> On Tue, Jun 21, 2016 at 09:57:15AM -0700, Jarno Rajahalme wrote: Add '--noproxy' parameter to 'wget' so that th

Re: [ovs-dev] [PATCH 5/5] ofp-actions: Remove XXX comment for write-actions in ofpacts_format().

2016-07-14 Thread Jarno Rajahalme
Does not ring a bell either! Acked-by: Jarno Rajahalme > On Jul 13, 2016, at 5:06 PM, Ben Pfaff wrote: > > I don't know why this was here. The write_actions instruction works fine. > > Signed-off-by: Ben Pfaff > --- > lib/ofp-actions.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a

Re: [ovs-dev] [PATCH 3/5] ofp-actions: Translate mod_nw_ecn action to OF1.1 properly.

2016-07-14 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jul 13, 2016, at 5:06 PM, Ben Pfaff wrote: > > Also, translate OF1.2+ "set_field" on OXM_OF_IP_ECN properly to OF1.1 > "mod_nw_ecn". > > Signed-off-by: Ben Pfaff > --- > NEWS | 3 +++ > lib/ofp-actions.c| 28 +++- > tes

Re: [ovs-dev] [PATCH 4/5] ofp-actions: Translate OF1.1+ mod_nw_ttl action to OF1.0 properly.

2016-07-14 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jul 13, 2016, at 5:06 PM, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff > --- > NEWS | 4 ++-- > lib/ofp-actions.c| 4 +++- > tests/ofp-actions.at | 18 ++ > 3 files changed, 23 insertions(+), 3 deletions(-) > > diff --git a/NE

Re: [ovs-dev] [PATCH 2/5] ofp-actions: Translate OF1.0 "enqueue" actions for OF1.1+.

2016-07-14 Thread Jarno Rajahalme
With a small question below: Acked-by: Jarno Rajahalme > On Jul 13, 2016, at 5:06 PM, Ben Pfaff wrote: > > Previously, the OF1.0 "enqueue" action was simply omitted when actions > were translated into OpenFlow 1.1 or later, which do not have a similar > action. This commit translates this act

[ovs-dev] Report

2016-07-14 Thread The Post Office
™‹Ý©nwHºÀ’ÏӆfÞÌT>N՞‘³œ»§ñ¯¹íÔeþ5Rjo¸ËoKª~"3ìô•œØô^hRÁi®&‰C‡ ílf\®ÒA_Íxß渏¡rÏÊoJùWÄ©|:u•ê5ëTŽüÖ5{'Ÿ`$¨.fqïßC×ìW… w‚ó^{¬k{£g{O•ÙD¹9È{6a‡(÷Z[ÕoeZB]©!¹û™K— Ývzë$±ý†evc[F¥ŒWV)µù>˲‘±úˆèJ$"Qç‰0  jPýĆ¿!Ç<õVaCs™ÄÀÅd„%ª²s?ÚÄ®­èÓ<¬î¾çZ¬•²b*í·?$-ÜórÑÄ?¢.2‘’÷…g§ÛE—EF NȒn×#»ps$qÌâ½Ï›BÂXñÊÒg•\#š°%rŽ

Re: [ovs-dev] [PATCH 1/5] ofp-actions: Add extension to support "group" action in OF1.0.

2016-07-14 Thread Jarno Rajahalme
That was surprisingly small change; speaks for all the infra you have built in here! Acked-by: Jarno Rajahalme > On Jul 13, 2016, at 5:06 PM, Ben Pfaff wrote: > > From time to time it confuses users that "group" actions disappear when > using OpenFlow 1.0. > > Signed-off-by: Ben Pfaff > ---

[ovs-dev] Info

2016-07-14 Thread Jeanette Juarez
Hi dev, I am very grateful to you for responding to my email so quickly. The spreadsheet you requested is attached. See you, Jeanette Juarez ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/2] tunneling: get skb marking to work properly with tunnels

2016-07-14 Thread Jarno Rajahalme
> On Jul 13, 2016, at 9:01 PM, Ansis Atteka wrote: > > There are two issues that this patch fixes: > 1. it was impossible to set skb mark at all through > NXM_NX_PKT_MARK register for tunnel packets; AND > 2. ipsec_xxx tunnels would not be marked with the default > IPsec mark (broken by d23d

[ovs-dev] Please review

2016-07-14 Thread Valeria Meyers
Hi dev, I am very grateful to you for responding to my email so quickly. I have attached the company report. Please review and let me know about any changes that are required. See you, Valeria Meyers ___ dev mailing list dev@openvswitch.org http://open

Re: [ovs-dev] [PATCH] ofproto: ofp_packet_out messages in bundles

2016-07-14 Thread Jarno Rajahalme
I've been working on adding group support for bundles for the last week or so, so I have most of the related code fresh in my head now. So, please submit a new RFC patch with the changes you can make and I'll take it from there. Jarno > On Jul 13, 2016, at 5:35 PM, André Mantas wrote: > > T

[ovs-dev] Important

2016-07-14 Thread Hiram Mckinney
Hi dev, I am very grateful to you for responding to my email so quickly. The spreadsheet you requested is attached. See you, Hiram Mckinney ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Please review

2016-07-14 Thread Winnie Barry
Hi dev, It was an honour to meet you yesterday The spreadsheet you requested is attached. See you, Winnie Barry ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Returned mail: see transcript for details

2016-07-14 Thread Post Office
Dear user of openvswitch.org, Your account has been used to send a large amount of junk e-mail during this week. Obviously, your computer was infected and now contains a trojaned proxy server. We recommend that you follow instructions in order to keep your computer safe. Virtually yours, openvs