Re: [Qemu-devel] [RFC PATCH] vl.c: add -semihosting-config "arg" sub-argument

[Leon Alrae <leon.alrae@imgtec.com>]

On 15/04/2015 17:08, Liviu Ionescu wrote:
> 
>> On 15 Apr 2015, at 15:49, Leon Alrae <leon.alrae@imgtec.com> wrote:
>>
>> ... If there aren't any objections for the proposed solution, then let's
>> just cook up the patch.
> 
> the changes related to moving --semihosting-cmdline to the end of the line are in:
> 
>   git://git.code.sf.net/p/gnuarmeclipse/qemu gnuarmeclipse-qemu
> 
> the gnuarmeclipse-dev branch, commit 8010d2 
> 
> although I did not group the semihosting related variables in a structure, that code may still be a good starting point (especially since it is already functional in my fork).
> 
> I suggest you compare the gnuarmeclipse-dev branch with master and get the semihosting related patches.
> 
> if you need any help, please don't hesitate to ask. 
> 
> alternatively I can create the patch, but the changes must also touch non-arm code, and I have no experience there.

Since the new option and your ARM semi-hosting improvements already work
for you I believe it would be much easier if you sent the patches first
rather than us figuring out the changes from your branch. Then we could
go from there reviewing / refining / testing etc.

Thanks,
Leon