[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config ar
From: |
Liviu Ionescu |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument |
Date: |
Wed, 20 May 2015 14:12:59 +0300 |
> 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
- [Qemu-devel] [PATCH v3 0/2] semihosting: clean up and add --semihosting-config arg, Leon Alrae, 2015/05/08
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Peter Maydell, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Leon Alrae, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Liviu Ionescu, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Peter Maydell, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Liviu Ionescu, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Peter Maydell, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Liviu Ionescu, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Peter Maydell, 2015/05/20
- Re: [Qemu-devel] [PATCH v3 2/2] semihosting: add --semihosting-config arg sub-argument, Liviu Ionescu, 2015/05/20