netxen: minor suspend resume fixes
[safe/jmp/linux-2.6] / drivers / net / netxen / netxen_nic_main.c
index 1399102..41cb0af 100644 (file)
@@ -1,5 +1,6 @@
 /*
  * Copyright (C) 2003 - 2009 NetXen, Inc.
+ * Copyright (C) 2009 - QLogic Corporation.
  * All rights reserved.
  *
  * This program is free software; you can redistribute it and/or
  * The full GNU General Public License is included in this distribution
  * in the file called LICENSE.
  *
- * Contact Information:
- *    info@netxen.com
- * NetXen Inc,
- * 18922 Forge Drive
- * Cupertino, CA 95014-0701
- *
  */
 
 #include <linux/vmalloc.h>
 #include <net/ip.h>
 #include <linux/ipv6.h>
 #include <linux/inetdevice.h>
+#include <linux/sysfs.h>
 
-MODULE_DESCRIPTION("NetXen Multi port (1/10) Gigabit Network Driver");
+MODULE_DESCRIPTION("QLogic/NetXen (1/10) GbE Converged Ethernet Driver");
 MODULE_LICENSE("GPL");
 MODULE_VERSION(NETXEN_NIC_LINUX_VERSIONID);
+MODULE_FIRMWARE(NX_P2_MN_ROMIMAGE_NAME);
+MODULE_FIRMWARE(NX_P3_CT_ROMIMAGE_NAME);
+MODULE_FIRMWARE(NX_P3_MN_ROMIMAGE_NAME);
+MODULE_FIRMWARE(NX_UNIFIED_ROMIMAGE_NAME);
 
 char netxen_nic_driver_name[] = "netxen_nic";
-static char netxen_nic_driver_string[] = "NetXen Network Driver version "
+static char netxen_nic_driver_string[] = "QLogic/NetXen Network Driver v"
     NETXEN_NIC_LINUX_VERSIONID;
 
 static int port_mode = NETXEN_PORT_MODE_AUTO_NEG;
@@ -57,7 +57,10 @@ static int use_msi = 1;
 
 static int use_msi_x = 1;
 
-/* Local functions to NetXen NIC driver */
+static int auto_fw_reset = AUTO_FW_RESET_ENABLED;
+module_param(auto_fw_reset, int, 0644);
+MODULE_PARM_DESC(auto_fw_reset,"Auto firmware reset (0=disabled, 1=enabled");
+
 static int __devinit netxen_nic_probe(struct pci_dev *pdev,
                const struct pci_device_id *ent);
 static void __devexit netxen_nic_remove(struct pci_dev *pdev);
@@ -66,7 +69,7 @@ static int netxen_nic_close(struct net_device *netdev);
 static netdev_tx_t netxen_nic_xmit_frame(struct sk_buff *,
                                               struct net_device *);
 static void netxen_tx_timeout(struct net_device *netdev);
-static void netxen_reset_task(struct work_struct *work);
+static void netxen_tx_timeout_task(struct work_struct *work);
 static void netxen_fw_poll_work(struct work_struct *work);
 static void netxen_schedule_work(struct netxen_adapter *adapter,
                work_func_t func, int delay);
@@ -78,6 +81,8 @@ static void netxen_nic_poll_controller(struct net_device *netdev);
 
 static void netxen_create_sysfs_entries(struct netxen_adapter *adapter);
 static void netxen_remove_sysfs_entries(struct netxen_adapter *adapter);
+static void netxen_create_diag_entries(struct netxen_adapter *adapter);
+static void netxen_remove_diag_entries(struct netxen_adapter *adapter);
 
 static int nx_decr_dev_ref_cnt(struct netxen_adapter *adapter);
 static int netxen_can_start_firmware(struct netxen_adapter *adapter);
@@ -442,6 +447,7 @@ netxen_read_mac_addr(struct netxen_adapter *adapter)
                netdev->dev_addr[i] = *(p + 5 - i);
 
        memcpy(netdev->perm_addr, netdev->dev_addr, netdev->addr_len);
+       memcpy(adapter->mac_addr, netdev->dev_addr, netdev->addr_len);
 
        /* set station address */
 
