Re: [PATCH v5 17/22] tests/guest-debug: add a simple test runner

2020-01-15 Thread Richard Henderson
On 1/14/20 5:09 AM, Alex Bennée wrote: > +if "system" in args.qemu: > +cmd = "%s %s %s -s -S" % (args.qemu, args.qargs, args.binary) > +else: > +cmd = "%s %s -g 1234 %s" % (args.qemu, args.qargs, args.binary) Oh, hard-coding of the port is going to cause failures. Multiple

Re: [PATCH v5 17/22] tests/guest-debug: add a simple test runner

2020-01-15 Thread Richard Henderson
On 1/14/20 5:09 AM, Alex Bennée wrote: > +gdb_cmd = "%s %s -ex 'target remote localhost:1234' -x %s" % (args.gdb, > args.binary, args.test) It'd be nice to wrap this line. Otherwise, Reviewed-by: Richard Henderson r~

[PATCH v5 17/22] tests/guest-debug: add a simple test runner

2020-01-14 Thread Alex Bennée
The test runners job is to start QEMU with guest debug enabled and then spawn a gdb process running a test script that exercises the functionality it wants to test. Signed-off-by: Alex Bennée --- tests/guest-debug/run-test.py | 57 +++ 1 file changed, 57 insertion