From mboxrd@z Thu Jan  1 00:00:00 1970
From: sophian.yoma@gmail.com (yoma sophian)
Date: Wed, 6 May 2015 01:50:22 +0800
Subject: About bootmem allocation/freeing flow
In-Reply-To: <CADUS3omZWb1OjWpc=rYibswsQGZqbSMOLrewqt5BK1MtL8njBQ@mail.gmail.com>
References: <CADUS3omZWb1OjWpc=rYibswsQGZqbSMOLrewqt5BK1MtL8njBQ@mail.gmail.com>
Message-ID: <CADUS3onK-GG=7_6wWUtPFY58Et2EeUdND+s0qzTGB8te-eeEVA@mail.gmail.com>
To: kernelnewbies@lists.kernelnewbies.org
List-Id: kernelnewbies.lists.kernelnewbies.org

2015-04-18 15:28 GMT+08:00 yoma sophian <sophian.yoma@gmail.com>:
> hi all:
> I have several questions about free_all_bootmem_core:
>
> 1.
> In __free_pages_bootmem, we set set_page_count(p, 0) while looping nr_pages,
> why we need to set_page_refcounted(page) before calling __free_pages?
below is excerpted from mm/page_alloc.c  and mm/internal.h
the reason why we use set_page_refcounted(page) is because
set_page_refcounted(page) will calling VM_BUG_ON
to checking page property?

static inline void set_page_refcounted(struct page *page)
{
        VM_BUG_ON(PageTail(page));
        VM_BUG_ON(atomic_read(&page->_
count));
        set_page_count(page, 1);
}

void __meminit __free_pages_bootmem(struct page *page, unsigned int order)
{
        unsigned int nr_pages = 1 << order;
        unsigned int loop;

        prefetchw(page);
        for (loop = 0; loop < nr_pages; loop++) {
                struct page *p = &page[loop];

                if (loop + 1 < nr_pages)
                        prefetchw(p + 1);
                __ClearPageReserved(p);
                set_page_count(p, 0);
        }

        page_zone(page)->managed_pages += 1 << order;
        set_page_refcounted(page);
        __free_pages(page, order);
}

thanks for your help,