@@ -464,6 +470,7 @@ int netxen_nic_set_mac(struct net_device *netdev, void *p)
                netxen_napi_disable(adapter);
        }
 
+       memcpy(adapter->mac_addr, addr->sa_data, netdev->addr_len);
        memcpy(netdev->dev_addr, addr->sa_data, netdev->addr_len);
        adapter->macaddr_set(adapter, addr->sa_data);
 
@@ -600,7 +607,8 @@ netxen_setup_pci_map(struct netxen_adapter *adapter)
        void __iomem *mem_ptr2 = NULL;
        void __iomem *db_ptr = NULL;
 
-       unsigned long mem_base, mem_len, db_base, db_len = 0, pci_len0 = 0;
+       resource_size_t mem_base, db_base;
+       unsigned long mem_len, db_len = 0, pci_len0 = 0;
 
        struct pci_dev *pdev = adapter->pdev;
        int pci_func = adapter->ahw.pci_func;
@@ -611,14 +619,12 @@ netxen_setup_pci_map(struct netxen_adapter *adapter)
         * Set the CRB window to invalid. If any register in window 0 is
         * accessed it should set the window to 0 and then reset it to 1.
         */
-       adapter->curr_window = 255;
-       adapter->ahw.qdr_sn_window = -1;
-       adapter->ahw.ddr_mn_window = -1;
+       adapter->ahw.crb_win = -1;
+       adapter->ahw.ocm_win = -1;
 
        /* remap phys address */
        mem_base = pci_resource_start(pdev, 0); /* 0 is for BAR 0 */
        mem_len = pci_resource_len(pdev, 0);
-       pci_len0 = 0;
 
        /* 128 Meg of memory */
        if (mem_len == NETXEN_PCI_128MB_SIZE) {
@@ -627,6 +633,7 @@ netxen_setup_pci_map(struct netxen_adapter *adapter)
                                SECOND_PAGE_GROUP_SIZE);
                mem_ptr2 = ioremap(mem_base + THIRD_PAGE_GROUP_START,
                                THIRD_PAGE_GROUP_SIZE);
+               pci_len0 = FIRST_PAGE_GROUP_SIZE;
        } else if (mem_len == NETXEN_PCI_32MB_SIZE) {
                mem_ptr1 = ioremap(mem_base, SECOND_PAGE_GROUP_SIZE);
                mem_ptr2 = ioremap(mem_base + THIRD_PAGE_GROUP_START -
@@ -639,18 +646,6 @@ netxen_setup_pci_map(struct netxen_adapter *adapter)
                        return -EIO;
                }
                pci_len0 = mem_len;
-
-               adapter->ahw.ddr_mn_window = 0;
-               adapter->ahw.qdr_sn_window = 0;
-
-               adapter->ahw.mn_win_crb = 0x100000 + PCIX_MN_WINDOW +
-                       (pci_func * 0x20);
-               adapter->ahw.ms_win_crb = 0x100000 + PCIX_SN_WINDOW;
-               if (pci_func < 4)
-                       adapter->ahw.ms_win_crb += (pci_func * 0x20);
-               else
-                       adapter->ahw.ms_win_crb +=
-                                       0xA0 + ((pci_func - 4) * 0x10);
        } else {
                return -EIO;
        }
@@ -664,6 +659,15 @@ netxen_setup_pci_map(struct netxen_adapter *adapter)
        adapter->ahw.pci_base1 = mem_ptr1;
        adapter->ahw.pci_base2 = mem_ptr2;
 
+       if (NX_IS_REVISION_P3P(adapter->ahw.revision_id)) {
+               adapter->ahw.ocm_win_crb = netxen_get_ioaddr(adapter,
+                       NETXEN_PCIX_PS_REG(PCIX_OCM_WINDOW_REG(pci_func)));
+
+       } else if (NX_IS_REVISION_P3(adapter->ahw.revision_id)) {
+               adapter->ahw.ocm_win_crb = netxen_get_ioaddr(adapter,
+                       NETXEN_PCIX_PS_REG(PCIE_MN_WINDOW_REG(pci_func)));
+       }
+
        if (NX_IS_REVISION_P3(adapter->ahw.revision_id))
                goto skip_doorbell;
 
