#
c566080c |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move syscall handling to new file
Our GDB syscall support is the last chunk of code that needs target specific support so move it to a new file. We take the opportunity to move the syscall
gdbstub: move syscall handling to new file
Our GDB syscall support is the last chunk of code that needs target specific support so move it to a new file. We take the opportunity to move the syscall state into its own singleton instance and add in a few helpers for the main gdbstub to interact with the module.
I also moved the gdb_exit() declaration into syscalls.h as it feels pretty related and most of the callers of it treat it as such.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-22-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-22-richard.henderson@linaro.org>
show more ...
|
#
505601d5 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: specialise stub_can_reverse
Currently we only support replay for softmmu mode so it is a constant false for user-mode.
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by:
gdbstub: specialise stub_can_reverse
Currently we only support replay for softmmu mode so it is a constant false for user-mode.
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-18-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-18-richard.henderson@linaro.org>
show more ...
|
#
7ea0c33d |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: introduce gdb_get_max_cpus
This is needed for handling vcont packets as the way of calculating max cpus vhanges between user and softmmu mode.
Reviewed-by: Richard Henderson <richard.hende
gdbstub: introduce gdb_get_max_cpus
This is needed for handling vcont packets as the way of calculating max cpus vhanges between user and softmmu mode.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-17-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-17-richard.henderson@linaro.org>
show more ...
|
#
589a5867 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: specialise target_memory_rw_debug
The two implementations are different enough to encourage having a specialisation and we can move some of the softmmu only stuff out of gdbstub.
Reviewed-
gdbstub: specialise target_memory_rw_debug
The two implementations are different enough to encourage having a specialisation and we can move some of the softmmu only stuff out of gdbstub.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20230302190846.2593720-16-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-16-richard.henderson@linaro.org>
show more ...
|
#
8a2025b3 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: specialise handle_query_attached
In both user and softmmu cases we are just replying with a constant.
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Richard Henderson
gdbstub: specialise handle_query_attached
In both user and softmmu cases we are just replying with a constant.
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-15-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-15-richard.henderson@linaro.org>
show more ...
|
#
a7e0f9bd |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: abstract target specific details from gdb_put_packet_binary
We unfortunately handle the checking of packet acknowledgement differently for user and softmmu modes. Abstract the user mode stu
gdbstub: abstract target specific details from gdb_put_packet_binary
We unfortunately handle the checking of packet acknowledgement differently for user and softmmu modes. Abstract the user mode stuff behind gdb_got_immediate_ack with a stub for softmmu.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20230302190846.2593720-14-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-14-richard.henderson@linaro.org>
show more ...
|
#
d96bf49b |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move chunks of user code into own files
The process was pretty similar to the softmmu move except we take the time to split stuff between user.c and user-target.c to avoid as much target sp
gdbstub: move chunks of user code into own files
The process was pretty similar to the softmmu move except we take the time to split stuff between user.c and user-target.c to avoid as much target specific compilation as possible. We also start to make use of our shiny new header scheme so the user-only helpers can be included without the rest of the exec/gsbstub.h cruft.
As before we split some functions into user and softmmu versions
Reviewed-by: Fabiano Rosas <farosas@suse.de> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-12-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-12-richard.henderson@linaro.org>
show more ...
|
#
b6fa2ec2 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move chunk of softmmu functionality to own file
This is mostly code motion but a number of things needed to be done for this minimal patch set:
- move shared structures to internals.h
gdbstub: move chunk of softmmu functionality to own file
This is mostly code motion but a number of things needed to be done for this minimal patch set:
- move shared structures to internals.h - splitting some functions into user and softmmu versions - fixing a few casting issues to keep softmmu common
More CONFIG_USER_ONLY stuff will be handled in a following patches.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Fabiano Rosas <farosas@suse.de> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-11-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-11-richard.henderson@linaro.org>
show more ...
|
#
36e067b2 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: make various helpers visible to the rest of the module
We will be needing to use these helpers between the user and softmmu files so declare them in the headers, add a system prefix and rem
gdbstub: make various helpers visible to the rest of the module
We will be needing to use these helpers between the user and softmmu files so declare them in the headers, add a system prefix and remove static from the implementations.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-10-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-10-richard.henderson@linaro.org>
show more ...
|
#
1678ea04 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move fromhex/tohex routines to internals
These will be needed from multiple places in the code. They are declared as inline so move to the header and fix up to modern coding style.
The onl
gdbstub: move fromhex/tohex routines to internals
These will be needed from multiple places in the code. They are declared as inline so move to the header and fix up to modern coding style.
The only other place that messes with hex stuff at the moment is the URI handling in utils but that would be more code churn so leave for now.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20230302190846.2593720-9-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-9-richard.henderson@linaro.org>
show more ...
|
#
9f56787c |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move GDBState to shared internals header
We are about to split softmmu and user mode helpers into different files. To facilitate this we will need to share access to the GDBState between th
gdbstub: move GDBState to shared internals header
We are about to split softmmu and user mode helpers into different files. To facilitate this we will need to share access to the GDBState between those files.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20230302190846.2593720-7-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-7-richard.henderson@linaro.org>
show more ...
|
#
97748558 |
| 03-Mar-2023 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub/internals.h: clean up include guard
Use something more specific to avoid name clashes.
Reviewed-by: Bin Meng <bmeng.cn@gmail.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Rev
gdbstub/internals.h: clean up include guard
Use something more specific to avoid name clashes.
Reviewed-by: Bin Meng <bmeng.cn@gmail.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20230302190846.2593720-2-alex.bennee@linaro.org> Message-Id: <20230303025805.625589-2-richard.henderson@linaro.org>
show more ...
|
Revision tags: v7.2.0, v7.2.0-rc4 |
|
#
55b5b8e9 |
| 06-Dec-2022 |
Philippe Mathieu-Daudé <philmd@linaro.org> |
gdbstub: Use vaddr type for generic insert/remove_breakpoint() API
Both insert/remove_breakpoint() handlers are used in system and user emulation. We can not use the 'hwaddr' type on user emulation,
gdbstub: Use vaddr type for generic insert/remove_breakpoint() API
Both insert/remove_breakpoint() handlers are used in system and user emulation. We can not use the 'hwaddr' type on user emulation, we have to use 'vaddr' which is defined as "wide enough to contain any #target_ulong virtual address".
gdbstub.c doesn't require to include "exec/hwaddr.h" anymore.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Fabiano Rosas <farosas@suse.de> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-Id: <20221216215519.5522-4-philmd@linaro.org>
show more ...
|
Revision tags: v7.2.0-rc3, v7.2.0-rc2, v7.2.0-rc1, v7.2.0-rc0 |
|
#
a48e7d9e |
| 29-Sep-2022 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move guest debug support check to ops
This removes the final hard coding of kvm_enabled() in gdbstub and moves the check to an AccelOps.
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.or
gdbstub: move guest debug support check to ops
This removes the final hard coding of kvm_enabled() in gdbstub and moves the check to an AccelOps.
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Mads Ynddal <mads@ynddal.dk> Message-Id: <20220929114231.583801-46-alex.bennee@linaro.org>
show more ...
|
#
ae7467b1 |
| 29-Sep-2022 |
Alex Bennée <alex.bennee@linaro.org> |
gdbstub: move breakpoint logic to accel ops
As HW virtualization requires specific support to handle breakpoints lets push out special casing out of the core gdbstub code and into AccelOpsClass. Thi
gdbstub: move breakpoint logic to accel ops
As HW virtualization requires specific support to handle breakpoints lets push out special casing out of the core gdbstub code and into AccelOpsClass. This will make it easier to add other accelerator support and reduces some of the stub shenanigans.
Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Mads Ynddal <mads@ynddal.dk> Message-Id: <20220929114231.583801-45-alex.bennee@linaro.org>
show more ...
|