diff options
| author | Benjamin Gray <bgray@linux.ibm.com> | 2022-11-09 15:51:05 +1100 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2022-11-30 21:46:48 +1100 |
| commit | 071c95c1acbd96e76bab8b25b5cad0d71a011f37 (patch) | |
| tree | ba49701bb0bb0f32c37250a8e535f7141c1df247 /tools/perf/scripts/python | |
| parent | powerpc: Allow clearing and restoring registers independent of saved breakpoi... (diff) | |
| download | linux-071c95c1acbd96e76bab8b25b5cad0d71a011f37.tar.gz linux-071c95c1acbd96e76bab8b25b5cad0d71a011f37.zip | |
powerpc/code-patching: Use WARN_ON and fix check in poking_init
BUG_ON() when failing to initialise the code patching window is
unnecessary, and use of BUG_ON is discouraged. We don't set
poking_init_done in this case, so failure to init the boot CPU will
result in a strict RWX error when a following patch_instruction uses
raw_patch_instruction. If it only fails for later CPUs, they won't be
onlined in the first place.
The return value of cpuhp_setup_state() is also >= 0 on success,
so check for < 0.
Signed-off-by: Benjamin Gray <bgray@linux.ibm.com>
Reviewed-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20221109045112.187069-3-bgray@linux.ibm.com
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions