Messages in this thread | | | Date | Tue, 19 Jun 2007 20:01:57 +0530 | From | "S. P. Prasanna" <> | Subject | Re: [patch 2/2] kprobes x86_64 quick fix mark-ro-data |
| |
On Tue, Jun 19, 2007 at 06:44:30AM -0700, Arjan van de Ven wrote: > On Tue, 2007-06-19 at 09:30 -0400, Mathieu Desnoyers wrote: > > * Arjan van de Ven (arjan@infradead.org) wrote: > > > On Tue, 2007-06-19 at 15:38 +0530, S. P. Prasanna wrote: > > > > This patch is a quick fix for x86_64 to enable kprobes only if DEBUG_RODATA is > > > > not configured. Since DEBUG_RODATA markes the kernel text pages as read-only. > > > > > > > > > it does?? > > > > > > I don't seem to be able to find this in the source code..... > > > > > > See arch/x86_64/mm/init.c:mark_rodata_ro(). > > eh woops > > > PATCH] x86: tighten kernel image page access rights > author > Jan Beulich <jbeulich@novell.com> > > Wed, 2 May 2007 17:27:10 +0000 > (19:27 +0200) > committer > Andi Kleen <andi@basil.nowhere.org> > > Wed, 2 May 2007 17:27:10 +0000 > (19:27 +0200) > commit > 6fb14755a676282a4e6caa05a08c92db8e45cfff > > > changed it to include text (even though Andi vetoed that before when I > asked for it on grounds of breaking kprobes)... sounds this really wants > to be a 2nd config option to seperatedly do code and data.
Something like having a seperate config option and a routine to mark kernel text as read execute only. And call mark_rwtext_ro() if CONFIG_DEBUG_ROTEXT is enabled below in mark_rodata_ro().
/* this code is for i386 architecture*/ static inline void mark_rwtext_ro(void) { unsigned long start = PFN_ALIGN(_text); unsigned long size = PFN_ALIGN(_etext) - start;
#ifdef CONFIG_HOTPLUG_CPU /* It must still be possible to apply SMP alternatives. */ if (num_possible_cpus() <= 1) #endif { change_page_attr(virt_to_page(start), size >> PAGE_SHIFT, PAGE_KERNEL_RX); printk("Write protecting the kernel text: %luk\n", size >> 10); } /* * global_flush_tlb() will be called after marking the data as readonly. */ }
#ifdef CONFIG_DEBUG_RODATA
void mark_rodata_ro(void) { ....
#ifdef CONFIG_DEBUG_ROTEXT mark_rwtext_ro(); #endif ....
}
Thanks Prasanna -- Prasanna S.P. Linux Technology Center India Software Labs, IBM Bangalore Email: prasanna@in.ibm.com Ph: 91-80-41776329 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |