Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument

[Liviu Ionescu]

> On 20 May 2015, at 11:51, Leon Alrae <address@hidden> wrote:
> 
> On 20/05/2015 09:30, Liviu Ionescu wrote:
>> 
>>> On 20 May 2015, at 11:11, Leon Alrae <address@hidden> wrote:
>>> 
>>> like "If both are specified, -kernel/-append are ignored (-kernel is
>>> used to load an image, but the path won't be passed to semihosting)" 
>> ...
>>> We could initialize semihosting.argv[0] with -kernel
>> 
>> 
>> here you have a small contradiction, both in the current ARM implementation, 
>> and in your proposal, the full kernel path is passed as argv[0].
> 
> These are two different cases: ...

yes, that's correct, my comment was related to the above quotes from your 
message, in the first part you mention "the path won't be passed to 
semihosting", which is not consistent with the actual behaviour, since the 
entire kernel path is passed unchanged.

if you update the manual text, please rephrase to make the text easier to 
understand and remove this notice related to no-path kernel.

regards,

Liviu