From: Ingo Molnar <mingo@elte.hu>
To: "Waskiewicz Jr, Peter P" <peter.p.waskiewicz.jr@intel.com>
Cc: "Allan, Bruce W" <bruce.w.allan@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com>,
"Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
"e1000-devel@lists.sourceforge.net"
<e1000-devel@lists.sourceforge.net>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"Rafael J. Wysocki" <rjw@sisk.pl>
Subject: Re: igb regression (interface hang) with latest -git
Date: Wed, 21 Jan 2009 22:20:05 +0100 [thread overview]
Message-ID: <20090121212005.GA26992@elte.hu> (raw)
In-Reply-To: <20090121182504.GA14259@elte.hu>
* Ingo Molnar <mingo@elte.hu> wrote:
> > One other piece of info (in addition to what Bruce requested) that
> > would be useful is after you've done ifconfig eht0 up, cat
> > /proc/interrupts | grep eth0 and send that output (I want to see your
> > MSI-X configuration).
>
> here it is:
>
> 79: 35 0 9836 7451 1197 1360 766 0 2196 995 770 0 2051 863 4714 1879 PCI-MSI-edge eth0-tx-0
> 80: 2335 0 2718 3651 5024 9850 533 0 1405 1095 0 0 4421 1229 0 2002 PCI-MSI-edge eth0-tx-1
> 81: 33 0 0 5478 1125 6585 4865 6306 0 0 0 714 3757 0 2662 1020 PCI-MSI-edge eth0-tx-2
> 82: 21 1428 3136 3783 4249 6833 0 0 4175 0 0 1465 1940 1780 6286 0 PCI-MSI-edge eth0-tx-3
> 83: 40 0 0 4788 0 1486 0 7185 3998 8637 1409 6785 0 0 0 10060 PCI-MSI-edge eth0-rx-0
> 84: 70 1441 2606 1342 0 1678 4301 1839 0 6507 6910 16470 0 1484 1004 0 PCI-MSI-edge eth0-rx-1
> 85: 40 1368 0 3560 5022 2040 5 1719 0 2037 956 40 0 17264 6456 2833 PCI-MSI-edge eth0-rx-2
> 86: 44 414 5 1164 30 4513 4595 1542 0 10033 1151 0 4961 4203 0 10703 PCI-MSI-edge eth0-rx-3
> 87: 1 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 PCI-MSI-edge eth0
btw., i turned off CONFIG_PCI_MSI and that worked around the interface
hang - the box has 40 minutes uptime now and still no hang. (it would hang
within 5 minutes previously)
can test the revert of any of these commits:
e42e4ba: igb: fix anoying type mismatch warning on rx/tx queue sizing
8d25332: igb: Fix build warning when DCA is disabled.
26bc19e: igb: re-order queues to support cleaner use of ivar on 82576
0e014cb: igb: defeature tx head writeback
678c610: drivers/net/igb: remove dead code (function 'igb_read_pci_cfg')
908a7a1: net: Remove unused netdev arg from some NAPI interfaces.
ea943d4: igb: fixup AER with proper error handling
5e8427e: igb: Correctly determine pci-e function number in virtual environment
b4557be: igb: update handling of RCTL for smaller buffer sizes
cb7b48f: igb/e1000e: Naming interrupt vectors
40a914f: igb: Add support for pci-e Advanced Error Reporting
527d47c: igb: link up/down messages must follow a specific format
5b9ab2e: Merge branch 'master' of master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6
69d728b: igb: loopback bits not correctly cleared from RCTL register
9b07f3d: igb: remove unneeded bit refrence when enabling jumbo frames
f5f4cf0: igb: do not use phy ops in ethtool test cleanup for non-copper parts
0082982: netdev: add more functions to netdevice ops
68fd991: igb: Fix tx/rx_ring_count parameters for igb on suspend/resume/ring resize
b2d5653: igb: simplify swap in clean_rx_irq if using packet split
2e5c692: igb: convert to net_device_ops
198d6ba: Merge branch 'master' of master.kernel.org:/pub/scm/linux/kernel/git/davem/net-2.6
4cf1653: netdevice: safe convert to netdev_priv() #part-2
babcda7: drivers/net: Kill now superfluous ->last_rx stores.
7c510e4: net: convert more to %pM
i had a quick look at all of them but none seem to have a direct
connection to MSI-X interrupt generation.
There's a couple of ones that could have side effects:
Maybe this one:
26bc19e: igb: re-order queues to support cleaner use of ivar on 82576
due to its sheer commit size and due to its impact on the hw programming.
or:
0e014cb: igb: defeature tx head writeback
Might have some side-effect on tx-completion IRQs and might tickle
firmware bugs?
Or:
5e8427e: igb: Correctly determine pci-e function number in virtual environment
while it should have no impact on a native kernel, it does change the PCI
config space access sequences.
Or maybe:
69d728b: igb: loopback bits not correctly cleared from RCTL register
as this impacts the PCI programming too.
But ... i dont really know this code so i'm guessing around.
Ingo
next prev parent reply other threads:[~2009-01-21 21:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-21 10:28 e1000e regression (interface hang) with latest -git Ingo Molnar
2009-01-21 17:35 ` Allan, Bruce W
2009-01-21 17:51 ` Ingo Molnar
2009-01-21 18:03 ` Ingo Molnar
2009-01-21 18:12 ` Waskiewicz Jr, Peter P
2009-01-21 18:25 ` Ingo Molnar
2009-01-21 21:20 ` Ingo Molnar [this message]
2009-01-21 21:34 ` igb " Rafael J. Wysocki
2009-01-21 21:48 ` Ingo Molnar
2009-01-21 22:22 ` Rafael J. Wysocki
2009-01-21 22:51 ` Duyck, Alexander H
2009-01-22 0:03 ` Stephen Hemminger
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=20090121212005.GA26992@elte.hu \
--to=mingo@elte.hu \
--cc=bruce.w.allan@intel.com \
--cc=e1000-devel@lists.sourceforge.net \
--cc=jeffrey.t.kirsher@intel.com \
--cc=jesse.brandeburg@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=peter.p.waskiewicz.jr@intel.com \
--cc=rjw@sisk.pl \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.