>>>>> On Fri, 24 Jun 2005 11:45:46 -0700, Keshavamurthy Anil S <[email protected]> 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 <[email protected]>
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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
