Re: [RFC PATCH 0/2] kmod: Support lockup option to make module un-removable

2014-10-12 Thread Rusty Russell
Lucas De Marchi  writes:
> On Tue, Aug 26, 2014 at 9:04 AM, Masami Hiramatsu
>  wrote:
>> Hi,
>>
>> Here is a pair of patches which adds --lockup option to
>> modprobe and libkmod.
>>
>> As I sent a series of patches which removes stop_machine()
>> from module removal: https://lkml.org/lkml/2014/8/25/142
>> it also adds lockup option which lock up the module in
>> the kernel and makes it un-removable.
>>
>> These patches enables us to use that option when loading
>> modules. Module lockup may be good for BIG SMP machines
>> since the kernel skips module refcounting if the module
>> is locked up :)
>>
>> Anyway, this is not needed if the lockup option is dropped
>> from the series. I send this for testing.
>
> Ok. I'm not sure it's clear... I'm waiting for feedback on the kernel
> patches in order to proceed with any review here. I'm not really
> convinced we want this option when loading a module.
>
> Rusty, what do you think?

I'm not convinced, I asked him to drop that patch.  If we have
significant performance issues, we'll have to do something smarter I
think anyway.

Cheers,
Rusty.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC PATCH 0/2] kmod: Support lockup option to make module un-removable

2014-10-12 Thread Rusty Russell
Lucas De Marchi lucas.de.mar...@gmail.com writes:
 On Tue, Aug 26, 2014 at 9:04 AM, Masami Hiramatsu
 masami.hiramatsu...@hitachi.com wrote:
 Hi,

 Here is a pair of patches which adds --lockup option to
 modprobe and libkmod.

 As I sent a series of patches which removes stop_machine()
 from module removal: https://lkml.org/lkml/2014/8/25/142
 it also adds lockup option which lock up the module in
 the kernel and makes it un-removable.

 These patches enables us to use that option when loading
 modules. Module lockup may be good for BIG SMP machines
 since the kernel skips module refcounting if the module
 is locked up :)

 Anyway, this is not needed if the lockup option is dropped
 from the series. I send this for testing.

 Ok. I'm not sure it's clear... I'm waiting for feedback on the kernel
 patches in order to proceed with any review here. I'm not really
 convinced we want this option when loading a module.

 Rusty, what do you think?

I'm not convinced, I asked him to drop that patch.  If we have
significant performance issues, we'll have to do something smarter I
think anyway.

Cheers,
Rusty.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC PATCH 0/2] kmod: Support lockup option to make module un-removable

2014-09-01 Thread Lucas De Marchi
On Tue, Aug 26, 2014 at 9:04 AM, Masami Hiramatsu
 wrote:
> Hi,
>
> Here is a pair of patches which adds --lockup option to
> modprobe and libkmod.
>
> As I sent a series of patches which removes stop_machine()
> from module removal: https://lkml.org/lkml/2014/8/25/142
> it also adds lockup option which lock up the module in
> the kernel and makes it un-removable.
>
> These patches enables us to use that option when loading
> modules. Module lockup may be good for BIG SMP machines
> since the kernel skips module refcounting if the module
> is locked up :)
>
> Anyway, this is not needed if the lockup option is dropped
> from the series. I send this for testing.

Ok. I'm not sure it's clear... I'm waiting for feedback on the kernel
patches in order to proceed with any review here. I'm not really
convinced we want this option when loading a module.

Rusty, what do you think?

-- 
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC PATCH 0/2] kmod: Support lockup option to make module un-removable

2014-09-01 Thread Lucas De Marchi
On Tue, Aug 26, 2014 at 9:04 AM, Masami Hiramatsu
masami.hiramatsu...@hitachi.com wrote:
 Hi,

 Here is a pair of patches which adds --lockup option to
 modprobe and libkmod.

 As I sent a series of patches which removes stop_machine()
 from module removal: https://lkml.org/lkml/2014/8/25/142
 it also adds lockup option which lock up the module in
 the kernel and makes it un-removable.

 These patches enables us to use that option when loading
 modules. Module lockup may be good for BIG SMP machines
 since the kernel skips module refcounting if the module
 is locked up :)

 Anyway, this is not needed if the lockup option is dropped
 from the series. I send this for testing.

Ok. I'm not sure it's clear... I'm waiting for feedback on the kernel
patches in order to proceed with any review here. I'm not really
convinced we want this option when loading a module.

Rusty, what do you think?

-- 
Lucas De Marchi
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[RFC PATCH 0/2] kmod: Support lockup option to make module un-removable

2014-08-26 Thread Masami Hiramatsu
Hi,

Here is a pair of patches which adds --lockup option to
modprobe and libkmod.

As I sent a series of patches which removes stop_machine()
from module removal: https://lkml.org/lkml/2014/8/25/142
it also adds lockup option which lock up the module in
the kernel and makes it un-removable.

These patches enables us to use that option when loading
modules. Module lockup may be good for BIG SMP machines
since the kernel skips module refcounting if the module
is locked up :)

Anyway, this is not needed if the lockup option is dropped
from the series. I send this for testing.

Thank you,

---

Masami Hiramatsu (2):
  libkmod: support lockup module option
  modprobe: Add --lockup option to make module unremovable


 libkmod/libkmod-module.c |3 +++
 libkmod/libkmod.h|4 +++-
 libkmod/missing.h|4 
 tools/modprobe.c |   10 +-
 4 files changed, 19 insertions(+), 2 deletions(-)

--

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[RFC PATCH 0/2] kmod: Support lockup option to make module un-removable

2014-08-26 Thread Masami Hiramatsu
Hi,

Here is a pair of patches which adds --lockup option to
modprobe and libkmod.

As I sent a series of patches which removes stop_machine()
from module removal: https://lkml.org/lkml/2014/8/25/142
it also adds lockup option which lock up the module in
the kernel and makes it un-removable.

These patches enables us to use that option when loading
modules. Module lockup may be good for BIG SMP machines
since the kernel skips module refcounting if the module
is locked up :)

Anyway, this is not needed if the lockup option is dropped
from the series. I send this for testing.

Thank you,

---

Masami Hiramatsu (2):
  libkmod: support lockup module option
  modprobe: Add --lockup option to make module unremovable


 libkmod/libkmod-module.c |3 +++
 libkmod/libkmod.h|4 +++-
 libkmod/missing.h|4 
 tools/modprobe.c |   10 +-
 4 files changed, 19 insertions(+), 2 deletions(-)

--

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/