@@ -728,7 +732,8 @@ netxen_check_options(struct netxen_adapter *adapter)
        if (adapter->portnum == 0) {
                get_brd_name_by_type(adapter->ahw.board_type, brd_name);
 
-               printk(KERN_INFO "NetXen %s Board S/N %s  Chip rev 0x%x\n",
+               pr_info("%s: %s Board S/N %s  Chip rev 0x%x\n",
+                               module_name(THIS_MODULE),
                                brd_name, serial_num, adapter->ahw.revision_id);
        }
 
@@ -820,7 +825,7 @@ netxen_start_firmware(struct netxen_adapter *adapter)
 
        if (first_boot != 0x55555555) {
                NXWR32(adapter, CRB_CMDPEG_STATE, 0);
-               netxen_pinit_from_rom(adapter, 0);
+               netxen_pinit_from_rom(adapter);
                msleep(1);
        }
 
@@ -859,8 +864,6 @@ netxen_start_firmware(struct netxen_adapter *adapter)
                | (_NETXEN_NIC_LINUX_SUBVERSION);
        NXWR32(adapter, CRB_DRIVER_VERSION, val);
 
-       NXWR32(adapter, NX_CRB_DEV_STATE, NX_DEV_READY);
-
 wait_init:
        /* Handshake with the card before we register the devices. */
        err = netxen_phantom_init(adapter, NETXEN_NIC_PEG_TUNE);
@@ -869,11 +872,15 @@ wait_init:
                goto err_out;
        }
 
+       NXWR32(adapter, NX_CRB_DEV_STATE, NX_DEV_READY);
+
        nx_update_dma_mask(adapter);
 
        netxen_check_options(adapter);
 
-       return 0;
+       adapter->need_fw_reset = 0;
+
+       /* fall through and release firmware */
 
 err_out:
        netxen_release_firmware(adapter);
@@ -941,8 +948,9 @@ netxen_nic_init_coalesce_defaults(struct netxen_adapter *adapter)
                NETXEN_DEFAULT_INTR_COALESCE_TX_PACKETS;
 }
 
+/* with rtnl_lock */
 static int
-netxen_nic_up(struct netxen_adapter *adapter, struct net_device *netdev)
+__netxen_nic_up(struct netxen_adapter *adapter, struct net_device *netdev)
 {
        int err;
 
@@ -956,7 +964,7 @@ netxen_nic_up(struct netxen_adapter *adapter, struct net_device *netdev)
                return err;
        }
        if (NX_IS_REVISION_P2(adapter->ahw.revision_id))
-               adapter->macaddr_set(adapter, netdev->dev_addr);
+               adapter->macaddr_set(adapter, adapter->mac_addr);
 
        adapter->set_multi(netdev);
        adapter->set_mtu(adapter, netdev->mtu);
@@ -983,14 +991,32 @@ netxen_nic_up(struct netxen_adapter *adapter, struct net_device *netdev)
        return 0;
 }
 
+/* Usage: During resume and firmware recovery module.*/
+
+static inline int
+netxen_nic_up(struct netxen_adapter *adapter, struct net_device *netdev)
+{
+       int err = 0;
+
+       rtnl_lock();
+       if (netif_running(netdev))
+               err = __netxen_nic_up(adapter, netdev);
+       rtnl_unlock();
+
+       return err;
+}
+
+/* with rtnl_lock */
 static void
-netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev)
+__netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev)
 {
        if (adapter->is_up != NETXEN_ADAPTER_UP_MAGIC)
                return;
 
-       clear_bit(__NX_DEV_UP, &adapter->state);
+       if (!test_and_clear_bit(__NX_DEV_UP, &adapter->state))
+               return;
 
+       smp_mb();
        spin_lock(&adapter->tx_clean_lock);
        netif_carrier_off(netdev);
        netif_tx_disable(netdev);
@@ -1009,6 +1035,17 @@ netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev)
        spin_unlock(&adapter->tx_clean_lock);
 }
 
