public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Pierre Rousselet <pierre.rousselet@wanadoo.fr>
To: Greg KH <greg@kroah.com>
Cc: lkml <linux-kernel@vger.kernel.org>,
	linux-usb-devel@lists.sourceforge.net
Subject: Re: 2.5.5-pre1 rmmod usb-uhci hangs
Date: Fri, 15 Feb 2002 18:50:16 +0100	[thread overview]
Message-ID: <3C6D4A58.6070401@wanadoo.fr> (raw)
In-Reply-To: <3C6D2130.1020103@wanadoo.fr> <20020215155636.GB1695@kroah.com>

Greg KH wrote:
> On Fri, Feb 15, 2002 at 03:54:40PM +0100, Pierre Rousselet wrote:
> 
>>with 2.5.5-pre1 usb-uhci module can't unload. rmmod hangs, leaving the 
>>system unstable. in one circumstance the box freezed with an oops 
>>involving swapper pid0 . this doesn't happen with 2.5.4
>>
> 
> Try this (untested, I haven't rebooted yet) patch:
> 
> thanks,
> 
> greg k-h
> 
> 
> diff -Nru a/drivers/usb/usb.c b/drivers/usb/usb.c
> --- a/drivers/usb/usb.c	Thu Feb 14 22:47:21 2002
> +++ b/drivers/usb/usb.c	Thu Feb 14 22:47:21 2002
> @@ -1979,11 +1979,11 @@
>  				if (driver->owner)
>  					__MOD_DEC_USE_COUNT(driver->owner);
>  				/* if driver->disconnect didn't release the interface */
> -				if (interface->driver) {
> -					put_device (&interface->dev);
> +				if (interface->driver)
>  					usb_driver_release_interface(driver, interface);
> -				}
>  			}
> +			/* remove our device node for this interface */
> +			put_device(&interface->dev);
>  		}
>  	}
>  
> 
> 
no, it doesn't solve the problem. i would like to test it whith 
preemtible kernel not set but it doesn't boot.

with both 2.5.4 and 2.5.5-pre1 when loading usb-uhci usb.c reports 2 
devices :
device 1 : USB UHCI Root Hub
device 2 : Alcatel Speed Touch USB (the driver module and firmware are 
not loaded at this stage)

when rmmoding usb-uhci with 2.5.4 usb.c reports:
usb.c: USB disconnect on device 1
usb.c: USB disconnect on device 2

with 2.5.5-pre1
usb.c: USB disconnect on device 1
  and nothing else.
Pierre
-- 
------------------------------------------------
  Pierre Rousselet <pierre.rousselet@wanadoo.fr>
------------------------------------------------


  reply	other threads:[~2002-02-15 17:53 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-02-15 14:54 2.5.5-pre1 rmmod usb-uhci hangs Pierre Rousselet
2002-02-15 15:56 ` Greg KH
2002-02-15 17:50   ` Pierre Rousselet [this message]
2002-02-15 17:57     ` Patrick Mochel
2002-02-15 18:22       ` Patrick Mochel
2002-02-15 19:11         ` Greg KH
2002-02-15 19:24         ` Pierre Rousselet
     [not found] <fa.n7cofbv.1him3j@ifi.uio.no>
     [not found] ` <fa.dsb79pv.on84ii@ifi.uio.no>
2002-02-24  2:54   ` Dan Hopper
2002-02-24  6:21     ` Greg KH
2002-02-24  6:39       ` Dan Hopper
2002-02-24  6:49         ` Greg KH
2002-02-24 17:37           ` Dan Hopper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3C6D4A58.6070401@wanadoo.fr \
    --to=pierre.rousselet@wanadoo.fr \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox