[v2] Fix gdb bugs and update gdb-xml

[PATCH for-7.1 v2 0/5] Fix gdb bugs and update gdb-xml

Song Gao posted 5 patches 1 year, 8 months ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220805033523.1416837-1-gaosong@loongson.cn

Maintainers: Xiaojuan Yang <yangxiaojuan@loongson.cn>, Song Gao <gaosong@loongson.cn>, "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <f4bug@amsat.org>, Laurent Vivier <laurent@vivier.eu>

configs/targets/loongarch64-softmmu.mak |  2 +-
gdb-xml/loongarch-base64.xml            | 13 +++---
gdb-xml/loongarch-fpu.xml               | 50 ++++++++++++++++++++++
gdb-xml/loongarch-fpu64.xml             | 57 -------------------------
linux-user/loongarch64/signal.c         | 24 +----------
target/loongarch/cpu.c                  |  8 +++-
target/loongarch/gdbstub.c              | 43 ++++++++++++++-----
target/loongarch/internals.h            |  3 ++
8 files changed, 103 insertions(+), 97 deletions(-)
create mode 100644 gdb-xml/loongarch-fpu.xml
delete mode 100644 gdb-xml/loongarch-fpu64.xml

Expand all Fold all

[PATCH for-7.1 v2 0/5] Fix gdb bugs and update gdb-xml

Posted by Song Gao 1 year, 8 months ago

Hi,All

This series fiex LoongArch GDB get the wrong pc, because the xml missing
the register orig_a0, and update loongarch gdb-xml to match GDB[1]

[1]:https://github.com/bminor/binutils-gdb/blob/master/gdb/features/loongarch

Please review!


V2:
    - Update orig_a0 value to 0;
    - Update fcc type to uint64;
    - Share write_fcc()/read_fcc();
    - Update patch2 commit message.

Thanks.
Song Gao


Song Gao (5):
  target/loongarch: Fix GDB get the wrong pc
  target/loongarch: add gdb_arch_name()
  target/loongarch: update loongarch-base64.xml
  target/loongarch: Update loongarch-fpu.xml
  target/loongarch: Update gdb_set_fpu() and gdb_get_fpu()

 configs/targets/loongarch64-softmmu.mak |  2 +-
 gdb-xml/loongarch-base64.xml            | 13 +++---
 gdb-xml/loongarch-fpu.xml               | 50 ++++++++++++++++++++++
 gdb-xml/loongarch-fpu64.xml             | 57 -------------------------
 linux-user/loongarch64/signal.c         | 24 +----------
 target/loongarch/cpu.c                  |  8 +++-
 target/loongarch/gdbstub.c              | 43 ++++++++++++++-----
 target/loongarch/internals.h            |  3 ++
 8 files changed, 103 insertions(+), 97 deletions(-)
 create mode 100644 gdb-xml/loongarch-fpu.xml
 delete mode 100644 gdb-xml/loongarch-fpu64.xml

-- 
2.31.1

Re: [PATCH for-7.1 v2 0/5] Fix gdb bugs and update gdb-xml

Posted by Richard Henderson 1 year, 8 months ago

On 8/4/22 20:35, Song Gao wrote:
> Hi,All
> 
> This series fiex LoongArch GDB get the wrong pc, because the xml missing
> the register orig_a0, and update loongarch gdb-xml to match GDB[1]
> 
> [1]:https://github.com/bminor/binutils-gdb/blob/master/gdb/features/loongarch
> 
> Please review!
> 
> 
> V2:
>      - Update orig_a0 value to 0;
>      - Update fcc type to uint64;
>      - Share write_fcc()/read_fcc();
>      - Update patch2 commit message.

Thanks, queued.


r~

Re: [PATCH for-7.1 v2 0/5] Fix gdb bugs and update gdb-xml

Posted by Alex Bennée 1 year, 8 months ago

Song Gao <gaosong@loongson.cn> writes:

> Hi,All
>
> This series fiex LoongArch GDB get the wrong pc, because the xml missing
> the register orig_a0, and update loongarch gdb-xml to match GDB[1]
>
> [1]:https://github.com/bminor/binutils-gdb/blob/master/gdb/features/loongarch
>
> Please review!

For the whole series:

Acked-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée