All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.