Re: [ovs-dev] [PATCH v5 1/4] ovn: ovn-ctl support for HA ovn DB servers

On Friday 11 November 2016 02:18 PM, Andy Zhou wrote: On Mon, Nov 7, 2016 at 11:55 PM, Babu Shanmugam > wrote: On Monday 07 November 2016 06:49 PM, Andy Zhou wrote: This version is better, I am able to apply them. Thanks. I got the system running,

[ovs-dev] Sample action for patch port

Hi Ben, As we know, NXAST_RAW_SAMPLE2 was introduced to OVS to make flow-based IPFIX to support tunnel information exporting. A field called “sampling_port” is added in NXAST_RAW_SAMPLE2 and it’s an ofp-port. This field is not only used for exporting tunnel information, but also used for judgin

Re: [ovs-dev] [PATCH ovs V1 0/9] Introducing HW offload support for openvswitch

Hi all, I have one doubt on hw-offload engine. May i know how synchronizing happening between kernel and hardware flow table? Is there any separate kernel thread for maintaining flow table synchronization? *With Warm Regards,* *Naveen M* Mobile : +91 8883356187 | Skype : nvn.m7 "THINK GOOD, D

Re: [ovs-dev] [PATCH] netdev-dpdk: Fix crash in QoS.

2016-11-02 10:59 GMT-07:00 Stokes, Ian : > > qos_conf can be NULL. This can be easily reproduced by setting egress > QoS > > on a port: > > > > ``` > > ovs-vsctl set port dpdk2 qos=@newqos -- --id=@newqos create qos > > type=egress-policer other-config:cir=4600 other-config:cbs=2048 ``` > > >

Re: [ovs-dev] [PATCH v3 1/2] logical-fields: Add ct.dnat and ct.snat symbols.

On Sun, Nov 13, 2016 at 11:15:26PM -0800, Gurucharan Shetty wrote: > Upcoming commits use it. > > Signed-off-by: Gurucharan Shetty > --- > v2->v3: no change Assuming the tests pass: Acked-by: Ben Pfaff ___ dev mailing list d...@openvswitch.org https:

[ovs-dev] Compras - Estrategias de Negociación

Obtenga las habilidades para hacer las mejores compras Estrategias de Negociación de Compras: Mejore Resultados Financieros y Utilidades 02 de diciembre - Zona Reforma (Ciudad de México) - 9:00 a 18:00 Imparte: MCE. Abdón Guzmán S. El área de compras en una empresa es, sin duda, uno

Re: [ovs-dev] [RFC PATCH] datapath: allow tunnels to be created with rtnetlink

On Thu, Oct 27, 2016 at 5:08 AM, Thadeu Lima de Souza Cascardo wrote: > On Wed, Oct 26, 2016 at 02:05:22PM -0700, Pravin Shelar wrote: >> On Wed, Oct 26, 2016 at 2:55 AM, Thadeu Lima de Souza Cascardo >> wrote: >> > On Tue, Oct 25, 2016 at 08:21:55PM -0700, Pravin Shelar wrote: >> >> > The fallba

Re: [ovs-dev] [PATCH v3 00/12] Improve performance of OVS-DPDK classifier.

2016-11-14 4:10 GMT-08:00 Bodireddy, Bhanuprakash < bhanuprakash.bodire...@intel.com>: > Hello daniele, > > Did you get a chance to review v4 of the remaining 4 patches in this > series? Also I have sent v5 of patch "dpcls: Use 32 packet batches for > lookups" separately based on your comments.

Re: [ovs-dev] [PATCH v2] ofproto: Return the OFPC_BUNDLES bit in switch features reply.

> On Nov 12, 2016, at 9:55 AM, Ben Pfaff wrote: > > On Fri, Nov 11, 2016 at 02:47:35PM -0800, Jarno Rajahalme wrote: >> Add definitions for the OpenFlow 1.4.1/1.5 specific capabilities bits >> OFPC14_BUNDLES and OFPC14_FLOW_MONITORING. Return the bundles >> capability bit in switch features rep

Re: [ovs-dev] [PATCH] ovn-sbctl: document logging and common options in man page

On Mon, Nov 14, 2016 at 3:57 PM, Russell Bryant wrote: > > On Mon, Nov 14, 2016 at 1:44 PM, Lance Richardson > wrote: > >> The ovn-sbctl is currently missing a description of logging and >> common (-h/--help/-V/--version) command-line options. Add them >> by including corresponding man page frag

