On Tue, 21 Dec 2004 12:52:22 -0800, Pete Zaitcev <[EMAIL PROTECTED]> wrote:
> On Sun, 12 Dec 2004 15:47:44 -0800, Nish Aravamudan <[EMAIL PROTECTED]> wrote:
>
> > > diff -urpN -X dontdiff linux-2.4.28-bk3/drivers/usb/serial/usbserial.c
> > > linux-2.4.28-bk3-sx4/drivers/usb/serial/usbserial.c
> > > --- linux-2.4.28-bk3/drivers/usb/serial/usbserial.c 2004-11-22
> > > 23:04:19.000000000 -0800
>
> > > @@ -1803,6 +1820,12 @@ static void __exit usb_serial_exit(void)
> > >
> > > usb_deregister(&usb_serial_driver);
> > > tty_unregister_driver(&serial_tty_driver);
> > > +
> > > + while (!list_empty(&usb_serial_driver_list)) {
> > > + err("%s - module is in use, hanging...\n", __FUNCTION__);
> > > + set_current_state(TASK_UNINTERRUPTIBLE);
> > > + schedule_timeout(5*HZ);
> > > + }
>
> > Please consider using msleep() here instead of schedule_timeout().
>
> No, Nish, it's 2.4. There's no msleep here. I can create something like
> "drivers/usb/serial/compat26.h", similar to include/linux/libata-compat.h,
> but I do not think it's worth the trouble at present juncture.
I agree that it's not worth the trouble. Sorry, I was under the
impression that Kernel-Janitors had pushed a series of patches to
backport msleep(). Maybe they haven't made it to mainline yet. Sorry
for the noise.
-Nish
-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://productguide.itmanagersjournal.com/
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
