Messages in this thread | | | Date | Wed, 28 Oct 2009 08:34:32 +0100 | From | Ingo Molnar <> | Subject | Re: [PATCH 06/10] bootmem: refactor free_all_bootmem_core |
| |
* 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
| |