mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Aritz Bastida <aritzbastida@gmail.com>
To: "linux-os (Dick Johnson)" <linux-os@analogic.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Stopping Kernel Threads at module unload time
Date: Wed, 9 Nov 2005 18:24:44 +0100	[thread overview]
Message-ID: <7d40d7190511090924k6ef493dbn@mail.gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.61.0511091158350.10894@chaos.analogic.com>

>
> > Now, if I call kthread_stop() in module unload time, does that code
> > run in user process context just like system calls do? That is
> > important, because if it cannot sleep, it would deadlock.
> >
>
> Not relevent. You have apparently made up your mind that you
> need to do it "your" way. Fine.

No, no, that is not MY way. That is the 2.6 way.
 (see http://lwn.net/Articles/64444/).

With the old API (kernel_thread) there was no way (at least, direct
way) to bind a thread to a cpu, which i actually need. And this is not
MY own approach , it is used throught the kernel:

* migration thread at sched.c:
           http://lxr.linux.no/source/kernel/sched.c#L4196

* ksoftirqd thread at softirq.c:
           http://lxr.linux.no/source/kernel/softirq.c#L350

* worker thread used in workqueues:
           http://lxr.linux.no/source/kernel/workqueue.c#L182

...and so on

All these threads are standard and well debugged, and all use the API
which I was talking about. Of course, as they are included in the
official kernel sources, none of them is stopped at module unload
time.

Thus, all I want to know is why kthread_stop() cant be called at
module unload time, if the cleanup code can sleep and if there is a
workaround for that.

I hope my questions are clearer now.
Thanks for your help

Aritz

  reply	other threads:[~2005-11-09 17:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-09 15:49 Aritz Bastida
2005-11-09 16:16 ` linux-os (Dick Johnson)
2005-11-09 16:56   ` Aritz Bastida
2005-11-09 17:05     ` linux-os (Dick Johnson)
2005-11-09 17:24       ` Aritz Bastida [this message]
2005-11-09 18:09         ` linux-os (Dick Johnson)
2005-11-09 17:27     ` Arnd Bergmann
2005-11-09 17:49       ` Aritz Bastida
2005-11-09 17:54         ` Aritz Bastida
2005-11-09 18:10         ` linux-os (Dick Johnson)

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=7d40d7190511090924k6ef493dbn@mail.gmail.com \
    --to=aritzbastida@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-os@analogic.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

all inboxes | Powered by JetHome®