Re: [ovs-dev] dpif-netdev:fix reload pmd's dead lock

2019-02-11 Thread Lilijun (Jerry, Cloud Networking)
Ok, I got it, thanks for your explanation. > -Original Message- > From: Ilya Maximets [mailto:i.maxim...@samsung.com] > Sent: Monday, February 11, 2019 9:55 PM > To: Lilijun (Jerry, Cloud Networking) ; Stokes, Ian > ; d...@openvswitch.org > Subject: Re: [ovs-dev] dpif

Re: [ovs-dev] dpif-netdev:fix reload pmd's dead lock

2019-02-10 Thread Lilijun (Jerry, Cloud Networking)
Yes, Thanks for your help again. Let's forget the patch :). But in my opinion, the function reload_affected_pmds() doesn't need be locked within dp->port_mutex. Is that true? > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] O

[ovs-dev] Question about using struct ofproto‘s port_by_name without lock protected

2019-01-31 Thread Lilijun (Jerry, Cloud Networking)
Hi all, Currently, when insert/delete/lookup the shash list struct ofproto‘s port_by_name, we have no lock to protect this list. This list was used lots of other functions. Is there something race issues? Can we make sure it's only used in the main ovs thread? Thanks. __

Re: [ovs-dev] dpif-netdev:fix reload pmd's dead lock

2019-01-31 Thread Lilijun (Jerry, Cloud Networking)
Maximets [mailto:i.maxim...@samsung.com] > Sent: Thursday, January 31, 2019 5:12 PM > To: Lilijun (Jerry, Cloud Networking) ; > d...@openvswitch.org > Cc: Stokes, Ian > Subject: Re: [ovs-dev] dpif-netdev:fix reload pmd's dead lock > > Hi. > > On 31.01.2019 8:57, Lilijun

[ovs-dev] Questions about OVS with P4

2019-01-17 Thread Lilijun (Jerry, Cloud Networking)
Hi all, I found one branch of OVS in github.com which have not been updated for 4 years. Are there any plan or work to support P4 in OVS? Thanks ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time

2018-12-12 Thread Lilijun (Jerry, Cloud Networking)
other threads have changed the global_seqno. > -Original Message- > From: Ben Pfaff [mailto:b...@ovn.org] > Sent: Monday, December 10, 2018 11:49 PM > To: Lilijun (Jerry, Cloud Networking) > Cc: d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU

Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time

2018-12-09 Thread Lilijun (Jerry, Cloud Networking)
8 AM > To: Lilijun (Jerry, Cloud Networking) > Cc: d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time > > If that's the case, then there's something wrong with seq_wait() or the use > of it in this context. The timer logic here is just to

Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time

2018-12-09 Thread Lilijun (Jerry, Cloud Networking)
Original Message- > From: Ben Pfaff [mailto:b...@ovn.org] > Sent: Friday, December 07, 2018 10:37 PM > To: Lilijun (Jerry, Cloud Networking) > Cc: d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time > > On Fri, Dec 07, 2018 at 02:03:27PM +08

Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time

2018-12-06 Thread Lilijun (Jerry, Cloud Networking)
...@ovn.org] > Sent: Thursday, December 06, 2018 10:58 PM > To: Lilijun (Jerry, Cloud Networking) > Cc: d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] [PATCH] rcu: reduce RCU block time > > On Thu, Dec 06, 2018 at 08:28:19PM +0800, Lilijun wrote: > > When calling ov

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-10 Thread Lilijun (Jerry, Cloud Networking)
VLAN. -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sent: Tuesday, September 11, 2018 3:33 AM To: Eric Garver ; Lilijun (Jerry, Cloud Networking) ; d...@openvswitch.org; ovs-disc...@openvswitch.org Subject: Re: [ovs-discuss] [PATCH v2] [ovs-dev] [PATCH] QinQ: support more vlan head

