Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-20 Thread Liviu Ionescu
Hi, with the latest submitted patches, the functionality I expect for qemu-system-arm is complete. (I have some more cosmetic suggestions, to be discussed later). to test the functionality, you can download an unit test application from:

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-19 Thread Liviu Ionescu
On 18 Nov 2014, at 21:46, Liviu Ionescu i...@livius.net wrote: once you integrate them... thank you for integrating them. so can I base my next patches on them? I'll work on another patch, to fix passing the semihosting command line down to the armv7m code, since currently this triggers

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-19 Thread Peter Maydell
On 19 November 2014 11:37, Liviu Ionescu i...@livius.net wrote: On 18 Nov 2014, at 21:46, Liviu Ionescu i...@livius.net wrote: once you integrate them... thank you for integrating them. so can I base my next patches on them? You can, yes. (Bear in mind that my target-arm.next branch is a

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-19 Thread Liviu Ionescu
On 19 Nov 2014, at 14:08, Peter Maydell peter.mayd...@linaro.org wrote: ... (Bear in mind that my target-arm.next branch is a rebasing branch, so you can't git merge or pull it; you need to fetch it and then rebase your patchstack on it.) I currently have two more branches in my git, for

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-19 Thread Peter Maydell
On 19 November 2014 13:06, Liviu Ionescu i...@livius.net wrote: I already explained, but will do it again. Sorry if I missed that; there's a lot of mail on qemu-devel... first, we are talking about two completely different use cases. on the A profile you have a kernel, and when you start it,

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-19 Thread Liviu Ionescu
On 19 Nov 2014, at 15:20, Peter Maydell peter.mayd...@linaro.org wrote: Yes, we should do this. (We have to make cmdline= optional anyway, to support old and previously working command lines.) ok, so I'll add the optional 'cmdline=' to the newly added '-semihosting-config'. since this was

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-18 Thread Liviu Ionescu
On 17 Nov 2014, at 14:32, Peter Maydell peter.mayd...@linaro.org wrote: You can start out by separating out one or two and submitting those. just did that, I submitted two patches, one to add the -semihosting-config and one to fix the exit code. if you have further comments, please let me

[Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-18 Thread Liviu Ionescu
The shortcomings addressed by this patch: - the semihosting trace messages disapeared when the GDB session was started - the semihosting exit code was not passed back to the host - the semihosting command line was not passed properly, because it generated a very large string, including the image

[Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-18 Thread Liviu Ionescu
Signed-off-by: Liviu Ionescu i...@livius.net --- gdbstub.c | 12 +++ hw/arm/armv7m.c | 27 +++- hw/arm/stellaris.c | 14 + include/hw/arm/arm.h| 8 --- include/hw/boards.h | 1 + include/sysemu/sysemu.h | 7 +++

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-17 Thread Peter Maydell
On 13 November 2014 23:55, Liviu Ionescu i...@livius.net wrote: The shortcomings addressed by this patch: - the semihosting trace messages disapeared when the GDB session was started - the semihosting exit code was not passed back to the host - the semihosting command line was not passed

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-17 Thread Peter Maydell
On 17 November 2014 12:32, Peter Maydell peter.mayd...@linaro.org wrote: This (the extra command line option to specify where semihosting should go) is definitely a feature we should add. I think it's possible to make use of the QemuOpts infrastructure to support -semihosting # current

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-17 Thread Liviu Ionescu
On 17 Nov 2014, at 14:32, Peter Maydell peter.mayd...@linaro.org wrote: that it's combining six different features and bug fixes into a single commit. Could you separate them out into their own patches? sure. in practical terms, this requires separate branches and each be applied to master,

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-17 Thread Peter Maydell
On 17 November 2014 14:32, Liviu Ionescu i...@livius.net wrote: On 17 Nov 2014, at 14:32, Peter Maydell peter.mayd...@linaro.org wrote: that it's combining six different features and bug fixes into a single commit. Could you separate them out into their own patches? sure. in practical terms,

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-17 Thread Liviu Ionescu
On 17 Nov 2014, at 14:35, Peter Maydell peter.mayd...@linaro.org wrote: Compare the handling of 'rtc' and similar options: you get to have an implied option name so -foo is treated like -foo enable=true, I already considered this in a previous tentative version, but the implementation

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-13 Thread Liviu Ionescu
to test the patch, you can download an unit test application from: https://dl.dropboxusercontent.com/u/78151643/gcm.elf there are two use cases: 1) standalone (usually unit tests integrated in a continuous integration system) $ ./qemu-system-arm -machine lm3s6965evb -image gcm.elf

Re: [Qemu-devel] [PATCH] functional ARM semihosting under GDB

2014-11-13 Thread Liviu Ionescu
On 14 Nov 2014, at 02:25, Liviu Ionescu i...@livius.net wrote: (gdb) system_reset in fact this is (gdb) mon system_reset Liviu