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

2014-11-20 Thread Liviu Ionescu
On 20 Nov 2014, at 17:39, Peter Maydell 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

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

2014-11-20 Thread Peter Maydell
On 20 November 2014 15:17, Liviu Ionescu wrote: > > On 20 Nov 2014, at 17:10, Peter Maydell 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_ACCE

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

2014-11-20 Thread Liviu Ionescu
On 20 Nov 2014, at 17:10, Peter Maydell 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. (on linux probably

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

2014-11-20 Thread Peter Maydell
On 20 November 2014 15:03, Liviu Ionescu wrote: > > On 20 Nov 2014, at 15:31, Peter Maydell 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 n

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

2014-11-20 Thread Liviu Ionescu
On 20 Nov 2014, at 15:31, Peter Maydell 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 = > qemu_opts_find(qemu_find

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

2014-11-20 Thread Peter Maydell
On 19 November 2014 22:05, Liviu Ionescu 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 used. > > Th

[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 streamli