Re: [ovs-dev] [PATCH v2] MAINTAINERS.rst: Move several people to emeritus status

e should any emeritus > committer desire to become active again. > > Per our policies, a vote of the majority of current committers (or > the list of maintainers prior to this change) is required to move a > committer to emeritus status. > > S

Re: [ovs-dev] Path MTU discovery on GRE interfaces

On Wed, Jun 23, 2021 at 10:06 AM Ben Pfaff wrote: > [updating Jesse's email address] > > On Wed, Jun 23, 2021 at 04:48:29PM +0200, Matthias May via dev wrote: > > I'm currently fighting with issues where TCP/UDP frames that are larger > than the MTU of a GRE tunnel are dropped. > > I'm aware of

Re: [ovs-dev] NSH Option 2 implementation

On Tue, Sep 20, 2016 at 3:52 AM, Jan Scheurich <jan.scheur...@ericsson.com> wrote: >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Friday, 16 September, 2016 01:38 >> >> I think the main issue is that when packets are being tunneled and NSH is in &g

Re: [ovs-dev] [PATCH v2 rebase] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

On Mon, Sep 19, 2016 at 8:48 AM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Sep 15, 2016 at 09:28:37AM -0700, Jesse Gross wrote: >> When using tunnel TLVs (at the moment, this means Geneve options), a >> controller must first map the class and type onto an appropriate OXM &g

Re: [ovs-dev] [PATCH] datapath: backport: openvswitch: use alias for genetlink family names

On Fri, Sep 16, 2016 at 11:52 AM, Thadeu Lima de Souza Cascardo wrote: > Upstream commit: > openvswitch: use alias for genetlink family names > > When userspace tries to create datapaths and the module is not loaded, > it will simply fail. With this patch, the

Re: [ovs-dev] NSH Option 2 implementation

peline or is generalized otherwise, e.g. when >> introducing support for P4. >> >> BR, Jan >> >> > -Original Message- >> > From: Ben Pfaff [mailto:b...@ovn.org] >> > Sent: Tuesday, 06 September, 2016 16:31 >> > To: Jesse Gross >

Re: [ovs-dev] [PATCH] datapath: compat: tunnels: Log error during initialization.

es about what went wrong. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovs-save: Use ${bridge} when dumping TLV map.

On Thu, Sep 15, 2016 at 10:43 AM, Ben Pfaff <b...@ovn.org> wrote: > On Sun, Sep 11, 2016 at 06:26:23PM -0700, Jesse Gross wrote: >> The script accidentally hardcoded the bridge name as 'br-int', >> a leftover from testing. >> >> Reported-by: Huanle Han <hanxue..

[ovs-dev] [PATCH v2 rebase] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

a <cha...@noironetworks.com> Signed-off-by: Jesse Gross <je...@kernel.org> --- This is a rebased version of the previously posted patch which no longer applies cleanly. v2: Pass 'struct tun_table' instead of 'struct ofproto' for functions that need to access tunnel metadata. --- i

Re: [ovs-dev] Open vSwitch 2.6 release scheduled for Sept. 15

On Tue, Sep 13, 2016 at 1:18 PM, Alin Serdean wrote: > On the Windows side we still have to integrate multiple internal/external > ports. As it was discussed before, I was wondering if they could be > integrated in 2.6. I think we're really past the time where

Re: [ovs-dev] [PATCH] datapath: compat: geneve: use ovs specific device type for compat geneve module.

On Mon, Sep 12, 2016 at 2:48 PM, Pravin B Shelar <pshe...@ovn.org> wrote: > This allows openvswitch and geneve module co-exist kernel on newer > kernels. > > Reported-by: Joe Stringer <j...@ovn.org> > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-

Re: [ovs-dev] procfs interface to openvswitch kernel module to help debugging

On Mon, Sep 12, 2016 at 10:22 AM, Ramu Ramamurthy wrote: > Hi, > > I dont see a /proc filesystem based interface to help debug the > openvswitch kernel module. Some of the knobs that may be useful are: > a) debug knobs for stats (for those stats are not already

[ovs-dev] [PATCH] ovs-save: Use ${bridge} when dumping TLV map.

The script accidentally hardcoded the bridge name as 'br-int', a leftover from testing. Reported-by: Huanle Han <hanxue...@gmail.com> Signed-off-by: Jesse Gross <je...@kernel.org> --- utilities/ovs-save | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utilitie

Re: [ovs-dev] [PATCH] datapath: Use pre-routing hook for conntrack.

On Fri, Sep 9, 2016 at 2:37 PM, Joe Stringer <j...@ovn.org> wrote: > On 8 September 2016 at 08:49, Jesse Gross <je...@kernel.org> wrote: >> On Wed, Sep 7, 2016 at 5:18 PM, Joe Stringer <j...@ovn.org> wrote: >>> On 1 September 2016 at 18:08, Jesse Gross <je..

Re: [ovs-dev] Open vSwitch 2.6 release scheduled for Sept. 15

On Fri, Sep 9, 2016 at 8:55 AM, Ben Pfaff wrote: > Our previously announced plan for OVS 2.6 is to release on Sept. 15, in > less than a week. In the OVN meeting in IRC yesterday, we already > discussed a few OVN bugs that should be fixed before the release. All > of them seem

Re: [ovs-dev] [PATCH] datapath: Use pre-routing hook for conntrack.

On Wed, Sep 7, 2016 at 5:18 PM, Joe Stringer <j...@ovn.org> wrote: > On 1 September 2016 at 18:08, Jesse Gross <je...@kernel.org> wrote: >> On Thu, Sep 1, 2016 at 5:01 PM, Joe Stringer <j...@ovn.org> wrote: >>> The upstream code uses NF_INET_PRE_ROUTING hoo

Re: [ovs-dev] [PATCH] datapath: Use pre-routing hook for conntrack.

kernel > panics were observed. Therefore it appears to be safe to bring this in > line with upstream without any other workarounds. > > Reported-by: Jesse Gross <je...@kernel.org> > Signed-off-by: Joe Stringer <j...@ovn.org> If you are confident that it doesn't cause problems o

[ovs-dev] [PATCH v2] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

a <cha...@noironetworks.com> Signed-off-by: Jesse Gross <je...@kernel.org> --- v2: Pass 'struct tun_table' instead of 'struct ofproto' for functions that need to access tunnel metadata. Rebase on top of ovs-ofctl metadata changes. --- include/openvswitch/match.h| 3 +- i

Re: [ovs-dev] [PATCH] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

On Tue, Aug 30, 2016 at 1:17 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Aug 23, 2016 at 02:28:27PM -0700, Jesse Gross wrote: >> When using tunnel TLVs (at the moment, this means Geneve options), a >> controller must first map the class and type onto an appropriate OXM &g

Re: [ovs-dev] [PATCH] ofproto: Do not override internal port MTU.

On Thu, Sep 1, 2016 at 10:17 AM, Daniele Di Proietto wrote: > Let me try to sum up the problem further > > 1) Behavior on master, before commit 47bf118665a3("ofproto: Always set MTU > for new internal ports."): > > a) When an internal interface is added, or its MTU

