public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Laurent Riffard <laurent.riffard@free.fr>
To: Patrick Mochel <mochel@digitalimplant.org>
Cc: Andrew Morton <akpm@osdl.org>,
	rjw@sisk.pl, rlrevell@joe-job.com,
	alsa-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org,
	Greg KH <greg@kroah.com>
Subject: Re: 2.6.12-rc1-mm2
Date: Fri, 25 Mar 2005 19:01:22 +0100	[thread overview]
Message-ID: <424451F2.5070201@free.fr> (raw)
In-Reply-To: <Pine.LNX.4.50.0503241658360.29178-100000@monsoon.he.net>

[-- Attachment #1: Type: text/plain, Size: 1170 bytes --]


Le 25.03.2005 02:00, Patrick Mochel a écrit :
> On Thu, 24 Mar 2005, Andrew Morton wrote:
>
>
>>Laurent Riffard <laurent.riffard@free.fr> wrote:
>>
>>> hello,
>>>
>>> Same kinds of problem here. It depends on the removed module. I
>>> mean: "rmmod loop" or "rmmod pcspkr" works. But "rmmod
>>> snd_ens1371" or "rmmod ohci1394" hangs.
>>>
>>> Sysrq-T when rmmoding snd_ens1371 :
>
> <snip>
>
>> It looks like we're getting stuck in the wait_for_completion() in
>> the new klist_remove().
>
> D'oh! It's getting hung while waiting to remove the current node from
> the list (which it can't remove because it's being used). The patch
> below should fix it.
>
> 	Pat
>
>
> ===== drivers/base/dd.c 1.3 vs edited =====
> --- 1.3/drivers/base/dd.c	2005-03-21 12:25:04 -08:00
> +++ edited/drivers/base/dd.c	2005-03-24 16:55:21 -08:00
> @@ -177,7 +177,7 @@
>
>  	sysfs_remove_link(&drv->kobj, kobject_name(&dev->kobj));
>  	sysfs_remove_link(&dev->kobj, "driver");
> -	klist_remove(&dev->knode_driver);
> +	klist_del(&dev->knode_driver);
>
>  	down(&dev->sem);
>  	device_detach_shutdown(dev);

Ok, I can confirm this patch solved the problem.

Thanks for your help.
--
laurent

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]

  parent reply	other threads:[~2005-03-25 18:01 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-24 12:41 2.6.12-rc1-mm2 Andrew Morton
2005-03-24 14:40 ` 2.6.12-rc1-mm2 Stefano Rivoir
2005-03-24 15:13   ` 2.6.12-rc1-mm2 Manuel Lauss
2005-03-24 15:18   ` 2.6.12-rc1-mm2 Brice Goglin
2005-03-24 15:31     ` 2.6.12-rc1-mm2 Stefano Rivoir
2005-03-24 20:05       ` 2.6.12-rc1-mm2 Andrew Morton
2005-03-24 20:21         ` 2.6.12-rc1-mm2 Brice Goglin
2005-03-25  1:19           ` 2.6.12-rc1-mm2 Alexey Dobriyan
2005-03-24 15:09 ` 2.6.12-rc1-mm2 (build error In function `zft_init') Steven Cole
2005-03-24 15:55   ` 2.6.12-rc1-mm2 (patch to fix build " Steven Cole
2005-03-24 21:49     ` Greg KH
2005-03-24 21:53       ` Andrew Morton
2005-03-24 16:46 ` 2.6.12-rc1-mm2 Lee Revell
2005-03-24 20:17   ` 2.6.12-rc1-mm2 Andrew Morton
2005-03-24 22:31     ` 2.6.12-rc1-mm2 Rafael J. Wysocki
2005-03-24 22:37       ` 2.6.12-rc1-mm2 Rafael J. Wysocki
2005-03-24 23:33       ` 2.6.12-rc1-mm2 Laurent Riffard
2005-03-24 23:49         ` 2.6.12-rc1-mm2 Andrew Morton
2005-03-25  1:00           ` 2.6.12-rc1-mm2 Patrick Mochel
2005-03-25  6:05             ` 2.6.12-rc1-mm2 Greg KH
2005-03-25 18:01             ` Laurent Riffard [this message]
2005-05-26  0:29       ` 2.6.12-rc1-mm2 Andrew Morton
2005-05-26 13:58         ` 2.6.12-rc1-mm2 Rafael J. Wysocki
2005-03-25  0:38 ` [2.6 patch] remove exports for oem modules Adrian Bunk
2005-03-25  4:12 ` OOPS running "ls -l /sys/class/i2c-adapter/*"-- 2.6.12-rc1-mm2 Miles Lane
2005-03-25  4:22   ` Andrew Morton
2005-03-25  7:38     ` Russell King
2005-03-25  7:45       ` Andrew Morton
2005-03-25  7:50         ` Russell King
2005-03-25  8:13           ` Russell King
2005-03-25 12:52             ` Miles Lane
2005-03-25 18:40               ` Andrew Morton
2005-03-25 19:50                 ` Jean Delvare
2005-03-26  4:44                   ` Miles Lane
2005-03-25 20:53       ` Lee Revell
2005-03-25 21:07         ` Russell King
2005-03-25 21:45           ` [PATCH] make Documentation/oops-tracing.txt relevant to 2.6 [was Re: OOPS running "ls -l /sys/class/i2c-adapter/*"-- 2.6.12-rc1-mm2] Lee Revell
2005-03-25 21:52             ` Russell King
2005-03-25 21:54               ` Lee Revell
2005-03-25 19:26 ` x86-64 preemption fix from IRQ and BKL in 2.6.12-rc1-mm2 Christophe Saout
2005-03-27  0:19   ` [PATCH] Fix preemption off of irq context on x86-64 with PREEMPT_BKL Christophe Saout
2005-03-27 17:28     ` Andi Kleen
2005-03-27 17:26   ` x86-64 preemption fix from IRQ and BKL in 2.6.12-rc1-mm2 Andi Kleen
2005-03-27 18:05     ` Christophe Saout
2005-03-28 15:26       ` Andi Kleen
     [not found] <fa.h0e9s21.ljie1j@ifi.uio.no>
2005-03-24 23:50 ` 2.6.12-rc1-mm2 Reuben Farrelly

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=424451F2.5070201@free.fr \
    --to=laurent.riffard@free.fr \
    --cc=akpm@osdl.org \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mochel@digitalimplant.org \
    --cc=rjw@sisk.pl \
    --cc=rlrevell@joe-job.com \
    /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