From: Andrew Morton <akpm@osdl.org>
To: Matthew Wilcox <matthew@wil.cx>
Cc: Val Henson <val_henson@linux.intel.com>,
Greg Kroah-Hartman <gregkh@suse.de>,
netdev@vger.kernel.org, linux-pci@atrey.karlin.mff.cuni.cz,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] [PCI] Check that MWI bit really did get set
Date: Sat, 14 Oct 2006 13:48:55 -0700 [thread overview]
Message-ID: <20061014134855.b66d7e65.akpm@osdl.org> (raw)
In-Reply-To: <20061014140249.GL11633@parisc-linux.org>
On Sat, 14 Oct 2006 08:02:49 -0600
Matthew Wilcox <matthew@wil.cx> wrote:
> On Fri, Oct 13, 2006 at 09:41:35PM -0700, Andrew Morton wrote:
> > Bisection shows that this patch
> > (pci-check-that-mwi-bit-really-did-get-set.patch in Greg's PCI tree) breaks
> > suspend-to-disk on my Vaio. It writes the suspend image and gets to the
> > point where it's supposed to power down, but doesn't.
>
> How odd. What driver is calling pci_set_mwi() on the suspend path?
ehci_pci_reinit(). I stuck a dump_stack() in there. See
http://userweb.kernel.org/~akpm/s5000342.jpg
> What drivers do you have loaded on the Vaio?
sony:/home/akpm> lsmod
Module Size Used by
ipw2200 163184 0
sonypi 21484 0
autofs4 19908 1
hidp 16192 2
l2cap 21764 5 hidp
bluetooth 49060 2 hidp,l2cap
sunrpc 154172 1
ip_conntrack_netbios_ns 3328 0
ipt_REJECT 4736 1
xt_state 2496 2
ip_conntrack 51020 2 ip_conntrack_netbios_ns,xt_state
nfnetlink 7128 1 ip_conntrack
xt_tcpudp 3392 4
iptable_filter 3264 1
ip_tables 12616 1 iptable_filter
x_tables 15428 4 ipt_REJECT,xt_state,xt_tcpudp,ip_tables
video 16836 0
sony_acpi 7312 0
sbs 15968 0
i2c_ec 5504 1 sbs
button 7184 0
battery 10692 0
asus_acpi 17564 0
backlight 6464 2 sony_acpi,asus_acpi
ac 5636 0
nvram 8072 0
ohci1394 33264 0
ehci_hcd 30088 0
ieee1394 291896 1 ohci1394
uhci_hcd 22092 0
sg 33628 0
joydev 9920 0
evbug 3392 0
snd_hda_intel 18968 0
snd_hda_codec 161536 1 snd_hda_intel
snd_seq_dummy 4228 0
snd_seq_oss 31744 0
snd_seq_midi_event 7360 1 snd_seq_oss
snd_seq 48208 5 snd_seq_dummy,snd_seq_oss,snd_seq_midi_event
snd_seq_device 8524 3 snd_seq_dummy,snd_seq_oss,snd_seq
ieee80211 30920 1 ipw2200
snd_pcm_oss 41504 0
snd_mixer_oss 16640 1 snd_pcm_oss
ieee80211_crypt 6016 1 ieee80211
snd_pcm 74632 3 snd_hda_intel,snd_hda_codec,snd_pcm_oss
snd_timer 21316 2 snd_seq,snd_pcm
snd 50980 9 snd_hda_intel,snd_hda_codec,snd_seq_oss,snd_seq,snd_seq_device,snd_pcm_oss,snd_mixer_oss,snd_pcm,snd_timer
soundcore 7968 1 snd
snd_page_alloc 10376 2 snd_hda_intel,snd_pcm
piix 9604 0 [permanent]
i2c_i801 7820 0
pcspkr 3136 0
i2c_core 21840 2 i2c_ec,i2c_i801
generic 5252 0 [permanent]
ext3 127688 1
jbd 52712 1 ext3
ide_disk 16000 0
ide_core 114780 3 piix,generic,ide_disk
next prev parent reply other threads:[~2006-10-14 20:49 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-10-06 19:05 Matthew Wilcox
2006-10-06 19:05 ` [PATCH 2/2] [TULIP] Check the return value from pci_set_mwi() Matthew Wilcox
2006-10-06 19:15 ` Jeff Garzik
2006-10-06 19:28 ` Matthew Wilcox
2006-10-06 19:59 ` Jeff Garzik
2006-10-07 5:34 ` Grant Grundler
2006-10-07 14:44 ` Jeff Garzik
2006-10-06 19:16 ` [PATCH 1/2] [PCI] Check that MWI bit really did get set Jeff Garzik
2006-10-14 4:41 ` Andrew Morton
2006-10-14 5:21 ` Greg KH
2006-10-14 14:02 ` Matthew Wilcox
2006-10-14 20:48 ` Andrew Morton [this message]
2006-10-15 3:20 ` Matthew Wilcox
2006-10-15 6:53 ` Andrew Morton
2006-10-15 13:54 ` Matthew Wilcox
2006-10-15 17:47 ` Andrew Morton
2006-10-15 7:08 ` [Bulk] " David Brownell
2006-10-15 13:52 ` Matthew Wilcox
2006-10-15 14:21 ` Alan Cox
2006-10-15 13:57 ` Matthew Wilcox
2006-10-15 17:45 ` Andrew Morton
2006-10-15 19:16 ` David Brownell
2006-10-15 19:34 ` Andrew Morton
2006-10-15 22:45 ` David Brownell
2006-10-15 23:18 ` Andrew Morton
2006-10-16 0:02 ` Alan Cox
2006-10-15 23:44 ` Andrew Morton
2006-10-16 0:44 ` Paul Mackerras
2006-10-16 1:10 ` Andrew Morton
2006-10-16 2:07 ` David Brownell
2006-10-16 10:58 ` Alan Cox
2006-10-16 11:02 ` Alan Cox
2006-10-16 0:16 ` David Brownell
2006-10-16 0:31 ` Andrew Morton
2006-10-16 10:59 ` Alan Cox
2006-10-15 21:52 ` [Bulk] " Alan Cox
2006-10-16 0:00 ` Paul Mackerras
2006-10-16 0:15 ` Andrew Morton
2006-10-16 0:21 ` David Brownell
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=20061014134855.b66d7e65.akpm@osdl.org \
--to=akpm@osdl.org \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@atrey.karlin.mff.cuni.cz \
--cc=matthew@wil.cx \
--cc=netdev@vger.kernel.org \
--cc=val_henson@linux.intel.com \
/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
Powered by JetHome