Re: [ovs-dev] [PATCH] datapath-windows: Append flow attribute key

2015-09-14 Thread Sairam Venugopal
Thanks for clarifying that. Acked-by: Sairam Venugopal On 9/14/15, 1:53 PM, "Alin Serdean" wrote: >The idea is there are two type of errors: >- those returned by the system: i.e. the result code that you pointed >below which gets interpreted by >(https://urldefense.proofpoint.com/v2/url?u=h

Re: [ovs-dev] [PATCH] datapath-windows: Append flow attribute key

2015-09-14 Thread Alin Serdean
The idea is there are two type of errors: - those returned by the system: i.e. the result code that you pointed below which gets interpreted by (https://github.com/openvswitch/ovs/blob/master/lib/netlink-socket.c#L567). Which means that was a system error. - netlink errors which will determine

Re: [ovs-dev] [PATCH] datapath-windows: Append flow attribute key

2015-09-14 Thread Sairam Venugopal
Thread-Topic: [ovs-dev] [PATCH] datapath-windows: Append flow attribute key Thread-Index: AQHQ7xepL84xz91rMEqut1rnlan04A== Date: Mon, 14 Sep 2015 18:03:29 + Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport

Re: [ovs-dev] [PATCH] datapath-windows: Append flow attribute key

2015-09-11 Thread Sorin Vinturis
I had a minor comment inline. Looks good otherwise. Acked-by: Sorin Vinturis -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Alin Serdean Sent: Friday, 11 September, 2015 02:15 To: dev@openvswitch.org Subject: [ovs-dev] [PATCH] datapath-windows: Append

[ovs-dev] [PATCH] datapath-windows: Append flow attribute key

2015-09-10 Thread Alin Serdean
Currently when running the vswitch daemon we get a lot of messages of the form: 2015-09-10T23:04:21Z|07255|dpif(revalidator11)|WARN|system@ovs-system: failed to flow_del (Invalid argument). The userspace expects after sending a delete flow command, to receive the flow key of the deleted flow. Cur