Re: [ovs-dev] [ovs-discuss] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-09 Thread Lilijun (Jerry, Cloud Networking)
Hi Eric, Yes, I agree with that effect. But how about this issue of QinQ that we can only support at most 2 VLANs ? Do you have any ideas? Thanks. -Original Message- From: Eric Garver [mailto:e...@garver.life] Sent: Friday, September 07, 2018 10:14 PM To: Lilijun (Jerry, Cloud

Re: [ovs-dev] [PATCH] vswitch.xml: Better explain vlan-limit.

2018-09-09 Thread Lilijun (Jerry, Cloud Networking)
Yes, That's OK, thanks. -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sent: Saturday, September 08, 2018 1:59 AM To: Eric Garver ; d...@openvswitch.org; Lilijun (Jerry, Cloud Networking) Subject: Re: [PATCH] vswitch.xml: Better explain vlan-limit. On Fri, Sep 07, 20

Re: [ovs-dev] [PATCH] vswitch.xml: Better explain vlan-limit.

2018-09-07 Thread Lilijun (Jerry, Cloud Networking)
That’s more clear to me. thanks. 发件人: Ben Pfaff 收件人: Eric Garvermailto:e...@garver.life>>;devmailto:d...@openvswitch.org>>;Lilijun (Jerry, Cloud Networking)mailto:jerry.lili...@huawei.com>> 主题: Re: [PATCH] vswitch.xml: Better explain vlan-limit. 时间: 2018-09-08 01:59:34 On Fr

Re: [ovs-dev] [ovs-discuss] [PATCH] [PATCH] QinQ: support more vlan headers.

2018-09-07 Thread Lilijun (Jerry, Cloud Networking)
, September 07, 2018 12:13 PM To: Lilijun (Jerry, Cloud Networking) Cc: ovs-disc...@openvswitch.org; d...@openvswitch.org; Eric Garver Subject: Re: [ovs-discuss] [PATCH] [ovs-dev] [PATCH] QinQ: support more vlan headers. On Fri, Sep 07, 2018 at 02:51:25AM +, Lilijun (Jerry, Cloud Networking

[ovs-dev] [PATCH v2] [PATCH] QinQ: support more vlan headers.

2018-09-07 Thread Lilijun (Jerry, Cloud Networking)
In my test, vlan-limit is set to 0 that means unlimited the count of vlan headers. ovs-vsctl set Open_vSwitch . other_config:vlan-limit=0 But in fact the macro FLOW_MAX_VLAN_HEADERS is defined as 2, so we can only support max two vlan headers. It doesn't work as the config vlan-limit's description

Re: [ovs-dev] [ovs-discuss] [PATCH] [PATCH] QinQ: support more vlan headers.

2018-09-06 Thread Lilijun (Jerry, Cloud Networking)
Sorry, I will push it again to fix this error. -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sent: Friday, September 07, 2018 12:20 PM To: Lilijun (Jerry, Cloud Networking) Cc: ovs-disc...@openvswitch.org; d...@openvswitch.org; Eric Garver Subject: Re: [ovs-discuss] [PATCH

[ovs-dev] [PATCH] [PATCH] QinQ: support more vlan headers.

2018-09-06 Thread Lilijun (Jerry, Cloud Networking)
Hi all, In my test, vlan-limit is set to 0 that means unlimited the count of vlan headers. ovs-vsctl set Open_vSwitch . other_config:vlan-limit=0 But in fact the macro FLOW_MAX_VLAN_HEADERS is defined as 2, so we can only support max two vlan headers. It doesn't work as the config vlan-limit's de

Re: [ovs-dev] [ovs-discuss] fix the mod_vlan_vid actions with OpenFlow13.

2018-08-22 Thread Lilijun (Jerry, Cloud Networking)
OK, Thanks for your reply. Maybe we need adapt to the change currently if no better something happen. -Original Message- From: Eric Garver [mailto:e...@erig.me] Sent: Thursday, August 23, 2018 3:00 AM To: Lilijun (Jerry, Cloud Networking) Cc: wangyunjian ; d...@openvswitch.org

Re: [ovs-dev] [ovs-discuss] fix the mod_vlan_vid actions with OpenFlow13.

2018-08-21 Thread Lilijun (Jerry, Cloud Networking)
Hi Eric and all, Thanks for Eric's advice very much. But in my opinion, this issue reported from wangyunjian is something about feature's compatibility. Using the rule: ovs-ofctl -O OpenFlow13 add-flow ovsbr0 " table=0,priority=2,in_port=1 actions=mod_vlan_vid:3,NORMAL", the action mod_vlan_

[ovs-dev] 答复: [PATCH net-next 0/6] Add dpdk-bond support

2018-06-14 Thread Lilijun (Jerry, Cloud Networking)
I think this dpdkbond option is worthy of being supported. In my view, ovs bonding works based on openflow upcall processing. When some slave gets link down, the datapath flows related it will be flushed an revalidated. As a result, that may cause a great deal of upcall requests to be handled a

[ovs-dev] 答复: 答复: Question: why ovs flush all datapath flows when destroying br/ofproto?

2018-05-24 Thread Lilijun (Jerry, Cloud Networking)
apath flows. Is that ok? Thanks. -邮件原件- 发件人: Ben Pfaff [mailto:b...@ovn.org] 发送时间: 2018年5月25日 1:13 收件人: Lilijun (Jerry, Cloud Networking) 抄送: d...@openvswitch.org; ovs-disc...@openvswitch.org; liucheng (J) ; lixiao (H) 主题: Re: 答复: Question: why ovs flush all datapath flows

[ovs-dev] 答复: Question: why ovs flush all datapath flows when destroying br/ofproto?

2018-05-23 Thread Lilijun (Jerry, Cloud Networking)
Are there anybody has some ideas about this question? Thanks. -邮件原件- 发件人: Lilijun (Jerry, Cloud Networking) 发送时间: 2018年5月18日 11:02 收件人: d...@openvswitch.org; 'Ben Pfaff' ; ovs-disc...@openvswitch.org 抄送: liucheng (J) ; lixiao (H) 主题: Question: why ovs flush all datapath

[ovs-dev] Question: why ovs flush all datapath flows when destroying br/ofproto?

2018-05-17 Thread Lilijun (Jerry, Cloud Networking)
Hi all, In my test, there are many VMs sending and receiving random flows using ovs. Per VM has one virtio-net NIC and connect to ovs bridge named such as br-plyXXX. These ovs bridges br-plyXXX are all connected to br-int using patch port. All VM's traffic can generate about 200 thousand datapa

[ovs-dev] 答复: Question about ovs megaflow with move NXM_OF_IP_DST actions

2018-04-04 Thread Lilijun (Jerry, Cloud Networking)
What's the megaflow match policy about move:NXM_OF_IP_DST? Why we don't using megaflow wildcard on the openflow actions of move:NXM_OF_IP_DST[]? Thank you! B.R. Lilijun -邮件原件- 发件人: Lilijun (Jerry, Cloud Networking) 发送时间: 2018年4月4日 11:56 收件人: d...@openvswitch.org; '

[ovs-dev] Question about ovs megaflow with move NXM_OF_IP_DST actions

2018-04-03 Thread Lilijun (Jerry, Cloud Networking)
Hi all, In my test, add one flow to br-ovs, and running one vm connected this br-ovs: cookie=0x0, duration=286.272s, table=31, n_packets=2, n_bytes=196, idle_age=161, priority=61010,ip,metadata=0x2b193,dl_dst=00:00:00:04:06:03 actions=move:NXM_OF_IP_DST[]->NXM_NX_REG0[],load:0x8439ce602f->OXM_OF

[ovs-dev] 答复: [PATCH 2/3] xlate: fix packets loopback caused by duplicate read of xcfgp.

2018-01-24 Thread Lilijun (Jerry)
I have tested your patch and can fix my problem described in the following patch. https://mail.openvswitch.org/pipermail/ovs-dev/2018-January/343423.html Thanks. -邮件原件- 发件人: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] 代表 Ben Pfaff 发送时间: 2018年1月25日 1:41 收件人:

[ovs-dev] 答复: [PATCH] Ofproto: fix the bug wrong datapath flow with same in_port and output port was generated

2018-01-23 Thread Lilijun (Jerry)
Yes, I also found this four related functions just again. I will do some test with your patch, thanks. -邮件原件- 发件人: Huanle Han [mailto:hanxue...@gmail.com] 发送时间: 2018年1月23日 12:33 收件人: Ben Pfaff ; Lilijun (Jerry) 抄送: ; Liuyongan ; Zhoujingbin 主题: Re: [ovs-dev] [PATCH] Ofproto: fix

[ovs-dev] [PATCH] Ofproto: fix the bug wrong datapath flow with same in_port and output port was generated

2018-01-19 Thread Lilijun (Jerry)
Hi all, In my test, the new datapath flow which has the same in_port and actions output port was found using ovs-appctl dpctl/dump-flows. Then the mac address will move from one port to another and back it again in the physical switch. This problem result in the VM's traffic become abnormal.