Date: Mon, 10 Mar 2003 05:16:49 -0600 From: Scott Johnson <scottj@insane.com> To: freebsd-stable@FREEBSD.ORG Subject: USB code change needed? Message-ID: <5.2.0.9.2.20030310050138.01a126a8@mail.ffanet.com>
next in thread | raw e-mail | index | archive | help
Hello, About a month ago, I realized that I could not boot with the new pseudo-device crypto in my kernel. After quite a bit of email was exchanged with various parties, Ian Dowse recommended that I patch uhci.c. Ian originally said something along the lines of "Is it possible that interrupts are inadvertently getting enabled too early during device probe time? The loop in uhci_waitintr() looks as if it will do the wrong thing if a real interrupt comes in and invokes uhci_intr(), as the interrupt info will not be found because it has already been handled." Based on this theory, Ian suggested the patch below. Without this patch, I cannot boot my system if I'm trying to use a kernel with crypto. With the patch, everything seems to be working ok. OpenSSL is using my crypto hardware just fine. But I haven't tested any of the USB functionality. And I don't have the resources to do so either. So my question is this: Is this patch something that needs to be incorporated into the tree? Or is my hardware just strange? I've been staying current with -STABLE, and as of March 7, -STABLE with crypto enabled still would not boot on my machine. Thanks in advance for any helpful advice. -Scott Here's the patch: =================================================================== RCS file: /home/ncvs/src/sys/dev/usb/uhci.c,v retrieving revision 1.132 diff -u -p -r1.132 uhci.c --- uhci.c 2003/02/28 19:28:29 1.132 +++ uhci.c 2003/03/10 11:01:08 @@ -1511,11 +1511,10 @@ uhci_waitintr(uhci_softc_t *sc, usbd_xfe for (; timo >= 0; timo--) { usb_delay_ms(&sc->sc_bus, 1); DPRINTFN(20,("uhci_waitintr: 0x%04x\n", UREAD2(sc, UHCI_STS))); - if (UREAD2(sc, UHCI_STS) & UHCI_STS_USBINT) { + if (UREAD2(sc, UHCI_STS) & UHCI_STS_USBINT) uhci_intr1(sc); - if (xfer->status != USBD_IN_PROGRESS) + if (xfer->status != USBD_IN_PROGRESS) return; - } } /* Timeout */ And here's my dmesg (patched kernel): Copyright (c) 1992-2003 The FreeBSD Project. Copyright (c) 1979, 1980, 1983, 1986, 1988, 1989, 1991, 1992, 1993, 1994 The Regents of the University of California. All rights reserved. FreeBSD 4.8-RC #1: Fri Mar 7 01:41:57 CST 2003 root@test01.insane.com:/usr/obj/usr/src/sys/CRYPTO Timecounter "i8254" frequency 1193182 Hz CPU: AMD Athlon(tm) Processor (1008.99-MHz 686-class CPU) Origin = "AuthenticAMD" Id = 0x642 Stepping = 2 Features=0x183f9ff<FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,SEP,MTRR,PGE,MCA,CMOV,PAT,PSE36,MMX,FXSR> AMD Features=0xc0440000<RSVD,AMIE,DSP,3DNow!> real memory = 536788992 (524208K bytes) avail memory = 516943872 (504828K bytes) Preloaded elf kernel "kernel" at 0xc053d000. Pentium Pro MTRR support enabled md0: Malloc disk Using $PIR table, 9 entries at 0xc00f1750 npx0: <math processor> on motherboard npx0: INT 16 interface pcib0: <Host to PCI bridge> on motherboard pci0: <PCI bus> on pcib0 agp0: <VIA 82C8363 (Apollo KT133A) host to PCI bridge> mem 0xe4000000-0xe7ffffff at device 0.0 on pci0 pcib1: <VIA 8363 (Apollo KT133) PCI-PCI (AGP) bridge> at device 1.0 on pci0 pci1: <PCI bus> on pcib1 isab0: <VIA 82C686 PCI-ISA bridge> at device 4.0 on pci0 isa0: <ISA bus> on isab0 atapci0: <VIA 82C686 ATA66 controller> port 0xd800-0xd80f at device 4.1 on pci0 ata0: at 0x1f0 irq 14 on atapci0 ata1: at 0x170 irq 15 on atapci0 uhci0: <VIA 83C572 USB controller> port 0xd400-0xd41f irq 10 at device 4.2 on pci0 usb0: <VIA 83C572 USB controller> on uhci0 usb0: USB revision 1.0 uhub0: VIA UHCI root hub, class 9/0, rev 1.00/1.00, addr 1 uhub0: 2 ports with 2 removable, self powered uhci1: <VIA 83C572 USB controller> port 0xd000-0xd01f irq 10 at device 4.3 on pci0 usb1: <VIA 83C572 USB controller> on uhci1 usb1: USB revision 1.0 uhub1: VIA UHCI root hub, class 9/0, rev 1.00/1.00, addr 1 uhub1: 2 ports with 2 removable, self powered uhub2: ALCOR Generic USB Hub, class 9/0, rev 1.10/1.00, addr 2 uhub2: 4 ports with 4 removable, self powered pci0: <unknown card> (vendor=0x1106, dev=0x3057) at 4.4 hifn0 mem 0xe2800000-0xe2800fff,0xe3000000-0xe3000fff irq 10 at device 9.0 on pci0 hifn0: Hifn 7951, rev 0, 128KB sram, 193 sessions fxp0: <Intel Pro 10/100B/100+ Ethernet> port 0xa400-0xa43f mem 0xe1800000-0xe18fffff,0xe2000000-0xe2000fff irq 10 at device 13.0 on pci0 fxp0: Ethernet address 00:02:b3:10:7a:39 inphy0: <i82555 10/100 media interface> on miibus0 inphy0: 10baseT, 10baseT-FDX, 100baseTX, 100baseTX-FDX, auto atapci1: <Promise ATA100 controller> port 0x8800-0x883f,0x9000-0x9003,0x9400-0x9407,0x9800-0x9803,0xa000-0xa007 mem 0xe1000000-0xe101ffff irq 11 at device 17.0 on pci0 ata2: at 0xa000 on atapci1 ata3: at 0x9400 on atapci1 orm0: <Option ROM> at iomem 0xc8000-0xc8fff on isa0 fdc0: ready for input in output fdc0: cmd 3 failed at out byte 1 of 3 atkbdc0: <Keyboard controller (i8042)> at port 0x60,0x64 on isa0 sio0 at port 0x3f8-0x3ff irq 4 flags 0x10 on isa0 sio0: type 16550A, console sio1 at port 0x2f8-0x2ff irq 3 on isa0 sio1: type 16550A ppc0: <Parallel port> at port 0x378-0x37f irq 7 on isa0 ppc0: SMC-like chipset (ECP/EPP/PS2/NIBBLE) in COMPATIBLE mode ppc0: FIFO with 16/16/8 bytes threshold plip0: <PLIP network interface> on ppbus0 lpt0: <Printer> on ppbus0 lpt0: Interrupt-driven port ppi0: <Parallel I/O> on ppbus0 IPsec: Initialized Security Association Processing. ata2-slave: ATA identify retries exceeded ata2-master: ATA identify retries exceeded ata3-slave: ATA identify retries exceeded ata3-master: ATA identify retries exceeded ad0: 3681MB <QUANTUM FIREBALL_TM3840A> [7480/16/63] at ata0-master WDMA2 acd0: CDROM <Compaq CRD-8402B> at ata1-slave PIO4 Mounting root from ufs:/dev/ad0s1a To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-stable" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5.2.0.9.2.20030310050138.01a126a8>