linux-usb.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Brian Morrison <bdm@fenrir.org.uk>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Mathias Nyman <mathias.nyman@linux.intel.com>,
	linux-usb@vger.kernel.org,
	Linux kernel regressions list <regressions@lists.linux.dev>,
	Chen Xingdi <chenxingdi@huawei.com>, Takashi Iwai <tiwai@suse.de>,
	Moritz Fischer <mdf@kernel.org>,
	Christian Lamparter <chunkeey@googlemail.com>,
	Vinod Koul <vkoul@kernel.org>
Subject: Re: USB regression in kernel 6.2.2
Date: Fri, 17 Mar 2023 18:37:13 +0000	[thread overview]
Message-ID: <20230317183713.7bef83c2@deangelis.fenrir.org.uk> (raw)
In-Reply-To: <3d0a0429-a634-4c18-82e8-72028d8226d0@rowland.harvard.edu>

On Wed, 15 Mar 2023 10:53:23 -0400
Alan Stern <stern@rowland.harvard.edu> wrote:

> On Wed, Mar 15, 2023 at 01:19:16PM +0200, Mathias Nyman wrote:
> > On 14.3.2023 16.00, Brian Morrison wrote:  
> > > On Mon, 13 Mar 2023 12:06:59 +0200
> > > Mathias Nyman <mathias.nyman@linux.intel.com> wrote:  
>  [...]  
> > > 
> > > I have now reverted the above commit, it's only the one line in
> > > xhci-pci.c and it took a couple of hours to rebuild my kernel
> > > rpms which wasn't too bad.
> > > 
> > > With this change all of my USB devices are present again and the 3
> > > /dev/ttyUSB* nodes are all present and usable.
> > >   
> > 
> > Thanks for testing.
> > So setting PROBE_PREFER_ASYNCHRONOUS does trigger this issue for
> > Renesas xHCI.
> > 
> > Was it so that with the devices connected to the Intel host
> > everything worked on 6.2 kernel?
> > 
> > Just to make sure that this is a vendor specific host issue and not
> > generic xhci driver issue.

I will see if I can test this, but it may be difficult. The add-on
Renesas card allows my USB cables (with quite large ferrites to keep RF
out of the PC) to fit in, the Intel host ports are in a different
orientation and so physically too close together for the ferrite-laden
cables to fit in.

If I can manage to test it I will report, but don't hold your breath.

> > 
> > If we can't quickly figure out the real reason for this then we
> > just have to revert that patch.

It's certainly working for me, but as I don't know much about how the
xhci driver initialises and finds the two bus host controllers I don't
know about any consequences beyond the boot delay issue that prompted
the async change in the first place.

> >   
> > > I found this in the linux-usb list archives:
> > > 
> > > https://www.spinics.net/lists/kernel/msg4569289.html
> > > 
> > > and the first part of this patch series here:
> > > 
> > > https://www.spinics.net/lists/kernel/msg4569288.html
> > > 
> > > Should both of these patches be reverted? I assume so but I don't
> > > think I have anything that uses an ehci device to test it.
> > >   
> > 
> > Probably just the xhci one. I haven't heard of any ehci issues.
> > 
> > Alan (cc) would know better if there are any new odd ehci issues
> > that can be traced back to the async probe change.  
> 
> I haven't heard of any problems with EHCI.

I think that EHCI and UHCI are older standards, I don't know if the
hardware those drivers work with is still common. I also have a VIA PCI
USB card on another machine that also uses the xhci driver and it's much
older than the machine with the Renesas card. I don't think I have the
hardware that would allow me to test those drivers.

> 
> Alan Stern
> 

Please ask if there is any extra patch you would like me to try.

-- 

Brian Morrison


      reply	other threads:[~2023-03-17 18:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-07 13:21 USB regression in kernel 6.2.2 Brian Morrison
2023-03-08  9:52 ` Linux regression tracking #adding (Thorsten Leemhuis)
2023-03-08 15:16 ` Mathias Nyman
2023-03-08 16:45   ` Brian Morrison
2023-03-09 20:04   ` Brian Morrison
2023-03-12  0:03     ` Brian Morrison
2023-03-13 10:06       ` Mathias Nyman
2023-03-14 14:00         ` Brian Morrison
2023-03-15 11:19           ` Mathias Nyman
2023-03-15 14:53             ` Alan Stern
2023-03-17 18:37               ` Brian Morrison [this message]

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=20230317183713.7bef83c2@deangelis.fenrir.org.uk \
    --to=bdm@fenrir.org.uk \
    --cc=chenxingdi@huawei.com \
    --cc=chunkeey@googlemail.com \
    --cc=linux-usb@vger.kernel.org \
    --cc=mathias.nyman@linux.intel.com \
    --cc=mdf@kernel.org \
    --cc=regressions@lists.linux.dev \
    --cc=stern@rowland.harvard.edu \
    --cc=tiwai@suse.de \
    --cc=vkoul@kernel.org \
    /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).