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
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
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
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
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
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
>
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
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..
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
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
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-
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
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
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..
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
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
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
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
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
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
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
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
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
>>
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
. 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
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) {
> +
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
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
-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
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
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>
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
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
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>
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
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
| 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
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 @@
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>
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
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
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
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(-)
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
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
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
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>
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
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-
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
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
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
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
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
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
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
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.
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
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
>>
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
>
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
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
-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
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
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
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
;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
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>
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
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
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
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
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
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,
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
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
.@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
..@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>
>
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
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:
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 <
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
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
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
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>
___
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>
_
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
-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
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
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
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
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
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
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:
>
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
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(-)
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>
___
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
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
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 - 100 of 3071 matches
Mail list logo