[ovs-dev] [PATCH v2] ovn-nbctl: Fix the ovn-nbctl test "LBs - daemon" which fails during rpm build

2018-10-23 Thread nusiddiq
From: Numan Siddique The test "ovn-nbctl: LBs - daemon" fails when it runs the command "ovn-nbctl lb-add lb0 30.0.0.1a 192.168.10.10:80,192.168.10.20:80". ovn-nbctl extracts the vip by calling the socket util function 'inet_parse_active()', and this function blocks when it calls dns_resolve().

Re: [ovs-dev] [PATCH] ovn-nbctl: Fix the ovn-nbctl test "LBs - daemon" which fails during rpm build

2018-10-23 Thread Numan Siddique
On Mon, Oct 22, 2018 at 11:21 PM Mark Michelson wrote: > On 10/22/2018 11:49 AM, nusid...@redhat.com wrote: > > From: Numan Siddique > > > > The test "ovn-nbctl: LBs - daemon" fails when it runs the command > > "ovn-nbctl lb-add lb0 30.0.0.1a 192.168.10.10:80,192.168.10.20:80". > ovn-nbctl > >

Re: [ovs-dev] [PATCH v1 0/3] Policy-based routing

2018-10-23 Thread Numan Siddique
On Tue, Oct 23, 2018 at 3:54 AM Mary Manohar wrote: > This patch series implements policy-based routing. > Policy-based routing (PBR) provides a mechanism to configure permit/deny > and reroute policies on the router. > Permit/deny policies are similar to OVN ACLs, but exist on the >

Re: [ovs-dev] OVN based distributed virtual routing for VLAN backed networks

2018-10-23 Thread Miguel Angel Ajo Pelayo
Hi, Nice document, you are capturing the nuisances of handling VLAN very well, I agree that we need a chassis specific mac for the distributed routers, when handling E/W traffic, otherwise you end up with the situation described in [1] making the router port MAC flip. That is also an

[ovs-dev] [PATCH] OVN: introduce mac_prefix support to IPAM

2018-10-23 Thread Lorenzo Bianconi
Add the possibility to specify a given mac address prefix for dynamically generated mac address. Mac address prefix can be specified in nbdb NB_Global table, options:mac_prefix= This patch fix a possible issue of L2 address duplication if multiple OVN deployments share a single broadcast domain

Re: [ovs-dev] Issues configuring OVS-DPDK in openstack queens

2018-10-23 Thread O Mahony, Billy
Hi Manojawa, So is there any remaining entry br-flat entry in the ovsdb? Does it give any clue to the reason – there may be a free-form ‘status’ or ‘info’ field for that purpose. I can understand the situation where a bridge might get incorrectly configured but I can’t understand why it is

Re: [ovs-dev] [PATCH] OVN: introduce mac_prefix support to IPAM

2018-10-23 Thread Aaron Conole
Hi Lorenzo, Lorenzo Bianconi writes: > Add the possibility to specify a given mac address prefix for > dynamically generated mac address. Mac address prefix can be > specified in nbdb NB_Global table, options:mac_prefix= > This patch fix a possible issue of L2 address duplication if > multiple

Re: [ovs-dev] [PATCH] OVN: introduce mac_prefix support to IPAM

2018-10-23 Thread Aaron Conole
Aaron Conole writes: > Hi Lorenzo, > > Lorenzo Bianconi writes: > >> Add the possibility to specify a given mac address prefix for >> dynamically generated mac address. Mac address prefix can be >> specified in nbdb NB_Global table, options:mac_prefix= >> This patch fix a possible issue of L2

Re: [ovs-dev] Issues configuring OVS-DPDK in openstack queens

2018-10-23 Thread O Mahony, Billy
Hi, I don't see any errors relating to the dpdk interfaces. But it is also not clear where the user-space drivers are bound and the hugepage memory is set up. So double check those two items. Is the dpdk-p0 interface being attached to br-flat? Even if there are issues with the dpdk port the

Re: [ovs-dev] Routing policies, add routing-policies in ovn-nbctl

2018-10-23 Thread 0-day Robot
Bleep bloop. Greetings Mary Manohar, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Mary Manohar needs to sign off. WARNING: Line lacks whitespace around operator

Re: [ovs-dev] Routing policies, add config in schema

2018-10-23 Thread 0-day Robot
Bleep bloop. Greetings Mary Manohar, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Mary Manohar needs to sign off. WARNING: Line is 82 characters long

