From: Peter Hurley <peter@hurleysoftware.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Lan Tianyu <tianyu.lan@intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jiri Kosina <jkosina@suse.cz>,
linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-next@vger.kernel.org
Subject: Re: [-next-20130204] usb/hcd: irq 18: nobody cared
Date: Fri, 08 Feb 2013 06:57:11 -0500 [thread overview]
Message-ID: <1360324631.3385.9.camel@thor.lan> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1302051520220.2245-100000@iolanthe.rowland.org>
On Tue, 2013-02-05 at 15:26 -0500, Alan Stern wrote:
> On Tue, 5 Feb 2013, Peter Hurley wrote:
>
> > With -next-20130204:
> >
> > [ 33.855570] irq 18: nobody cared (try booting with the "irqpoll" option)
> > [ 33.855580] Pid: 0, comm: swapper/4 Not tainted 3.8.0-next-20130204-xeon #20130204
> > [ 33.855582] Call Trace:
> > [ 33.855585] <IRQ> [<ffffffff810f1076>] __report_bad_irq+0x36/0xe0
> > [ 33.855600] [<ffffffff810f152a>] note_interrupt+0x1aa/0x200
> > [ 33.855606] [<ffffffff8101edf2>] ? mwait_idle+0x82/0x1b0
> > [ 33.855610] [<ffffffff810eed89>] handle_irq_event_percpu+0xc9/0x260
> > [ 33.855614] [<ffffffff810eef68>] handle_irq_event+0x48/0x70
> > [ 33.855618] [<ffffffff810f20ba>] handle_fasteoi_irq+0x5a/0x100
> > [ 33.855624] [<ffffffff810182f2>] handle_irq+0x22/0x40
> > [ 33.855630] [<ffffffff816e2a9a>] do_IRQ+0x5a/0xd0
> > [ 33.855636] [<ffffffff816d97ad>] common_interrupt+0x6d/0x6d
> > [ 33.855638] <EOI> [<ffffffff810f8e2a>] ? rcu_eqs_enter_common+0x4a/0x320
> > [ 33.855646] [<ffffffff8101edf2>] ? mwait_idle+0x82/0x1b0
> > [ 33.855649] [<ffffffff8101ed99>] ? mwait_idle+0x29/0x1b0
> > [ 33.855653] [<ffffffff8101f8a6>] cpu_idle+0x116/0x130
> > [ 33.855658] [<ffffffff816c0c0f>] start_secondary+0x251/0x258
> > [ 33.855660] handlers:
> > [ 33.855664] [<ffffffff814f98a0>] usb_hcd_irq
> > [ 33.855667] Disabling IRQ #18
> >
> > From earlier in the boot log:
> > [ 1.297020] uhci_hcd 0000:00:1d.2: setting latency timer to 64
> > [ 1.297032] uhci_hcd 0000:00:1d.2: UHCI Host Controller
> > [ 1.297040] uhci_hcd 0000:00:1d.2: new USB bus registered, assigned bus number 4
> > [ 1.297076] uhci_hcd 0000:00:1d.2: irq 18, io base 0x0000ff40
> > [ 1.297213] hub 4-0:1.0: USB hub found
> > [ 1.297221] hub 4-0:1.0: 2 ports detected
> >
> > lsusb:
> > ...
> > Bus 004 Device 002: ID 0764:0501 Cyber Power System, Inc. CP1500 AVR UPS
> > Bus 004 Device 003: ID 046d:c52b Logitech, Inc. Unifying Receiver
> > ...
> >
> > Didn't have this problem with -next-20120125. One of these commits, maybe?
> > next/drivers/usb/core$ git log --oneline next-20130125..next-20130204 -- *
> > 2c2e865 usb: forbid memory allocation with I/O during bus reset
> > e9121a8 Merge remote-tracking branch 'usb/usb-next'
> > 3b2ab2b Revert "usb: Register usb port's acpi power resources"
> > da0aa71 USB: add usb_hcd_{start,end}_port_resume
> > 192fef1 usb: enable usb port device's async suspend.
> > f6cced1 usb: expose usb port's pm qos flags to user space
> > ad493e5 usb: add usb port auto power off mechanism
> > 971fcd4 usb: add runtime pm support for usb port device
> > 88bb965 usb: Register usb port's acpi power resources
> > 54a3ac0 usb: Using correct way to clear usb3.0 device's remote wakeup feature.
>
> None of those commits seems likely to have caused the problem. The
> most likely is da0aa71, combined with one you didn't mention:
>
> 840008bb USB: UHCI: notify usbcore about port resumes
>
> If you revert these two commits, does that make any difference? What
> if you revert all of these commits?
None of these commits are directly causing this. I tested Greg's usb
master and usb-next and both run fine for me.
> > I didn't see any changes in the drivers/hid/hid-logitech-dj.c or usbhid
> > but maybe it's doing something wrong anyway?
> >
> > I'll open a bugzilla if a bunch more info is necessary.
>
> If the suggestion above doesn't work out, bisection may be the best way
> to find the answer.
Any pointers on how to one of) build a linear history for -next, get
bisect to work on an integration tree, or third option I haven't thought
of?
Thanks,
Peter Hurley
next prev parent reply other threads:[~2013-02-08 11:57 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-05 17:51 [-next-20130204] usb/hcd: irq 18: nobody cared Peter Hurley
2013-02-05 20:26 ` Alan Stern
2013-02-08 11:57 ` Peter Hurley [this message]
2013-02-10 3:14 ` [Bisected] " Peter Hurley
2013-02-10 14:23 ` Peter Hurley
2013-02-10 20:33 ` Yinghai Lu
2013-02-11 6:40 ` Yinghai Lu
2013-02-11 10:25 ` Jiri Slaby
2013-02-11 13:02 ` Peter Hurley
2013-02-11 19:19 ` Yinghai Lu
2013-02-11 19:45 ` Sedat Dilek
2013-02-11 19:53 ` Yinghai Lu
2013-02-11 19:57 ` Yinghai Lu
2013-02-11 21:06 ` Yinghai Lu
2013-02-11 22:41 ` Bjorn Helgaas
2013-02-12 0:08 ` Yinghai Lu
2013-02-12 3:21 ` Yinghai Lu
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=1360324631.3385.9.camel@thor.lan \
--to=peter@hurleysoftware.com \
--cc=gregkh@linuxfoundation.org \
--cc=jkosina@suse.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=stern@rowland.harvard.edu \
--cc=tianyu.lan@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;
as well as URLs for NNTP newsgroup(s).