[ovs-dev] ov q

2015-10-27 Thread Returned mail
Dear user of openvswitch.org, Mail system administrator of openvswitch.org would like to inform you Your account was used to send a large amount of spam messages during the last week. Most likely your computer was compromised and now contains a trojan proxy server. Please follow instructions i

[ovs-dev] [PATCH v4] ovsdb-idl: Add support for change tracking

2015-10-27 Thread Ansari, Shad
Ovsdb-idl notifies a client that something changed; it does not track which table, row changed in what way (insert, modify or delete). As a result, a client has to scan or reconfigure the entire idl after ovsdb_idl_run(). This is presumably fine for typical ovs schemas where tables are relative

Re: [ovs-dev] [PATCH 1/1] ovsdb-idl: Add additional support for change tracking.

2015-10-27 Thread Ansari, Shad
> I see a few style errors, e.g. the { should be on a line of its own > here: > static bool > ovsdb_idl_track_is_set(struct ovsdb_idl_table *table) { > > ovsdb_idl_track_is_set() seems expensive given that it is being executed > on every row destroy. > > I'm not really pleased with the id

[ovs-dev] [PATCH v3] ovsdb-idl: Add support for change tracking

2015-10-27 Thread Ansari, Shad
Ovsdb-idl notifies a client that something changed; it does not track which table, row changed in what way (insert, modify or delete). As a result, a client has to scan or reconfigure the entire idl after ovsdb_idl_run(). This is presumably fine for typical ovs schemas where tables are relatively s

[ovs-dev] [PATCH v2] ovsdb-idl: Add support for change tracking

2015-10-27 Thread Ansari, Shad
Ovsdb-idl notifies a client that something changed; it does not track which table, row changed in what way (insert, modify or delete). As a result, a client has to scan or reconfigure the entire idl after ovsdb_idl_run(). This is presumably fine for typical ovs schemas where tables are relatively

Re: [ovs-dev] [PATCH 1/4] ovn : Setup a controller for br-int

2015-10-27 Thread Babu Shanmugam
Hi Ben, I understand that when an openflow controller is set on the switch, the switch will try to connect to the controller target and keeps alive the connection by a series of echo_requests and echo_replies. So, I thought, we need a passive connection that listens on the target that is confi

Re: [ovs-dev] [PATCH] datapath-windows: STT - Offload inner checksum calculation

2015-10-27 Thread Nithin Raju
Thanks for doing this! Acked-by: Nithin Raju -Original Message- From: Sairam Venugopal Date: Tuesday, October 27, 2015 at 2:36 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH] datapath-windows: STT - Offload inner checksumcalculation >Offload the inner checksum comput

[ovs-dev] [PATCH] datapath-windows: STT - Offload inner checksum calculation

2015-10-27 Thread Sairam Venugopal
Offload the inner checksum computation to NDIS in OvsDecapStt function. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Stt.c | 133 ++ 1 file changed, 58 insertions(+), 75 deletions(-) diff --git a/datapath-windows/ovsext/Stt.c b/datapath-win

Re: [ovs-dev] [PATCH v2 3/3] datapath-windows: STT - Enable support for TCP Segmentation offloads

2015-10-27 Thread Gurucharan Shetty
> Acked-by: Nithin Raju Series applied. > > -Original Message- > From: Sairam Venugopal > Date: Tuesday, October 27, 2015 at 10:20 AM > To: Nithin Raju > Subject: Fw: [PATCH v2 3/3] datapath-windows: STT - Enable support for TCP > Segmentation offloads > >> >>__

Re: [ovs-dev] [PATCH] Revert "datapath-windows: Support attribute OVS_KEY_ATTR_TCP_FLAGS"

2015-10-27 Thread Gurucharan Shetty
On Tue, Oct 27, 2015 at 1:07 PM, Sairam Venugopal wrote: > Acked-by: Sairam Venugopal Applied, thanks. > > > On 10/27/15, 12:50 PM, "Alin Serdean" > wrote: > >>This reverts commit a26b2023ce33fed1ef962012dc2c03765d2e92cb. >> >>This patch punishes performance without the implementation of >>mega

Re: [ovs-dev] [PATCH] Revert "datapath-windows: Support attribute OVS_KEY_ATTR_TCP_FLAGS"

2015-10-27 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 10/27/15, 12:50 PM, "Alin Serdean" wrote: >This reverts commit a26b2023ce33fed1ef962012dc2c03765d2e92cb. > >This patch punishes performance without the implementation of >megaflows on Windows. > >Once megaflows is implemented in the flow logic this patch will be >

[ovs-dev] [PATCH] Revert "datapath-windows: Support attribute OVS_KEY_ATTR_TCP_FLAGS"

2015-10-27 Thread Alin Serdean
This reverts commit a26b2023ce33fed1ef962012dc2c03765d2e92cb. This patch punishes performance without the implementation of megaflows on Windows. Once megaflows is implemented in the flow logic this patch will be revisited. Signed-off-by: Alin Gabriel Serdean --- datapath-windows/ovsext/DpInte

Re: [ovs-dev] [PATCH v2 3/3] datapath-windows: STT - Enable support for TCP Segmentation offloads

2015-10-27 Thread Nithin Raju
Thanks for working on this. It is really awesome to have TSO support with STT now! Acked-by: Nithin Raju -Original Message- From: Sairam Venugopal Date: Tuesday, October 27, 2015 at 10:20 AM To: Nithin Raju Subject: Fw: [PATCH v2 3/3] datapath-windows: STT - Enable support for TCP Segm

Re: [ovs-dev] [PATCH net-next V18 3/3] 802.1AD: Flow handling, actions, vlan parsing and netlink attributes

2015-10-27 Thread Pravin Shelar
On Tue, Oct 27, 2015 at 9:45 AM, Thomas F Herbert wrote: > On 10/26/15 10:10 PM, Pravin Shelar wrote: > Thanks for the review. >> >> On Sun, Oct 25, 2015 at 5:11 PM, Thomas F Herbert >> wrote: >>> >>> Add support for 802.1ad including the ability to push and pop double >>> tagged vlans. Add suppo

Re: [ovs-dev] [PATCH 2/3] datapath-windows: STT - Add support for TCP Segmentation Offload

2015-10-27 Thread Nithin Raju
Acked-by: Nithin Raju -Original Message- From: Sairam Venugopal Date: Saturday, October 24, 2015 at 1:40 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH 2/3] datapath-windows: STT - Add support for TCP Segmentation Offload >Create and initialize the background thread and buf

