Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933751AbdCWRMj (ORCPT ); Thu, 23 Mar 2017 13:12:39 -0400 Received: from mail-vk0-f68.google.com ([209.85.213.68]:33643 "EHLO mail-vk0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751710AbdCWRMf (ORCPT ); Thu, 23 Mar 2017 13:12:35 -0400 MIME-Version: 1.0 In-Reply-To: <58D3FF97.8000202@linux.intel.com> References: <58CFE56C.8050906@linux.intel.com> <58D00409.7060205@linux.intel.com> <58D2B98D.90909@linux.intel.com> <58D3FF97.8000202@linux.intel.com> From: Diego Viola Date: Thu, 23 Mar 2017 14:12:33 -0300 Message-ID: Subject: Re: Dell Inspiron 5558/0VNM2T hangs at resume from suspend when USB 3 is enabled To: Mathias Nyman Cc: mathias.nyman@intel.com, Roger , Ulf Hansson , Greg KH , Wei WANG , "linux-kernel@vger.kernel.org" , Linux USB List , Alan Stern Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2962 Lines: 98 On Thu, Mar 23, 2017 at 2:02 PM, Mathias Nyman 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 >>> wrote: >>>> >>>> On Tue, Mar 21, 2017 at 10:04 AM, Diego Viola >>>> wrote: >>>>> >>>>> On Mon, Mar 20, 2017 at 8:15 PM, Diego Viola >>>>> wrote: >>>>>> >>>>>> On Mon, Mar 20, 2017 at 3:27 PM, Diego Viola >>>>>> wrote: >>>>>>> >>>>>>> On Mon, Mar 20, 2017 at 1:32 PM, Mathias Nyman >>>>>>> wrote: >>>>>>>> >>>>>>>> On 20.03.2017 17:39, Diego Viola wrote: >>>>>>>>> >>>>>>>>> >>>>>>>>> On Mon, Mar 20, 2017 at 11:21 AM, Mathias Nyman >>>>>>>>> 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