All of lore.kernel.org
 help / color / mirror / Atom feed
From: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
To: ksummit-discuss@lists.linuxfoundation.org
Subject: Re: [Ksummit-discuss] No more module removal -- Unconference track
Date: Mon, 25 Aug 2014 20:01:59 +0900	[thread overview]
Message-ID: <53FB17A7.9090209@hitachi.com> (raw)
In-Reply-To: <20140819145547.GB18536@roeck-us.net>

(2014/08/19 23:55), Guenter Roeck wrote:
> On Tue, Aug 19, 2014 at 10:48:39AM -0400, Theodore Ts'o wrote:
>> This has been scheduled at 2pm at the request of Rusty.  (Reminder: if
>> you're going to propose a topic, please send e-mail to start a thread
>> on ksummit-discuss).
>>
> Do we have a context ? I am using insert/remove module a lot during testing,
> and would hate to see it go. It also permits module updates without having to
> reboot the kernel. There must be lots of other reasons to support module
> removal. So I would really dislike if it was no longer available, and I don't
> really see the point.

I have to explain why, since I asked Rusty to improving removing.

What I found is that the module unloading involving 2 stop_machine()s
for each module removing. It must not be needed. However, since the
module's ref-counter is over-optimized for BIG SMP machine, we can't
remove it without replacing it. But it means some performance
regression can happen on such big-scale SMP machines (not the laptop nor
normal smp machine).
So I asked him to introduce something like lock-up option which locks
up the given module, and the kernel skips ref-counting on that module.

Anyway, I sent the series right now :)

https://lkml.org/lkml/2014/8/25/142

Please check it if it is good for your use-cases.

Thank you,

-- 
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@hitachi.com

  parent reply	other threads:[~2014-08-25 11:02 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-19 14:48 [Ksummit-discuss] No more module removal -- Unconference track Theodore Ts'o
2014-08-19 14:55 ` Guenter Roeck
2014-08-19 15:23   ` Theodore Ts'o
2014-08-19 15:40     ` Dan Carpenter
2014-08-19 15:47       ` Guenter Roeck
2014-08-19 16:09         ` Dan Carpenter
2014-08-19 16:34           ` Martin K. Petersen
2014-08-19 16:59           ` Rik van Riel
2014-08-19 17:19           ` Guenter Roeck
2014-08-19 16:43         ` David Woodhouse
2014-08-19 17:13           ` Grant Likely
2014-08-19 17:23             ` David Woodhouse
2014-08-19 15:54     ` Takashi Iwai
2014-08-25 11:01   ` Masami Hiramatsu [this message]
2014-08-25 11:05     ` Jiri Kosina
2014-08-26  5:24       ` Masami Hiramatsu
2014-08-27 23:05         ` Theodore Ts'o
2014-08-26 21:39     ` David Howells
2014-08-26 21:45       ` Jiri Kosina
2014-08-27  6:43         ` Masami Hiramatsu

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=53FB17A7.9090209@hitachi.com \
    --to=masami.hiramatsu.pt@hitachi.com \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.