Re: [PATCH 06/10] bootmem: refactor free_all_bootmem_core

Ingo Molnar
2009-10-28T07:37:03Z

* FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> wrote:

> From: Chris Wright <chrisw@sous-sol.org>
>
> Move the loop that frees all bootmem pages back to page allocator into
> its own function. This should have not functional effect and allows the
> function to be reused later.
>
> Reviewed-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
> Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Small signoff-handling sidenote: in such cases (when you are handling
other people's patches) the expected workflow is to add your SOB line
after Chris's line, instead of the Reviewed-by line.

I.e. it should look like this:

Signed-off-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>

Showing the path of the patch - ending with your signoff.

Thanks,

Ingo