Re: [ovs-dev] Issues configuring OVS-DPDK in openstack queens

2018-10-23 Thread Manojawa Paritala
Hi Billy, Thank you for your reply. 1. Huge pages are properly set. Based on the dpdk configuration dpdk-socket-mem="4096,4096", 8 pages were created under /dev/hugepages. 2. dpdk-p0 is not attached to br-flat1. Actually I defined the bridge as br-flat1. 3. Yes, 'ovs-vsctl show' does not show

[ovs-dev] [branch 2.9][PATCH] ovn: Fix IPv6 DAD failure for container ports

2018-10-23 Thread nusiddiq
From: Numan Siddique When a container port is created inside a VM, the below kernel message is seen and IPv6 doesn't work on that interface. [ 138.000753] IPv6: vlan4: IPv6 duplicate address detected! When a container port sends a ethernet broadcast packet, OVN delivers the same packet back

Re: [ovs-dev] [PATCH v11 05/14] dp-packet: Fix data_len handling multi-seg mbufs.

2018-10-23 Thread Lam, Tiago
On 22/10/2018 11:31, Stokes, Ian wrote: >> When a dp_packet is from a DPDK source, and it contains multi-segment >> mbufs, the data_len is not equal to the packet size, pkt_len. Instead, the >> data_len of each mbuf in the chain should be considered while distributing >> the new (provided) size.

[ovs-dev] US $15.5 million Cash Payment/Home Delivery. From Nat West Bank London United Kingdom.

2018-10-23 Thread ahmed idris via dev
Natwest Bank Plc London I35 Bishops gate Plaza London, United Kingdom (Offshore Office) Attention: It was resolved and agreed that your inheritance/contract payment sum of US $15.5 million should be release and send to you on a special method of payment tagged Cash Payment/Home Delivery in

[ovs-dev] US $15.5 million Cash Payment/Home Delivery. From Nat West Bank London United Kingdom.

2018-10-23 Thread ahmed idris via dev
Natwest Bank Plc London I35 Bishops gate Plaza London, United Kingdom (Offshore Office) Attention: It was resolved and agreed that your inheritance/contract payment sum of US $15.5 million should be release and send to you on a special method of payment tagged Cash Payment/Home Delivery in

[ovs-dev] d...@openvswitch.org LPO/Payment

2018-10-23 Thread Javaid Rizvi Groups
Hi Customer d...@openvswitch.org ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] ofp-port: Free memory on error in ofp_print_ofpst_port_reply

2018-10-23 Thread Ben Pfaff
On Thu, Oct 18, 2018 at 02:10:47PM -0700, Yifeng Sun wrote: > Counters in ops->custom_stats may already be valid at this error point. > This patch frees the leaked memory. > > Reported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=10322 > Signed-off-by: Yifeng Sun Thanks, applied

[ovs-dev] d...@openvswitch.org LPO/Payment

2018-10-23 Thread Javaid Rizvi Groups
Hi Customer d...@openvswitch.org ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] odp-util: Initialize nsh_hdr in odp_nsh_hdr_from_attr

2018-10-23 Thread Ben Pfaff
On Thu, Oct 18, 2018 at 02:10:49PM -0700, Yifeng Sun wrote: > For function odp_nsh_hdr_from_attr, paramemter `attr` may not contain > all necessary OVS_NSH_KEY_ATTR_ to fully initialize data of nsh_hdr. > On return, nsh_hdr can contain uninitialized values in its fields. > > This patch zeroes out

[ovs-dev] [PATCH] ovs-ofctl: Update documentation for MPLS actions.

2018-10-23 Thread Ben Pfaff
Reported-by: lidejun Signed-off-by: Ben Pfaff --- utilities/ovs-ofctl.8.in | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in index aefea16ee34d..659406de57b4 100644 --- a/utilities/ovs-ofctl.8.in +++

Re: [ovs-dev] Issues configuring OVS-DPDK in openstack queens

2018-10-23 Thread Manojawa Paritala
Hi Billy, There are no br-flat1 entries in ovsdb. As suggested I increased the log level to debug and then tried the same scenario again. Though the result was same (br-flat1 getting deleted), I observed the below 2 issues (i assume). Issue-1 :- 1. Everything is up and running. That is all the

Re: [ovs-dev] [PATCH v1] Docs: Remove zero-copy QEMU limitation.