Re: [ovs-dev] [PATCH 3/3] dpif-netdev: Drop unused function.

On 11 November 2016 at 05:02, Simon Horman wrote: > On Wed, Nov 09, 2016 at 05:00:22PM -0800, Joe Stringer wrote: >> Signed-off-by: Joe Stringer > > Acked-by: Simon Horman Thanks, applied. ___ dev mailing list d...@openvswitch.org https://mail.openvsw

Re: [ovs-dev] [PATCH 2/3] dpif-provider: Update port_add documentation.

On 11 November 2016 at 05:11, Simon Horman wrote: > On Wed, Nov 09, 2016 at 05:00:21PM -0800, Joe Stringer wrote: >> The implementations these days use ODPP_NONE rather than UINT32_MAX. >> >> Signed-off-by: Joe Stringer > > Acked-by: Simon Horman Thanks, applied. ___

Re: [ovs-dev] [PATCH 1/3] lib: Use nl_attr_get_odp_port().

On 11 November 2016 at 05:01, Simon Horman wrote: > Hi Joe, > > On Wed, Nov 09, 2016 at 05:00:20PM -0800, Joe Stringer wrote: >> This helper is a little tidier than the alternative. Use it treewide. >> >> Signed-off-by: Joe Stringer >> --- >> lib/dpif-netdev.c| 2 +- >> lib/odp-util.

[ovs-dev] [PATCHv2] lib: Use nl_attr_get_odp_port().

This helper is a little tidier than the alternative. Use it treewide. Signed-off-by: Joe Stringer --- lib/dpif-netdev.c| 4 ++-- lib/odp-util.c | 2 +- ofproto/ofproto-dpif-sflow.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/dpif-netdev.c

Re: [ovs-dev] [PATCH] dpif-provider: Use ODPP_NONE in dp_netdev_flow_add()

On 11 November 2016 at 05:31, Simon Horman wrote: > From: Simon Horman > > This appears to be the only place where ODPP_NONE is not used but could be. > > Signed-off-by: Simon Horman Acked-by: Joe Stringer ___ dev mailing list d...@openvswitch.org ht

Re: [ovs-dev] [PATCH] ovn-sbctl: document logging and common options in man page

On Mon, Nov 14, 2016 at 1:44 PM, Lance Richardson wrote: > The ovn-sbctl is currently missing a description of logging and > common (-h/--help/-V/--version) command-line options. Add them > by including corresponding man page fragments. > > Signed-off-by: Lance Richardson > --- > manpages.mk

Re: [ovs-dev] [PATCH ovs V1 7/9] dpif-hw-acc: operate implementation

On 1 November 2016 at 07:53, Paul Blakey wrote: > added operate implemenation using tc for flow offload, > supporting flow get, flow put, and flow del. > > Signed-off-by: Paul Blakey > Signed-off-by: Shahar Klein > --- > lib/dpif-hw-acc.c | 821 > +++

Re: [ovs-dev] [PATCH ovs V1 3/9] dpif-hw-acc: Add new hash maps ufid <-> tc flow and ovs port <-> netdev

On 1 November 2016 at 07:53, Paul Blakey wrote: > A flow in ovs can be identified by its ufid, and in tc by its > flow handle + priority. > We use the flow protocol number as the priority when offloading to tc. > So we save a map that maps from ovs ufid <-> tc flow handle + protocol. > Also added

Re: [ovs-dev] [PATCH v3 1/2] logical-fields: Add ct.dnat and ct.snat symbols.

On Mon, Nov 14, 2016 at 2:15 AM, Gurucharan Shetty wrote: > Upcoming commits use it. > > Signed-off-by: Gurucharan Shetty > --- > v2->v3: no change > --- > ovn/lib/logical-fields.c | 2 ++ > ovn/ovn-sb.xml | 19 +-- > tests/ovn.at | 4 +++- > 3 files chan

Re: [ovs-dev] [PATCH ovs V1 2/9] dpif-hw-acc: Add tc interface

There are various style variations that are unusual in this whole series (compared against the standard OVS style), I've pointed out a few of them in this patch along with some specific feedback on this patch. It's not comprehensive though, there is too much for me to comment on. On 1 November 201

Re: [ovs-dev] [PATCH ovs V1 1/9] dpif-hw-acc: New dpif provider