+/* Usage: During suspend and firmware recovery module */
+
+static inline void
+netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev)
+{
+       rtnl_lock();
+       if (netif_running(netdev))
+               __netxen_nic_down(adapter, netdev);
+       rtnl_unlock();
+
+}
 
 static int
 netxen_nic_attach(struct netxen_adapter *adapter)
@@ -1117,14 +1154,14 @@ netxen_nic_reset_context(struct netxen_adapter *adapter)
                netif_device_detach(netdev);
 
                if (netif_running(netdev))
-                       netxen_nic_down(adapter, netdev);
+                       __netxen_nic_down(adapter, netdev);
 
                netxen_nic_detach(adapter);
 
                if (netif_running(netdev)) {
                        err = netxen_nic_attach(adapter);
                        if (!err)
-                               err = netxen_nic_up(adapter, netdev);
+                               err = __netxen_nic_up(adapter, netdev);
 
                        if (err)
                                goto done;
@@ -1181,7 +1218,7 @@ netxen_setup_netdev(struct netxen_adapter *adapter,
 
        netdev->irq = adapter->msix_entries[0].vector;
 
-       INIT_WORK(&adapter->tx_timeout_task, netxen_reset_task);
+       INIT_WORK(&adapter->tx_timeout_task, netxen_tx_timeout_task);
 
        if (netxen_read_mac_addr(adapter))
                dev_warn(&pdev->dev, "failed to read mac addr\n");
@@ -1207,16 +1244,10 @@ netxen_nic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        int pci_func_id = PCI_FUNC(pdev->devfn);
        uint8_t revision_id;
 
-       if (pdev->class != 0x020000) {
-               printk(KERN_DEBUG "NetXen function %d, class %x will not "
-                               "be enabled.\n",pci_func_id, pdev->class);
-               return -ENODEV;
-       }
-
        if (pdev->revision >= NX_P3_A0 && pdev->revision < NX_P3_B1) {
-               printk(KERN_WARNING "NetXen chip revisions between 0x%x-0x%x"
+               pr_warning("%s: chip revisions between 0x%x-0x%x"
                                "will not be enabled.\n",
-                               NX_P3_A0, NX_P3_B1);
+                               module_name(THIS_MODULE), NX_P3_A0, NX_P3_B1);
                return -ENODEV;
        }
 
@@ -1250,7 +1281,9 @@ netxen_nic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        revision_id = pdev->revision;
        adapter->ahw.revision_id = revision_id;
 
-       rwlock_init(&adapter->adapter_lock);
+       rwlock_init(&adapter->ahw.crb_lock);
+       spin_lock_init(&adapter->ahw.mem_lock);
+
        spin_lock_init(&adapter->tx_clean_lock);
        INIT_LIST_HEAD(&adapter->mac_list);
 
@@ -1280,7 +1313,7 @@ netxen_nic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 
        err = netxen_start_firmware(adapter);
        if (err)
-               goto err_out_iounmap;
+               goto err_out_decr_ref;
 
        /*
         * See if the firmware gave us a virtual-physical port mapping.
@@ -1315,6 +1348,8 @@ netxen_nic_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
                break;
        }
 
+       netxen_create_diag_entries(adapter);
+
        return 0;
 
 err_out_disable_msi:
@@ -1322,6 +1357,7 @@ err_out_disable_msi:
 
        netxen_free_dummy_dma(adapter);
 
+err_out_decr_ref:
        nx_decr_dev_ref_cnt(adapter);
 
 err_out_iounmap:
@@ -1367,6 +1403,8 @@ static void __devexit netxen_nic_remove(struct pci_dev *pdev)
 
        netxen_teardown_intr(adapter);
 
+       netxen_remove_diag_entries(adapter);
+
        netxen_cleanup_pci_map(adapter);
 
        netxen_release_firmware(adapter);
@@ -1410,14 +1448,14 @@ static int __netxen_nic_shutdown(struct pci_dev *pdev)
                pci_enable_wake(pdev, PCI_D3hot, 1);
        }
 
-       pci_disable_device(pdev);
-
        return 0;
 }
 static void netxen_nic_shutdown(struct pci_dev *pdev)
 {
        if (__netxen_nic_shutdown(pdev))
                return;
+
+       pci_disable_device(pdev);
 }
 #ifdef CONFIG_PM
 static int
@@ -1430,6 +1468,8 @@ netxen_nic_suspend(struct pci_dev *pdev, pm_message_t state)
                return retval;
 
        pci_set_power_state(pdev, pci_choose_state(pdev, state));
+
+       pci_disable_device(pdev);
        return 0;
 }
 
@@ -1440,14 +1480,16 @@ netxen_nic_resume(struct pci_dev *pdev)
        struct net_device *netdev = adapter->netdev;
        int err;
 
-       pci_set_power_state(pdev, PCI_D0);
-       pci_restore_state(pdev);
-
        err = pci_enable_device(pdev);
        if (err)
                return err;
 
-       adapter->curr_window = 255;
+       pci_set_power_state(pdev, PCI_D0);
+       pci_set_master(pdev);
+       pci_restore_state(pdev);
+
+       adapter->ahw.crb_win = -1;
+       adapter->ahw.ocm_win = -1;
 
        err = netxen_start_firmware(adapter);
        if (err) {
@@ -1464,12 +1506,12 @@ netxen_nic_resume(struct pci_dev *pdev)
                if (err)
                        goto err_out_detach;
 
-               netif_device_attach(netdev);
-
                netxen_config_indev_addr(netdev, NETDEV_UP);
        }
 
+       netif_device_attach(netdev);
        netxen_schedule_work(adapter, netxen_fw_poll_work, FW_POLL_DELAY);
+       return 0;
 
 err_out_detach:
        netxen_nic_detach(adapter);
@@ -1491,7 +1533,7 @@ static int netxen_nic_open(struct net_device *netdev)
        if (err)
                return err;
 
-       err = netxen_nic_up(adapter, netdev);
+       err = __netxen_nic_up(adapter, netdev);
        if (err)
                goto err_out;
 
@@ -1511,7 +1553,7 @@ static int netxen_nic_close(struct net_device *netdev)
 {
        struct netxen_adapter *adapter = netdev_priv(netdev);
 
-       netxen_nic_down(adapter, netdev);
+       __netxen_nic_down(adapter, netdev);
        return 0;
 }
 
@@ -1714,7 +1756,7 @@ netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
        /* 4 fragments per cmd des */
        no_of_desc = (frag_count + 3) >> 2;
 
-       if (unlikely(no_of_desc + 2) > netxen_tx_avail(tx_ring)) {
+       if (unlikely(no_of_desc + 2 > netxen_tx_avail(tx_ring))) {
                netif_stop_queue(netdev);
                return NETDEV_TX_BUSY;
        }
@@ -1880,7 +1922,7 @@ static void netxen_tx_timeout(struct net_device *netdev)
        schedule_work(&adapter->tx_timeout_task);
 }
 
-static void netxen_reset_task(struct work_struct *work)
+static void netxen_tx_timeout_task(struct work_struct *work)
 {
        struct netxen_adapter *adapter =
                container_of(work, struct netxen_adapter, tx_timeout_task);
@@ -1888,21 +1930,43 @@ static void netxen_reset_task(struct work_struct *work)
        if (!netif_running(adapter->netdev))
                return;
 
-       if (test_bit(__NX_RESETTING, &adapter->state))
+       if (test_and_set_bit(__NX_RESETTING, &adapter->state))
                return;
 
-       netxen_napi_disable(adapter);
+       if (++adapter->tx_timeo_cnt >= NX_MAX_TX_TIMEOUTS)
+               goto request_reset;
 
-       adapter->netdev->trans_start = jiffies;
+       if (NX_IS_REVISION_P2(adapter->ahw.revision_id)) {
+               /* try to scrub interrupt */
+               netxen_napi_disable(adapter);
 
-       netxen_napi_enable(adapter);
-       netif_wake_queue(adapter->netdev);
+               adapter->netdev->trans_start = jiffies;
+
+               netxen_napi_enable(adapter);
+
+               netif_wake_queue(adapter->netdev);
+
+               clear_bit(__NX_RESETTING, &adapter->state);
+
+       } else {
+               clear_bit(__NX_RESETTING, &adapter->state);
+               if (!netxen_nic_reset_context(adapter)) {
+                       adapter->netdev->trans_start = jiffies;
+                       return;
+               }
+
+               /* context reset failed, fall through for fw reset */
+       }
+
+request_reset:
+       adapter->need_fw_reset = 1;
+       clear_bit(__NX_RESETTING, &adapter->state);
 }
 
 struct net_device_stats *netxen_nic_get_stats(struct net_device *netdev)
 {
        struct netxen_adapter *adapter = netdev_priv(netdev);
-       struct net_device_stats *stats = &adapter->net_stats;
+       struct net_device_stats *stats = &netdev->stats;
 
        memset(stats, 0, sizeof(*stats));
 
@@ -1995,7 +2059,7 @@ static int netxen_nic_poll(struct napi_struct *napi, int budget)
 
        if ((work_done < budget) && tx_complete) {
                napi_complete(&sds_ring->napi);
-               if (netif_running(adapter->netdev))
+               if (test_bit(__NX_DEV_UP, &adapter->state))
                        netxen_nic_enable_int(sds_ring);
        }
 
@@ -2046,6 +2110,22 @@ nx_decr_dev_ref_cnt(struct netxen_adapter *adapter)
        return count;
 }
 
+static void
+nx_dev_request_reset(struct netxen_adapter *adapter)
+{
+       u32 state;
+
+       if (netxen_api_lock(adapter))
+               return;
+
+       state = NXRD32(adapter, NX_CRB_DEV_STATE);
+
+       if (state != NX_DEV_INITALIZING)
+               NXWR32(adapter, NX_CRB_DEV_STATE, NX_DEV_NEED_RESET);
+
+       netxen_api_unlock(adapter);
+}
+
 static int
 netxen_can_start_firmware(struct netxen_adapter *adapter)
 {
@@ -2131,9 +2211,11 @@ netxen_fwinit_work(struct work_struct *work)
        switch (dev_state) {
        case NX_DEV_COLD:
        case NX_DEV_READY:
-               netxen_start_firmware(adapter);
-               netxen_schedule_work(adapter, netxen_attach_work, 0);
-               return;
+               if (!netxen_start_firmware(adapter)) {
+                       netxen_schedule_work(adapter, netxen_attach_work, 0);
+                       return;
+               }
+               break;
 
        case NX_DEV_INITALIZING:
                if (++adapter->fw_wait_cnt < FW_POLL_THRESH) {
@@ -2141,14 +2223,13 @@ netxen_fwinit_work(struct work_struct *work)
                                        netxen_fwinit_work, 2 * FW_POLL_DELAY);
                        return;
                }
-               break;
 
        case NX_DEV_FAILED:
        default:
+               nx_incr_dev_ref_cnt(adapter);
                break;
        }
 
-       nx_incr_dev_ref_cnt(adapter);
        clear_bit(__NX_RESETTING, &adapter->state);
 }
 
@@ -2163,25 +2244,29 @@ netxen_detach_work(struct work_struct *work)
 
        netif_device_detach(netdev);
 
-       if (netif_running(netdev))
-               netxen_nic_down(adapter, netdev);
+       netxen_nic_down(adapter, netdev);
 
        netxen_nic_detach(adapter);
 
        status = NXRD32(adapter, NETXEN_PEG_HALT_STATUS1);
 
-       ref_cnt = nx_decr_dev_ref_cnt(adapter);
-
        if (status & NX_RCODE_FATAL_ERROR)
-               return;
+               goto err_ret;
 
        if (adapter->temp == NX_TEMP_PANIC)
-               return;
+               goto err_ret;
+
+       ref_cnt = nx_decr_dev_ref_cnt(adapter);
 
        delay = (ref_cnt == 0) ? 0 : (2 * FW_POLL_DELAY);
 
        adapter->fw_wait_cnt = 0;
        netxen_schedule_work(adapter, netxen_fwinit_work, delay);
+
+       return;
+
+err_ret:
+       clear_bit(__NX_RESETTING, &adapter->state);
 }
 
 static int