2018-10-23 Thread Loftus, Ciara
> > Remove note regarding zero-copy compatibility with QEMU >= 2.7. > > When zero-copy was introduced to OVS it was incompatible with QEMU >= > 2.7. This issue has since been fixed in DPDK with commit > 803aeecef123 ("vhost: fix dequeue zero copy with virtio1") and > backported to DPDK LTS

Re: [ovs-dev] Sync ipv6 route info from kernel

2018-10-23 Thread Ben Pfaff
On Sat, Oct 20, 2018 at 06:10:52AM +, lidejun wrote: > Has anybody found this issue: when using userspace ovs + dpdk with ipv6 vxlan > tunnel, if too many ipv6 route changes in a short time, ovs main thread will > stuck at recvmsg? > Call chain is: > route_table_run ---> nln_run --->

Re: [ovs-dev] [PATCH v2] ovn-nbctl: Fix the ovn-nbctl test "LBs - daemon" which fails during rpm build

2018-10-23 Thread Ben Pfaff
On Tue, Oct 23, 2018 at 11:48:58AM +0530, nusid...@redhat.com wrote: > From: Numan Siddique > > The test "ovn-nbctl: LBs - daemon" fails when it runs the command > "ovn-nbctl lb-add lb0 30.0.0.1a 192.168.10.10:80,192.168.10.20:80". ovn-nbctl > extracts the vip by calling the socket util function

Re: [ovs-dev] [PATCH v2] ovn-nbctl: Fix the ovn-nbctl test "LBs - daemon" which fails during rpm build

2018-10-23 Thread Ben Pfaff
On Tue, Oct 23, 2018 at 09:49:15AM -0700, Ben Pfaff wrote: > On Tue, Oct 23, 2018 at 11:48:58AM +0530, nusid...@redhat.com wrote: > > From: Numan Siddique > > > > The test "ovn-nbctl: LBs - daemon" fails when it runs the command > > "ovn-nbctl lb-add lb0 30.0.0.1a

Re: [ovs-dev] [PATCH v2] ovn-nbctl: Fix the ovn-nbctl test "LBs - daemon" which fails during rpm build

2018-10-23 Thread Numan Siddique
On Tue, Oct 23, 2018 at 10:20 PM Ben Pfaff wrote: > On Tue, Oct 23, 2018 at 09:49:15AM -0700, Ben Pfaff wrote: > > On Tue, Oct 23, 2018 at 11:48:58AM +0530, nusid...@redhat.com wrote: > > > From: Numan Siddique > > > > > > The test "ovn-nbctl: LBs - daemon" fails when it runs the command > > >

[ovs-dev] [PATCH] ofp-msgs: Added NXT_REQUESTFORWARD for OF1.0-1.3

2018-10-23 Thread Zak Whittington
Backported OFPT14_REQUESTFORWARD to OF1.0-1.3 as a Nicira extension. Will only forward GROUP_MOD messages in OF 1.0 through 1.2, since METER_MOD was only added in 1.3. VMware-BZ: 2136594 Signed-off-by: Zak Whittington --- include/openvswitch/ofp-monitor.h | 1 + include/openvswitch/ofp-msgs.h

Re: [ovs-dev] [PATCH v2] ovn-nbctl: Fix the ovn-nbctl test "LBs - daemon" which fails during rpm build

2018-10-23 Thread Ben Pfaff
On Tue, Oct 23, 2018 at 11:24:39PM +0530, Numan Siddique wrote: > On Tue, Oct 23, 2018 at 10:20 PM Ben Pfaff wrote: > > > On Tue, Oct 23, 2018 at 09:49:15AM -0700, Ben Pfaff wrote: > > > On Tue, Oct 23, 2018 at 11:48:58AM +0530, nusid...@redhat.com wrote: > > > > From: Numan Siddique > > > > >

Re: [ovs-dev] [PATCH] ofp-msgs: Added NXT_REQUESTFORWARD for OF1.0-1.3

2018-10-23 Thread Ben Pfaff
On Tue, Oct 23, 2018 at 11:23:32AM -0700, Zak Whittington wrote: > Backported OFPT14_REQUESTFORWARD to OF1.0-1.3 as a Nicira > extension. Will only forward GROUP_MOD messages in OF > 1.0 through 1.2, since METER_MOD was only added in 1.3. > > VMware-BZ: 2136594 > Signed-off-by: Zak Whittington

