[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 08/22] target/i386: use gdb_get_reg helpers
From: |
Richard Henderson |
Subject: |
Re: [PATCH v6 08/22] target/i386: use gdb_get_reg helpers |
Date: |
Thu, 6 Feb 2020 11:14:01 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 2/5/20 5:10 PM, Alex Bennée wrote:
> This is cleaner than poking memory directly and will make later
> clean-ups easier.
>
> Signed-off-by: Alex Bennée <address@hidden>
> ---
> target/i386/gdbstub.c | 22 ++++++++--------------
> 1 file changed, 8 insertions(+), 14 deletions(-)
>
> diff --git a/target/i386/gdbstub.c b/target/i386/gdbstub.c
> index 572ead641ca..208b3de70da 100644
> --- a/target/i386/gdbstub.c
> +++ b/target/i386/gdbstub.c
> @@ -95,29 +95,26 @@ int x86_cpu_gdb_read_register(CPUState *cs, uint8_t
> *mem_buf, int n)
> if (env->hflags & HF_CS64_MASK) {
> return gdb_get_reg64(mem_buf, env->regs[gpr_map[n]]);
> } else if (n < CPU_NB_REGS32) {
> +
> return gdb_get_reg64(mem_buf,
> env->regs[gpr_map[n]] & 0xffffffffUL);
Watch the stray spaces.
> } else if (n >= IDX_FP_REGS && n < IDX_FP_REGS + 8) {
> -#ifdef USE_X86LDOUBLE
> - /* FIXME: byteswap float values - after fixing fpregs layout. */
> - memcpy(mem_buf, &env->fpregs[n - IDX_FP_REGS], 10);
> -#else
> - memset(mem_buf, 0, 10);
> -#endif
> + guint8 *data = (guint8 *) &env->fpregs[n - IDX_FP_REGS];
> + memcpy(mem_buf, data, 10);
> + /* byte swap? this is at least correct of x86-on-x86 */
Yes, byte swap.
typedef struct {
uint64_t low;
uint16_t high;
} floatx80;
Output little-endian low, followed by little-endian high.
> @@ -290,10 +287,7 @@ int x86_cpu_gdb_write_register(CPUState *cs, uint8_t
> *mem_buf, int n)
> return 4;
> }
> } else if (n >= IDX_FP_REGS && n < IDX_FP_REGS + 8) {
> -#ifdef USE_X86LDOUBLE
> - /* FIXME: byteswap float values - after fixing fpregs layout. */
> memcpy(&env->fpregs[n - IDX_FP_REGS], mem_buf, 10);
> -#endif
> return 10;
Similarly.
r~
- [PATCH v6 00/22] gdbstub refactor and SVE support, Alex Bennée, 2020/02/05
- [PATCH v6 04/22] gdbstub: move mem_buf to GDBState and use GByteArray, Alex Bennée, 2020/02/05
- [PATCH v6 03/22] gdbstub: move str_buf to GDBState and use GString, Alex Bennée, 2020/02/05
- [PATCH v6 05/22] gdbstub: add helper for 128 bit registers, Alex Bennée, 2020/02/05
- [PATCH v6 01/22] gdbstub: make GDBState static and have common init function, Alex Bennée, 2020/02/05
- [PATCH v6 07/22] target/m68k: use gdb_get_reg helpers, Alex Bennée, 2020/02/05
- [PATCH v6 08/22] target/i386: use gdb_get_reg helpers, Alex Bennée, 2020/02/05
- Re: [PATCH v6 08/22] target/i386: use gdb_get_reg helpers,
Richard Henderson <=
- [PATCH v6 06/22] target/arm: use gdb_get_reg helpers, Alex Bennée, 2020/02/05
- [PATCH v6 10/22] target/arm: prepare for multiple dynamic XMLs, Alex Bennée, 2020/02/05
- [PATCH v6 02/22] gdbstub: stop passing GDBState * around and use global, Alex Bennée, 2020/02/05
- [PATCH v6 16/22] configure: allow user to specify what gdb to use, Alex Bennée, 2020/02/05
- [PATCH v6 14/22] target/arm: don't bother with id_aa64pfr0_read for USER_ONLY, Alex Bennée, 2020/02/05
- [PATCH v6 13/22] target/arm: generate xml description of our SVE registers, Alex Bennée, 2020/02/05
- [PATCH v6 17/22] tests/guest-debug: add a simple test runner, Alex Bennée, 2020/02/05
- [PATCH v6 09/22] gdbstub: extend GByteArray to read register helpers, Alex Bennée, 2020/02/05