Re: port-i386/57662: StarTech ICUSB23208FD 8-Port USB-to-Serial Adapter Hub fails on Alix with NetBSD/i386 9.3

[Alexander Schreiber <als%thangorodrim.ch@localhost>]

On Sat, Oct 28, 2023 at 02:05:02AM +0000, Taylor R Campbell wrote:
> Also: Can you try this patch on top as well?

> >From fe71f2d88628e0a801d04fc4ad4b4667272c065f Mon Sep 17 00:00:00 2001
> From: Taylor R Campbell <riastradh%NetBSD.org@localhost>
> Date: Sat, 28 Oct 2023 02:01:10 +0000
> Subject: [PATCH] ehci(4): Fix bug causing missed wakeups since ehci.c 1.308.
> 
> For reasons beyond me now, I used cv_signal on the same cv that is
> used to wait for the doorbell to be available _and_ to wait for the
> host controller to acknowledge the doorbell.  Which means when the
> host controller acknowledges the doorbell, we might wake some thread
> waiting for the doorbell to be available -- and leave the thread
> waiting for the doorbell acknowledgment hanging indefinitely.
> 
> PR port-i386/57662
> 
> XXX pullup-10
> ---
>  sys/dev/usb/ehci.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Both patches applied and things do look better. Now, instead of hanging
forever, after the second (so, at the third) connection attempt, cu
errors out with: "cu: link down" after a few seconds. This behavior then
sticks.

Relevant snippet from the trace:

  1699   1699 cu       0.000010616 CALL  open(0xbfb9a56e,6,0)
  1699   1699 cu       0.000011733 NAMI  "/dev/ttyU1"
  1699   1699 cu       11.991814027 RET   open -1 errno 5 Input/output error

vmstat usbhist dump at:

https://www.thangorodrim.ch/tmp/NetBSD10BETA_serial_vmstat-usbhist_patch-1

Kind regards,
           Alex.
-- 
"Opportunity is missed by most people because it is dressed in overalls and
 looks like work."                                      -- Thomas A. Edison