mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Alex Scheele" <alex@ccbeheer.nl>
To: <linux-kernel@vger.kernel.org>
Cc: <rhl@traceroute.dk>
Subject: Re: e1000 (on-board CSA) lockup with 2.6.3 on ifconfig
Date: Thu, 19 Feb 2004 12:38:35 +0100	[thread overview]
Message-ID: <000001c3f6dc$ed8ee9b0$3364a8c0@lapas> (raw)

Hi,

The lockup is probably caused by a patch that made it in the tree 
which adds an interrupt disable/enable to keep interrupt assertion 
state synced between 82547 and APIC. This patch was in the tree before 
but was reverted a while back because it locked up 82547 CSA-based LOMs. 
Apparently it made it back in. Below I have inserted a patch (against 2.6.3)
which 
removes it again. Feel free to test if it stops the lockups.


Regards,

Alex Scheele



--- e1000_main.c        2004-02-18 04:57:26.000000000 +0100
+++ e1000_main_new.c    2004-02-19 12:28:15.000000000 +0100
@@ -2124,26 +2124,10 @@
                __netif_rx_schedule(netdev);
        }
 #else
-        /* Writing IMC and IMS is needed for 82547.
-          Due to Hub Link bus being occupied, an interrupt
-          de-assertion message is not able to be sent.
-          When an interrupt assertion message is generated later,
-          two messages are re-ordered and sent out.
-          That causes APIC to think 82547 is in de-assertion
-          state, while 82547 is in assertion state, resulting
-          in dead lock. Writing IMC forces 82547 into
-          de-assertion state.
-        */
-       if(hw->mac_type == e1000_82547 || hw->mac_type == e1000_82547_rev_2)
-               e1000_irq_disable(adapter);
-
        for(i = 0; i < E1000_MAX_INTR; i++)
                if(!e1000_clean_rx_irq(adapter) &
                   !e1000_clean_tx_irq(adapter))
                        break;
-
-       if(hw->mac_type == e1000_82547 || hw->mac_type == e1000_82547_rev_2)
-               e1000_irq_enable(adapter);
 #endif

        return IRQ_HANDLED;



             reply	other threads:[~2004-02-19 11:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-02-19 11:38 Alex Scheele [this message]
2004-02-19 18:04 ` Rene H. Larsen
  -- strict thread matches above, loose matches on Subject: below --
2004-02-18 19:23 Rene H. Larsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='000001c3f6dc$ed8ee9b0$3364a8c0@lapas' \
    --to=alex@ccbeheer.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rhl@traceroute.dk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®