@@ -2193,6 +2278,11 @@ netxen_check_health(struct netxen_adapter *adapter)
        if (netxen_nic_check_temp(adapter))
                goto detach;
 
+       if (adapter->need_fw_reset) {
+               nx_dev_request_reset(adapter);
+               goto detach;
+       }
+
        state = NXRD32(adapter, NX_CRB_DEV_STATE);
        if (state == NX_DEV_NEED_RESET)
                goto detach;
@@ -2215,7 +2305,8 @@ netxen_check_health(struct netxen_adapter *adapter)
        dev_info(&netdev->dev, "firmware hang detected\n");
 
 detach:
-       if (!test_and_set_bit(__NX_RESETTING, &adapter->state))
+       if ((auto_fw_reset == AUTO_FW_RESET_ENABLED) &&
+                       !test_and_set_bit(__NX_RESETTING, &adapter->state))
                netxen_schedule_work(adapter, netxen_detach_work, 0);
        return 1;
 }
@@ -2293,6 +2384,161 @@ static struct device_attribute dev_attr_bridged_mode = {
        .store = netxen_store_bridged_mode,
 };
 
+static ssize_t
+netxen_store_diag_mode(struct device *dev,
+               struct device_attribute *attr, const char *buf, size_t len)
+{
+       struct netxen_adapter *adapter = dev_get_drvdata(dev);
+       unsigned long new;
+
+       if (strict_strtoul(buf, 2, &new))
+               return -EINVAL;
+
+       if (!!new != !!(adapter->flags & NETXEN_NIC_DIAG_ENABLED))
+               adapter->flags ^= NETXEN_NIC_DIAG_ENABLED;
+
+       return len;
+}
+
+static ssize_t
+netxen_show_diag_mode(struct device *dev,
+               struct device_attribute *attr, char *buf)
+{
+       struct netxen_adapter *adapter = dev_get_drvdata(dev);
+
+       return sprintf(buf, "%d\n",
+                       !!(adapter->flags & NETXEN_NIC_DIAG_ENABLED));
+}
+
+static struct device_attribute dev_attr_diag_mode = {
+       .attr = {.name = "diag_mode", .mode = (S_IRUGO | S_IWUSR)},
+       .show = netxen_show_diag_mode,
+       .store = netxen_store_diag_mode,
+};
+
+static int
+netxen_sysfs_validate_crb(struct netxen_adapter *adapter,
+               loff_t offset, size_t size)
+{
+       if (!(adapter->flags & NETXEN_NIC_DIAG_ENABLED))
+               return -EIO;
+
+       if ((size != 4) || (offset & 0x3))
+               return  -EINVAL;
+
+       if (offset < NETXEN_PCI_CRBSPACE)
+               return -EINVAL;
+
+       return 0;
+}
+
+static ssize_t
+netxen_sysfs_read_crb(struct kobject *kobj, struct bin_attribute *attr,
+               char *buf, loff_t offset, size_t size)
+{
+       struct device *dev = container_of(kobj, struct device, kobj);
+       struct netxen_adapter *adapter = dev_get_drvdata(dev);
+       u32 data;
+       int ret;
+
+       ret = netxen_sysfs_validate_crb(adapter, offset, size);
+       if (ret != 0)
+               return ret;
+
+       data = NXRD32(adapter, offset);
+       memcpy(buf, &data, size);
+       return size;
+}
+
+static ssize_t
+netxen_sysfs_write_crb(struct kobject *kobj, struct bin_attribute *attr,
+               char *buf, loff_t offset, size_t size)
+{
+       struct device *dev = container_of(kobj, struct device, kobj);
+       struct netxen_adapter *adapter = dev_get_drvdata(dev);
+       u32 data;
+       int ret;
+
+       ret = netxen_sysfs_validate_crb(adapter, offset, size);
+       if (ret != 0)
+               return ret;
+
+       memcpy(&data, buf, size);
+       NXWR32(adapter, offset, data);
+       return size;
+}
+
+static int
+netxen_sysfs_validate_mem(struct netxen_adapter *adapter,
+               loff_t offset, size_t size)
+{
+       if (!(adapter->flags & NETXEN_NIC_DIAG_ENABLED))
+               return -EIO;
+
+       if ((size != 8) || (offset & 0x7))
+               return  -EIO;
+
+       return 0;
+}
+
+static ssize_t
+netxen_sysfs_read_mem(struct kobject *kobj, struct bin_attribute *attr,
+               char *buf, loff_t offset, size_t size)
+{
+       struct device *dev = container_of(kobj, struct device, kobj);
+       struct netxen_adapter *adapter = dev_get_drvdata(dev);
+       u64 data;
+       int ret;
+
+       ret = netxen_sysfs_validate_mem(adapter, offset, size);
+       if (ret != 0)
+               return ret;
+
+       if (adapter->pci_mem_read(adapter, offset, &data))
+               return -EIO;
+
+       memcpy(buf, &data, size);
+
+       return size;
+}
+
+ssize_t netxen_sysfs_write_mem(struct kobject *kobj,
+               struct bin_attribute *attr, char *buf,
+               loff_t offset, size_t size)
+{
+       struct device *dev = container_of(kobj, struct device, kobj);
+       struct netxen_adapter *adapter = dev_get_drvdata(dev);
+       u64 data;
+       int ret;
+
+       ret = netxen_sysfs_validate_mem(adapter, offset, size);
+       if (ret != 0)
+               return ret;
+
+       memcpy(&data, buf, size);
+
+       if (adapter->pci_mem_write(adapter, offset, data))
+               return -EIO;
+
+       return size;
+}
+
+
+static struct bin_attribute bin_attr_crb = {
+       .attr = {.name = "crb", .mode = (S_IRUGO | S_IWUSR)},
+       .size = 0,
+       .read = netxen_sysfs_read_crb,
+       .write = netxen_sysfs_write_crb,
+};
+
+static struct bin_attribute bin_attr_mem = {
+       .attr = {.name = "mem", .mode = (S_IRUGO | S_IWUSR)},
+       .size = 0,
+       .read = netxen_sysfs_read_mem,
+       .write = netxen_sysfs_write_mem,
+};
+
+
 static void
 netxen_create_sysfs_entries(struct netxen_adapter *adapter)
 {
@@ -2318,6 +2564,33 @@ netxen_remove_sysfs_entries(struct netxen_adapter *adapter)
                device_remove_file(dev, &dev_attr_bridged_mode);
 }
 
