Date: Sat, 11 Oct 2003 12:50:01 +0100 From: Josef Karthauser <joe@FreeBSD.org> To: Tijl Coosemans <tijl@ulyssis.org> Cc: freebsd-stable@freebsd.org Subject: Re: hang of new usb stack [was Re: Patch for boot-time USB hangs in 4.9-PRERELEASE] Message-ID: <20031011115001.GE1028@genius.tao.org.uk> In-Reply-To: <20031006211517.64c5d916.tijl@ulyssis.org> References: <200309271405.aa30238@salmon.maths.tcd.ie> <20031005190822.GC4548@genius.tao.org.uk> <20031006211517.64c5d916.tijl@ulyssis.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--z+pzSjdB7cqptWpS Content-Type: multipart/mixed; boundary="PGNNI9BzQDUtgA2J" Content-Disposition: inline --PGNNI9BzQDUtgA2J Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 06, 2003 at 09:15:17PM +0200, Tijl Coosemans wrote: > On Sun, 5 Oct 2003 20:08:22 +0100, Josef Karthauser wrote: >=20 > > On Sat, Sep 27, 2003 at 02:05:51PM +0100, Ian Dowse wrote: > > >=20 > > > Could people who are experiencing boot-time hangs in 4.9-PRERELEASE > > > try the following patch to see if it helps? I've had one positive > > > report so far, but it would be helpful to get more feedback to > > > determine if this is the right fix to be committed. > >=20 > > I'm interested in whether this fix also fixes the boot time handing of > > the new usb stack. Perhaps any interested parties could test it and > > let me know? > >=20 > > The new usb patch to RELENG_4 as of today is at: > >=20 > > http://www.josef-k.net/misc/RELENG_4-USB-20031005.patch.gz >=20 > I get the following error during a make buildworld: >=20 > =3D=3D=3D> usr.sbin/usbdevs > cc -O2 -pipe -march=3Dpentium -I/usr/src/usr.sbin/usbdevs/../../sys -c > /usr/src/usr.sbin/usbdevs/usbdevs.c > /usr/src/usr.sbin/usbdevs/usbdevs.c: In function `usbdev': > /usr/src/usr.sbin/usbdevs/usbdevs.c:91: structure has no member named > `udi_lowspeed' > *** Error code 1 You're right! I didn't add the usb tools to that patch set. I'll make sure that they are there for the next one. In the mean time I've attached the patch to usbdevs.c that you want. Joe --=20 Josef Karthauser (joe@tao.org.uk) http://www.josef-k.net/ FreeBSD (cvs meister, admin and hacker) http://www.uk.FreeBSD.org/ Physics Particle Theory (student) http://www.pact.cpes.sussex.ac.uk/ =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D An eclectic mix of fact an= d theory. =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D --PGNNI9BzQDUtgA2J Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="usbdevs.c-patch" Index: usbdevs.c =================================================================== RCS file: /home/ncvs/src/usr.sbin/usbdevs/usbdevs.c,v retrieving revision 1.5.2.3 diff -u -r1.5.2.3 usbdevs.c --- usbdevs.c 13 Nov 2002 15:15:21 -0000 1.5.2.3 +++ usbdevs.c 11 Oct 2003 11:49:07 -0000 @@ -64,7 +64,7 @@ void usage() { - fprintf(stderr, "Usage: %s [-a addr] [-d] [-f dev] [-v]\n", + fprintf(stderr, "usage: %s [-a addr] [-d] [-f dev] [-v]\n", getprogname()); exit(1); } @@ -88,8 +88,12 @@ printf("addr %d: ", a); done[a] = 1; if (verbose) { - if (di.udi_lowspeed) - printf("low speed, "); + switch (di.udi_speed) { + case USB_SPEED_LOW: printf("low speed, "); break; + case USB_SPEED_FULL: printf("full speed, "); break; + case USB_SPEED_HIGH: printf("high speed, "); break; + default: break; + } if (di.udi_power) printf("power %d mA, ", di.udi_power); else --PGNNI9BzQDUtgA2J-- --z+pzSjdB7cqptWpS Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.3 (FreeBSD) iEYEARECAAYFAj+H7mgACgkQXVIcjOaxUBb1cwCgndNlD+q6z6Uz/vG0JKpbMiRz X4YAn2sedfse6g5VnftJRmXBRelykeb+ =BizG -----END PGP SIGNATURE----- --z+pzSjdB7cqptWpS--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20031011115001.GE1028>