Merge branch 'e1000-fixes' of master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/netdev-2.6
* 'e1000-fixes' of master.kernel.org:/pub/scm/linux/kernel/git/jgarzik/netdev-2.6: e1000: FIX: Stop raw interrupts disabled nag from RT e1000: FIX: firmware handover bits e1000: FIX: be ready for incoming irq at pci_request_irq
This commit is contained in:
@@ -409,25 +409,21 @@ e1000_release_hw_control(struct e1000_adapter *adapter)
|
|||||||
{
|
{
|
||||||
uint32_t ctrl_ext;
|
uint32_t ctrl_ext;
|
||||||
uint32_t swsm;
|
uint32_t swsm;
|
||||||
uint32_t extcnf;
|
|
||||||
|
|
||||||
/* Let firmware taken over control of h/w */
|
/* Let firmware taken over control of h/w */
|
||||||
switch (adapter->hw.mac_type) {
|
switch (adapter->hw.mac_type) {
|
||||||
case e1000_82571:
|
|
||||||
case e1000_82572:
|
|
||||||
case e1000_80003es2lan:
|
|
||||||
ctrl_ext = E1000_READ_REG(&adapter->hw, CTRL_EXT);
|
|
||||||
E1000_WRITE_REG(&adapter->hw, CTRL_EXT,
|
|
||||||
ctrl_ext & ~E1000_CTRL_EXT_DRV_LOAD);
|
|
||||||
break;
|
|
||||||
case e1000_82573:
|
case e1000_82573:
|
||||||
swsm = E1000_READ_REG(&adapter->hw, SWSM);
|
swsm = E1000_READ_REG(&adapter->hw, SWSM);
|
||||||
E1000_WRITE_REG(&adapter->hw, SWSM,
|
E1000_WRITE_REG(&adapter->hw, SWSM,
|
||||||
swsm & ~E1000_SWSM_DRV_LOAD);
|
swsm & ~E1000_SWSM_DRV_LOAD);
|
||||||
|
break;
|
||||||
|
case e1000_82571:
|
||||||
|
case e1000_82572:
|
||||||
|
case e1000_80003es2lan:
|
||||||
case e1000_ich8lan:
|
case e1000_ich8lan:
|
||||||
extcnf = E1000_READ_REG(&adapter->hw, CTRL_EXT);
|
ctrl_ext = E1000_READ_REG(&adapter->hw, CTRL_EXT);
|
||||||
E1000_WRITE_REG(&adapter->hw, CTRL_EXT,
|
E1000_WRITE_REG(&adapter->hw, CTRL_EXT,
|
||||||
extcnf & ~E1000_CTRL_EXT_DRV_LOAD);
|
ctrl_ext & ~E1000_CTRL_EXT_DRV_LOAD);
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
break;
|
break;
|
||||||
@@ -450,26 +446,21 @@ e1000_get_hw_control(struct e1000_adapter *adapter)
|
|||||||
{
|
{
|
||||||
uint32_t ctrl_ext;
|
uint32_t ctrl_ext;
|
||||||
uint32_t swsm;
|
uint32_t swsm;
|
||||||
uint32_t extcnf;
|
|
||||||
|
|
||||||
/* Let firmware know the driver has taken over */
|
/* Let firmware know the driver has taken over */
|
||||||
switch (adapter->hw.mac_type) {
|
switch (adapter->hw.mac_type) {
|
||||||
case e1000_82571:
|
|
||||||
case e1000_82572:
|
|
||||||
case e1000_80003es2lan:
|
|
||||||
ctrl_ext = E1000_READ_REG(&adapter->hw, CTRL_EXT);
|
|
||||||
E1000_WRITE_REG(&adapter->hw, CTRL_EXT,
|
|
||||||
ctrl_ext | E1000_CTRL_EXT_DRV_LOAD);
|
|
||||||
break;
|
|
||||||
case e1000_82573:
|
case e1000_82573:
|
||||||
swsm = E1000_READ_REG(&adapter->hw, SWSM);
|
swsm = E1000_READ_REG(&adapter->hw, SWSM);
|
||||||
E1000_WRITE_REG(&adapter->hw, SWSM,
|
E1000_WRITE_REG(&adapter->hw, SWSM,
|
||||||
swsm | E1000_SWSM_DRV_LOAD);
|
swsm | E1000_SWSM_DRV_LOAD);
|
||||||
break;
|
break;
|
||||||
|
case e1000_82571:
|
||||||
|
case e1000_82572:
|
||||||
|
case e1000_80003es2lan:
|
||||||
case e1000_ich8lan:
|
case e1000_ich8lan:
|
||||||
extcnf = E1000_READ_REG(&adapter->hw, EXTCNF_CTRL);
|
ctrl_ext = E1000_READ_REG(&adapter->hw, CTRL_EXT);
|
||||||
E1000_WRITE_REG(&adapter->hw, EXTCNF_CTRL,
|
E1000_WRITE_REG(&adapter->hw, CTRL_EXT,
|
||||||
extcnf | E1000_EXTCNF_CTRL_SWFLAG);
|
ctrl_ext | E1000_CTRL_EXT_DRV_LOAD);
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
break;
|
break;
|
||||||
@@ -522,14 +513,15 @@ e1000_release_manageability(struct e1000_adapter *adapter)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
int
|
/**
|
||||||
e1000_up(struct e1000_adapter *adapter)
|
* e1000_configure - configure the hardware for RX and TX
|
||||||
|
* @adapter = private board structure
|
||||||
|
**/
|
||||||
|
static void e1000_configure(struct e1000_adapter *adapter)
|
||||||
{
|
{
|
||||||
struct net_device *netdev = adapter->netdev;
|
struct net_device *netdev = adapter->netdev;
|
||||||
int i;
|
int i;
|
||||||
|
|
||||||
/* hardware has been reset, we need to reload some things */
|
|
||||||
|
|
||||||
e1000_set_multi(netdev);
|
e1000_set_multi(netdev);
|
||||||
|
|
||||||
e1000_restore_vlan(adapter);
|
e1000_restore_vlan(adapter);
|
||||||
@@ -548,14 +540,20 @@ e1000_up(struct e1000_adapter *adapter)
|
|||||||
}
|
}
|
||||||
|
|
||||||
adapter->tx_queue_len = netdev->tx_queue_len;
|
adapter->tx_queue_len = netdev->tx_queue_len;
|
||||||
|
}
|
||||||
|
|
||||||
#ifdef CONFIG_E1000_NAPI
|
int e1000_up(struct e1000_adapter *adapter)
|
||||||
netif_poll_enable(netdev);
|
{
|
||||||
#endif
|
/* hardware has been reset, we need to reload some things */
|
||||||
e1000_irq_enable(adapter);
|
e1000_configure(adapter);
|
||||||
|
|
||||||
clear_bit(__E1000_DOWN, &adapter->flags);
|
clear_bit(__E1000_DOWN, &adapter->flags);
|
||||||
|
|
||||||
|
#ifdef CONFIG_E1000_NAPI
|
||||||
|
netif_poll_enable(adapter->netdev);
|
||||||
|
#endif
|
||||||
|
e1000_irq_enable(adapter);
|
||||||
|
|
||||||
/* fire a link change interrupt to start the watchdog */
|
/* fire a link change interrupt to start the watchdog */
|
||||||
E1000_WRITE_REG(&adapter->hw, ICS, E1000_ICS_LSC);
|
E1000_WRITE_REG(&adapter->hw, ICS, E1000_ICS_LSC);
|
||||||
return 0;
|
return 0;
|
||||||
@@ -640,15 +638,15 @@ e1000_down(struct e1000_adapter *adapter)
|
|||||||
* reschedule our watchdog timer */
|
* reschedule our watchdog timer */
|
||||||
set_bit(__E1000_DOWN, &adapter->flags);
|
set_bit(__E1000_DOWN, &adapter->flags);
|
||||||
|
|
||||||
|
#ifdef CONFIG_E1000_NAPI
|
||||||
|
netif_poll_disable(netdev);
|
||||||
|
#endif
|
||||||
e1000_irq_disable(adapter);
|
e1000_irq_disable(adapter);
|
||||||
|
|
||||||
del_timer_sync(&adapter->tx_fifo_stall_timer);
|
del_timer_sync(&adapter->tx_fifo_stall_timer);
|
||||||
del_timer_sync(&adapter->watchdog_timer);
|
del_timer_sync(&adapter->watchdog_timer);
|
||||||
del_timer_sync(&adapter->phy_info_timer);
|
del_timer_sync(&adapter->phy_info_timer);
|
||||||
|
|
||||||
#ifdef CONFIG_E1000_NAPI
|
|
||||||
netif_poll_disable(netdev);
|
|
||||||
#endif
|
|
||||||
netdev->tx_queue_len = adapter->tx_queue_len;
|
netdev->tx_queue_len = adapter->tx_queue_len;
|
||||||
adapter->link_speed = 0;
|
adapter->link_speed = 0;
|
||||||
adapter->link_duplex = 0;
|
adapter->link_duplex = 0;
|
||||||
@@ -1410,21 +1408,17 @@ e1000_open(struct net_device *netdev)
|
|||||||
return -EBUSY;
|
return -EBUSY;
|
||||||
|
|
||||||
/* allocate transmit descriptors */
|
/* allocate transmit descriptors */
|
||||||
if ((err = e1000_setup_all_tx_resources(adapter)))
|
err = e1000_setup_all_tx_resources(adapter);
|
||||||
|
if (err)
|
||||||
goto err_setup_tx;
|
goto err_setup_tx;
|
||||||
|
|
||||||
/* allocate receive descriptors */
|
/* allocate receive descriptors */
|
||||||
if ((err = e1000_setup_all_rx_resources(adapter)))
|
err = e1000_setup_all_rx_resources(adapter);
|
||||||
goto err_setup_rx;
|
|
||||||
|
|
||||||
err = e1000_request_irq(adapter);
|
|
||||||
if (err)
|
if (err)
|
||||||
goto err_req_irq;
|
goto err_setup_rx;
|
||||||
|
|
||||||
e1000_power_up_phy(adapter);
|
e1000_power_up_phy(adapter);
|
||||||
|
|
||||||
if ((err = e1000_up(adapter)))
|
|
||||||
goto err_up;
|
|
||||||
adapter->mng_vlan_id = E1000_MNG_VLAN_NONE;
|
adapter->mng_vlan_id = E1000_MNG_VLAN_NONE;
|
||||||
if ((adapter->hw.mng_cookie.status &
|
if ((adapter->hw.mng_cookie.status &
|
||||||
E1000_MNG_DHCP_COOKIE_STATUS_VLAN_SUPPORT)) {
|
E1000_MNG_DHCP_COOKIE_STATUS_VLAN_SUPPORT)) {
|
||||||
@@ -1437,12 +1431,33 @@ e1000_open(struct net_device *netdev)
|
|||||||
e1000_check_mng_mode(&adapter->hw))
|
e1000_check_mng_mode(&adapter->hw))
|
||||||
e1000_get_hw_control(adapter);
|
e1000_get_hw_control(adapter);
|
||||||
|
|
||||||
|
/* before we allocate an interrupt, we must be ready to handle it.
|
||||||
|
* Setting DEBUG_SHIRQ in the kernel makes it fire an interrupt
|
||||||
|
* as soon as we call pci_request_irq, so we have to setup our
|
||||||
|
* clean_rx handler before we do so. */
|
||||||
|
e1000_configure(adapter);
|
||||||
|
|
||||||
|
err = e1000_request_irq(adapter);
|
||||||
|
if (err)
|
||||||
|
goto err_req_irq;
|
||||||
|
|
||||||
|
/* From here on the code is the same as e1000_up() */
|
||||||
|
clear_bit(__E1000_DOWN, &adapter->flags);
|
||||||
|
|
||||||
|
#ifdef CONFIG_E1000_NAPI
|
||||||
|
netif_poll_enable(netdev);
|
||||||
|
#endif
|
||||||
|
|
||||||
|
e1000_irq_enable(adapter);
|
||||||
|
|
||||||
|
/* fire a link status change interrupt to start the watchdog */
|
||||||
|
E1000_WRITE_REG(&adapter->hw, ICS, E1000_ICS_LSC);
|
||||||
|
|
||||||
return E1000_SUCCESS;
|
return E1000_SUCCESS;
|
||||||
|
|
||||||
err_up:
|
|
||||||
e1000_power_down_phy(adapter);
|
|
||||||
e1000_free_irq(adapter);
|
|
||||||
err_req_irq:
|
err_req_irq:
|
||||||
|
e1000_release_hw_control(adapter);
|
||||||
|
e1000_power_down_phy(adapter);
|
||||||
e1000_free_all_rx_resources(adapter);
|
e1000_free_all_rx_resources(adapter);
|
||||||
err_setup_rx:
|
err_setup_rx:
|
||||||
e1000_free_all_tx_resources(adapter);
|
e1000_free_all_tx_resources(adapter);
|
||||||
@@ -3361,12 +3376,9 @@ e1000_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
|
|||||||
(adapter->hw.mac_type == e1000_82573))
|
(adapter->hw.mac_type == e1000_82573))
|
||||||
e1000_transfer_dhcp_info(adapter, skb);
|
e1000_transfer_dhcp_info(adapter, skb);
|
||||||
|
|
||||||
local_irq_save(flags);
|
if (!spin_trylock_irqsave(&tx_ring->tx_lock, flags))
|
||||||
if (!spin_trylock(&tx_ring->tx_lock)) {
|
|
||||||
/* Collision - tell upper layer to requeue */
|
/* Collision - tell upper layer to requeue */
|
||||||
local_irq_restore(flags);
|
|
||||||
return NETDEV_TX_LOCKED;
|
return NETDEV_TX_LOCKED;
|
||||||
}
|
|
||||||
|
|
||||||
/* need: count + 2 desc gap to keep tail from touching
|
/* need: count + 2 desc gap to keep tail from touching
|
||||||
* head, otherwise try next time */
|
* head, otherwise try next time */
|
||||||
|
Reference in New Issue
Block a user