Re: [ovs-dev] [PATCH v2 1/5] ofproto-dpif-xlate: Don't unwildcard tunnel attributes on set.

On Tue, Aug 30, 2016 at 6:47 PM, Daniele Di Proietto wrote: > When translating a set action we also unwildcard the field in question. > This is done to correctly translate set actions with the value identical > to the ingress flow, like in the following example: > > flow

Re: [ovs-dev] one issue about vhost xstats with/without CRC

On Wed, Aug 31, 2016 at 2:30 AM, Yang, Zhiyong wrote: > Hi, all: > > Physical NIC has a set of hardware counters, such as > u64 prc64; > u64 prc127; > u64 prc255; etc. > DPDK counts the prc64 in two ways. Physical NIC counts prc64 with CRC by > hardware. Virtio computes

Re: [ovs-dev] [PATCH] ovs-save: Restore tunnel TLV map before flows.

On Tue, Aug 30, 2016 at 3:34 PM, Ben Pfaff <b...@ovn.org> wrote: > On Tue, Aug 30, 2016 at 02:16:36PM -0700, Jesse Gross wrote: >> Scripts that integrate OVS with a distribution often save and >> restore flows across distruptive events, such as an upgrade. The >>

Re: [ovs-dev] [PATCH 1/2] ovs-ofctl: Fix crash with replace-flows and diff-flows with tunnel metadata.

On Tue, Aug 30, 2016 at 3:22 PM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Aug 29, 2016 at 11:58:14AM -0700, Jesse Gross wrote: >> When flows are read by ovs-ofctl (either from a switch or a file), >> tunnel metadata space is dynamically allocated since there isn't a >&g

[ovs-dev] [PATCH] ovs-save: Restore tunnel TLV map before flows.

. Otherwise, the instance of OVS receiving the new flows won't know the meaning of these fields and will ignore them. Signed-off-by: Jesse Gross <je...@kernel.org> --- utilities/ovs-save | 6 ++ 1 file changed, 6 insertions(+) diff --git a/utilities/ovs-save b/utilities/ovs-save index d

Re: [ovs-dev] [PATCH] ovn-controller: Convert encaps module back to full processing

