From mboxrd@z Thu Jan 1 00:00:00 1970 From: Masami Hiramatsu Subject: Re: Re: [PATCH -tip RFC v2 01/22] kprobes: Prohibit probing on .entry.text code Date: Sun, 17 Nov 2013 16:21:27 +0900 Message-ID: <52886E77.1000707@hitachi.com> References: <20131115045312.27580.95902.stgit@kbuild-fedora.novalocal> <20131115045318.27580.69554.stgit@kbuild-fedora.novalocal> <20131115114306.5d14db66@gandalf.local.home> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Geert Uytterhoeven Cc: Linux-Arch , Ananth N Mavinakayanahalli , "David S. Miller" , Peter Zijlstra , Frederic Weisbecker , the arch/x86 maintainers , lkml , Steven Rostedt , virtualization@lists.linux-foundation.org, Ingo Molnar , Al Viro , "H. Peter Anvin" , Thomas Gleixner , Seiji Aguchi , Ingo Molnar List-Id: virtualization@lists.linuxfoundation.org (2013/11/16 2:46), Geert Uytterhoeven wrote: > On Fri, Nov 15, 2013 at 5:43 PM, Steven Rostedt wrote: >> On Fri, 15 Nov 2013 04:53:18 +0000 >> Masami Hiramatsu wrote: >> >>> .entry.text is a code area which is used for interrupt/syscall >>> entries, and there are many sensitive codes. >>> Thus, it is better to prohibit probing on all of such codes >>> instead of a part of that. >>> Since some symbols are already registered on kprobe blacklist, >>> this also removes them from the blacklist. >> >> This change only works with x86. On other archs, I get this: >> >> kernel/built-in.o: In function `register_kprobe': >> (.kprobes.text+0x9f4): undefined reference to `__entry_text_start' >> kernel/built-in.o: In function `register_kprobe': >> (.kprobes.text+0x9f8): undefined reference to `__entry_text_end' >> make[1]: *** [vmlinux] Error 1 >> make: *** [sub-make] Error 2 > > X86 is the sole architecture that has ENTRY_TEXT in its > arch/*/kernel/vmlinux.lds.S. Ah, thanks! I'll fix that. BTW, should I add CONFIG_HAVE_ENTRY_TEXT for x86 and use it or just use CONFIG_X86 to detect it? Thank you, -- Masami HIRAMATSU IT Management Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com