Le 4/29/20 à 11:38 AM, Laurent Vivier a écrit :
Le 29/04/2020 à 11:28, Alex Bennée a écrit :

Laurent Vivier <laur...@vivier.eu> writes:

Le 28/04/2020 à 19:17, KONRAD Frederic a écrit :
Currently "cf-core.xml" is sent to GDB when using any m68k flavor.  Thing is
it uses the "org.gnu.gdb.coldfire.core" feature name and gdb 8.3 then expects
a coldfire FPU instead of the default m68881 FPU.

This is not OK because the m68881 floats registers are 96 bits wide so it
crashes GDB with the following error message:

(gdb) target remote localhost:7960
Remote debugging using localhost:7960
warning: Register "fp0" has an unsupported size (96 bits)
warning: Register "fp1" has an unsupported size (96 bits)
...
Remote 'g' packet reply is too long (expected 148 bytes, got 180 bytes):    \
   00000000000[...]0000

With this patch: qemu-system-m68k -M none -cpu m68020 -s -S

(gdb) tar rem :1234
Remote debugging using :1234
warning: No executable has been specified and target does not support
determining executable automatically.  Try using the "file" command.
0x00000000 in ?? ()
(gdb) p $fp0
$1 = nan(0xffffffffffffffff)

Signed-off-by: KONRAD Frederic <frederic.kon...@adacore.com>
---
  configure             |  2 +-
  gdb-xml/m68k-core.xml | 29 ++++++++++++++++++++++++++++
  target/m68k/cpu.c     | 52 ++++++++++++++++++++++++++++++++++++---------------
  3 files changed, 67 insertions(+), 16 deletions(-)
  create mode 100644 gdb-xml/m68k-core.xml

Reviewed-by: Laurent Vivier <laur...@vivier.eu>

Are you going to take this through your tree or do you want me to add it
to my small pile of gdbstub fixes?


Please add it to your pile.

Thanks,
Laurent


Thanks!

Reply via email to