Re: [Qemu-devel] [PATCH v2] Add the "-semihosting-config" option.

[Peter Maydell]

On 19 November 2014 10:31, Peter Maydell <address@hidden> wrote:
> On 18 November 2014 20:19, Liviu Ionescu <address@hidden> wrote:
>> The usual semihosting behaviour is to process the system calls locally and
>> return; unfortuantelly the initial implementation dinamically changed the
>> target to GDB during debug sessions, which, for the usual arm-none-eabi-gdb,
>> is not implemented. The result was that during debug sessions the semihosting
>> calls were discarded.
>>
>> This patch adds a configuration variable and an option to set it on the
>> command line:
>>
>>     -semihosting-config [enable=on|off,]target=native|gdb|auto
>>
>> This option enables semihosting and defines where the semihosting calls will
>> be addressed, to QEMU ('native') or to GDB ('gdb'). The default is auto, 
>> which
>> means 'gdb' during debug sessions and 'native' otherwise.
>>
>> Signed-off-by: Liviu Ionescu <address@hidden>

> Applied to target-arm.next, thanks.

...but I should have tested it a bit better. I had to move the
declaration and definition of semihosting_target to gdbstub.h
and gdbstub.c, because otherwise the linux-user targets won't
compile. (They don't compile vl.c.)

thanks
-- PMM