Re: [Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-20 Thread Peter Maydell
On 19 November 2014 22:05, Liviu Ionescu i...@livius.net wrote: A new sub-option was added to -semihosting-config to define the entire semihosting command line (cmdline=string). This string is passed down to armv7m.c; if not defined, for compatibility reasons, the -kernel -append values are

Re: [Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-20 Thread Liviu Ionescu
On 20 Nov 2014, at 15:31, Peter Maydell peter.mayd...@linaro.org wrote: I think you can avoid having to plumb the command line string into the MachineState and arm_boot_info structures, because you can just have the semihosting code look the option up by name: QemuOpts *opts =

Re: [Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-20 Thread Peter Maydell
On 20 November 2014 15:03, Liviu Ionescu i...@livius.net wrote: On 20 Nov 2014, at 15:31, Peter Maydell peter.mayd...@linaro.org wrote: I think you can avoid having to plumb the command line string into the MachineState and arm_boot_info structures, because you can just have the semihosting

Re: [Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-20 Thread Liviu Ionescu
On 20 Nov 2014, at 17:10, Peter Maydell peter.mayd...@linaro.org wrote: something is missing with the way options are handled, this call fails with BAD_ACCESS. What's this? It's not an error code in QEMU... no, it is a system condition, EXC_BAD_ACCESS, generally caused by a bad pointer.

Re: [Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-20 Thread Peter Maydell
On 20 November 2014 15:17, Liviu Ionescu i...@livius.net wrote: On 20 Nov 2014, at 17:10, Peter Maydell peter.mayd...@linaro.org wrote: something is missing with the way options are handled, this call fails with BAD_ACCESS. What's this? It's not an error code in QEMU... no, it is a

Re: [Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-20 Thread Liviu Ionescu
On 20 Nov 2014, at 17:39, Peter Maydell peter.mayd...@linaro.org wrote: NB: you'll want to stick this inside the !CONFIG_USER_ONLY did this. now it works for me, but please check if it compiles properly for other targets. regards, Liviu

[Qemu-devel] [PATCH] Add -semihosting-config ....cmdline=string.

2014-11-19 Thread Liviu Ionescu
A new sub-option was added to -semihosting-config to define the entire semihosting command line (cmdline=string). This string is passed down to armv7m.c; if not defined, for compatibility reasons, the -kernel -append values are used. The armv7m_init() and stellaris_init() interfaces were