Re: kern/22646 (Panic in ohci_add_done())

[Andreas Gustafsson <gson%gson.org@localhost>]

The following reply was made to PR kern/22646; it has been noted by GNATS.

From: Andreas Gustafsson <gson%gson.org@localhost>
To: Nick Hudson <nick.hudson%gmx.co.uk@localhost>
Cc: gnats-bugs%NetBSD.org@localhost
Subject: Re: kern/22646 (Panic in ohci_add_done())
Date: Mon, 25 Apr 2016 12:07:49 +0300

 Nick Hudson wrote:
 > Can you do the test again with the modified patch attached and also 
 > provide UGEN_DEBUG/ugendebug=10 output?
 
 Doing just that, I was unable to reproduce the previous behavior;
 gphoto2 just kept hanging and never got into the non-hanging state.
 I figured this was because of all those ugen debug printfs going to the
 9600 bps serial console and slowing things down, so I redirected the
 ugen debug logs to the message buffer only by changing the lines
 
   #define DPRINTF(x)      if (ugendebug) printf x
   #define DPRINTFN(n,x)   if (ugendebug>(n)) printf x
 
 to
 
   #define DPRINTF(x)      if (ugendebug) printf_tolog x
   #define DPRINTFN(n,x)   if (ugendebug>(n)) printf_tolog x
 
 With that change, I was again able to get the behavior where gphoto2
 stops hanging after a few runs.  The following logs are again from a
 number of hanging gphoto2 runs followed by a single non-hanging one.
 
   http://www.gson.org/netbsd/bugs/22646/dmesg3.txt
 
 > vmstat -u usbhist as well
 
   http://www.gson.org/netbsd/bugs/22646/usbhist3.txt
 
 -- 
 Andreas Gustafsson, gson%gson.org@localhost