From: Mathias Nyman <mathias.nyman@linux.intel.com>
To: Diego Viola <diego.viola@gmail.com>
Cc: mathias.nyman@intel.com, Roger <rogerable@realtek.com>,
Ulf Hansson <ulf.hansson@linaro.org>,
Greg KH <gregkh@linuxfoundation.org>,
Wei WANG <wei_wang@realsil.com.cn>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Linux USB List <linux-usb@vger.kernel.org>,
Alan Stern <stern@rowland.harvard.edu>
Subject: Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3 is enabled
Date: Thu, 23 Mar 2017 19:02:15 +0200 [thread overview]
Message-ID: <58D3FF97.8000202@linux.intel.com> (raw)
In-Reply-To: <58D2B98D.90909@linux.intel.com>
On 22.03.2017 19:51, Mathias Nyman wrote:
> On 22.03.2017 00:52, Diego Viola wrote:
>> On Tue, Mar 21, 2017 at 12:29 PM, Diego Viola <diego.viola@gmail.com> wrote:
>>> On Tue, Mar 21, 2017 at 10:04 AM, Diego Viola <diego.viola@gmail.com> wrote:
>>>> On Mon, Mar 20, 2017 at 8:15 PM, Diego Viola <diego.viola@gmail.com> wrote:
>>>>> On Mon, Mar 20, 2017 at 3:27 PM, Diego Viola <diego.viola@gmail.com> wrote:
>>>>>> On Mon, Mar 20, 2017 at 1:32 PM, Mathias Nyman
>>>>>> <mathias.nyman@linux.intel.com> wrote:
>>>>>>> On 20.03.2017 17:39, Diego Viola wrote:
>>>>>>>>
>>>>>>>> On Mon, Mar 20, 2017 at 11:21 AM, Mathias Nyman
>>>>>>>> <mathias.nyman@linux.intel.com> wrote:
>>>>>>>>>
>>>>>>>>> On 19.03.2017 23:29, Diego Viola wrote:
>>>>>>>>>>
>>>>>>>>>>> Still a problem with 4.11.0-rc2-ARCH+
>>>>>>>>>
>>>>>>>>> xhci tracing can be added with:
>>>>>>>>>
>>>>>>>>> mount -t debugfs none /sys/kernel/debug
>>>>>>>>> echo xhci-hcd >> /sys/kernel/debug/tracing/set_event
>>
>> Here's the log I was able to obtain today, dmesg + ftrace at the time
>> of the crash:
>>
>> https://bugzilla.kernel.org/attachment.cgi?id=255419
>>
>> USB keyboard and mouse was plugged when I reproduced this.
>>
>> Please let me know if you need more info.
>>
>
> Thanks, I'm looking at the logs and so far the most suspicious looking entry is:
>
> [ 257.060941] rtsx_usb-254 0.... 119946155us : xhci_urb_enqueue: ep1out-bulk: urb ffff880105a93300 pipe 3221259520 length 0/12 sgs 0/0 stream 0 flags 00010000
> [ 257.063601] rtsx_usb-254 0.... 119946162us : xhci_urb_enqueue: ep0out-control: urb ffff880105a93300 pipe 2147484928 length 0/0 sgs 0/0 stream 0 flags 00100000
>
> It enqueues the same URB, without ever giving it back or actually queuing any trbs for
> the urb, wel,l it might just fail to enqueue it in the first place.
>
> I need to search for a URB that has been dequeued but never given back in the trace
Ok, found a much more likely candidate:
[ 258.004078] kworker/-544 0d..1 121599183us : xhci_urb_dequeue: ep1out-bulk: urb ffff880105a930c0 pipe 3221259520...
We try to kill this URB "ffff880105a930c0", twice, and its never given back.
Trace is missing "xhci_dbg_cancel_urb: Cancel URB..." entry in log after
xhci_urb_dequeue, so it never got added to the list for cancellation in xhci driver.
xhci_urb_dequeue() has one place where it just returns an error without
giving back the urb or queuing it for cancellation.
This is in my opinion a bug in xhci_urb_dequeue()
rtsx_usb_ms is a good test for usb, it seems to be constantly queuing urbs at all
inappropriate times.
If I write a patch can you try it out?
-Mathias
next prev parent reply other threads:[~2017-03-23 17:00 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-08 18:49 Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3 is enabled Diego Viola
2017-03-08 19:59 ` Diego Viola
2017-03-08 20:15 ` Greg KH
2017-03-08 20:40 ` Diego Viola
2017-03-09 14:11 ` Diego Viola
2017-03-09 17:15 ` Diego Viola
2017-03-14 17:20 ` Diego Viola
2017-03-14 19:15 ` Diego Viola
2017-03-15 14:00 ` Diego Viola
2017-03-16 14:03 ` Ulf Hansson
2017-03-16 15:07 ` Alan Stern
2017-03-16 15:45 ` Diego Viola
2017-03-16 15:51 ` Diego Viola
2017-03-16 16:02 ` Diego Viola
2017-03-16 16:19 ` Diego Viola
2017-03-16 17:14 ` Diego Viola
2017-03-16 18:47 ` Diego Viola
2017-03-17 15:55 ` Diego Viola
2017-03-17 16:24 ` Alan Stern
2017-03-17 16:57 ` Diego Viola
2017-03-17 20:18 ` Diego Viola
2017-03-19 21:29 ` Diego Viola
2017-03-20 14:21 ` Mathias Nyman
2017-03-20 15:39 ` Diego Viola
2017-03-20 16:32 ` Mathias Nyman
2017-03-20 18:27 ` Diego Viola
2017-03-20 23:15 ` Diego Viola
2017-03-21 13:04 ` Diego Viola
2017-03-21 15:29 ` Diego Viola
2017-03-21 22:52 ` Diego Viola
2017-03-22 17:51 ` Mathias Nyman
2017-03-23 17:02 ` Mathias Nyman [this message]
2017-03-23 17:12 ` Diego Viola
2017-03-24 13:12 ` [RFT PATCH] xhci: Manually give back cancelled URBs if queuing for cancel fails Mathias Nyman
2017-03-24 16:25 ` Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3 is enabled Diego Viola
2017-03-27 7:04 ` Mathias Nyman
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=58D3FF97.8000202@linux.intel.com \
--to=mathias.nyman@linux.intel.com \
--cc=diego.viola@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mathias.nyman@intel.com \
--cc=rogerable@realtek.com \
--cc=stern@rowland.harvard.edu \
--cc=ulf.hansson@linaro.org \
--cc=wei_wang@realsil.com.cn \
/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).