Re: [PATCH v3 0/2] sparse_init rewrite

2018-07-02 Thread Pavel Tatashin
On Mon, Jul 2, 2018 at 12:20 PM Dave Hansen wrote: > > On 07/01/2018 07:04 PM, Pavel Tatashin wrote: > > include/linux/mm.h | 9 +- > > mm/sparse-vmemmap.c | 44 --- > > mm/sparse.c | 279 +++- > > 3 files changed, 125 insertions(+), 207 del

Re: [PATCH v3 0/2] sparse_init rewrite

2018-07-02 Thread Dave Hansen
On 07/01/2018 07:04 PM, Pavel Tatashin wrote: > include/linux/mm.h | 9 +- > mm/sparse-vmemmap.c | 44 --- > mm/sparse.c | 279 +++- > 3 files changed, 125 insertions(+), 207 deletions(-) FWIW, I'm not a fan of rewrites, but this is an awful

[PATCH v3 0/2] sparse_init rewrite

2018-07-01 Thread Pavel Tatashin
Changelog: v3 - v1 - Fixed two issues found by Baoquan He v1 - v2 - Addressed comments from Oscar Salvador In sparse_init() we allocate two large buffers to temporary hold usemap and memmap for the whole machine. However, we can avoid doing that if we changed sparse_init() to opera