linux-kernel - Re: [Patch 003/005](memory hotplug) make alloc_bootmem

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <86802c440804022252q71af08fere6619271dfe17ee@mail.gmail.com>
Date:	Wed, 2 Apr 2008 22:52:50 -0700
From:	"Yinghai Lu" <yhlu.kernel@...il.com>
To:	"Yasunori Goto" <y-goto@...fujitsu.com>
Cc:	"Badari Pulavarty" <pbadari@...ibm.com>,
	"Andrew Morton" <akpm@...l.org>,
	"Linux Kernel ML" <linux-kernel@...r.kernel.org>,
	linux-mm <linux-mm@...ck.org>
Subject: Re: [Patch 003/005](memory hotplug) make alloc_bootmem_section()

On Wed, Apr 2, 2008 at 10:41 PM, Yasunori Goto <y-goto@...fujitsu.com> wrote:
> alloc_bootmem_section() can allocate specified section's area.
>  This is used for usemap to keep same section with pgdat by later patch.
>
>  Signed-off-by: Yasunori Goto <y-goto@...fujitsu.com>
>
>   include/linux/bootmem.h |    2 ++
>   mm/bootmem.c            |   25 +++++++++++++++++++++++++
>   2 files changed, 27 insertions(+)
>
>  Index: current/include/linux/bootmem.h
>  ===================================================================
>  --- current.orig/include/linux/bootmem.h        2008-04-01 20:56:39.000000000 +0900
>  +++ current/include/linux/bootmem.h     2008-04-01 20:59:02.000000000 +0900
>  @@ -101,6 +101,8 @@
>   extern void free_bootmem_node(pg_data_t *pgdat,
>                               unsigned long addr,
>                               unsigned long size);
>  +extern void *alloc_bootmem_section(unsigned long size,
>  +                                  unsigned long section_nr);
>
>   #ifndef CONFIG_HAVE_ARCH_BOOTMEM_NODE
>   #define alloc_bootmem_node(pgdat, x) \
>  Index: current/mm/bootmem.c
>  ===================================================================
>  --- current.orig/mm/bootmem.c   2008-04-01 20:56:39.000000000 +0900
>  +++ current/mm/bootmem.c        2008-04-01 20:59:02.000000000 +0900
>  @@ -540,6 +540,31 @@
>         return __alloc_bootmem(size, align, goal);
>   }
>
>  +void * __init alloc_bootmem_section(unsigned long size,
>  +                                   unsigned long section_nr)
>  +{
>  +       void *ptr;
>  +       unsigned long limit, goal, start_nr, end_nr, pfn;
>  +       struct pglist_data *pgdat;
>  +
>  +       pfn = section_nr_to_pfn(section_nr);
>  +       goal = PFN_PHYS(pfn);
>  +       limit = PFN_PHYS(section_nr_to_pfn(section_nr + 1)) - 1;
>  +       pgdat = NODE_DATA(early_pfn_to_nid(pfn));
>  +       ptr = __alloc_bootmem_core(pgdat->bdata, size, SMP_CACHE_BYTES, goal,
>  +                                  limit);
>  +
>  +       start_nr = pfn_to_section_nr(PFN_DOWN(__pa(ptr)));
>  +       end_nr = pfn_to_section_nr(PFN_DOWN(__pa(ptr) + size));
>  +       if (start_nr != section_nr || end_nr != section_nr) {
>  +               printk(KERN_WARNING "alloc_bootmem failed on section %ld.\n",
>  +                      section_nr);
>  +               free_bootmem_core(pgdat->bdata, __pa(ptr), size);
>  +               ptr = NULL;
>  +       }

how about __alloc_bootmem_core return NULL?

YH
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/