OpenE2K
/
binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

gdb/testsuite: add simavr.exp board 

This patch adds a board file for against a simavr target (so, for the
AVR architecture).

simavr, when started with option -g, runs a GDB stub on port 1234.  In
the current latest release (1.6), the port is hardcoded to 1234.  But in
master, there is the option to choose another port.  So while the board
file hardcodes the port today, in the future it should be possible to
let the user choose a port, or automatically select a free port.

It is easy enough to run, make sure you have avr-gcc/avr-g++ and simavr
installed, and as usual just run:

    make check RUNTESTFLAGS="--target_board=simavr"

The following environment variables influence the behavior of the board
file:

  - SIMAVR_MCU: type of chip to simulate
  - SIMAVR_PATH: path to simavr binary (useful if you build your own
    simavr or for some reason it is not simply called `simavr`.

As expected, there are a lot of failures.  Many tests use some features
not supported by such a target, and I suppose there are real GDB bugs
too.  But a lot also passes (including tests that actually run stuff),
so this board file should still help to validate changes to the AVR
architecture support.

These are the results I got of running tests gdb.base/*.exp:

    # of expected passes            20926
    # of unexpected failures        2257
    # of expected failures          14
    # of unknown successes          1
    # of known failures             13
    # of unresolved testcases       592
    # of untested testcases         156
    # of unsupported tests          30
    # of paths in test names        3
    # of duplicate test names       56

gdb/testsuite/ChangeLog:

	* boards/simavr.exp: New file.

Change-Id: Ib7fa8c4e2e90b08b104bb9b552df37779de3bc21
This commit is contained in:
Simon Marchi 2020-05-25 11:55:21 -04:00 committed by Simon Marchi
parent 6cf66e763a
commit 462f72c552
2 changed files with 99 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gdb/testsuite/ChangeLog
View File

@ -1,3 +1,7 @@
2020-05-25 Simon Marchi <simon.marchi@efficios.com>
* boards/simavr.exp: New file.
2020-05-25 Simon Marchi <simon.marchi@efficios.com>
* lib/gdb.exp (gdb_run_cmd): Return success or failure.

95
gdb/testsuite/boards/simavr.exp Normal file
View File

@ -0,0 +1,95 @@
# Copyright 2020 Free Software Foundation, Inc.
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 3 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
# Let the user override the path to the simavr binary with the SIMAVR_PATH
# environment variable.
if { [info exists ::env(SIMAVR_PATH)] } {
set simavr_path $::env(SIMAVR_PATH)
} else {
set simavr_path simavr
}
# Let the user override the simulated AVR chip with the SIMAVR_PATH environment
# variable.
#
# The value passed here must be supported by avr-gcc (see the -mmcu flag in the
# `AVR Options` section of the gcc(1) man page) and by simavr (see output of
# `simavr --list-cores`).
if { [info exists ::env(SIMAVR_MCU)] } {
set simavr_mcu $::env(SIMAVR_MCU)
} else {
set simavr_mcu atmega2560
}
set simavr_last_load_file ""
set simavr_spawn_id ""
set_board_info compiler avr-gcc
set_board_info c++compiler avr-g++
set_board_info cflags "-mmcu=${simavr_mcu}"
set_board_info ldflags "-mmcu=${simavr_mcu}"
set_board_info use_gdb_stub 1
set_board_info gdb_protocol "remote"
set_board_info gdb,do_reload_on_run 1
set_board_info noargs 1
set_board_info gdb,noinferiorio 1
set_board_info gdb,nofileio 1
set_board_info gdb,noresults 1
set_board_info gdb,nosignals 1
proc gdb_load { file } {
global simavr_last_load_file
global simavr_spawn_id
global simavr_mcu
global simavr_path
global gdb_prompt
if { $file == "" } {
set file $simavr_last_load_file
} else {
set simavr_last_load_file $file
}
gdb_file_cmd $file
# Close any previous simavr instance.
if { $simavr_spawn_id != "" } {
close -i $simavr_spawn_id
set simavr_spawn_id ""
}
# Run simavr.
set cmd "spawn -noecho ${simavr_path} --mcu ${simavr_mcu} -g $file"
verbose -log "Spawning simavr: $cmd"
eval $cmd
set simavr_spawn_id $spawn_id
gdb_expect {
-i $simavr_spawn_id -re ".*avr_gdb_init listening on port 1234" {}
timeout { error "unable to start simavr" }
}
# Connect to simavr.
send_gdb "target remote :1234\n"
gdb_expect {
-re ".*Remote debugging using :1234.*\[\r\n\]+$gdb_prompt $" {}
timeout { error "unable to connect to simavr stub" }
}
return 0
}