On 1 November 2016 at 07:53, Paul Blakey wrote: > Added infrastructure for a new provider that will be able > to send some flows to supporting HW for offloading. > > Signed-off-by: Paul Blakey > Signed-off-by: Shahar Klein The amount of boilerplate makes me wonder if this could be structured a

[ovs-dev] [PATCH] ovn-sbctl: document logging and common options in man page

The ovn-sbctl is currently missing a description of logging and common (-h/--help/-V/--version) command-line options. Add them by including corresponding man page fragments. Signed-off-by: Lance Richardson --- manpages.mk | 4 ovn/utilities/ovn-sbctl.8.in | 3 +++ 2 files c

Re: [ovs-dev] [PATCH] netdev-dpdk: Use instant sending instead of queueing of packets.

Hi Cascardo, I agree, the tx queueing mechanism in 2.5 never really worked because it made wrong assumptions about the rxq distribution and introduced latency (at least), as you point out. Thanks for doing the backport, since it was non trivial. Usually when we do a backport we use the commit me

[ovs-dev] [PATCH v3 2/2] ovn-northd: Force SNAT for multiple gateway routers.

When multiple gateway routers exist, a packet can enter any gateway router. Once the packet reaches its destination, its reverse direction should be via the same gateway router. This is achieved by doing a SNAT of the packet in the inward direction (towards logical space) with a IP address of the

[ovs-dev] [PATCH v3 1/2] logical-fields: Add ct.dnat and ct.snat symbols.

Upcoming commits use it. Signed-off-by: Gurucharan Shetty --- v2->v3: no change --- ovn/lib/logical-fields.c | 2 ++ ovn/ovn-sb.xml | 19 +-- tests/ovn.at | 4 +++- 3 files changed, 22 insertions(+), 3 deletions(-) diff --git a/ovn/lib/logical-fields.c b/

[ovs-dev] [PATCH v3 2/3] ovn-sbctl: commands for managing connection configuration

Add ovn-sbctl commands to manage connection target configuration, including read-only connection attributes. Since there is now a valid reason to use this utility in production environments, Remove statement that this should not be used in production from man page and run-time help text. Added co

[ovs-dev] [PATCH v3 3/3] ovn-sb: add support for SSL configuration in database

Add SSL configuration to the southbound database schema and add commands to ovn-sbctl to allow management of these entries. Added commands: Print SSL configuration: ovn-sbctl get-ssl Delete SSL configuration: ovn-sbctl del-ssl Set SSL configuration: ovn-sbctl [--bootst

[ovs-dev] [PATCH v3 0/3] ovn-sb: support for managing connections in db

(Resending in hopes patchwork will pick up this series.) - Add SSL configuration information to OVN_Southbound schema. - Add commands to ovn-sbctl for managing remote connection configuration. - Add commands to ovn-sbctl for managing SSL configuration. - Add a test case to exercise the above. Fol

[ovs-dev] [PATCH v3 1/3] ovn-sb: add SSL configuration to southbound db schema

Augment OVN southbound database schema to allow SSL connection configuration information to be stored and managed within the southbound database. Signed-off-by: Lance Richardson --- ovn/ovn-sb.ovsschema | 21 ++--- ovn/ovn-sb.xml | 48 +++

Re: [ovs-dev] what is OVS NSH patchset merge status?

Hi Yi Yang, It is great that you establish contact. We have just started a new initiative to revive the work on upstreaming support for flexible (aka L3) tunneling and NSH support in OVS. Please have a look at the mail below. https://mail.openvswitch.org/pipermail/ovs-dev/2016-November/324788.ht

Re: [ovs-dev] [PATCH 2/5] lib/dp-packet: copy additional packet info when do packet copy

> > > >2016/10/28 17:47, Kavanagh, Mark B : >>> Currently, when doing packet copy, lots of DPDK mbuf's info >>> will be missed, like packet type, ol_flags, etc. >>> Those information is very important for DPDK to do >>> packets processing. >>> >>> Signed-off-by: Michael Qiu >>> Signed-off-by: Jiji

Re: [ovs-dev] [CudaMailTagged] [PATCH 1/4] ovn: Support DNAT with port.

On 14 November 2016 at 01:59, nickcooper-zhangtonghao wrote: > Hi Guru Shetty > > Because of public ip restrictions , I have different private IPs sharing a > single outbound internet IP. > The VMs with different private IPs will run different services(e.g. web, > email, dns service). > > I hope

Re: [ovs-dev] [PATCH 1/1] : Add Clear statistics support at interface level

writes: > Hi All, > > Below are the configuration and patch details which provides the support > for clearing statistics at Interface level. > Description & Configuration: > OVS by default has the support for the command ovs-vsctl clear interface > statistics > But in OVS the command functional

Re: [ovs-dev] [PATCH v2 1/4] ovs-tcpdump: Remove the newline from readline().

nickcooper-zhangtonghao writes: > When using ovs-tcpdump, we always get the white line. > Remove newline from readline() and ovs-tcpdump will > format the output as tcpdump tool. > > Signed-off-by: nickcooper-zhangtonghao > --- I acked this with V1, so: Acked-by: Aaron Conole

Re: [ovs-dev] [PATCH v2 4/4] ovs-tcpdump: Fix error handling when parsing parameter.

Nick, nickcooper-zhangtonghao writes: > Signed-off-by: nickcooper-zhangtonghao > --- > utilities/ovs-tcpdump.in | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/utilities/ovs-tcpdump.in b/utilities/ovs-tcpdump.in > index 96aa1e2..d984a39 100755 > --- a/utilities/ovs-

Re: [ovs-dev] [PATCH v2 3/4] ovs-tcpdump: destroy_mirror should really destroy the mirror.

nickcooper-zhangtonghao writes: > Signed-off-by: nickcooper-zhangtonghao > --- Acked-by: Aaron Conole ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH v2 2/4] ovs-tcpdump: should't halt when dumping traffic.

