Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net

Pull networking fixes from David Miller:

 1) ping_err() ICMP error handler looks at wrong ICMP header, from Li
    Wei.

 2) TCP socket hash function on ipv6 is too weak, from Eric Dumazet.

 3) netif_set_xps_queue() forgets to drop mutex on errors, fix from
    Alexander Duyck.

 4) sum_frag_mem_limit() can deadlock due to lack of BH disabling, fix
    from Eric Dumazet.

 5) TCP SYN data is miscalculated in tcp_send_syn_data(), because the
    amount of TCP option space was not taken into account properly in
    this code path.  Fix from yuchung Cheng.

 6) MLX4 driver allocates device queues with the wrong size, from Kleber
    Sacilotto.

 7) sock_diag can access past the end of the sock_diag_handlers[] array,
    from Mathias Krause.

 8) vlan_set_encap_proto() makes incorrect assumptions about where
    skb->data points, rework the logic so that it works regardless of
    where skb->data happens to be.  From Jesse Gross.

 9) Fix gianfar build failure with NET_POLL enabled, from Paul
    Gortmaker.

10) Fix Ipv4 ID setting and checksum calculations in GRE driver, from
   Pravin B Shelar.

11) bgmac driver does:

        int i;

        for (i = 0; ...; ...) {
                ...
                for (i = 0; ...; ...) {

    effectively corrupting the outer loop index, use a seperate
    variable for the inner loops.  From Rafał Miłecki.

12) Fix suspend bugs in smsc95xx driver, from Ming Lei.

* git://git.kernel.org/pub/scm/linux/kernel/git/davem/net: (35 commits)
  usbnet: smsc95xx: rename FEATURE_AUTOSUSPEND
  usbnet: smsc95xx: fix broken runtime suspend
  usbnet: smsc95xx: fix suspend failure
  bgmac: fix indexing of 2nd level loops
  b43: Fix lockdep splat on module unload
  Revert "ip_gre: propogate target device GSO capability to the tunnel device"
  IP_GRE: Fix GRE_CSUM case.
  VXLAN: Use tunnel_ip_select_ident() for tunnel IP-Identification.
  IP_GRE: Fix IP-Identification.
  net/pasemi: Fix missing coding style
  vmxnet3: fix ethtool ring buffer size setting
  vmxnet3: make local function static
  bnx2x: remove dead code and make local funcs static
  gianfar: fix compile fail for NET_POLL=y due to struct packing
  vlan: adjust vlan_set_encap_proto() for its callers
  sock_diag: Simplify sock_diag_handlers[] handling in __sock_diag_rcv_msg
  sock_diag: Fix out-of-bounds access to sock_diag_handlers[]
  vxlan: remove depends on CONFIG_EXPERIMENTAL
  mlx4_en: fix allocation of CPU affinity reverse-map
  mlx4_en: fix allocation of device tx_cq
  ...

1 files changed, 17 insertions, 19 deletions

diff --git a/drivers/net/usb/smsc95xx.c b/drivers/net/usb/smsc95xx.c
index ff4fa37dfd1d..e6d2dea1373c 100644
--- a/drivers/net/usb/smsc95xx.c
+++ b/drivers/net/usb/smsc95xx.c
@@ -53,7 +53,7 @@
 
 #define FEATURE_8_WAKEUP_FILTERS        (0x01)
 #define FEATURE_PHY_NLP_CROSSOVER       (0x02)
-#define FEATURE_AUTOSUSPEND             (0x04)
+#define FEATURE_REMOTE_WAKEUP           (0x04)
 
 #define SUSPEND_SUSPEND0                (0x01)
 #define SUSPEND_SUSPEND1                (0x02)
@@ -1146,7 +1146,7 @@ static int smsc95xx_bind(struct usbnet *dev, struct usb_interface *intf)
             (val == ID_REV_CHIP_ID_89530_) || (val == ID_REV_CHIP_ID_9730_))
                 pdata->features = (FEATURE_8_WAKEUP_FILTERS |
                         FEATURE_PHY_NLP_CROSSOVER |
-                        FEATURE_AUTOSUSPEND);
+                        FEATURE_REMOTE_WAKEUP);
         else if (val == ID_REV_CHIP_ID_9512_)
                 pdata->features = FEATURE_8_WAKEUP_FILTERS;
 
