[PATCH v5] rtems-debugger: Fixed 32bit pointers
chrisj at rtems.org
Sun May 9 01:50:25 UTC 2021
On 8/5/21 6:13 am, Stephen Clark wrote:
> Using 32bit types like uint32_t for pointers creates issues on 64 bit
> architectures like AArch64. Replaced occurrences of these with uintptr_t,
> which will work for both 32 and 64 bit architectures. Added hex_decode_addr
> function to rtems-debugger. Changed rtems_debugger_target_swbreak_control()
> parameter to DB_ADDR.
The change seems to mix things. I originally added the DB_* defines in case
there needed to be a way to handle arch specifics and it looks like there does
not need to be.
If the interface to gdb can be handled by suitable types then I suggest the
macros be removed and only the types be used. I think mixing things like only
adds a layer of confusion.
More information about the devel