diff options
author | Pierre-Clément Tosi <ptosi@google.com> | 2024-03-18 19:35:49 +0000 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2024-04-10 08:41:00 -0600 |
commit | 4b36f01201bfb62aba2669230436e5012dc60533 (patch) | |
tree | ec9de780e35f3870266325aefd17566ad9e585f7 /cmd/sysboot.c | |
parent | 2d357e4dbea0dd2fd23002bdf2e18f901aec2d99 (diff) |
arm64: Fix map_range() not splitting mapped blocks
The implementation of map_range() creates the requested mapping by
walking the page tables, iterating over multiple PTEs and/or descending
into existing table mappings as needed. When doing so, it assumes any
pre-existing valid PTE to be a table mapping. This assumption is wrong
if the platform code attempts to successively map two overlapping ranges
where the latter intersects a block mapping created for the former.
As a result, map_range() treats the existing block mapping as a table
mapping and descends into it i.e. starts interpreting the
previously-mapped range as an array of PTEs, writing to them and
potentially even descending further (extra fun with MMIO ranges!).
Instead, pass any valid non-table mapping to split_block(), which
ensures that it actually was a block mapping (calls panic() otherwise)
before splitting it.
Fixes: 41e2787f5ec4 ("arm64: Reduce add_map() complexity")
Signed-off-by: Pierre-Clément Tosi <ptosi@google.com>
Tested-by: Fabio Estevam <festevam@gmail.com>
Tested-by: Hiago De Franco <hiago.franco@toradex.com> # Toradex Verdin AM62
Reviewed-by: Marc Zyngier <maz@kernel.org>
Diffstat (limited to 'cmd/sysboot.c')
0 files changed, 0 insertions, 0 deletions