diff options
| author | Michael Ellerman <[email protected]> | 2020-12-17 11:53:06 +1100 |
|---|---|---|
| committer | Michael Ellerman <[email protected]> | 2021-01-30 11:39:25 +1100 |
| commit | 7613f5a66becfd0e43a0f34de8518695888f5458 (patch) | |
| tree | 79cd8aeccdf14be69adf62f05fce5b3238ffffa4 /tools/perf/scripts/python/bin | |
| parent | 52f6b0a90bcf573ba8a33e97544c7b6f292376a4 (diff) | |
powerpc/64s/kuap: Use mmu_has_feature()
In commit 8150a153c013 ("powerpc/64s: Use early_mmu_has_feature() in
set_kuap()") we switched the KUAP code to use early_mmu_has_feature(),
to avoid a bug where we called set_kuap() before feature patching had
been done, leading to recursion and crashes.
That path, which called probe_kernel_read() from printk(), has since
been removed, see commit 2ac5a3bf7042 ("vsprintf: Do not break early
boot with probing addresses").
Additionally probe_kernel_read() no longer invokes any KUAP routines,
since commit fe557319aa06 ("maccess: rename probe_kernel_{read,write}
to copy_{from,to}_kernel_nofault") and c33165253492 ("powerpc: use
non-set_fs based maccess routines").
So it should now be safe to use mmu_has_feature() in the KUAP
routines, because we shouldn't invoke them prior to feature patching.
This is essentially a revert of commit 8150a153c013 ("powerpc/64s: Use
early_mmu_has_feature() in set_kuap()"), but we've since added a
second usage of early_mmu_has_feature() in get_kuap(), so we convert
that to use mmu_has_feature() as well.
Reported-by: Christophe Leroy <[email protected]>
Signed-off-by: Michael Ellerman <[email protected]>
Depends-on: c33165253492 ("powerpc: use non-set_fs based maccess routines").
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'tools/perf/scripts/python/bin')
0 files changed, 0 insertions, 0 deletions