nickcooper-zhangtonghao writes: > when sys.stdin is passed to tcpdump via pipe, > it will cause tcpdump to halt. > > Signed-off-by: nickcooper-zhangtonghao > --- On further investigation, there are other issues around supporting this properly, so: Acked-by: Aaron Conole __

Re: [ovs-dev] [PATCH 4/4] ovs-tcpdump: destroy_mirror should really destroy the mirror.

nickcooper-zhangtonghao writes: > Signed-off-by: nickcooper-zhangtonghao > --- Acked-by: Aaron Conole ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 3/4] ovs-tcpdump: Fix error handling when parsing parameter.

nickcooper-zhangtonghao writes: > Hi Aaron > You ignore an error case. Run the tcpdump with invalid option or > argument. I agree, that error case is being skipped currently. > In this case, process will raise directly KeyboardInterrupt exception, not > run the while loop any more. We should r

Re: [ovs-dev] [PATCH 1/4] ovs-tcpdump: Remove the newline from readline().

nickcooper-zhangtonghao writes: > When using ovs-tcpdump, we always get the white line. > Remove newline from readline() and ovs-tcpdump will > format the output as tcpdump tool. > > Signed-off-by: nickcooper-zhangtonghao > --- Acked-by: Aaron Conole ___

Re: [ovs-dev] [PATCH 2/4] ovs-tcpdump: should't halt when dumping traffic.

