0195-net-broadcast-hub-packets-if-at-least-one-port-can-r.patch

   1 From 25ade2154673396e432de2b3c413865d1552c9b3 Mon Sep 17 00:00:00 2001
   2 From: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
   3 Date: Fri, 24 Aug 2012 13:50:30 +0100
   4 Subject: [PATCH] net: broadcast hub packets if at least one port can receive
   5
   6 In commit 60c07d933c66c4b30a83b7ccbc8a0cb3df1b2d0e ("net: fix
   7 qemu_can_send_packet logic") the "VLAN" broadcast behavior was changed
   8 to queue packets if any net client cannot receive.  It turns out that
   9 this was not actually the right fix and just hides the real bug that
  10 hw/usb/dev-network.c:usbnet_receive() clobbers its receive buffer when
  11 called multiple times in a row.  The commit also introduced a new bug
  12 that "VLAN" packets would not be sent if one of multiple net clients was
  13 down.
  14
  15 The hw/usb/dev-network.c bug has since been fixed, so this patch reverts
  16 broadcast behavior to send packets as long as one net client can
  17 receive.  Packets simply get queued for the net clients that are
  18 temporarily unable to receive.
  19
  20 Reported-by: Roy.Li <rongqing.li@windriver.com>
  21 Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
  22 (cherry picked from commit 61518a74ca98870e8ff132f91dd5dda252e31f58)
  23
  24 Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
  25 ---
  26  net/hub.c | 6 +++---
  27  1 file changed, 3 insertions(+), 3 deletions(-)
  28
  29 diff --git a/net/hub.c b/net/hub.c
  30 index ac157e3..650a8b4 100644
  31 --- a/net/hub.c
  32 +++ b/net/hub.c
  33 @@ -97,12 +97,12 @@ static int net_hub_port_can_receive(NetClientState *nc)
  34              continue;
  35          }
  36
  37 -        if (!qemu_can_send_packet(&port->nc)) {
  38 -            return 0;
  39 +        if (qemu_can_send_packet(&port->nc)) {
  40 +            return 1;
  41          }
  42      }
  43
  44 -    return 1;
  45 +    return 0;
  46  }
  47
  48  static ssize_t net_hub_port_receive(NetClientState *nc,
  49 --
  50 1.7.12.1
  51