diff options
| author | Ada Couprie Diaz <[email protected]> | 2025-07-07 11:41:04 +0000 |
|---|---|---|
| committer | Will Deacon <[email protected]> | 2025-07-08 12:27:42 +0000 |
| commit | 80691d35523de3292b64c2ffa444aab3d55e51ba (patch) | |
| tree | ee62bb1240ff0bd6f33816a32aba2158b7779f5b /drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | |
| parent | arm64: debug: split hardware breakpoint exception entry (diff) | |
| download | kernel-80691d35523de3292b64c2ffa444aab3d55e51ba.tar.gz kernel-80691d35523de3292b64c2ffa444aab3d55e51ba.zip | |
arm64: debug: refactor reinstall_suspended_bps()
`reinstall_suspended_bps()` plays a key part in the stepping process
when we have hardware breakpoints and watchpoints enabled.
It checks if we need to step one, will re-enable it if it has
been handled and will return whether or not we need to proceed with
a single-step.
However, the current naming and return values make it harder to understand
the logic and goal of the function.
Rename it `try_step_suspended_breakpoints()` and change the return value
to a boolean, aligning it with similar functions used in
`do_el0_undef()` like `try_emulate_mrs()`, and making its behaviour
more obvious.
Signed-off-by: Ada Couprie Diaz <[email protected]>
Tested-by: Luis Claudio R. Goncalves <[email protected]>
Reviewed-by: Anshuman Khandual <[email protected]>
Reviewed-by: Will Deacon <[email protected]>
Acked-by: Mark Rutland <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Will Deacon <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c')
0 files changed, 0 insertions, 0 deletions
