[ovs-dev] Mail System Error - Returned Mail

2015-08-18 Thread Mail Delivery Subsystem
The message could not be delivered ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] Query regarding OF1.5 Egress tables (EXT-306)

2015-08-18 Thread niti Rohilla
Hi Jean, Thanks for your reply. Based on your reply please validate our understanding. 1. From all the commands only OFPTFC_MODIFY needs to be implemented and only for a single table in a request. This command is used to modify flow table capabilities and will not modify pipeline processing or an

[ovs-dev] Can OVS forward TTP messages to Helium ODL

2015-08-18 Thread tech_kals Kals
Hi Experts, I am working on the openflow switch (hardware switch) which have its own Openflow agent functionalities -- It uses OFDPA for data plane programming. But, it uses OVS (ofproto lib in OVS) to interact with the controller. All communication between our switch and the controller happe

[ovs-dev] [PATCH v3] dpif-netdev: proper tx queue id

2015-08-18 Thread Ilya Maximets
Currently tx_qid is equal to pmd->core_id. This leads to unexpected behavior if pmd-cpu-mask different from '/(0*)(1|3|7)?(f*)/', e.g. if core_ids are not sequential, or doesn't start from 0, or both. Example: starting 2 pmd threads with 1 port, 2 rxqs per port, pmd-cpu-mask =

Re: [ovs-dev] [PATCH v2] dpif-netdev: proper tx queue id

2015-08-18 Thread Ilya Maximets
Thanks for review. I sent new version of patch. http://openvswitch.org/pipermail/dev/2015-August/058926.html Best regards, Ilya Maximets. On 17.08.2015 19:45, Daniele Di Proietto wrote: > Sorry about the long wait. > > I think this is the right direction, I like the idea of having a thread > lo

Re: [ovs-dev] [PATCH 1/3] ovn-controller-vtep: Fix dereference of NULL pointer.

2015-08-18 Thread Russell Bryant
On 08/17/2015 11:29 PM, Alex Wang wrote: > Found by inspection. > > Signed-off-by: Alex Wang Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/3] ovn-controller-vtep: Call binding_cleanup() before gateway_cleanup().

2015-08-18 Thread Russell Bryant
On 08/17/2015 11:29 PM, Alex Wang wrote: > Otherwise, binding_cleanup() will be a no-op since all related chassis > entries are deleted in gateway_cleanup(). > > Found by inspection. > > Signed-off-by: Alex Wang > --- > ovn/controller-vtep/ovn-controller-vtep.c |4 ++-- > 1 file changed, 2

Re: [ovs-dev] [PATCH 3/3] ovn-controller-vtep: Make 'ovnsb_remote' and 'vtep_remote' free-able.

2015-08-18 Thread Russell Bryant
On 08/17/2015 11:29 PM, Alex Wang wrote: > Found by inspection. > > Signed-off-by: Alex Wang > --- > ovn/controller-vtep/ovn-controller-vtep.c |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/ovn/controller-vtep/ovn-controller-vtep.c > b/ovn/controller-vtep/o

[ovs-dev] [PATCH] ovn: Free default db befor exit.

2015-08-18 Thread Russell Bryant
The static result of default_db() was malloc'd but not freed before exit. Make the static result global and free it before exit. Signed-off-by: Russell Bryant --- ovn/northd/ovn-northd.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/ovn/northd/ovn-northd.c b/o

Re: [ovs-dev] Query regarding OF1.5 Egress tables (EXT-306)

2015-08-18 Thread Jean Tourrilhes
On Tue, Aug 18, 2015 at 05:40:34PM +0530, niti Rohilla wrote: > Hi Jean, > > Thanks for your reply. Based on your reply please validate our > understanding. > > 1. From all the commands only OFPTFC_MODIFY needs to be implemented and > only for a single table in a request. This command is used to

Re: [ovs-dev] [PATCH 2/3] ovn-controller-vtep: Call binding_cleanup() before gateway_cleanup().

2015-08-18 Thread Alex Wang
On Tue, Aug 18, 2015 at 8:10 AM, Russell Bryant wrote: > On 08/17/2015 11:29 PM, Alex Wang wrote: > > Otherwise, binding_cleanup() will be a no-op since all related chassis > > entries are deleted in gateway_cleanup(). > > > > Found by inspection. > > > > Signed-off-by: Alex Wang > > --- > > ov

[ovs-dev] [PATCH 3/3 V2] ovn-controller-vtep: Make 'ovnsb_remote' and 'vtep_remote' free-able.

2015-08-18 Thread Alex Wang
Found by inspection. Signed-off-by: Alex Wang --- PATCH->V2: - free 'default_db_'. --- ovn/controller-vtep/ovn-controller-vtep.c | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/ovn/controller-vtep/ovn-controller-vtep.c b/ovn/controller-vtep/ovn-contro

Re: [ovs-dev] [PATCH 3/3 V2] ovn-controller-vtep: Make 'ovnsb_remote' and 'vtep_remote' free-able.

2015-08-18 Thread Russell Bryant
On 08/18/2015 11:02 AM, Alex Wang wrote: > Found by inspection. > > Signed-off-by: Alex Wang Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn: Free default db befor exit.

2015-08-18 Thread Alex Wang
Acked-by: Alex Wang On Tue, Aug 18, 2015 at 8:24 AM, Russell Bryant wrote: > The static result of default_db() was malloc'd but not freed before > exit. Make the static result global and free it before exit. > > Signed-off-by: Russell Bryant > --- > ovn/northd/ovn-northd.c | 12

Re: [ovs-dev] [PATCH] ovn: Free default db befor exit.

2015-08-18 Thread Alex Wang
Thx, applied to master, On Tue, Aug 18, 2015 at 11:06 AM, Alex Wang wrote: > Acked-by: Alex Wang > > On Tue, Aug 18, 2015 at 8:24 AM, Russell Bryant > wrote: > >> The static result of default_db() was malloc'd but not freed before >> exit. Make the static result global and free it before exit

Re: [ovs-dev] [PATCH 3/3 V2] ovn-controller-vtep: Make 'ovnsb_remote' and 'vtep_remote' free-able.

2015-08-18 Thread Alex Wang
Thanks, applied to master,~ On Tue, Aug 18, 2015 at 11:05 AM, Russell Bryant wrote: > On 08/18/2015 11:02 AM, Alex Wang wrote: > > Found by inspection. > > > > Signed-off-by: Alex Wang > > Acked-by: Russell Bryant > > -- > Russell Bryant > ___ dev ma

[ovs-dev] [PATCH] hmap: Ensure iterator is NULL after iteration.

2015-08-18 Thread Russell Bryant
The HMAP_FOR_EACH_()* macros had a usability issue where the iterator was only NULL at the completion of iteration if the hmap_node was the first struct member. This change ensures that the iterator is set to NULL when iteration ends normally without a 'break'. Signed-off-by: Russell Bryant ---

Re: [ovs-dev] [PATCH] tunneling: Remove gre64 tunnel support.

2015-08-18 Thread Pravin Shelar
On Mon, Jul 27, 2015 at 1:33 PM, Jesse Gross wrote: > On Tue, Jul 21, 2015 at 3:35 PM, Pravin B Shelar wrote: >> diff --git a/datapath-windows/ovsext/Debug.h >> b/datapath-windows/ovsext/Debug.h >> index 4b7b526..e4e55ac 100644 >> --- a/datapath-windows/ovsext/Debug.h >> +++ b/datapath-windows/o

[ovs-dev] [PATCH v2] tunneling: Remove gre64 tunnel support.

2015-08-18 Thread Pravin B Shelar
GRE64 was introduced to extend gre key from 32-bit to 64-bit using gre-key and sequence number field. But GRE64 is not standard protocol. There are not many users of this protocol. Therefore we have decided to remove it. Signed-off-by: Pravin B Shelar --- - Removed bit hole from OVS_DB_ flags. -

[ovs-dev] [PATCH 1/2] odp-util: Fix put_nd_key().

2015-08-18 Thread Jarno Rajahalme
Actually copy the 'nd_target' from the key. Found by inspection. Signed-off-by: Jarno Rajahalme --- lib/odp-util.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/odp-util.c b/lib/odp-util.c index f142f03..b50d87b 100644 --- a/lib/odp-util.c +++ b/lib/odp-util.c @@ -4711

[ovs-dev] [PATCH 2/2] meta-flow: Fix ip_frag handling in mf_set_wild().

2015-08-18 Thread Jarno Rajahalme
The wildcard bits were set when they should have been cleared. Found by inspection. Signed-off-by: Jarno Rajahalme --- lib/meta-flow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/meta-flow.c b/lib/meta-flow.c index 4c7cf2c..528c109 100644 --- a/lib/meta-flow.c +++ b/

[ovs-dev] [PATCH] ovn-controller-vtep.at: Check the successful exit of ovn-controller-vtep.

2015-08-18 Thread Alex Wang
Signed-off-by: Alex Wang --- tests/ovn-controller-vtep.at |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at index de97bf9..fe2fd4d 100644 --- a/tests/ovn-controller-vtep.at +++ b/tests/ovn-controller-vtep.at @@ -8

[ovs-dev] [ovn-controller-vtep V7 1/3] ovn-controller-vtep: Add vtep module.

2015-08-18 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 --- V6->V7: - change the assertion to VLOG_ERR in vtep_lswitch_run(). - refine the vtep_lswi

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

2015-08-18 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 --- V6->V7: - rebase. - adopt suggestions from Russell. V5->V6: - rebase. V4->V5: - rebase on top of master. -

[ovs-dev] [ovn-controller-vtep V7 3/3] ovn-controller-vtep: Update related documentation.

2015-08-18 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

[ovs-dev] Mail System Error - Returned Mail

2015-08-18 Thread The Post Office
Îí÷hƒs?”CG¡3S7^¢¦§ãIÙe/Ž¬NÆ{­^‹%É'ª{Òn,ÅíË3¦g'“ …ŒäNNXþ>‘Ìý~AæNɟ•·Ä(30Aµ®«©ÙW¬Hçü•^¥ŽØãÝ1ºGm¼Óìʒ§ zs¸B˜RÙýS‚Þ- Äð¨0,"H”`#ÒÔ0“¦"n4’̱ç´Pï[Nt«Kgo¡EÚ¾Bì™W§¡I£BbÓ>Œma¨ób¥\¿ý… ØNäû½å†ÊDԟïûÛ8ˁò±öâ [WÕµç/Ìyg$„}¨mXOÐ** 2—µÁŽdNTO yª^ï°ÇýؼéŽGÃg ÌßQk¢êþV•ÓX5Z\O`6föÆÏ㱏vڄ|…FpýÈ®"ylƒ¤)£³›{WRÄ[A¶ba

[ovs-dev] [PATCH v5 2/2] ofproto: Implement OF1.4 Group & Meter change notification messages

2015-08-18 Thread niti1489
From: Niti Rohilla This patch adds support for Openflow1.4 Group & meter change notification messages. In a multi controller environment, when a controller modifies the state of group and meter table, the request that successfully modifies this state is forwarded to other controllers. Other contr