Re: [ovs-dev] [PATCH v2] odp-util: Move ufid handling to odp_flow_from_string

2018-10-23 Thread Ben Pfaff
On Thu, Oct 18, 2018 at 02:10:48PM -0700, Yifeng Sun wrote: > When parse_odp_key_mask_attr runs into ufid, it returns length of ufid > without further parsing. This causes problem to set action in function > parse_odp_action where nested nlattrs are expected from ofpbuf after. > > Since in all

Re: [ovs-dev] [PATCH v2] connmgr: Fix vswitchd abort when a port is added and the controller is down

2018-10-23 Thread Ben Pfaff
On Thu, Oct 18, 2018 at 04:47:05PM +0530, nusid...@redhat.com wrote: > From: Numan Siddique > > We see the below trace when a port is added to a bridge and the configured > controller is down > > 0x7fb002f8b207 in raise () from /lib64/libc.so.6 > 0x7fb002f8c8f8 in abort () from

Re: [ovs-dev] [PATCH v1 0/3] Policy-based routing

2018-10-23 Thread Ben Pfaff
On Mon, Oct 22, 2018 at 10:24:03PM +, Mary Manohar wrote: > This patch series implements policy-based routing. > Policy-based routing (PBR) provides a mechanism to configure permit/deny and > reroute policies on the router. > Permit/deny policies are similar to OVN ACLs, but exist on the

Re: [ovs-dev] [PATCH] ossfuzz: Add ofctl parse target

2018-10-23 Thread Ben Pfaff
On Mon, Oct 15, 2018 at 11:23:33AM +0200, bshas...@sect.tu-berlin.de wrote: > From: Bhargava Shastry > > This patch adds a new target called ofctl_parse_target to > ossfuzz. The main idea is to begin to fuzz APIs from the ofctl utility > program. At a later point, these may be added. For the

Re: [ovs-dev] [PATCH 3/8] ofp-table: Ignore bits that have to change according to OpenFlow.

2018-10-23 Thread Justin Pettit
> On Aug 30, 2018, at 1:00 PM, Ben Pfaff wrote: > > OpenFlow table feature replies contain a per-table bitmap that indicates > which tables a flow can point to in goto_table actions. OpenFlow requires > that a table only be able to go to higher-numbered tables. This means that > a switch

Re: [ovs-dev] [PATCH] ovs-ofctl: Update documentation for MPLS actions.

2018-10-23 Thread Justin Pettit
> On Oct 23, 2018, at 9:26 AM, Ben Pfaff wrote: > > Reported-by: lidejun > Signed-off-by: Ben Pfaff Acked-by: Justin Pettit --Justin ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 1/8] vconn: Avoid null dereference on error path.

2018-10-23 Thread Justin Pettit
> On Aug 30, 2018, at 1:00 PM, Ben Pfaff wrote: > > Sometimes the 'errors' list is passed as null, and in that case it should > not be used. > > Found by inspection. > > Signed-off-by: Ben Pfaff Acked-by: Justin Pettit --Justin ___ dev

Re: [ovs-dev] [PATCH 2/8] ofp-table: Better summarize table features and statistics.

2018-10-23 Thread Justin Pettit
> On Aug 30, 2018, at 1:00 PM, Ben Pfaff wrote: > > Before this patch, most dump-table-stats outputs would contain about > 250 lines of the form: > > table #: ditto > > With this patch, they have one line like this: > > tables 2...254: ditto > > which is much easier to read. > >

Re: [ovs-dev] [PATCH 4/8] ofp-table: Always format the table number in table features.

2018-10-23 Thread Justin Pettit
> On Aug 30, 2018, at 1:00 PM, Ben Pfaff wrote: > > Table features should indicate the table number as well as the table > name. Before this, the first line for each table looked like this: > table myname ("myname"): > but it's more useful if it's: > table 123 ("myname"): > >

[ovs-dev] [PATCH] ofp-msgs: Added NXT_REQUESTFORWARD for OF1.0-1.3

2018-10-23 Thread Zak Whittington
Backported OFPT14_REQUESTFORWARD to OF1.0-1.3 as a Nicira extension. VMware-BZ: 2136594 Signed-off-by: Zak Whittington --- include/openvswitch/ofp-monitor.h | 1 + include/openvswitch/ofp-msgs.h| 5 +- lib/ofp-monitor.c | 11 +++-- ofproto/connmgr.c | 6