From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Mosberger Date: Fri, 24 Jun 2005 19:10:38 +0000 Subject: Re: [patch][ia64]Refuse kprobe on ivt code Message-Id: <17084.23214.237084.224128@napali.hpl.hp.com> List-Id: References: <20050623172832.B26121@unix-os.sc.intel.com> <17083.25625.991516.736507@napali.hpl.hp.com> <20050624114545.A4826@unix-os.sc.intel.com> In-Reply-To: <20050624114545.A4826@unix-os.sc.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Keshavamurthy Anil S Cc: davidm@hpl.hp.com, akpm@osdl.org, Linux Kernel , Linux IA64 >>>>> On Fri, 24 Jun 2005 11:45:46 -0700, Keshavamurthy Anil S said: Anil> On Thu, Jun 23, 2005 at 06:38:33PM -0700, David Mosberger wrote: >> Please do the checking based on the .text.ivt section instead (and add >> the necessary labels to vmlinux.S and asm-ia64/sections.h). Anil> Subject: Refuse kprobe insert on IVT code Anil> Not safe to insert kprobes on IVT code. Anil> This patch checks to see if the address on which Kprobes is being Anil> inserted is in ivt code and if it is in ivt code then Anil> refuse to register kprobe. Anil> Take 1: This patch is based on review comments from David Mosberger, Anil> now checking based on .text.ivt Anil> Signed-off-by: Anil S Keshavamurthy Looks fine, except: Anil> +/* Returns non-zero if the addr is in the Interrupt Vector Table */ Anil> +static inline int in_ivt_functions(unsigned long addr) Anil> +{ Anil> + extern char __start_ivt_text[], __end_ivt_text[]; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Anil> + return (addr >= (unsigned long)__start_ivt_text Anil> + && addr < (unsigned long)__end_ivt_text); Anil> +} Surely you meant to use the declaration from sections.h instead? Thanks, --david