On Sun, Aug 28, 2016 at 3:51 PM, Ryan Moats wrote: > diff --git a/ovn/controller/encaps.c b/ovn/controller/encaps.c > index d99ba05..87f 100644 > --- a/ovn/controller/encaps.c > +++ b/ovn/controller/encaps.c > +SBREC_CHASSIS_FOR_EACH (chassis_rec, ctx->ovnsb_idl) { > +

Re: [ovs-dev] [PATCH] ipsec: Do not allow ipsec_gre tunnel traffic to exit unencrypted

574ff4aa (tunneling: get skb marking to work >properly with tunnels) > 2. ca3574d5 (IPsec: refactor out some code in >OVS_MONITOR_IPSEC_START macro) > > Signed-off-by: Ansis Atteka <aatt...@ovn.org> > Reported-by: Steffen Birke

[ovs-dev] [PATCH 2/2] ovs-ofctl: Extract tunnel metadata correctly when sorting flows.

and returned field is simply zeroed. This makes sorting based on tunnel metadata non-deterministic. We have the tunnel allocation stored in match metadata with each flow, so we can simply extract the data from there rather than trying to build and populate a global mapping table. Signed-off-by: Jesse Gross

[ovs-dev] [PATCH 1/2] ovs-ofctl: Fix crash with replace-flows and diff-flows with tunnel metadata.

-by: Jesse Gross <je...@kernel.org> --- tests/ovs-ofctl.at| 28 +++ utilities/ovs-ofctl.c | 209 +- 2 files changed, 218 insertions(+), 19 deletions(-) diff --git a/tests/ovs-ofctl.at b/tests/ovs-ofctl.at index 00db247..d966fc2 100644 --- a/tes

Re: [ovs-dev] [RFC PATCHv6] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

mprovement on VxLAN tunneling > decapsulation even though the SSE vector Rx function is disabled in DPDK poll > mode driver. > > Signed-off-by: Sugesh Chandran <sugesh.chand...@intel.com> Thanks for all of the revisions! This one looks good to me. I'll look forward to se

Re: [ovs-dev] [RFC PATCHv5] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Fri, Aug 26, 2016 at 2:39 AM, Chandran, Sugesh <sugesh.chand...@intel.com> wrote: >> -Original Message----- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Thursday, August 25, 2016 10:09 PM >> To: Chandran, Sugesh <sugesh.chand...@intel.com>

Re: [ovs-dev] [RFC PATCHv5] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Thu, Aug 25, 2016 at 2:43 AM, Sugesh Chandran wrote: > diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c > index 6d334db..46c4045 100644 > --- a/lib/netdev-dpdk.c > +++ b/lib/netdev-dpdk.c > static void > +dpdk_eth_checksum_offload_configure(struct netdev_dpdk

Re: [ovs-dev] [PATCH] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Wed, Aug 24, 2016 at 7:54 AM, Sugesh Chandran wrote: > diff --git a/lib/netdev-native-tnl.c b/lib/netdev-native-tnl.c > index ce2582f..31a12d6 100644 > --- a/lib/netdev-native-tnl.c > +++ b/lib/netdev-native-tnl.c > @@ -179,20 +181,26 @@ udp_extract_tnl_md(struct

Re: [ovs-dev] [RFC PATCHv3] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Tue, Aug 23, 2016 at 2:57 AM, Chandran, Sugesh <sugesh.chand...@intel.com> wrote: >> -Original Message----- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Monday, August 22, 2016 7:50 PM >> To: Chandran, Sugesh <sugesh.chand...@intel.com>

Re: [ovs-dev] NSH Option 2 implementation

On Tue, Aug 23, 2016 at 9:12 AM, Jan Scheurich <jan.scheur...@ericsson.com> wrote: >> -Original Message----- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Monday, 15 August, 2016 19:12 >> >> On Fri, Aug 12, 2016 at 1:01 AM, Jan Scheurich &l

[ovs-dev] [PATCH] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

a <cha...@noironetworks.com> Signed-off-by: Jesse Gross <je...@kernel.org> --- include/openvswitch/match.h| 4 +- include/openvswitch/ofp-parse.h| 6 +- include/openvswitch/ofp-util.h | 17 +- include/openvswitch/tun-metadata.h | 2 +- lib/classifier.c

Re: [ovs-dev] [PATCH] datapath: Add support for kernel 4.7

| 2 +- > acinclude.m4 | 4 ++-- > 5 files changed, 17 insertions(+), 4 deletions(-) Acked-by: Jesse Gross <je...@kernel.org> ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [RFC PATCHv3] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Mon, Aug 22, 2016 at 6:40 AM, Sugesh Chandran wrote: > diff --git a/lib/netdev-native-tnl.c b/lib/netdev-native-tnl.c > index ce2582f..78ce0c9 100644 > --- a/lib/netdev-native-tnl.c > +++ b/lib/netdev-native-tnl.c > @@ -85,11 +85,17 @@

Re: [ovs-dev] [RFC PATCHv2] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Mon, Aug 22, 2016 at 6:38 AM, Chandran, Sugesh <sugesh.chand...@intel.com> wrote: >> -Original Message----- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Saturday, August 20, 2016 2:07 AM >> To: Chandran, Sugesh <sugesh.chand...@intel.com>

Re: [ovs-dev] [RFC PATCHv2] netdev-dpdk: Enable Rx checksum offloading feature on DPDK physical ports.

On Fri, Aug 19, 2016 at 3:40 AM, Sugesh Chandran wrote: > diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c > index e5f2cdd..113e6d8 100644 > --- a/lib/netdev-dpdk.c > +++ b/lib/netdev-dpdk.c > @@ -1090,6 +1127,15 @@ netdev_dpdk_set_config(struct netdev *netdev, const

Re: [ovs-dev] [PATCH] match: Only print external tunnel flags.

On Fri, Aug 19, 2016 at 8:54 AM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Aug 18, 2016 at 05:25:25PM -0700, Jesse Gross wrote: >> Some tunnel flags are purely internal implementation details (primarily >> FLOW_TNL_F_UDPIF). These shouldn't be output when we format tunnel &g

Re: [ovs-dev] Will openvSwitch-2.5.0 supports VXLAN over IPSEC ?

On Thu, Aug 18, 2016 at 9:23 PM, santhu vaddepally wrote: > Hi, > > Please , Can any one tell me whether openvSwitch-2.5.0 supports VXLAN > over IPSEC. I didn't find any stuff regarding support of VXLAN over IPSEC > in README or other. Is this feasible with the

[ovs-dev] [PATCH] match: Only print external tunnel flags.

Some tunnel flags are purely internal implementation details (primarily FLOW_TNL_F_UDPIF). These shouldn't be output when we format tunnel flows, so this masks them out. Signed-off-by: Jesse Gross <je...@kernel.org> --- lib/match.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

Re: [ovs-dev] [PATCH] tests: Avoid race in tunnel-push-pop tests.

On Thu, Aug 18, 2016 at 4:36 PM, pravin shelar <pshe...@ovn.org> wrote: > On Thu, Aug 18, 2016 at 12:26 PM, Jesse Gross <je...@kernel.org> wrote: >> These tests dump the flows currently in the datapath but they aren't >> the first to run and might have existing flows le

Re: [ovs-dev] [PATCH v2] datapath: compat: backport LCO optimization.

On Wed, Aug 17, 2016 at 10:17 PM, pravin shelar <pshe...@ovn.org> wrote: > On Wed, Aug 17, 2016 at 5:26 PM, Jesse Gross <je...@kernel.org> wrote: >> On Wed, Aug 17, 2016 at 10:23 AM, Pravin B Shelar <pshe...@ovn.org> wrote: >>> diff --git a/datapath/linux/compat

[ovs-dev] [PATCH] tests: Avoid race in tunnel-push-pop tests.

These tests dump the flows currently in the datapath but they aren't the first to run and might have existing flows left over. We don't really care if there are others flows present so filter on the ones we want to avoid being affected by the timing of flow eviction. Signed-off-by: Jesse Gross

Re: [ovs-dev] [PATCH RFC] netdev-dpdk: Rx checksum offloading feature on DPDK physical ports.

On Tue, Aug 16, 2016 at 3:06 AM, Chandran, Sugesh <sugesh.chand...@intel.com> wrote: >> -Original Message----- >> From: Jesse Gross [mailto:je...@kernel.org] >> Sent: Monday, August 15, 2016 6:44 PM >> To: Chandran, Sugesh <sugesh.chand...@intel.com>

Re: [ovs-dev] [PATCH v2] datapath: compat: backport LCO optimization.

On Wed, Aug 17, 2016 at 10:23 AM, Pravin B Shelar wrote: > diff --git a/datapath/linux/compat/gso.c b/datapath/linux/compat/gso.c > index 89df07f..48a56b9 100644 > --- a/datapath/linux/compat/gso.c > +++ b/datapath/linux/compat/gso.c > @@ -236,12 +236,12 @@ static int

Re: [ovs-dev] [PATCH] ovn: Set critical bit in Geneve option.

On Tue, Aug 16, 2016 at 2:04 PM, Russell Bryant <russ...@ovn.org> wrote: > > On Tue, Aug 16, 2016 at 4:58 PM, Jesse Gross <je...@kernel.org> wrote: >> >> Currently the Geneve option type that OVN uses is 0, which in >> Geneve marks this as non-critical. Non-

Re: [ovs-dev] [PATCH 3/3] datapath: compat: gso: use sg and csum features for segmentation.

On Mon, Aug 15, 2016 at 2:11 PM, Pravin B Shelar wrote: > With use of LCO support we can finally enable SG and CSUM > features for GSO backport. > > Signed-off-by: Pravin B Shelar I don't know if this is safe. The kernels where this function is called are the

Re: [ovs-dev] [PATCH 2/3] datapath: compat: handle_offloads: remove csum_help param.

ller <da...@davemloft.net> > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> (Though it is dependent on the previous patch, of course.) ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/3] datapath: compat: backport LCO optimization.

On Mon, Aug 15, 2016 at 2:11 PM, Pravin B Shelar wrote: > diff --git a/datapath/linux/compat/gso.c b/datapath/linux/compat/gso.c > index 89df07f..f973fb4 100644 > --- a/datapath/linux/compat/gso.c > +++ b/datapath/linux/compat/gso.c > @@ -236,9 +236,6 @@ static int

[ovs-dev] [PATCH] ovn: Set critical bit in Geneve option.

the critical bit to properly mark the intention. Obviously, this will cause interoperability issues with any existing deployments but it should be fine while OVN is still labeled as experimental. Signed-off-by: Jesse Gross <je...@kernel.org> --- ovn/controller/physical.h | 2 +- ovn/ovn-architec

Re: [ovs-dev] [PATCH RFC] netdev-dpdk: Rx checksum offloading feature on DPDK physical ports.

On Mon, Aug 8, 2016 at 3:03 AM, Sugesh Chandran wrote: > To enable checksum offloading at rx side while adding a port, add the > 'rx-checksum-offload' option to the 'ovs-vsctl add-port' command-line as > below, > > 'ovs-vsctl add-port br0 dpdk0 -- \ > set Interface

Re: [ovs-dev] NSH Option 2 implementation

On Fri, Aug 12, 2016 at 1:01 AM, Jan Scheurich wrote: > The only clean way to avoid that now would be to maintain the monolithic > push/pop_nsh semantic of the Yi Yang patch and always push/pop NSH header and > outer MAC header together. Together with Simon's patch

Re: [ovs-dev] [PATCH v2] datapath: backport: openvswitch: do not ignore netdev errors when creating tunnel vports

ed-by: Pravin B Shelar <pshe...@ovn.org> > Signed-off-by: David S. Miller <da...@davemloft.net> > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Use passed ctx in XLATE_REPORT_ERROR.

On Mon, Aug 15, 2016 at 9:22 AM, Ben Pfaff <b...@ovn.org> wrote: > On Mon, Aug 15, 2016 at 08:21:19AM -0700, Jesse Gross wrote: >> XLATE_REPORT_ERROR is a macro that takes struct xlate_ctx as an >> argument but also implicitly uses 'ctx' from the local function >> scope.

[ovs-dev] [PATCH] ofproto-dpif-xlate: Use passed ctx in XLATE_REPORT_ERROR.

XLATE_REPORT_ERROR is a macro that takes struct xlate_ctx as an argument but also implicitly uses 'ctx' from the local function scope. This is works with current uses but it really should be using the argument. Signed-off-by: Jesse Gross <je...@kernel.org> --- ofproto/ofproto-dpif-xlate

Re: [ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

On Fri, Aug 12, 2016 at 1:06 PM, Ben Pfaff <b...@ovn.org> wrote: > On Thu, Aug 11, 2016 at 05:20:34PM -0700, Jesse Gross wrote: >> Currently metadata transmitted by OVN over Geneve tunnels is >> unprotected by any checksum other than the one provided by the link >>

Re: [ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

On Thu, Aug 11, 2016 at 7:10 PM, Ryan Moats <rmo...@us.ibm.com> wrote: > "dev" <dev-boun...@openvswitch.org> wrote on 08/11/2016 07:20:34 PM: > >> From: Jesse Gross <je...@kernel.org> >> To: dev@openvswitch.org >> Date: 08/11/2016 07:22 PM >

Re: [ovs-dev] [PATCH 2/2] datapath: backport: openvswitch: do not ignore netdev errors when creating tunnel vports

On Fri, Aug 12, 2016 at 10:49 AM, Pravin B Shelar wrote: > Upstream commit: > commit 4b5b9ba553f9aa5f484ab972fc9b58061885ceca > Author: Martynas Pumputis > Date: Tue Aug 9 16:24:50 2016 +0100 > > openvswitch: do not ignore netdev errors

Re: [ovs-dev] [PATCH 1/2] datapath: backport: OVS: Ignore negative headroom value

om/show_bug.cgi?id=1361414 > Signed-off-by: Ian Wienand <iwien...@redhat.com> > Signed-off-by: David S. Miller <da...@davemloft.net> > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

-by: Jesse Gross <je...@kernel.org> --- ovn/controller-vtep/gateway.c | 9 + ovn/controller/chassis.c | 8 ovn/controller/encaps.c | 12 +++- ovn/ovn-sb.xml| 10 -- ovn/utilities/ovn-sbctl.c | 2 ++ tests/ovn-controller-vtep.at

[ovs-dev] [PATCH rebase 2/3] ovn-controller: Make encap processing more robust against changes.

work incrementally.") Signed-off-by: Jesse Gross <je...@kernel.org> --- ovn/controller/encaps.c | 471 +--- ovn/controller/ovn-controller.c | 1 + 2 files changed, 248 insertions(+), 224 deletions(-) diff --git a/ovn/controller/encaps.c b/ovn/con

[ovs-dev] [PATCH rebase 1/3] ovn-controller: Fix memory leak when updating tunnels.

When a tunnel possibly needs to be updated, we are currently allocating a new name for it. This is not necessary and in fact nothing uses the name, which then results in the memory being leaked. Fixes: 1d45d5a9 ("ovn-controller: Change encaps_run to work incrementally.") Signed-off

[ovs-dev] [PATCH rebase 0/3] Enable Geneve checksums in OVN

of incremental processing which the first two patches fix. I'm targetting this for OVS 2.6 since the bulk of the changes are bug fixes and the benefits from the UDP checksums are significant. Jesse Gross (3): ovn-controller: Fix memory leak when updating tunnels. ovn-controller: Make encap

Re: [ovs-dev] [PATCH v2] datapath: compat: keep skb encapsulation zero on older kernel.

;pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> And, yes, I'm aware that we've now gone full circle on where to put clearing this flag... ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 3/3] datapath: compat: vxlan: fix vxlan_notify_add_rx_port().

On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar <pshe...@ovn.org> wrote: > Same as earlier patch this fixes vxlan recieve offload implementation. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org>

Re: [ovs-dev] [PATCH 2/3] datapath: compat: geneve: fix geneve_notify_add_rx_port()

On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar <pshe...@ovn.org> wrote: > Remove mutual exclusion between udp-gro registration and geneve receive port > registration. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse

Re: [ovs-dev] [PATCH 1/3] datapath: compat: keep skb encapsulation zero on older kernel.

On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar wrote: > When using compat GSO there is no need to turn on skb > encapsulation bit since OVS does not use any tunnel GSO > functionality from the networking stack. > > Signed-off-by: Pravin B Shelar I'm not sure

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

On Tue, Aug 9, 2016 at 7:52 AM, Simon Horman <simon.hor...@netronome.com> wrote: > Hi Jesse, > > On Wed, Jul 13, 2016 at 07:35:59AM -0700, Jesse Gross wrote: >> On Wed, Jul 13, 2016 at 4:04 AM, Brady Allen Johnson >> <brady.allen.john...@ericsson.com> wrote

Re: [ovs-dev] [PATCH 2/2] datapath: backport: net: vxlan: lwt: Use source ip address during route lookup.

other LWT implementations, > like GENEVE and GRE. > > Fixes: ee122c79d42 ("vxlan: Flow based tunneling"). > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> > Acked-by: Jiri Benc <jb...@redhat.com> > Signed-off-by: David S. Miller <da

Re: [ovs-dev] [PATCH 1/2] datapath: backport: net: vxlan: lwt: Fix vxlan local traffic.

id S. Miller <da...@davemloft.net> > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/6] datapath: compat: Use checksum offload for outer header.

On Wed, Aug 3, 2016 at 9:00 AM, pravin shelar <pshe...@ovn.org> wrote: > On Tue, Aug 2, 2016 at 4:55 PM, Jesse Gross <je...@kernel.org> wrote: >> On Tue, Aug 2, 2016 at 3:55 PM, pravin shelar <pshe...@ovn.org> wrote: >>> On Tue, Aug 2, 2016 at 3:11 PM,

Re: [ovs-dev] [PATCH 1/6] datapath: compat: Use udp-checksum function for compat case.

SE_UPSTREAM_TUNNEL. > Following patch couple this function to USE_UPSTREAM_TUNNEL symbol > rather than kernel version. > This is not bug, But it helps in code readability. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Since this is basically a subset of the other patch, with the

Re: [ovs-dev] [PATCH 6/6] datapath: compat: gso: tighen checks for compat GSO code.

On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar <pshe...@ovn.org> wrote: > Few function can be compiled out for non GSO case. This > patch make it bit cleaner to understand GSO compat code. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse

Re: [ovs-dev] [PATCH 5/6] datapath: backport: geneve: fix max_mtu setting

.@davemloft.net> > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 4/6] datapath: backport: openvswitch: fix conntrack netlink event delivery

..@nicira.com> > CC: Justin Pettit <jpet...@nicira.com> > CC: Andy Zhou <az...@nicira.com> > CC: Thomas Graf <tg...@suug.ch> > Signed-off-by: Samuel Gauthier <samuel.gauth...@6wind.com> > Acked-by: Joe Stringer <j...@ovn.org> >

Re: [ovs-dev] [PATCH 3/6] datapath: compat: vxlan: fix udp-csum typo

On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar <pshe...@ovn.org> wrote: > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> > --- > datapath/linux/compat/vxlan.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Jesse

Re: [ovs-dev] [PATCH 2/6] datapath: fix size of struct ovs_gso_cb

On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar <pshe...@ovn.org> wrote: > struct ovs_gso_cb is stored in skb->cd. avoid going beyond size > of skb->cb. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by:

Re: [ovs-dev] [PATCH 2/6] datapath: compat: Use checksum offload for outer header.

On Tue, Aug 2, 2016 at 3:55 PM, pravin shelar <pshe...@ovn.org> wrote: > On Tue, Aug 2, 2016 at 3:11 PM, Jesse Gross <je...@kernel.org> wrote: >> On Tue, Aug 2, 2016 at 2:17 PM, Pravin B Shelar <pshe...@ovn.org> wrote: >>> Signed-off-by: Pravin B Shelar <

Re: [ovs-dev] [PATCH 6/6] datapath: Add support for kernel 4.7

On Tue, Aug 2, 2016 at 2:18 PM, Pravin B Shelar wrote: > Signed-off-by: Pravin B Shelar > --- > FAQ.md | 2 +- > NEWS | 2 +- > acinclude.m4 | 4 ++-- > 3 files changed, 4 insertions(+), 4 deletions(-) Should we also add 4.7 to Travis? I noticed

Re: [ovs-dev] [PATCH 5/6] datapath: compat: remove duplicate check.

On Tue, Aug 2, 2016 at 2:18 PM, Pravin B Shelar <pshe...@ovn.org> wrote: > The check for tunnel GSO packet is done at ip-handle-offloads. Remove > same check from udp-handle-offloads. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse

Re: [ovs-dev] [PATCH 4/6] datapath: compat: fix SKB_INIT_FILL_METADATA_DST definition

On Tue, Aug 2, 2016 at 2:18 PM, Pravin B Shelar <pshe...@ovn.org> wrote: > In case of OVS using compat fill metadata dst implementation we need to > setup temperory dst. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse

Re: [ovs-dev] [PATCH 1/6] datapath: compat: Detect GSO support at ovs configure

t; configure stage. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> It looks like RHEL 7.2 has already backported ndo_features_check, so this should give an immediate improvement there - not just a theoretical benefit. Acked-by: Jesse Gross <je...@kernel.org> ___

Re: [ovs-dev] [PATCH 3/6] datapath: compat: cleanup ip-tunnels

On Tue, Aug 2, 2016 at 2:18 PM, Pravin B Shelar <pshe...@ovn.org> wrote: > Remove kernel version check related to unsupported kernel. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> _

Re: [ovs-dev] [PATCH 2/6] datapath: compat: Use checksum offload for outer header.

On Tue, Aug 2, 2016 at 2:17 PM, Pravin B Shelar wrote: > Signed-off-by: Pravin B Shelar > --- > datapath/linux/compat/include/net/udp.h | 2 +- > datapath/linux/compat/udp.c | 19 ++- > datapath/linux/compat/udp_tunnel.c | 17

[ovs-dev] [PATCH 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

-by: Jesse Gross <je...@kernel.org> --- ovn/controller-vtep/gateway.c | 9 + ovn/controller/chassis.c | 8 ovn/controller/encaps.c | 12 +++- ovn/ovn-sb.xml| 10 -- ovn/utilities/ovn-sbctl.c | 2 ++ tests/ovn-controller-vtep.at

[ovs-dev] [PATCH 1/3] ovn-controller: Fix memory leak when updating tunnels.

When a tunnel possibly needs to be updated, we are currently allocating a new name for it. This is not necessary and in fact nothing uses the name, which then results in the memory being leaked. Fixes: 1d45d5a9 ("ovn-controller: Change encaps_run to work incrementally.") Signed-off

[ovs-dev] [PATCH 2/3] ovn-controller: Make encap processing more robust against changes.

work incrementally.") Signed-off-by: Jesse Gross <je...@kernel.org> --- ovn/controller/encaps.c | 469 +--- ovn/controller/ovn-controller.c | 1 + 2 files changed, 247 insertions(+), 223 deletions(-) diff --git a/ovn/controller/encaps.c b/ovn/con

[ovs-dev] [PATCH 0/3] Enable Geneve checksums in OVN

of incremental processing which the first two patches fix. I'm targetting this for OVS 2.6 since the bulk of the changes are bug fixes and the benefits from the UDP checksums are significant. Jesse Gross (3): ovn-controller: Fix memory leak when updating tunnels. ovn-controller: Make encap

Re: [ovs-dev] [PATCH v5 13/16] system-tests: Run conntrack tests with userspace.

On Wed, Jul 27, 2016 at 1:40 PM, Daniele Di Proietto wrote: > On 27/07/2016 13:12, "Joe Stringer" wrote: > >>On 26 July 2016 at 17:58, Daniele Di Proietto wrote: >>> The userspace connection tracker doesn't support ALGs, frag

Re: [ovs-dev] Considering the possibility of integrating DPDK generic classifier APIs in OVS.

From: Gray, Mark D >> Sent: Wednesday, July 20, 2016 7:17 PM >> To: Jesse Gross <je...@kernel.org> >> Cc: Chandran, Sugesh <sugesh.chand...@intel.com>; >> dev@openvswitch.org; Giller, Robin <robin.gil...@intel.com> >> Subject: RE: [ovs-dev] Considering

Re: [ovs-dev] [RFC PATCH v2 00/13] Add Network Service Header Support

On Thu, Jul 21, 2016 at 3:40 PM, Jan Scheurich wrote: > 1. The pending question whether to model NSH headers as packet header match > fields, metadata fields, or both applies in particular to the MD2 TLVs. > > We have three main OpenFlow use cases for the MD2 TLVs: >

Re: [ovs-dev] [PATCH v2 1/3] datapath: compat: fix udp checksum calculation

On Tue, Jul 26, 2016 at 3:59 PM, pravin shelar <pshe...@ovn.org> wrote: > On Tue, Jul 26, 2016 at 3:53 PM, Jesse Gross <je...@kernel.org> wrote: >> On Tue, Jul 26, 2016 at 3:24 PM, Pravin B Shelar <pshe...@ovn.org> wrote: >>> diff --git a/datapath/linux/comp

Re: [ovs-dev] [PATCH v2 3/3] datapath: compat: simplify ip_local_out().

On Tue, Jul 26, 2016 at 3:24 PM, Pravin B Shelar <pshe...@ovn.org> wrote: > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> > --- > datapath/linux/compat/gso.c | 82 > ++--- > 1 file changed, 33 insertions(+), 49 deletions(-)

Re: [ovs-dev] [PATCH v2 2/3] datapath: compat: unset skb encapsulation bit

gt;> i40e :42:00.0: TX driver issue detected, PF reset issued > > Following patch resets this bit in case compat layer handles the packet. > > VMware-BZ: 1698877 > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse Gross <je...@kernel.org> ___

Re: [ovs-dev] releasing 2.6: branch Aug 1, release Sep 15

On Sun, Jul 24, 2016 at 10:53 AM, Ben Pfaff wrote: > On Sun, Jul 24, 2016 at 08:39:31AM -0300, Thadeu Lima de Souza Cascardo wrote: >> On Sat, Jul 23, 2016 at 08:59:35AM -0700, Ben Pfaff wrote: >> > The proposed Open vSwitch release schedule calls for branching 2.6 from >> > master

Re: [ovs-dev] [PATCH] datapath: Add support for kernel 4.6

On Mon, Jul 25, 2016 at 6:40 PM, Pravin B Shelar <pshe...@ovn.org> wrote: > Most of patch iron out USE_UPSTREAM_TUNNEL case where datapath > directly use upstream tunneling modules. > > Signed-off-by: Pravin B Shelar <pshe...@ovn.org> Acked-by: Jesse

Re: [ovs-dev] [PATCH 2/3] datapath: compat: unset skb encapsulation bit

On Tue, Jul 26, 2016 at 12:30 PM, pravin shelar <pshe...@ovn.org> wrote: > On Tue, Jul 26, 2016 at 11:14 AM, Jesse Gross <je...@kernel.org> wrote: >> On Tue, Jul 26, 2016 at 10:56 AM, pravin shelar <pshe...@ovn.org> wrote: >>> On Tue, Jul 26, 2016 at 10:06 AM,

  1   2   3   4   5   6   7   8   9   10   >