nickcooper-zhangtonghao writes: > when sys.stdin is passed to tcpdump via pipe, > it will cause tcpdump to halt. > > Signed-off-by: nickcooper-zhangtonghao > --- Strangely, I remember using this and having it work; it seems that it requires more effort though (ex: the .read() call needs to cha

[ovs-dev] [PATCH] FAQ: Fix current LTS version

Signed-off-by: Darragh O'Reilly --- FAQ.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FAQ.rst b/FAQ.rst index 0e53301..a933c78 100644 --- a/FAQ.rst +++ b/FAQ.rst @@ -143,7 +143,7 @@ Q: What does it mean for an Open vSwitch release to be LTS (long-term support)? If

Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: Fix duplicate multicast packets

Thanks. > From: Simon Horman [mailto:simon.hor...@netronome.com] > Sent: 14 November 2016 12:44 > Thanks, I have applied this to the master branch. > > Please let me know which if any of branch-2.* you feel it is appropriate for. Thanks. The customer is on branch-2.5. _

Re: [ovs-dev] [PATCH v2] ofproto-dpif-xlate: Fix duplicate multicast packets

On Fri, Nov 11, 2016 at 01:57:00PM +, O'Reilly, Darragh wrote: > When iterating the list of mrouters, skip any that are not on the same > vlan as the multicast packet to be forwarded. This bug was causing > duplicate packets when more than one mrouter was behind a trunk port. > > Reported-at:

Re: [ovs-dev] [PATCH 1/4] ovn: Support DNAT with port.

> > I understand that we can extend the current DNAT feature to include > DNAT:port. But is there is a use case where you want to use this? Any > extensions of the NAT table can be better designed if we understand the > end use-case for it. If not, I will just take a look at the first version of th

Re: [ovs-dev] [PATCH v3 00/12] Improve performance of OVS-DPDK classifier.

Hello daniele, Did you get a chance to review v4 of the remaining 4 patches in this series? Also I have sent v5 of patch "dpcls: Use 32 packet batches for lookups" separately based on your comments. Bhanu Prakash. >-Original Message- >From: dev [mailto:dev-boun...@openvswitch.org]

Re: [ovs-dev] All Faculteit \ Staff

From: Ulla Brandt Lasson Sent: Monday, November 14, 2016 2:19 AM To: Ulla Brandt Lasson Subject: All Faculteit \ Staff All Faculteit \ Staff Kennis te nemen van deze belangrijke update die onze nieuwe web-mail is verbeterd met een nieuwe messaging systeem van Owa / vooruitzichten die ook snel

Re: [ovs-dev] [CudaMailTagged] [PATCH 1/4] ovn: Support DNAT with port.

Hi Guru Shetty Because of public ip restrictions , I have different private IPs sharing a single outbound internet IP. The VMs with different private IPs will run different services(e.g. web, email, dns service). I hope OVN will support that feature. The patches I submitted may be not good eno

[ovs-dev] [PATCH v2 3/4] ovs-tcpdump: destroy_mirror should really destroy the mirror.

Signed-off-by: nickcooper-zhangtonghao --- utilities/ovs-tcpdump.in | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/utilities/ovs-tcpdump.in b/utilities/ovs-tcpdump.in index c189bc8..96aa1e2 100755 --- a/utilities/ovs-tcpdump.in +++ b/utilities/ovs-tcpdump.in @@ -230,7 +2

[ovs-dev] [PATCH v2 1/4] ovs-tcpdump: Remove the newline from readline().

When using ovs-tcpdump, we always get the white line. Remove newline from readline() and ovs-tcpdump will format the output as tcpdump tool. Signed-off-by: nickcooper-zhangtonghao --- utilities/ovs-tcpdump.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilities/ovs-tcpdu

[ovs-dev] [PATCH v2 4/4] ovs-tcpdump: Fix error handling when parsing parameter.

Signed-off-by: nickcooper-zhangtonghao --- utilities/ovs-tcpdump.in | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/utilities/ovs-tcpdump.in b/utilities/ovs-tcpdump.in index 96aa1e2..d984a39 100755 --- a/utilities/ovs-tcpdump.in +++ b/utilities/ovs-tcpdump.in @@ -433,7 +433

[ovs-dev] [PATCH v2 2/4] ovs-tcpdump: should't halt when dumping traffic.

when sys.stdin is passed to tcpdump via pipe, it will cause tcpdump to halt. Signed-off-by: nickcooper-zhangtonghao --- utilities/ovs-tcpdump.in | 3 --- 1 file changed, 3 deletions(-) diff --git a/utilities/ovs-tcpdump.in b/utilities/ovs-tcpdump.in index 01e1f6d..c189bc8 100755 --- a/utilities

[ovs-dev] [PATCH v2 0/4] Fix ovs-tcpdump bugs.

Each patch fixes a different issue. nickcooper-zhangtonghao (4): ovs-tcpdump: Remove the newline from readline(). ovs-tcpdump: should't halt when dumping traffic. ovs-tcpdump: destroy_mirror should really destroy the mirror. ovs-tcpdump: Fix error handling when parsing parameter. utiliti

Re: [ovs-dev] [PATCH 2/4] ovs-tcpdump: should't halt when dumping traffic.

Hi Aaron, In fedora 24, when running ovs-tcpdump to dump traffic, I press the enter key and there are not ovs-tcpdump output any more. Thanks. Nick > On Nov 9, 2016, at 1:43 AM, Aaron Conole wrote: > >> when sys.stdin is passed to tcpdump via pipe, >> it will cause tcpdump to halt. >> >> Sig

Re: [ovs-dev] [PATCH ovs V1 0/9] Introducing HW offload support for openvswitch

On 07/11/2016 02:02, Joe Stringer wrote: On 3 November 2016 at 06:19, Simon Horman wrote: Hi Paul, thanks for this very interesting work. I hope we can see offload support present in OvS in the not too distant future. Looking over the patches I have a few high-level comments: * Please run