From: riku voipio <riku.voipio@iki.fi>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Schildbach, Wolfgang" <WSCHI@dolby.com>,
qemu-devel@nongnu.org, Nathan Froyd <froydnj@codesourcery.com>
Subject: Re: [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling forsemihosted executables
Date: Fri, 07 Jan 2011 17:34:12 +0200 [thread overview]
Message-ID: <4D273274.40206@iki.fi> (raw)
In-Reply-To: <AANLkTimRuf_9r0ETTcz1zbYk0dj4-hgvMhnH8_c00=Pb@mail.gmail.com>
Hi,
Honestly, I have never tried ARM semihosting. Do you have some
recommended test instructions?
Riku
On 12/23/2010 04:09 PM, Peter Maydell wrote:
> Riku, did you want to pick this one up as a linux-user patch? Otherwise
> it's on my list of "patches which got no comments and will go into the
> next arm pull request" -- although that won't be until early next year
> and I expect 0.14 will have branched by then.
>
> -- PMM
>
> On 23 December 2010 12:58, Schildbach, Wolfgang<WSCHI@dolby.com> wrote:
>> Ping -- I have not seen any corresponding commit in the qemu master
>> repository. Anthony, could you please consider this for inclusion into
>> 0.14?
>>
>> Thanks,
>> - Wolfgang
>>
>>
>> -----Original Message-----
>> From: qemu-devel-bounces+wschi=dolby.com@nongnu.org
>> [mailto:qemu-devel-bounces+wschi=dolby.com@nongnu.org] On Behalf Of
>> Peter Maydell
>> Sent: Monday, December 06, 2010 4:06 PM
>> To: qemu-devel@nongnu.org
>> Cc: Riku Voipio; Schildbach, Wolfgang; Nathan Froyd
>> Subject: [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling
>> forsemihosted executables
>>
>> This patchset fixes the commandline handling for ARM semihosted
>> executables run under linux-user mode, and cleans up the resulting
>> unused variable in the linux-user image_info struct. The code is all by
>> Wolfgang but he is having difficulty sending properly formatted patches
>> to the list so I am retransmitting them (with my Reviewed-by: signoff).
>>
>> Wolfgang Schildbach (2):
>> Fix commandline handling for ARM semihosted executables
>> Remove dead code for ARM semihosting commandline handling
>>
>> arm-semi.c | 79
>> +++++++++++++++++++++++++++++------------------
>> bsd-user/bsdload.c | 2 -
>> bsd-user/qemu.h | 1 -
>> linux-user/linuxload.c | 2 -
>> linux-user/qemu.h | 1 -
>> 5 files changed, 49 insertions(+), 36 deletions(-)
>>
>>
>>
next prev parent reply other threads:[~2011-01-07 15:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-06 15:06 [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling for semihosted executables Peter Maydell
2010-12-06 15:06 ` [Qemu-devel] [PATCH 1/2] Fix commandline handling for ARM " Peter Maydell
2010-12-06 15:06 ` [Qemu-devel] [PATCH 2/2] Remove dead code for ARM semihosting commandline handling Peter Maydell
2010-12-23 12:58 ` [Qemu-devel] [PATCH 0/2] ARM: fix commandline handling forsemihosted executables Schildbach, Wolfgang
2010-12-23 14:09 ` Peter Maydell
2011-01-07 15:34 ` riku voipio [this message]
2011-01-07 15:46 ` Schildbach, Wolfgang
2011-01-07 15:56 ` Peter Maydell
2011-01-07 16:29 ` riku voipio
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4D273274.40206@iki.fi \
--to=riku.voipio@iki.fi \
--cc=WSCHI@dolby.com \
--cc=froydnj@codesourcery.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).