Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3 is enabled

Path	csiph.com!news.redatomik.org!aioe.org!news.servidellagleba.it!bofh.it!news.nic.it!robomod
From	Mathias Nyman <mathias.nyman@intel.com>
Newsgroups	linux.kernel
Subject	Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3 is enabled
Date	Mon, 27 Mar 2017 09:10:03 +0200
Message-ID	<tpwWS-4eZ-29@gated-at.bofh.it> (permalink)
References	<tm2Vj-Og-3@gated-at.bofh.it> <tm3om-Yu-3@gated-at.bofh.it> <tm6vT-3sM-9@gated-at.bofh.it> <tmQIq-3cW-17@gated-at.bofh.it> <tn6tQ-5Wi-35@gated-at.bofh.it> <tn7Jg-6KZ-29@gated-at.bofh.it> <tn8vE-7pZ-29@gated-at.bofh.it> <tnae6-cZ-27@gated-at.bofh.it> <tneKK-3jB-5@gated-at.bofh.it> <tnrHY-3KN-17@gated-at.bofh.it> <tnu37-5ee-1@gated-at.bofh.it> <tnAUV-1rR-7@gated-at.bofh.it> <tnSyu-6bg-41@gated-at.bofh.it> <toepk-5id-33@gated-at.bofh.it> <toez0-5me-5@gated-at.bofh.it> <toAgb-4c4-27@gated-at.bofh.it>
X-Original-To	Diego Viola <diego.viola@gmail.com>, Mathias Nyman <mathias.nyman@linux.intel.com>
X-Extloop1	1
X-Ironport-Av	E=Sophos;i="5.36,229,1486454400"; d="scan'208";a="948498863"
User-Agent	Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0
MIME-Version	1.0
Content-Type	text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding	7bit
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	119
Organization	linux.* mail to news gateway
X-Original-Cc	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>
X-Original-Date	Mon, 27 Mar 2017 10:04:48 +0300
X-Original-Message-ID	<58D8B990.9060006@intel.com>
X-Original-References	<CA+ToGPEquNetNW5PnTcB5p=jUwwKmp-=wgyJ7ftt+GAv=REfTQ@mail.gmail.com> <CA+ToGPFU=JfZTFSnXyofKkZ9n13hF_TXcQCpLt_t0J8QKeRFYw@mail.gmail.com> <CA+ToGPG=wyotbKczGgJvuSmzFdLVaSbED05qkbFLiLF9W=Kz1A@mail.gmail.com> <CA+ToGPGASfrDZyuv-p0V3yVFTQ33wi8kLE+DEQ=4Dt+VJvVG4w@mail.gmail.com> <58CFE56C.8050906@linux.intel.com> <CA+ToGPFXoqfn7PyM6iDthMN0jWVC5EKUtmzc3M-gJxcy1he_nw@mail.gmail.com> <58D00409.7060205@linux.intel.com> <CA+ToGPHd+pVc5v6-4p6XYGvqwRYzjGAg5RXQ+O=Yg6VvrtTevA@mail.gmail.com> <CA+ToGPEO3SZyqAC17rGSQifSi1S662OVJGE5wKSaMamFarK5AQ@mail.gmail.com> <CA+ToGPFEzMpTsntpFU0e=fD=EwMSR9CULZFb-PeE_LxdrgtFXg@mail.gmail.com> <CA+ToGPH=z2kmCmYmGjJ+kVZc2kMsjm-VMHqKqkrxe4xcSOz6QA@mail.gmail.com> <CA+ToGPH+YDh8TL1ODmRXOPCS_Sq4wsHYZrsOn33b9c0+56eB-w@mail.gmail.com> <58D2B98D.90909@linux.intel.com> <58D3FF97.8000202@linux.intel.com> <CA+ToGPFiAU1WvTQzKmzSVWtyWde+jh1e_MNv9aeNDBFopdf7vA@mail.gmail.com> <CA+ToGPErCfB0ZiFmKRD0C71qANsWDWpqZ0sqE6TNWzbRYbwd4Q@mail.gmail.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1609517

Show key headers only | View raw

On 24.03.2017 18:25, Diego Viola wrote:
> On Thu, Mar 23, 2017 at 2:12 PM, Diego Viola <diego.viola@gmail.com> wrote:
>> On Thu, Mar 23, 2017 at 2:02 PM, Mathias Nyman
>> <mathias.nyman@linux.intel.com> wrote:
>>> 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?
>>
>> Yes.
>>
>>>
>>> -Mathias
>>>
>>>
>>>
>>
>> Thanks,
>> Diego
>
> Hi Mathias,
>
> I tested your patch with Linux 4.11-rc3 and can confirm that it solves
> the problem.
>
> I've tested suspend and resume with i3lock 150 times and it works.
>
> Thank you, I appreciate it a lot.
>
> Diego
>


Great, I'll send it forward, it can still make 4.11 final.

Thanks for testing

-Mathias

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB  3 is enabled Alan Stern <stern@rowland.harvard.edu> - 2017-03-17 17:30 +0100
  Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-17 18:00 +0100
    Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-17 21:20 +0100
      Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-19 22:40 +0100
        Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Mathias Nyman <mathias.nyman@linux.intel.com> - 2017-03-20 15:30 +0100
          Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-20 16:50 +0100
            Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Mathias Nyman <mathias.nyman@linux.intel.com> - 2017-03-20 17:40 +0100
              Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-20 19:30 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-21 00:20 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-21 14:10 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-21 16:40 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-22 00:00 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Mathias Nyman <mathias.nyman@linux.intel.com> - 2017-03-22 18:50 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Mathias Nyman <mathias.nyman@linux.intel.com> - 2017-03-23 18:10 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-23 18:20 +0100
                [RFT PATCH] xhci: Manually give back cancelled URBs if queuing for cancel fails Mathias Nyman <mathias.nyman@linux.intel.com> - 2017-03-24 14:20 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Diego Viola <diego.viola@gmail.com> - 2017-03-24 17:30 +0100
                Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3  is enabled Mathias Nyman <mathias.nyman@intel.com> - 2017-03-27 09:10 +0200

csiph-web