Messages in this thread | | | Date | Thu, 31 Jul 2014 16:50:49 +0800 | From | Baoquan He <> | Subject | Re: [PATCH v2] not adding modules range to kcore if it's equal to vmcore range |
| |
On 07/15/14 at 03:16pm, Andrew Morton wrote: > On Sun, 13 Jul 2014 09:05:46 +0800 Baoquan He <bhe@redhat.com> wrote: > > > On some ARCHs modules range is eauql to vmalloc range. E.g on i686 > > "#define MODULES_VADDR VMALLOC_START" > > "#define MODULES_END VMALLOC_END" > > This will cause 2 duplicate program segments in /proc/kcore, makes > > user confused. In this patch a judgment added to check if modules > > range is equal to vmalloc range. If yes, just skip adding the modules > > range. > > > > ... > > > > --- a/fs/proc/kcore.c > > +++ b/fs/proc/kcore.c > > @@ -610,8 +610,10 @@ static void __init proc_kcore_text_init(void) > > struct kcore_list kcore_modules; > > static void __init add_modules_range(void) > > { > > - kclist_add(&kcore_modules, (void *)MODULES_VADDR, > > - MODULES_END - MODULES_VADDR, KCORE_VMALLOC); > > + if (MODULES_VADDR != VMALLOC_START) { > > + kclist_add(&kcore_modules, (void *)MODULES_VADDR, > > + MODULES_END - MODULES_VADDR, KCORE_VMALLOC); > > + } > > } > > #else > > static void __init add_modules_range(void) > > But if some application or script is using the modules range, won't > this patch cause breakage on some architectures?
Hi Andrew,
As I replied in previous mail, people usually check memory range but not which entry is available, so this should not cause breakage on the relevant architectures.
Surely if you don't like it, I am fine too.
Baoquan Thanks
>
| |