Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-03 Thread Heiko Carstens
On Thu, Jul 02, 2020 at 09:48:52PM -0700, Andrew Morton wrote: > On Thu, 2 Jul 2020 09:23:10 +0200 David Hildenbrand wrote: > > > >>> --- > > >>> arch/s390/kernel/crash_dump.c | 6 ++-- > > >>> include/linux/memblock.h | 28 ++--- > > >>> mm/memblock.c | 57

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-02 Thread Andrew Morton
On Thu, 2 Jul 2020 09:23:10 +0200 David Hildenbrand wrote: > >>> --- > >>> arch/s390/kernel/crash_dump.c | 6 ++-- > >>> include/linux/memblock.h | 28 ++--- > >>> mm/memblock.c | 57 ++- > >>> 3 files changed, 55 insertions(+), 3

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-02 Thread David Hildenbrand
On 01.07.20 17:31, Heiko Carstens wrote: > On Wed, Jul 01, 2020 at 06:06:43PM +0300, Mike Rapoport wrote: >> Hi David, >> >> On Wed, Jul 01, 2020 at 04:18:29PM +0200, David Hildenbrand wrote: >>> "physmem" in the memblock allocator is somewhat weird: it's not actually >>> used for allocation, it's

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-01 Thread Mike Rapoport
On Wed, Jul 01, 2020 at 05:31:57PM +0200, Heiko Carstens wrote: > On Wed, Jul 01, 2020 at 06:06:43PM +0300, Mike Rapoport wrote: > > Hi David, > > > > On Wed, Jul 01, 2020 at 04:18:29PM +0200, David Hildenbrand wrote: > > > "physmem" in the memblock allocator is somewhat weird: it's not actually >

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-01 Thread Heiko Carstens
> >> While fixing up the documentation, sneak in some related cleanups. We can > >> stop setting CONFIG_HAVE_MEMBLOCK_PHYS_MAP for s390x next. > > As you noted in the previous version it should have been > > CONFIG_ARCH_KEEP_MEMBLOCK ;-) > Grml :) maybe maintainers can fix that up when applying in

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-01 Thread David Hildenbrand
> Am 01.07.2020 um 17:07 schrieb Mike Rapoport : > > Hi David, > >> On Wed, Jul 01, 2020 at 04:18:29PM +0200, David Hildenbrand wrote: >> "physmem" in the memblock allocator is somewhat weird: it's not actually >> used for allocation, it's simply information collected during boot, which >> de

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-01 Thread Heiko Carstens
On Wed, Jul 01, 2020 at 06:06:43PM +0300, Mike Rapoport wrote: > Hi David, > > On Wed, Jul 01, 2020 at 04:18:29PM +0200, David Hildenbrand wrote: > > "physmem" in the memblock allocator is somewhat weird: it's not actually > > used for allocation, it's simply information collected during boot, whi

Re: [PATCH v2 1/2] mm/memblock: expose only miminal interface to add/walk physmem

2020-07-01 Thread Mike Rapoport
Hi David, On Wed, Jul 01, 2020 at 04:18:29PM +0200, David Hildenbrand wrote: > "physmem" in the memblock allocator is somewhat weird: it's not actually > used for allocation, it's simply information collected during boot, which > describes the unmodified physical memory map at boot time, without a