Re: [ovs-dev] [PATCH v2 1/3] datapath-windows: Move OvsAllocateNBLFromBuffer to BufferMgmt

2015-10-27 Thread Nithin Raju
Acked-by: Nithin Raju -Original Message- From: Sairam Venugopal Date: Monday, October 26, 2015 at 4:48 PM To: "dev@openvswitch.org" Subject: [ovs-dev] [PATCH v2 1/3] datapath-windows: MoveOvsAllocateNBLFromBuffer to BufferMgmt >Move the functionality around creating an NBL from Buf

Re: [ovs-dev] [PATCH net-next V18 3/3] 802.1AD: Flow handling, actions, vlan parsing and netlink attributes

2015-10-27 Thread Thomas F Herbert
On 10/26/15 10:10 PM, Pravin Shelar wrote: Thanks for the review. On Sun, Oct 25, 2015 at 5:11 PM, Thomas F Herbert wrote: Add support for 802.1ad including the ability to push and pop double tagged vlans. Add support for 802.1ad to netlink parsing and flow conversion. Uses double nested encap

Re: [ovs-dev] [PATCH net-next V17 2/3] Check for vlan ethernet types for 8021.q or 802.1ad

2015-10-27 Thread Thomas F Herbert
On 10/26/15 8:14 AM, Albino B Neto wrote: 2015-10-25 22:11 GMT-02:00 Thomas F Herbert : Signed-off-by: Thomas F Herbert --- include/linux/if_vlan.h | 16 1 file changed, 16 insertions(+) diff --git a/include/linux/if_vlan.h b/include/linux/if_vlan.h index 67ce5bd..d2494b5 1

[ovs-dev] (no subject)

2015-10-27 Thread MAILER-DAEMON
The original message was received at Tue, 27 Oct 2015 15:17:56 +0530 from openvswitch.org [164.192.2.237] - The following addresses had permanent fatal errors - - Transcript of session follows - ... while talking to 167.83.48.197: >>> RCPT To: <<< 550 5.1.1 ... Not known here __

[ovs-dev] [PATCH 3/3] ovn-northd: Fix table ID in a comment.

2015-10-27 Thread Russell Bryant
This changed from 2 to 3 when ACLs got implemented, as it turned out ACLs needed two tables (1 and 2). While we're at it, do a bit of OCD formatting cleanup by fixing the alignment of '\' at the end of each line in the logical flow table defininitions. Signed-off-by: Russell Bryant --- ovn/nort

[ovs-dev] [PATCH 1/3] ovn: Fix check on existing encap row.

2015-10-27 Thread Russell Bryant
This code does some checking to validate the existing encaps for a chassis to see if they need to be updated. This typo resulted in ovn-controller re-creating its encap(s) every time this code ran, making ovn-controller and ovsdb-server eat up a CPU in my testing. Signed-off-by: Russell Bryant -

[ovs-dev] [PATCH 2/3] ovn: Remove duplicate versions from schemas.

2015-10-27 Thread Russell Bryant
Since commit 5935835968c9d36ffe306863f0c8079d3b670e2a, the OVN nb and sb schema definitions have included duplicate version entries. In the nb case, the version has since been updated to 2.0.0, but only in one place. Remove the duplicate version entries that were at the bottom of the files. Sign