Re: Re: [PATCH -tip 3/3] x86: Remove unused text_poke_smp and text_poke_smp_batch

[Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>]

(2013/07/22 17:21), Ingo Molnar wrote:
> 
> * Ingo Molnar <mingo@kernel.org> wrote:
> 
>>
>> * Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote:
>>
>>> Since introducing the text_poke_bp for all text_poke_smp*
>>> callers, text_poke_smp* are now unused. This basically
>>> reverts 3d55cc8a, 7deb18dc and related commits.
>>>
>>> This also fixes a Kconfig dependency issue on STOP_MACHINE
>>> in the case of CONFIG_SMP && !CONFIG_MODULE_UNLOAD.
>>>
>>> Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
>>> ---
>>>  arch/x86/Kconfig                   |    5 --
>>>  arch/x86/include/asm/alternative.h |   11 ----
>>>  arch/x86/kernel/alternative.c      |   98 +-----------------------------------
>>>  3 files changed, 2 insertions(+), 112 deletions(-)
>>
>> Hm, it does not build with the attached config:
>>
>> arch/x86/kernel/kprobes/opt.c: In function 'setup_optimize_kprobe':
>> arch/x86/kernel/kprobes/opt.c:394:6: error: dereferencing pointer to incomplete type
>> arch/x86/kernel/kprobes/opt.c:395:6: error: dereferencing pointer to incomplete type
>> arch/x86/kernel/kprobes/opt.c:396:6: error: dereferencing pointer to incomplete type
> 
> 32-bit appears to be the key pattern.

Ingo, please make sure your x86/jumplabel tree picked all of my patches.
This 3/3 strongly depends on the previous 1/3 and 2/3.
Without it, kprobes/opt.c can not be build because it still uses the
text_poke_params data structure which 3/3 removes.

Thank you,

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