From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262022AbULVTsM (ORCPT ); Wed, 22 Dec 2004 14:48:12 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262023AbULVTsM (ORCPT ); Wed, 22 Dec 2004 14:48:12 -0500 Received: from parcelfarce.linux.theplanet.co.uk ([195.92.249.252]:9648 "EHLO www.linux.org.uk") by vger.kernel.org with ESMTP id S262022AbULVTsH (ORCPT ); Wed, 22 Dec 2004 14:48:07 -0500 Date: Wed, 22 Dec 2004 15:24:29 -0200 From: Marcelo Tosatti To: Nish Aravamudan Cc: Pete Zaitcev , greg@kroah.com, linux-usb-devel@lists.sourceforge.net, rwhite@casabyte.com, linux-kernel@vger.kernel.org, kingst@eecs.umich.edu, paulkf@microgate.com, oleksiy@kharkiv.com.ua, reg@dwf.com, clemens@dwf.com Subject: Re: Little rework of usbserial in 2.4\ Message-ID: <20041222172429.GK3088@logos.cnet> References: <20041127173558.4011b177@lembas.zaitcev.lan> <29495f1d0412121547c0c644d@mail.gmail.com> <20041221125222.5754cdb2@lembas.zaitcev.lan> <29495f1d04122208073d71914b@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <29495f1d04122208073d71914b@mail.gmail.com> User-Agent: Mutt/1.5.5.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 22, 2004 at 11:07:08AM -0500, Nish Aravamudan wrote: > On Tue, 21 Dec 2004 12:52:22 -0800, Pete Zaitcev wrote: > > On Sun, 12 Dec 2004 15:47:44 -0800, Nish Aravamudan 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, Pete, msleep() is now in the generic headers, megaraid2 wants it to fix a EH busywait condition which triggers the NMI watchdog (the EH path doesnt yield the CPU as it should), plus libata and forcedeth already had their own msleep() definitions which now have been removed. Patches to change current handcoded yields to msleep() wont be accepted (cleanups), those belong to v2.6. Thanks