+static void
+netxen_create_diag_entries(struct netxen_adapter *adapter)
+{
+       struct pci_dev *pdev = adapter->pdev;
+       struct device *dev;
+
+       dev = &pdev->dev;
+       if (device_create_file(dev, &dev_attr_diag_mode))
+               dev_info(dev, "failed to create diag_mode sysfs entry\n");
+       if (device_create_bin_file(dev, &bin_attr_crb))
+               dev_info(dev, "failed to create crb sysfs entry\n");
+       if (device_create_bin_file(dev, &bin_attr_mem))
+               dev_info(dev, "failed to create mem sysfs entry\n");
+}
+
+
+static void
+netxen_remove_diag_entries(struct netxen_adapter *adapter)
+{
+       struct pci_dev *pdev = adapter->pdev;
+       struct device *dev = &pdev->dev;
+
+       device_remove_file(dev, &dev_attr_diag_mode);
+       device_remove_bin_file(dev, &bin_attr_crb);
+       device_remove_bin_file(dev, &bin_attr_mem);
+}
+
 #ifdef CONFIG_INET
 
 #define is_netxen_netdev(dev) (dev->netdev_ops == &netxen_netdev_ops)
@@ -2476,7 +2749,6 @@ static int __init netxen_init_module(void)
        register_netdevice_notifier(&netxen_netdev_cb);
        register_inetaddr_notifier(&netxen_inetaddr_cb);
 #endif
-
        return pci_register_driver(&netxen_driver);
 }