From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=36168 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1Om6na-0001Gm-4g
	for qemu-devel@nongnu.org; Thu, 19 Aug 2010 11:12:59 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69)
	(envelope-from <christophe.lyon@st.com>) id 1Om6me-0004JE-3X
	for qemu-devel@nongnu.org; Thu, 19 Aug 2010 11:12:01 -0400
Received: from eu1sys200aog112.obsmtp.com ([207.126.144.133]:56098)
	by eggs.gnu.org with esmtp (Exim 4.69)
	(envelope-from <christophe.lyon@st.com>) id 1Om6md-0004IP-Ql
	for qemu-devel@nongnu.org; Thu, 19 Aug 2010 11:12:00 -0400
Received: from zeta.dmz-us.st.com (zeta.dmz-us.st.com [167.4.80.115])
	by beta.dmz-us.st.com (STMicroelectronics) with ESMTP id 2C0AA134
	for <qemu-devel@nongnu.org>; Thu, 19 Aug 2010 15:08:29 +0000 (GMT)
Received: from Webmail-eu.st.com (safex1hubcas3.st.com [10.75.90.18])
	by zeta.dmz-us.st.com (STMicroelectronics) with ESMTP id A987C2EC
	for <qemu-devel@nongnu.org>; Thu, 19 Aug 2010 15:11:38 +0000 (GMT)
Message-ID: <4C6D4994.2090209@st.com>
Date: Thu, 19 Aug 2010 17:11:16 +0200
From: Christophe Lyon <christophe.lyon@st.com>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH] ARM: semi-hosting support for stderr
References: <4C24CECA.1040308@st.com>
In-Reply-To: <4C24CECA.1040308@st.com>
Content-Type: text/plain; charset="ISO-8859-1"; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>

Ping?

On 25.06.2010 17:44, Christophe LYON wrote:
> Hello,
>
> I propose this small patch so that ARM semi-hosting handles stderr as
> expected when linking with Newlib/libgloss.
>
> diff --git a/arm-semi.c b/arm-semi.c
> index 9549e6c..6874036 100644
> --- a/arm-semi.c
> +++ b/arm-semi.c
> @@ -211,8 +211,11 @@ uint32_t do_arm_semihosting(CPUState *env)
>            if (strcmp(s, ":tt") == 0) {
>                if (ARG(1)<  4)
>                    return STDIN_FILENO;
> -            else
> +            else if (ARG(1) == 4)
>                    return STDOUT_FILENO;
> +           else
> +             return STDERR_FILENO; /* See newlib/libgloss
> +                                      implementation.  */
>            }
>            if (use_gdb_syscalls()) {
>                gdb_do_syscall(arm_semi_cb, "open,%s,%x,1a4", ARG(0),
>
>
> Christophe
>
>