Re: Re: modules, add_kallsyms() && DEFINE_PER_CPU

2013-11-27 Thread Masami Hiramatsu
(2013/11/27 22:35), Oleg Nesterov wrote: > On 11/27, Masami Hiramatsu wrote: >> >> (2013/11/27 2:50), Oleg Nesterov wrote: >>> On 11/26, Oleg Nesterov wrote: Note: this doesn't work for modules, but module's per-cpu data is not visible for kallsyms_lookup_name() anyway. >>> >>> Rusty

Re: modules, add_kallsyms() && DEFINE_PER_CPU

2013-11-27 Thread Oleg Nesterov
On 11/27, Masami Hiramatsu wrote: > > (2013/11/27 2:50), Oleg Nesterov wrote: > > On 11/26, Oleg Nesterov wrote: > >> > >> Note: this doesn't work for modules, but module's per-cpu data is > >> not visible for kallsyms_lookup_name() anyway. > > > > Rusty, I am just curious if it makes sense to chan

Re: modules, add_kallsyms() && DEFINE_PER_CPU

2013-11-27 Thread Masami Hiramatsu
(2013/11/27 17:20), Namhyung Kim wrote: > Hi Masami, > > On Wed, 27 Nov 2013 11:23:43 +0900, Masami Hiramatsu wrote: >> (2013/11/27 2:50), Oleg Nesterov wrote: >>> On 11/26, Oleg Nesterov wrote: Note: this doesn't work for modules, but module's per-cpu data is not visible for kallsy

Re: modules, add_kallsyms() && DEFINE_PER_CPU

2013-11-27 Thread Namhyung Kim
Hi Masami, On Wed, 27 Nov 2013 11:23:43 +0900, Masami Hiramatsu wrote: > (2013/11/27 2:50), Oleg Nesterov wrote: >> On 11/26, Oleg Nesterov wrote: >>> >>> Note: this doesn't work for modules, but module's per-cpu data is >>> not visible for kallsyms_lookup_name() anyway. >> >> Rusty, I am just cu

Re: modules, add_kallsyms() && DEFINE_PER_CPU

2013-11-26 Thread Masami Hiramatsu
(2013/11/27 2:50), Oleg Nesterov wrote: > On 11/26, Oleg Nesterov wrote: >> >> Note: this doesn't work for modules, but module's per-cpu data is >> not visible for kallsyms_lookup_name() anyway. > > Rusty, I am just curious if it makes sense to change this or not... > > But DEFINE_PER_CPU'ed symb

modules, add_kallsyms() && DEFINE_PER_CPU

2013-11-26 Thread Oleg Nesterov
On 11/26, Oleg Nesterov wrote: > > Note: this doesn't work for modules, but module's per-cpu data is > not visible for kallsyms_lookup_name() anyway. Rusty, I am just curious if it makes sense to change this or not... But DEFINE_PER_CPU'ed symbols are ignored by add_kallsyms(). I guess this is be