On 09/12/13 10:34, Martin Pieuchot wrote: > On 12/09/13(Thu) 08:16, RD Thrush wrote: >> On 09/12/13 05:15, Martin Pieuchot wrote: >>> [...] >>> >>> Could you try the diff below on the v1 machine and tell me if it helps? >> >> Thanks, I don't think it helped... > > By looking at your new log, I believe it did ;) > >> After booting the new kernel, upon first use of the kvm switch, the serial >> console began filling with diagnostics which (to my untrained eye) looked >> similar to the pre-patch session. v1-2. Since I was unable to remotely ssh, >> I >> interrupted ddb and set ehcidebug=1 but was unable to remotely ssh. >> ehcidebug=0 >> allowed remote ssh and normal operations resumed. >> >> <http://arp.thrush.com/openbsd/ehci_idone/v1/v1-3> has the associated serial >> console. >> >> Let me know if I can provide further info. > > Could you reproduce once again this manipulation with the diff below > applied on top of the previous one?
<http://arp.thrush.com/openbsd/ehci_idone/v1/v1-4> has the results after adding your diff. I'm assuming the continual diagnostic output is expected with ehcidebug=5. Interrupting ddb, setting ehcidebug=1 and continuing ddb still hits my panic hack; however, I'll assume that's due to the crudeness of my hack. I did subsequently reboot with the same custom kernel w/ ehcidebug=0 and didn't hit the panic hack. What's the next step? >>>> WRT to the other machine, x4, I installed the patch and have not yet had a >>>> problem. However, with ehcidebug=5, the following 2 line message is issued >>>> about once per second: >>>> ehci_intrlist_timeout >>>> ehci_check_intr: ex=0xffff800000238c00 >>> >>> That's expected, thanks for confirming the problem cannot be reproduced >>> with this diff with an ATI controller. >> >> Since the problem has been intermittent since Nov 2012, I'm not sure enough >> time/kvm usage has occurred for certainty about that fix. > > Since this diff is a backport of what has been done in FreeBSD and I > already had a similar report from another developer with the same > controller, I'm quite confident. > >> I will run a custom MP kernel with your pci/ehci_pci.c patch on the x4 >> machine >> whenever I install a new snapshot and let you know if I notice the >> ehci_idone issue. > > Please let me know. > > Index: uhidev.c > =================================================================== > RCS file: /home/ncvs/src/sys/dev/usb/uhidev.c,v > retrieving revision 1.44 > diff -u -p -r1.44 uhidev.c > --- uhidev.c 7 May 2013 08:44:38 -0000 1.44 > +++ uhidev.c 12 Sep 2013 14:25:52 -0000 > @@ -393,7 +393,7 @@ uhidev_intr(struct usbd_xfer *xfer, void > } > #endif > > - if (status == USBD_CANCELLED) > + if (status == USBD_CANCELLED || status == USBD_IOERROR) > return; > > if (status != USBD_NORMAL_COMPLETION) {
