Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-07-08 Thread Philip Li
gt;Cc: Wei Yang ; Williams, Dan J > > >; a...@linux-foundation.org; > > >kbuild-...@lists.01.org; > > >clang-built-li...@googlegroups.com; linux...@kvack.org; linux- > > >ker...@vger.kernel.org > > >Subject: Re: [PATCH] mm/sparse: only sub-section aligned range

Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-07-08 Thread Nick Desaulniers
gt;clang-built-li...@googlegroups.com; linux...@kvack.org; linux- > >ker...@vger.kernel.org > >Subject: Re: [PATCH] mm/sparse: only sub-section aligned range would be > >populated > > > >On Wed, Jul 01, 2020 at 02:11:10AM +0800, kernel test robot wrote: > >>H

Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-07-01 Thread Wei Yang
On Wed, Jul 01, 2020 at 02:11:10AM +0800, kernel test robot wrote: >Hi Wei, > >Thank you for the patch! Perhaps something to improve: > >[auto build test WARNING on mmotm/master] > >url: >https://github.com/0day-ci/linux/commits/Wei-Yang/mm-sparse-only-sub-section-aligned-range-would-be-populat

Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-07-01 Thread David Hildenbrand
On 01.07.20 04:32, Wei Yang wrote: > On Tue, Jun 30, 2020 at 02:52:35PM +0200, David Hildenbrand wrote: >> On 30.06.20 04:14, Wei Yang wrote: >>> There are two code path which invoke __populate_section_memmap() >>> >>> * sparse_init_nid() >>> * sparse_add_section() >>> >>> For both case, we are

Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-06-30 Thread Wei Yang
On Tue, Jun 30, 2020 at 02:52:35PM +0200, David Hildenbrand wrote: >On 30.06.20 04:14, Wei Yang wrote: >> There are two code path which invoke __populate_section_memmap() >> >> * sparse_init_nid() >> * sparse_add_section() >> >> For both case, we are sure the memory range is sub-section align

Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-06-30 Thread kernel test robot
Hi Wei, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on mmotm/master] url: https://github.com/0day-ci/linux/commits/Wei-Yang/mm-sparse-only-sub-section-aligned-range-would-be-populated/20200630-101713 base: git://git.cmpxchg.org/linux-mmotm.git master con

Re: [PATCH] mm/sparse: only sub-section aligned range would be populated

2020-06-30 Thread David Hildenbrand
On 30.06.20 04:14, Wei Yang wrote: > There are two code path which invoke __populate_section_memmap() > > * sparse_init_nid() > * sparse_add_section() > > For both case, we are sure the memory range is sub-section aligned. > > * we pass PAGES_PER_SECTION to sparse_init_nid() > * we check

[PATCH] mm/sparse: only sub-section aligned range would be populated

2020-06-29 Thread Wei Yang
There are two code path which invoke __populate_section_memmap() * sparse_init_nid() * sparse_add_section() For both case, we are sure the memory range is sub-section aligned. * we pass PAGES_PER_SECTION to sparse_init_nid() * we check range by check_pfn_span() before calling sparse_