diff mbox

[net-next] sunvnet: only use connected ports when sending

Message ID 53D01FD9.5000804@oracle.com
State Changes Requested, archived
Delegated to: David Miller
Headers show

Commit Message

David L Stevens July 23, 2014, 8:49 p.m. UTC
The sunvnet driver doesn't check whether or not a port is connected when
transmitting packets, which results in failures if a port fails to connect
(e.g., due to a version mismatch). The original code also assumes
unnecessarily that the first port is up and a switch, even though there is
a flag for switch ports.

This patch only matches a port if it is connected, and otherwise uses the
switch_port flag to send the packet to a switch port that is up.

Signed-off-by: David L Stevens <david.stevens@oracle.com>
---
 drivers/net/ethernet/sun/sunvnet.c |   22 +++++++++++++++++-----
 1 files changed, 17 insertions(+), 5 deletions(-)

Comments

David Miller July 25, 2014, 6:31 a.m. UTC | #1
From: David L Stevens <david.stevens@oracle.com>
Date: Wed, 23 Jul 2014 16:49:29 -0400

> The sunvnet driver doesn't check whether or not a port is connected when
> transmitting packets, which results in failures if a port fails to connect
> (e.g., due to a version mismatch). The original code also assumes
> unnecessarily that the first port is up and a switch, even though there is
> a flag for switch ports.
> 
> This patch only matches a port if it is connected, and otherwise uses the
> switch_port flag to send the packet to a switch port that is up.
> 
> Signed-off-by: David L Stevens <david.stevens@oracle.com>

This doesn't apply because your email client mangled the patch, it turned
TAB characters into sequences of SPACE characters.

Secondly, this is a bonafide bug fix, and therefore should be targetted
at 'net' instead of 'net-next'.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/ethernet/sun/sunvnet.c b/drivers/net/ethernet/sun/sunvnet.c
index fd411d6..d813bfb 100644
--- a/drivers/net/ethernet/sun/sunvnet.c
+++ b/drivers/net/ethernet/sun/sunvnet.c
@@ -610,6 +610,13 @@  static int __vnet_tx_trigger(struct vnet_port *port)
        return err;
 }

+static inline bool port_is_up(struct vnet_port *vnet)
+{
+       struct vio_driver_state *vio = &vnet->vio;
+
+       return !!(vio->hs_state & VIO_HS_COMPLETE);
+}
+
 struct vnet_port *__tx_port_find(struct vnet *vp, struct sk_buff *skb)
 {
        unsigned int hash = vnet_hashfn(skb->data);
@@ -617,14 +624,19 @@  struct vnet_port *__tx_port_find(struct vnet *vp, struct sk_buff *skb)
        struct vnet_port *port;

        hlist_for_each_entry(port, hp, hash) {
+               if (!port_is_up(port))
+                       continue;
                if (ether_addr_equal(port->raddr, skb->data))
                        return port;
        }
-       port = NULL;
-       if (!list_empty(&vp->port_list))
-               port = list_entry(vp->port_list.next, struct vnet_port, list);
-
-       return port;
+       list_for_each_entry(port, &vp->port_list, list) {
+               if (!port->switch_port)
+                       continue;
+               if (!port_is_up(port))
+                       continue;
+               return port;
+       }
+       return NULL;
 }

 struct vnet_port *tx_port_find(struct vnet *vp, struct sk_buff *skb)