linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] memblock, arm: fix crashes caused by holes in the memory map
@ 2021-07-04 15:08 Mike Rapoport
  2021-07-04 20:09 ` pr-tracker-bot
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Rapoport @ 2021-07-04 15:08 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Andrew Morton, Kefeng Wang, Mike Rapoport, Mike Rapoport,
	Russell King, Tony Lindgren, linux-arm-kernel, linux-kernel,
	linux-mm

Hi Linus,

The following changes since commit c4681547bcce777daf576925a966ffa824edd09d:

  Linux 5.13-rc3 (2021-05-23 11:42:48 -1000)

are available in the Git repository at:

  ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/rppt/memblock.git tags/memblock-v5.14-rc1

for you to fetch changes up to a4d5613c4dc6d413e0733e37db9d116a2a36b9f3:

  arm: extend pfn_valid to take into account freed memory map alignment (2021-06-30 11:39:00 +0300)

----------------------------------------------------------------
memblock, arm: fix crashes caused by holes in the memory map

The coordination between freeing of unused memory map, pfn_valid() and core
mm assumptions about validity of the memory map in various ranges was not
designed for complex layouts of the physical memory with a lot of holes all
over the place.

Kefen Wang reported crashes in move_freepages() on a system with the
following memory layout [1]:

  node   0: [mem 0x0000000080a00000-0x00000000855fffff]
  node   0: [mem 0x0000000086a00000-0x0000000087dfffff]
  node   0: [mem 0x000000008bd00000-0x000000008c4fffff]
  node   0: [mem 0x000000008e300000-0x000000008ecfffff]
  node   0: [mem 0x0000000090d00000-0x00000000bfffffff]
  node   0: [mem 0x00000000cc000000-0x00000000dc9fffff]
  node   0: [mem 0x00000000de700000-0x00000000de9fffff]
  node   0: [mem 0x00000000e0800000-0x00000000e0bfffff]
  node   0: [mem 0x00000000f4b00000-0x00000000f6ffffff]
  node   0: [mem 0x00000000fda00000-0x00000000ffffefff]

These crashes can be mitigated by enabling CONFIG_HOLES_IN_ZONE on ARM and
essentially turning pfn_valid_within() to pfn_valid() instead of having it
hardwired to 1 on that architecture, but this would require to keep
CONFIG_HOLES_IN_ZONE solely for this purpose.

A cleaner approach is to update ARM's implementation of pfn_valid() to take
into accounting rounding of the freed memory map to pageblock boundaries
and make sure it returns true for PFNs that have memory map entries even if
there is no physical memory backing those PFNs.

[1] https://lore.kernel.org/lkml/2a1592ad-bc9d-4664-fd19-f7448a37edc0@huawei.com

----------------------------------------------------------------
Mike Rapoport (4):
      memblock: free_unused_memmap: use pageblock units instead of MAX_ORDER
      memblock: align freed memory map on pageblock boundaries with SPARSEMEM
      memblock: ensure there is no overflow in memblock_overlaps_region()
      arm: extend pfn_valid to take into account freed memory map alignment

 arch/arm/mm/init.c | 13 ++++++++++++-
 mm/memblock.c      | 26 ++++++++++++++------------
 2 files changed, 26 insertions(+), 13 deletions(-)

-- 
Sincerely yours,
Mike.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GIT PULL] memblock, arm: fix crashes caused by holes in the memory map
  2021-07-04 15:08 [GIT PULL] memblock, arm: fix crashes caused by holes in the memory map Mike Rapoport
@ 2021-07-04 20:09 ` pr-tracker-bot
  0 siblings, 0 replies; 2+ messages in thread
From: pr-tracker-bot @ 2021-07-04 20:09 UTC (permalink / raw)
  To: Mike Rapoport
  Cc: Linus Torvalds, Andrew Morton, Kefeng Wang, Mike Rapoport,
	Mike Rapoport, Russell King, Tony Lindgren, linux-arm-kernel,
	linux-kernel, linux-mm

The pull request you sent on Sun, 4 Jul 2021 18:08:03 +0300:

> ssh://git@gitolite.kernel.org/pub/scm/linux/kernel/git/rppt/memblock.git tags/memblock-v5.14-rc1

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/a412897fb546fbb291095be576165ce757eff70b

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-04 20:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-04 15:08 [GIT PULL] memblock, arm: fix crashes caused by holes in the memory map Mike Rapoport
2021-07-04 20:09 ` pr-tracker-bot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).