@@ -1247,10 +1247,12 @@ static int smsc95xx_enter_suspend0(struct usbnet *dev)
 
         /* read back PM_CTRL */
         ret = smsc95xx_read_reg_nopm(dev, PM_CTRL, &val);
+        if (ret < 0)
+                return ret;
 
         pdata->suspend_flags |= SUSPEND_SUSPEND0;
 
-        return ret;
+        return 0;
 }
 
 static int smsc95xx_enter_suspend1(struct usbnet *dev)
@@ -1293,10 +1295,12 @@ static int smsc95xx_enter_suspend1(struct usbnet *dev)
         val |= (PM_CTL_WUPS_ED_ | PM_CTL_ED_EN_);
 
         ret = smsc95xx_write_reg_nopm(dev, PM_CTRL, val);
+        if (ret < 0)
+                return ret;
 
         pdata->suspend_flags |= SUSPEND_SUSPEND1;
 
-        return ret;
+        return 0;
 }
 
 static int smsc95xx_enter_suspend2(struct usbnet *dev)
@@ -1313,10 +1317,12 @@ static int smsc95xx_enter_suspend2(struct usbnet *dev)
         val |= PM_CTL_SUS_MODE_2;
 
         ret = smsc95xx_write_reg_nopm(dev, PM_CTRL, val);
+        if (ret < 0)
+                return ret;
 
         pdata->suspend_flags |= SUSPEND_SUSPEND2;
 
-        return ret;
+        return 0;
 }
 
 static int smsc95xx_enter_suspend3(struct usbnet *dev)
@@ -1372,7 +1378,7 @@ static int smsc95xx_autosuspend(struct usbnet *dev, u32 link_up)
         if (!link_up) {
                 /* link is down so enter EDPD mode, but only if device can
                  * reliably resume from it.  This check should be redundant
-                 * as current FEATURE_AUTOSUSPEND parts also support
+                 * as current FEATURE_REMOTE_WAKEUP parts also support
                  * FEATURE_PHY_NLP_CROSSOVER but it's included for clarity */
                 if (!(pdata->features & FEATURE_PHY_NLP_CROSSOVER)) {
                         netdev_warn(dev->net, "EDPD not supported\n");
@@ -1412,15 +1418,6 @@ static int smsc95xx_suspend(struct usb_interface *intf, pm_message_t message)
         u32 val, link_up;
         int ret;
 
-        /* TODO: don't indicate this feature to usb framework if
-         * our current hardware doesn't have the capability
-         */
-        if ((message.event == PM_EVENT_AUTO_SUSPEND) &&
-            (!(pdata->features & FEATURE_AUTOSUSPEND))) {
-                netdev_warn(dev->net, "autosuspend not supported\n");
-                return -EBUSY;
-        }
-
         ret = usbnet_suspend(intf, message);
         if (ret < 0) {
                 netdev_warn(dev->net, "usbnet_suspend error\n");
@@ -1435,7 +1432,8 @@ static int smsc95xx_suspend(struct usb_interface *intf, pm_message_t message)
         /* determine if link is up using only _nopm functions */
         link_up = smsc95xx_link_ok_nopm(dev);
 
-        if (message.event == PM_EVENT_AUTO_SUSPEND) {
+        if (message.event == PM_EVENT_AUTO_SUSPEND &&
+            (pdata->features & FEATURE_REMOTE_WAKEUP)) {
                 ret = smsc95xx_autosuspend(dev, link_up);
                 goto done;
         }
@@ -1872,11 +1870,11 @@ static int smsc95xx_manage_power(struct usbnet *dev, int on)
 
         dev->intf->needs_remote_wakeup = on;
 
-        if (pdata->features & FEATURE_AUTOSUSPEND)
+        if (pdata->features & FEATURE_REMOTE_WAKEUP)
                 return 0;
 
-        /* this chip revision doesn't support autosuspend */
-        netdev_info(dev->net, "hardware doesn't support USB autosuspend\n");
+        /* this chip revision isn't capable of remote wakeup */
+        netdev_info(dev->net, "hardware isn't capable of remote wakeup\n");
 
         if (on)
                 usb_autopm_get_interface_no_resume(dev->intf);