All of lore.kernel.org
 help / color / mirror / Atom feed
* [GIT net] Open vSwitch
@ 2013-02-07  3:14 Jesse Gross
       [not found] ` <1360206896-43022-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
  2013-02-08  4:39 ` [GIT net] Open vSwitch David Miller
  0 siblings, 2 replies; 35+ messages in thread
From: Jesse Gross @ 2013-02-07  3:14 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

One bug fix for net/3.8 for a long standing problem that was reported a few
times recently.

The following changes since commit a49f0d1ea3ec94fc7cf33a7c36a16343b74bd565:

  Linux 3.8-rc1 (2012-12-21 17:19:00 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to d9d59089c43fc33eb403cbb928e56c131f191dd5:

  openvswitch: Move LRO check from transmit to receive. (2013-01-21 23:57:26 -0800)

----------------------------------------------------------------
Jesse Gross (1):
      openvswitch: Move LRO check from transmit to receive.

 net/openvswitch/vport-netdev.c |   16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [PATCH] openvswitch: Move LRO check from transmit to receive.
       [not found] ` <1360206896-43022-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
@ 2013-02-07  3:14   ` Jesse Gross
  0 siblings, 0 replies; 35+ messages in thread
From: Jesse Gross @ 2013-02-07  3:14 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

The check for LRO packets was incorrectly put in the transmit path
instead of on receive.  Since this check is supposed to protect OVS
(and other parts of the system) from packets that it cannot handle
it is obviously not useful on egress.  Therefore, this commit moves
it back to the receive side.

The primary problem that this caused is upcalls to userspace tried
to segment the packet even though no segmentation information is
available.  This would later cause NULL pointer dereferences when
skb_gso_segment() did nothing.

Signed-off-by: Jesse Gross <jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
---
 net/openvswitch/vport-netdev.c |   16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/net/openvswitch/vport-netdev.c b/net/openvswitch/vport-netdev.c
index a9327e2..670cbc3 100644
--- a/net/openvswitch/vport-netdev.c
+++ b/net/openvswitch/vport-netdev.c
@@ -35,10 +35,11 @@
 /* Must be called with rcu_read_lock. */
 static void netdev_port_receive(struct vport *vport, struct sk_buff *skb)
 {
-	if (unlikely(!vport)) {
-		kfree_skb(skb);
-		return;
-	}
+	if (unlikely(!vport))
+		goto error;
+
+	if (unlikely(skb_warn_if_lro(skb)))
+		goto error;
 
 	/* Make our own copy of the packet.  Otherwise we will mangle the
 	 * packet for anyone who came before us (e.g. tcpdump via AF_PACKET).
@@ -50,6 +51,10 @@ static void netdev_port_receive(struct vport *vport, struct sk_buff *skb)
 
 	skb_push(skb, ETH_HLEN);
 	ovs_vport_receive(vport, skb);
+	return;
+
+error:
+	kfree_skb(skb);
 }
 
 /* Called with rcu_read_lock and bottom-halves disabled. */
@@ -169,9 +174,6 @@ static int netdev_send(struct vport *vport, struct sk_buff *skb)
 		goto error;
 	}
 
-	if (unlikely(skb_warn_if_lro(skb)))
-		goto error;
-
 	skb->dev = netdev_vport->dev;
 	len = skb->len;
 	dev_queue_xmit(skb);
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-02-07  3:14 [GIT net] Open vSwitch Jesse Gross
       [not found] ` <1360206896-43022-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
@ 2013-02-08  4:39 ` David Miller
  1 sibling, 0 replies; 35+ messages in thread
From: David Miller @ 2013-02-08  4:39 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Wed,  6 Feb 2013 19:14:55 -0800

> One bug fix for net/3.8 for a long standing problem that was reported a few
> times recently.
> 
> The following changes since commit a49f0d1ea3ec94fc7cf33a7c36a16343b74bd565:
> 
>   Linux 3.8-rc1 (2012-12-21 17:19:00 -0800)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2014-11-15  1:37 Pravin B Shelar
@ 2014-11-16 20:02 ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2014-11-16 20:02 UTC (permalink / raw)
  To: pshelar; +Cc: netdev

From: Pravin B Shelar <pshelar@nicira.com>
Date: Fri, 14 Nov 2014 17:37:24 -0800

> Following fixes are accumulated in ovs-repo.
> Three of them are related to protocol processing, one is
> related to memory leak in case of error and one is to
> fix race.
> Patch "Validate IPv6 flow key and mask values" has conflicts
> with net-next, Let me know if you want me to send the patch
> for net-next.

Pulled, thanks a lot.

I'll sort out the merge conflicts next time I merge net into
net-next, thanks for offering.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2014-11-15  1:37 Pravin B Shelar
  2014-11-16 20:02 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Pravin B Shelar @ 2014-11-15  1:37 UTC (permalink / raw)
  To: davem; +Cc: netdev

Following fixes are accumulated in ovs-repo.
Three of them are related to protocol processing, one is
related to memory leak in case of error and one is to
fix race.
Patch "Validate IPv6 flow key and mask values" has conflicts
with net-next, Let me know if you want me to send the patch
for net-next.

----------------------------------------------------------------
The following changes since commit b23dc5a7cc6ebc9a0d57351da7a0e8454c9ffea3:

  Merge tag 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost (2014-11-13 18:07:52 -0800)

are available in the git repository at:


  git://git.kernel.org/pub/scm/linux/kernel/git/pshelar/openvswitch.git net_ovs

for you to fetch changes up to fecaef85f7188ad1822210e2c7a7625c9a32a8e4:

  openvswitch: Validate IPv6 flow key and mask values. (2014-11-14 15:13:26 -0800)

----------------------------------------------------------------
Daniele Di Proietto (1):
      openvswitch: Fix NDP flow mask validation

Jarno Rajahalme (1):
      openvswitch: Validate IPv6 flow key and mask values.

Jesse Gross (1):
      openvswitch: Fix checksum calculation when modifying ICMPv6 packets.

Pravin B Shelar (2):
      openvswitch: Fix memory leak.
      openvswitch: Convert dp rcu read operation to locked operations

 net/openvswitch/actions.c      | 10 ++++++----
 net/openvswitch/datapath.c     | 14 +++++++-------
 net/openvswitch/flow_netlink.c |  9 ++++++++-
 3 files changed, 21 insertions(+), 12 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2014-07-01  6:33 Pravin B Shelar
@ 2014-07-08  2:39 ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2014-07-08  2:39 UTC (permalink / raw)
  To: pshelar; +Cc: dev, netdev

From: Pravin B Shelar <pshelar@nicira.com>
Date: Mon, 30 Jun 2014 23:33:16 -0700

> A set of fixes for net.
> First bug is related flow-table management.  Second one is in sample
> action. Third is related flow stats and last one add gre-err handler for ovs.

Pulled, thanks Pravin.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2014-07-01  6:33 Pravin B Shelar
  2014-07-08  2:39 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Pravin B Shelar @ 2014-07-01  6:33 UTC (permalink / raw)
  To: davem-fT/PcQaiUtIeIZ0/mPfg9Q
  Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

A set of fixes for net.
First bug is related flow-table management.  Second one is in sample
action. Third is related flow stats and last one add gre-err handler for ovs.

The following changes since commit dba63115ce0c888fcb4cdec3f8a4ba97d144afaf:

  powerpc: bpf: Fix the broken LD_VLAN_TAG_PRESENT test (2014-06-27 16:14:12 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/pshelar/openvswitch.git net_ovs_fixes

for you to fetch changes up to 4a46b24e147dfa9b858026da02cad0bdd4e149d2:

  openvswitch: Use exact lookup for flow_get and flow_del. (2014-06-30 20:47:15 -0700)

----------------------------------------------------------------
Alex Wang (1):
      openvswitch: Use exact lookup for flow_get and flow_del.

Andy Zhou (1):
      openvswitch: Fix a double free bug for the sample action

Ben Pfaff (1):
      openvswitch: Fix tracking of flags seen in TCP flows.

Wei Zhang (1):
      openvswitch: supply a dummy err_handler of gre_cisco_protocol to prevent kernel crash

 net/openvswitch/actions.c    |  2 ++
 net/openvswitch/datapath.c   | 27 +++++++++++++--------------
 net/openvswitch/flow.c       |  4 ++--
 net/openvswitch/flow.h       |  5 +++--
 net/openvswitch/flow_table.c | 16 ++++++++++++++++
 net/openvswitch/flow_table.h |  3 ++-
 net/openvswitch/vport-gre.c  | 17 +++++++++++++++++
 7 files changed, 55 insertions(+), 19 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
       [not found] ` <1395338954-26206-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
@ 2014-03-20 21:30   ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2014-03-20 21:30 UTC (permalink / raw)
  To: jesse-l0M0P4e3n4LQT0dZR+AlfA
  Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

From: Jesse Gross <jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
Date: Thu, 20 Mar 2014 11:09:10 -0700

> Four small fixes for net/3.14. I realize that these are late in the cycle - just got back from vacation.
> 
> The following changes since commit c14e0953ca51dbcb8d1ac92acbdcff23d0caa158:
> 
>   openvswitch: Suppress error messages on megaflow updates (2014-02-04 22:32:38 -0800)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2014-03-20 18:09 Jesse Gross
       [not found] ` <1395338954-26206-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2014-03-20 18:09 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

Four small fixes for net/3.14. I realize that these are late in the cycle - just got back from vacation.

The following changes since commit c14e0953ca51dbcb8d1ac92acbdcff23d0caa158:

  openvswitch: Suppress error messages on megaflow updates (2014-02-04 22:32:38 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to f9b8c4c8baded129535d82d74df8e87a7a369f54:

  openvswitch: Correctly report flow used times for first 5 minutes after boot. (2014-03-20 10:45:21 -0700)

----------------------------------------------------------------
Ben Pfaff (1):
      openvswitch: Correctly report flow used times for first 5 minutes after boot.

Jarno Rajahalme (2):
      openvswitch: Read tcp flags only then the tranport header is present.
      openvswitch: Fix race.

Jiri Pirko (1):
      ovs: fix dp check in ovs_dp_reset_user_features

 net/openvswitch/datapath.c | 9 +++++----
 net/openvswitch/flow.c     | 3 ++-
 2 files changed, 7 insertions(+), 5 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
       [not found]     ` <CAJZOPZK17fAaDfYj21i0F=cOg0uX52X9BPfdxZaExGyNTQqtdA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-02-06  7:32       ` Jesse Gross
  0 siblings, 0 replies; 35+ messages in thread
From: Jesse Gross @ 2014-02-06  7:32 UTC (permalink / raw)
  To: Or Gerlitz
  Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA,
	David Miller, Dan Carpenter

On Wed, Feb 5, 2014 at 11:09 PM, Or Gerlitz <or.gerlitz-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> On Wed, Feb 5, 2014 at 8:59 AM, Jesse Gross <jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org> wrote:
>>
>> A handful of bug fixes for net/3.14. High level fixes are:
>>  * Regressions introduced by the zerocopy changes, particularly with
>>    old userspaces.
>
> Hi, so this post was the 2nd version of the five patches you posted
> earlier, right? it would be very helpful if you
> denote that on the subject line (e.g just use --subject-prefix="PATCH
> net V1" for git format-patch) and provide
> crash diff listing from V0. Also, on a related note to the patch that
> deals with locking, I see these two smatch
> complaints, which might be false-positives, what's your thinking?
>
> net/openvswitch/flow.c:127 ovs_flow_stats_get() warn: returning with
> unbalanced local_bh_disable
> net/openvswitch/flow.c:160 ovs_flow_stats_clear() warn: returning with
> unbalanced local_bh_disable

Yes, it looks like a false positive to me.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
       [not found] ` <1391583561-25399-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
@ 2014-02-06  7:09   ` Or Gerlitz
       [not found]     ` <CAJZOPZK17fAaDfYj21i0F=cOg0uX52X9BPfdxZaExGyNTQqtdA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 35+ messages in thread
From: Or Gerlitz @ 2014-02-06  7:09 UTC (permalink / raw)
  To: Jesse Gross, Dan Carpenter
  Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA, David Miller

On Wed, Feb 5, 2014 at 8:59 AM, Jesse Gross <jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org> wrote:
>
> A handful of bug fixes for net/3.14. High level fixes are:
>  * Regressions introduced by the zerocopy changes, particularly with
>    old userspaces.

Hi, so this post was the 2nd version of the five patches you posted
earlier, right? it would be very helpful if you
denote that on the subject line (e.g just use --subject-prefix="PATCH
net V1" for git format-patch) and provide
crash diff listing from V0. Also, on a related note to the patch that
deals with locking, I see these two smatch
complaints, which might be false-positives, what's your thinking?

net/openvswitch/flow.c:127 ovs_flow_stats_get() warn: returning with
unbalanced local_bh_disable
net/openvswitch/flow.c:160 ovs_flow_stats_clear() warn: returning with
unbalanced local_bh_disable

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2014-02-05  6:59 Jesse Gross
@ 2014-02-06  0:26 ` David Miller
       [not found] ` <1391583561-25399-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
  1 sibling, 0 replies; 35+ messages in thread
From: David Miller @ 2014-02-06  0:26 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Tue,  4 Feb 2014 22:59:16 -0800

> A handful of bug fixes for net/3.14. High level fixes are:
>  * Regressions introduced by the zerocopy changes, particularly with
>    old userspaces.
>  * A few bugs lingering from the introduction of megaflows.
>  * Overly zealous error checking that is now being triggered frequently
>    in common cases.

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2014-02-05  6:59 Jesse Gross
  2014-02-06  0:26 ` David Miller
       [not found] ` <1391583561-25399-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
  0 siblings, 2 replies; 35+ messages in thread
From: Jesse Gross @ 2014-02-05  6:59 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, dev

A handful of bug fixes for net/3.14. High level fixes are:
 * Regressions introduced by the zerocopy changes, particularly with
   old userspaces.
 * A few bugs lingering from the introduction of megaflows.
 * Overly zealous error checking that is now being triggered frequently
   in common cases.

The following changes since commit bce3ea81d5b2a33ed0e275d58a45f10ce95cbd50:

  xen-netfront: handle backend CLOSED without CLOSING (2014-02-04 20:42:36 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to c14e0953ca51dbcb8d1ac92acbdcff23d0caa158:

  openvswitch: Suppress error messages on megaflow updates (2014-02-04 22:32:38 -0800)

----------------------------------------------------------------
Andy Zhou (2):
      openvswitch: Fix kernel panic on ovs_flow_free
      openvswitch: Suppress error messages on megaflow updates

Daniele Di Proietto (1):
      openvswitch: Fix ovs_dp_cmd_msg_size()

Pravin B Shelar (1):
      openvswitch: Fix ovs_flow_free() ovs-lock assert.

Thomas Graf (1):
      openvswitch: Pad OVS_PACKET_ATTR_PACKET if linear copy was performed

 net/openvswitch/datapath.c   | 23 ++++++++----
 net/openvswitch/flow_table.c | 88 ++++++++++++++++++++++----------------------
 net/openvswitch/flow_table.h |  2 +-
 3 files changed, 60 insertions(+), 53 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2014-02-03  1:08 Jesse Gross
  0 siblings, 0 replies; 35+ messages in thread
From: Jesse Gross @ 2014-02-03  1:08 UTC (permalink / raw)
  To: David Miller; +Cc: netdev-u79uwXL29TY76Z2rM5mHXA, dev-yBygre7rU0SM8Zsap4Y0gw

A handful of bug fixes for net/3.14. High level fixes are:
 * Regression introduced by the zerocopy changes, particularly with
   old userspaces.
 * A few bugs lingering from the introduction of megaflows.
 * Overly zealous error checking that is now being triggered frequently
   in common cases.

The following changes since commit 9895c503ef5b32c1ff4c4c224d6e8db2935dc3c0:

  ksz884x: delete useless variable (2014-01-15 13:43:03 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to dd3bceef108f99e21900c872d8b7ec632cfb88ac:

  openvswitch: Suppress error messages on megaflow updates (2014-02-02 16:39:48 -0800)

----------------------------------------------------------------
Andy Zhou (2):
      openvswitch: Fix kernel panic on ovs_flow_free
      openvswitch: Suppress error messages on megaflow updates

Daniele Di Proietto (1):
      openvswitch: Fix ovs_dp_cmd_msg_size()

Pravin B Shelar (1):
      openvswitch: Fix ovs_flow_free() ovs-lock assert.

Thomas Graf (1):
      openvswitch: Pad OVS_PACKET_ATTR_PACKET if linear copy was performed

 net/openvswitch/datapath.c   | 22 +++++++----
 net/openvswitch/flow_table.c | 87 +++++++++++++++++++++-----------------------
 net/openvswitch/flow_table.h |  2 +-
 3 files changed, 57 insertions(+), 54 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-10-28 18:04 Jesse Gross
@ 2013-10-29 21:37 ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2013-10-29 21:37 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Mon, 28 Oct 2013 11:04:50 -0700

> One patch for net/3.12 fixing an issue where devices could be in an
> invalid state they are removed while still attached to OVS.

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2013-10-28 18:04 Jesse Gross
  2013-10-29 21:37 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2013-10-28 18:04 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

One patch for net/3.12 fixing an issue where devices could be in an
invalid state they are removed while still attached to OVS.

The following changes since commit 272b98c6455f00884f0350f775c5342358ebb73f:

  Linux 3.12-rc1 (2013-09-16 16:17:51 -0400)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to b07c26511e94ab856f3700c56d582c0da36d5b4d:

  openvswitch: fix vport-netdev unregister (2013-10-16 14:50:22 -0700)

----------------------------------------------------------------
Alexei Starovoitov (1):
      openvswitch: fix vport-netdev unregister

 net/openvswitch/dp_notify.c    |  7 +++++--
 net/openvswitch/vport-netdev.c | 16 +++++++++++++---
 net/openvswitch/vport-netdev.h |  1 +
 3 files changed, 19 insertions(+), 5 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-08-14 23:19 Jesse Gross
@ 2013-08-15  8:41 ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2013-08-15  8:41 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Wed, 14 Aug 2013 16:19:04 -0700

> Three bug fixes that are fairly small either way but resolve obviously
> incorrect code. For net/3.11.

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2013-08-14 23:19 Jesse Gross
  2013-08-15  8:41 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2013-08-14 23:19 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

Three bug fixes that are fairly small either way but resolve obviously
incorrect code. For net/3.11.

The following changes since commit ad81f0545ef01ea651886dddac4bef6cec930092:

  Linux 3.11-rc1 (2013-07-14 15:18:27 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to 36bf5cc66d60868bcc10aff209defed5a7b71c1d:

  openvswitch: Reset tunnel key between input and output. (2013-08-14 15:50:36 -0700)

----------------------------------------------------------------
Jesse Gross (2):
      openvswitch: Fix bad merge resolution.
      openvswitch: Reset tunnel key between input and output.

Pravin B Shelar (1):
      openvswitch: Use correct type while allocating flex array.

 net/openvswitch/actions.c  | 1 +
 net/openvswitch/datapath.c | 3 ---
 net/openvswitch/flow.c     | 2 +-
 3 files changed, 2 insertions(+), 4 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-04-18  2:37     ` David Miller
@ 2013-04-18  4:22       ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2013-04-18  4:22 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: David Miller <davem@davemloft.net>
Date: Wed, 17 Apr 2013 22:37:41 -0400 (EDT)

> From: Jesse Gross <jesse@nicira.com>
> Date: Wed, 17 Apr 2013 19:31:41 -0700
> 
>> On Tue, Apr 16, 2013 at 1:53 PM, David Miller <davem@davemloft.net> wrote:
>>> From: Jesse Gross <jesse@nicira.com>
>>> Date: Tue, 16 Apr 2013 13:17:27 -0700
>>>
>>>> Two small bug fixes for net/3.9 including the issue previously
>>>> discussed where allocation of netlink notifications can fail after
>>>> changes have been committed.
>>>
>>> Looks good, pulled, thanks!
>> 
>> Sorry, but can you check whether this was pulled correctly? I'm not
>> seeing it in your net tree.
> 
> I don't see it either.
> 
> It might be sitting locally on a computer I'm not physically in front
> of right now, I'll check when I next go there (either tonight or
> tomorrow).
> 
> Sorry!

Yes, that's exactly what happened, I forgot to push out right before
leaving this machine yesterday.

I've fixed this up and pushed it all back out to 'net'.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-04-18  2:31   ` Jesse Gross
@ 2013-04-18  2:37     ` David Miller
  2013-04-18  4:22       ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: David Miller @ 2013-04-18  2:37 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Wed, 17 Apr 2013 19:31:41 -0700

> On Tue, Apr 16, 2013 at 1:53 PM, David Miller <davem@davemloft.net> wrote:
>> From: Jesse Gross <jesse@nicira.com>
>> Date: Tue, 16 Apr 2013 13:17:27 -0700
>>
>>> Two small bug fixes for net/3.9 including the issue previously
>>> discussed where allocation of netlink notifications can fail after
>>> changes have been committed.
>>
>> Looks good, pulled, thanks!
> 
> Sorry, but can you check whether this was pulled correctly? I'm not
> seeing it in your net tree.

I don't see it either.

It might be sitting locally on a computer I'm not physically in front
of right now, I'll check when I next go there (either tonight or
tomorrow).

Sorry!

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-04-16 20:53 ` David Miller
@ 2013-04-18  2:31   ` Jesse Gross
  2013-04-18  2:37     ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2013-04-18  2:31 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, dev

On Tue, Apr 16, 2013 at 1:53 PM, David Miller <davem@davemloft.net> wrote:
> From: Jesse Gross <jesse@nicira.com>
> Date: Tue, 16 Apr 2013 13:17:27 -0700
>
>> Two small bug fixes for net/3.9 including the issue previously
>> discussed where allocation of netlink notifications can fail after
>> changes have been committed.
>
> Looks good, pulled, thanks!

Sorry, but can you check whether this was pulled correctly? I'm not
seeing it in your net tree.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-04-16 20:17 Jesse Gross
@ 2013-04-16 20:53 ` David Miller
  2013-04-18  2:31   ` Jesse Gross
  0 siblings, 1 reply; 35+ messages in thread
From: David Miller @ 2013-04-16 20:53 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Tue, 16 Apr 2013 13:17:27 -0700

> Two small bug fixes for net/3.9 including the issue previously
> discussed where allocation of netlink notifications can fail after
> changes have been committed.

Looks good, pulled, thanks!

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2013-04-16 20:17 Jesse Gross
  2013-04-16 20:53 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2013-04-16 20:17 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, dev

Two small bug fixes for net/3.9 including the issue previously
discussed where allocation of netlink notifications can fail after
changes have been committed.

The following changes since commit 330305cc4a6b0cb75c22fc01b8826f0ad755550f:

  ipv4: Fix ip-header identification for gso packets. (2013-03-26 13:50:05 -0400)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to d3e1101c9b75574e68380b5cb10c9395fd8855de:

  openvswitch: correct an invalid BUG_ON (2013-03-27 09:07:41 -0700)

----------------------------------------------------------------
Hong Zhiguo (1):
      openvswitch: correct an invalid BUG_ON

Jesse Gross (1):
      openvswitch: Preallocate reply skb in ovs_vport_cmd_set().

 net/openvswitch/datapath.c |   30 ++++++++++++++++++------------
 net/openvswitch/flow.c     |    2 +-
 2 files changed, 19 insertions(+), 13 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2013-03-15  0:15 Jesse Gross
@ 2013-03-15 13:01 ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2013-03-15 13:01 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Thu, 14 Mar 2013 17:15:00 -0700

> A few different bug fixes, including several for issues with userspace
> communication that have gone unnoticed up until now.  These are intended
> for net/3.9.

Pulled, thanks.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2013-03-15  0:15 Jesse Gross
  2013-03-15 13:01 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2013-03-15  0:15 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

A few different bug fixes, including several for issues with userspace
communication that have gone unnoticed up until now.  These are intended
for net/3.9.

The following changes since commit 19f949f52599ba7c3f67a5897ac6be14bfcb1200:

  Linux 3.8 (2013-02-18 15:58:34 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to d176ca2a48ff2b5d7becfacdcbd1d72c73bd22d1:

  openvswitch: remove some useless comments (2013-02-22 17:02:49 -0800)

----------------------------------------------------------------
Cong Wang (2):
      openvswitch: fix the calculation of checksum for vlan header
      openvswitch: remove some useless comments

Rich Lane (4):
      openvswitch: Fix ovs_vport_cmd_del return value on success
      openvswitch: Fix ovs_vport_cmd_new return value on success
      openvswitch: Call genlmsg_end in queue_userspace_packet
      openvswitch: Fix parsing invalid LLC/SNAP ethertypes

 net/openvswitch/actions.c      |    4 ++--
 net/openvswitch/datapath.c     |    3 +++
 net/openvswitch/flow.c         |    6 +++++-
 net/openvswitch/vport-netdev.c |    3 +--
 net/openvswitch/vport.c        |    3 +--
 5 files changed, 12 insertions(+), 7 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2012-11-27 18:37 Jesse Gross
  2012-11-28 18:48 ` David Miller
@ 2012-11-28 23:04 ` David Miller
  1 sibling, 0 replies; 35+ messages in thread
From: David Miller @ 2012-11-28 23:04 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Tue, 27 Nov 2012 10:37:01 -0800

> These two small bug fixes are intended for 3.7/net if there is still time.

Pulled, thanks.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2012-11-28 18:48 ` David Miller
@ 2012-11-28 19:08   ` Jesse Gross
  0 siblings, 0 replies; 35+ messages in thread
From: Jesse Gross @ 2012-11-28 19:08 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, dev

On Wed, Nov 28, 2012 at 10:48 AM, David Miller <davem@davemloft.net> wrote:
>
> Date: Tue, 27 Nov 2012 10:37:01 -0800
>
> Is it really Tuesday morning where you are?

Unfortunately, no.  It's VM clock slew, which I've fixed now.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2012-11-27 18:37 Jesse Gross
@ 2012-11-28 18:48 ` David Miller
  2012-11-28 19:08   ` Jesse Gross
  2012-11-28 23:04 ` David Miller
  1 sibling, 1 reply; 35+ messages in thread
From: David Miller @ 2012-11-28 18:48 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev


Date: Tue, 27 Nov 2012 10:37:01 -0800

Is it really Tuesday morning where you are?

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2012-11-27 18:37 Jesse Gross
  2012-11-28 18:48 ` David Miller
  2012-11-28 23:04 ` David Miller
  0 siblings, 2 replies; 35+ messages in thread
From: Jesse Gross @ 2012-11-27 18:37 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

These two small bug fixes are intended for 3.7/net if there is still time.

The following changes since commit ddffeb8c4d0331609ef2581d84de4d763607bd37:

  Linux 3.7-rc1 (2012-10-14 14:41:04 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to d04d382980c86bdee9960c3eb157a73f8ed230cc:

  openvswitch: Store flow key len if ARP opcode is not request or reply. (2012-10-30 17:17:09 -0700)

----------------------------------------------------------------
Jesse Gross (1):
      openvswitch: Print device when warning about over MTU packets.

Mehak Mahajan (1):
      openvswitch: Store flow key len if ARP opcode is not request or reply.

 net/openvswitch/flow.c         |   14 +++++---------
 net/openvswitch/vport-netdev.c |    2 +-
 2 files changed, 6 insertions(+), 10 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
       [not found] ` <1346785688-2910-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
@ 2012-09-04 19:18   ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2012-09-04 19:18 UTC (permalink / raw)
  To: jesse-l0M0P4e3n4LQT0dZR+AlfA
  Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

From: Jesse Gross <jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
Date: Tue,  4 Sep 2012 12:08:05 -0700

> A few bug fixes intended for net/3.6.
> 
> The following changes since commit 0d7614f09c1ebdbaa1599a5aba7593f147bf96ee:
> 
>   Linux 3.6-rc1 (2012-08-02 16:38:10 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes
> 
> for you to fetch changes up to c303aa94cdae483a7577230e61720e126e600a52:
> 
>   openvswitch: Fix FLOW_BUFSIZE definition. (2012-09-03 19:06:27 -0700)

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2012-09-04 19:08 Jesse Gross
       [not found] ` <1346785688-2910-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2012-09-04 19:08 UTC (permalink / raw)
  To: David Miller; +Cc: netdev, dev

A few bug fixes intended for net/3.6.

The following changes since commit 0d7614f09c1ebdbaa1599a5aba7593f147bf96ee:

  Linux 3.6-rc1 (2012-08-02 16:38:10 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to c303aa94cdae483a7577230e61720e126e600a52:

  openvswitch: Fix FLOW_BUFSIZE definition. (2012-09-03 19:06:27 -0700)

----------------------------------------------------------------
Jesse Gross (2):
      openvswitch: Relax set header validation.
      openvswitch: Fix FLOW_BUFSIZE definition.

Joe Stringer (1):
      openvswitch: Fix typo

 net/openvswitch/actions.c  |    2 +-
 net/openvswitch/datapath.c |    6 +++---
 net/openvswitch/flow.h     |    8 +++++---
 3 files changed, 9 insertions(+), 7 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
       [not found] ` <1336441885-11085-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
@ 2012-05-08 23:32   ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2012-05-08 23:32 UTC (permalink / raw)
  To: jesse-l0M0P4e3n4LQT0dZR+AlfA
  Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

From: Jesse Gross <jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
Date: Mon,  7 May 2012 18:51:22 -0700

> A few patches for net/3.4.
> 
> The following changes since commit dd775ae2549217d3ae09363e3edb305d0fa19928:
> 
>   Linux 3.4-rc1 (2012-03-31 16:24:09 -0700)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

Pulled, thanks Jesse.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2012-05-08  1:51 Jesse Gross
       [not found] ` <1336441885-11085-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2012-05-08  1:51 UTC (permalink / raw)
  To: David Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

A few patches for net/3.4.

The following changes since commit dd775ae2549217d3ae09363e3edb305d0fa19928:

  Linux 3.4-rc1 (2012-03-31 16:24:09 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

for you to fetch changes up to 072ae6314a191e3a9fc309b1e4e539ac7abc48ad:

  openvswitch: Validation of IPv6 set port action uses IPv4 header (2012-05-07 17:23:10 -0700)

----------------------------------------------------------------
Ansis Atteka (1):
      openvswitch: Release rtnl_lock if ovs_vport_cmd_build_info() failed.

Jesse Gross (1):
      openvswitch: Add length check when retrieving TCP flags.

Pravin B Shelar (1):
      openvswitch: Validation of IPv6 set port action uses IPv4 header

 net/openvswitch/datapath.c |   27 +++++++++++++++++----------
 net/openvswitch/flow.c     |    3 ++-
 2 files changed, 19 insertions(+), 11 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

* Re: [GIT net] Open vSwitch
  2012-03-08  2:56 Jesse Gross
@ 2012-03-08  6:49 ` David Miller
  0 siblings, 0 replies; 35+ messages in thread
From: David Miller @ 2012-03-08  6:49 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev

From: Jesse Gross <jesse@nicira.com>
Date: Wed,  7 Mar 2012 18:56:06 -0800

> A few last minute patches for net/3.3 if there is time.
> 
> The following changes since commit dcd6c92267155e70a94b3927bce681ce74b80d1f:
> 
>   Linux 3.3-rc1 (2012-01-19 15:04:48 -0800)
> 
> are available in the git repository at:
>   git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

Pulled, thanks.

^ permalink raw reply	[flat|nested] 35+ messages in thread

* [GIT net] Open vSwitch
@ 2012-03-08  2:56 Jesse Gross
  2012-03-08  6:49 ` David Miller
  0 siblings, 1 reply; 35+ messages in thread
From: Jesse Gross @ 2012-03-08  2:56 UTC (permalink / raw)
  To: David S. Miller; +Cc: dev-yBygre7rU0TnMu66kgdUjQ, netdev-u79uwXL29TY76Z2rM5mHXA

A few last minute patches for net/3.3 if there is time.

The following changes since commit dcd6c92267155e70a94b3927bce681ce74b80d1f:

  Linux 3.3-rc1 (2012-01-19 15:04:48 -0800)

are available in the git repository at:
  git://git.kernel.org/pub/scm/linux/kernel/git/jesse/openvswitch.git fixes

Ben Pfaff (1):
      openvswitch: Honor dp_ifindex, when specified, for vport lookup by name.

Jesse Gross (1):
      openvswitch: Fix checksum update for actions on UDP packets.

 net/openvswitch/actions.c  |   44 ++++++++++++++++++++++++++++++++------------
 net/openvswitch/datapath.c |    3 +++
 2 files changed, 35 insertions(+), 12 deletions(-)

^ permalink raw reply	[flat|nested] 35+ messages in thread

end of thread, other threads:[~2014-11-16 20:02 UTC | newest]

Thread overview: 35+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-07  3:14 [GIT net] Open vSwitch Jesse Gross
     [not found] ` <1360206896-43022-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2013-02-07  3:14   ` [PATCH] openvswitch: Move LRO check from transmit to receive Jesse Gross
2013-02-08  4:39 ` [GIT net] Open vSwitch David Miller
  -- strict thread matches above, loose matches on Subject: below --
2014-11-15  1:37 Pravin B Shelar
2014-11-16 20:02 ` David Miller
2014-07-01  6:33 Pravin B Shelar
2014-07-08  2:39 ` David Miller
2014-03-20 18:09 Jesse Gross
     [not found] ` <1395338954-26206-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2014-03-20 21:30   ` David Miller
2014-02-05  6:59 Jesse Gross
2014-02-06  0:26 ` David Miller
     [not found] ` <1391583561-25399-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2014-02-06  7:09   ` Or Gerlitz
     [not found]     ` <CAJZOPZK17fAaDfYj21i0F=cOg0uX52X9BPfdxZaExGyNTQqtdA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-02-06  7:32       ` Jesse Gross
2014-02-03  1:08 Jesse Gross
2013-10-28 18:04 Jesse Gross
2013-10-29 21:37 ` David Miller
2013-08-14 23:19 Jesse Gross
2013-08-15  8:41 ` David Miller
2013-04-16 20:17 Jesse Gross
2013-04-16 20:53 ` David Miller
2013-04-18  2:31   ` Jesse Gross
2013-04-18  2:37     ` David Miller
2013-04-18  4:22       ` David Miller
2013-03-15  0:15 Jesse Gross
2013-03-15 13:01 ` David Miller
2012-11-27 18:37 Jesse Gross
2012-11-28 18:48 ` David Miller
2012-11-28 19:08   ` Jesse Gross
2012-11-28 23:04 ` David Miller
2012-09-04 19:08 Jesse Gross
     [not found] ` <1346785688-2910-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2012-09-04 19:18   ` David Miller
2012-05-08  1:51 Jesse Gross
     [not found] ` <1336441885-11085-1-git-send-email-jesse-l0M0P4e3n4LQT0dZR+AlfA@public.gmane.org>
2012-05-08 23:32   ` David Miller
2012-03-08  2:56 Jesse Gross
2012-03-08  6:49 ` David Miller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.