From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B212DC04AB6 for ; Fri, 31 May 2019 11:29:48 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 7A64526839 for ; Fri, 31 May 2019 11:29:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7A64526839 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([127.0.0.1]:41390 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hWfjH-0007as-Jq for qemu-devel@archiver.kernel.org; Fri, 31 May 2019 07:29:47 -0400 Received: from eggs.gnu.org ([209.51.188.92]:38198) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hWfiS-0007IF-NF for qemu-devel@nongnu.org; Fri, 31 May 2019 07:28:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hWfiN-0004Ow-1h for qemu-devel@nongnu.org; Fri, 31 May 2019 07:28:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46360) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hWfhz-0003yO-8y; Fri, 31 May 2019 07:28:31 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 57CDF3082E10; Fri, 31 May 2019 11:28:15 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 31BB54EE0D; Fri, 31 May 2019 11:28:14 +0000 (UTC) Received: from zmail17.collab.prod.int.phx2.redhat.com (zmail17.collab.prod.int.phx2.redhat.com [10.5.83.19]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 3CA051833001; Fri, 31 May 2019 11:28:13 +0000 (UTC) Date: Fri, 31 May 2019 07:28:09 -0400 (EDT) From: Miroslav Rezanina To: Alex =?utf-8?Q?Benn=C3=A9e?= Message-ID: <833530119.25503992.1559302089822.JavaMail.zimbra@redhat.com> In-Reply-To: <87sgsu51bd.fsf@zen.linaroharston> References: <20190514155301.16123-1-alex.bennee@linaro.org> <20190514155301.16123-7-alex.bennee@linaro.org> <20190531091204.tjmq622gw457xbdr@lws.brq.redhat.com> <87sgsu51bd.fsf@zen.linaroharston> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Originating-IP: [10.40.204.19, 10.4.195.2] Thread-Topic: target/arm: use the common interface for WRITE0/WRITEC in arm-semi Thread-Index: xTt/YjrPnnl2GhGx0+HqPIz9t1252A== X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.46]); Fri, 31 May 2019 11:28:20 +0000 (UTC) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: Re: [Qemu-devel] [RFC PATCH 06/11] target/arm: use the common interface for WRITE0/WRITEC in arm-semi X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Riku Voipio , qemu-devel@nongnu.org, qemu-arm@nongnu.org, Laurent Vivier Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" ----- Original Message ----- > From: "Alex Benn=C3=A9e" > To: "Miroslav Rezanina" > Cc: qemu-devel@nongnu.org, "Peter Maydell" , "R= iku Voipio" , > qemu-arm@nongnu.org, "Laurent Vivier" > Sent: Friday, May 31, 2019 1:08:06 PM > Subject: Re: [Qemu-devel] [RFC PATCH 06/11] target/arm: use the common in= terface for WRITE0/WRITEC in arm-semi >=20 >=20 > Miroslav Rezanina writes: >=20 > > On Tue, May 14, 2019 at 04:52:56PM +0100, Alex Benn=C3=A9e wrote: > >> Now we have a common semihosting console interface use that for our > >> string output. However ARM is currently unique in also supporting > >> semihosting for linux-user so we need to replicate the API in > >> linux-user. If other architectures gain this support we can move the > >> file later. > >> > >> Signed-off-by: Alex Benn=C3=A9e > >> --- > >> linux-user/Makefile.objs | 2 ++ > >> linux-user/arm/semihost.c | 24 ++++++++++++++++++++++++ > >> target/arm/arm-semi.c | 31 ++++++------------------------- > >> 3 files changed, 32 insertions(+), 25 deletions(-) > >> create mode 100644 linux-user/arm/semihost.c > >> > >> diff --git a/linux-user/Makefile.objs b/linux-user/Makefile.objs > >> index 769b8d83362..285c5dfa17a 100644 > >> --- a/linux-user/Makefile.objs > >> +++ b/linux-user/Makefile.objs > >> @@ -6,4 +6,6 @@ obj-y =3D main.o syscall.o strace.o mmap.o signal.o \ > >> obj-$(TARGET_HAS_BFLT) +=3D flatload.o > >> obj-$(TARGET_I386) +=3D vm86.o > >> obj-$(TARGET_ARM) +=3D arm/nwfpe/ > >> +obj-$(TARGET_ARM) +=3D arm/semihost.o > >> +obj-$(TARGET_AARCH64) +=3D arm/semihost.o > >> obj-$(TARGET_M68K) +=3D m68k-sim.o > >> diff --git a/linux-user/arm/semihost.c b/linux-user/arm/semihost.c > >> new file mode 100644 > >> index 00000000000..9554102a855 > >> --- /dev/null > >> +++ b/linux-user/arm/semihost.c > >> @@ -0,0 +1,24 @@ > >> +/* > >> + * ARM Semihosting Console Support > >> + * > >> + * Copyright (c) 2019 Linaro Ltd > >> + * > >> + * Currently ARM is unique in having support for semihosting support > >> + * in linux-user. So for now we implement the common console API but > >> + * just for arm linux-user. > >> + * > >> + * SPDX-License-Identifier: GPL-2.0-or-later > >> + */ > >> + > >> +#include "qemu/osdep.h" > >> +#include "cpu.h" > >> +#include "hw/semihosting/console.h" > >> +#include "qemu.h" > >> + > >> +int qemu_semihosting_console_out(CPUArchState *env, target_ulong addr= , > >> int len) > >> +{ > >> + void *s =3D lock_user_string(addr); > >> + len =3D write(STDERR_FILENO, s, len ? len : strlen(s)); > >> + unlock_user(s, addr, 0); > >> + return len; > >> +} > >> diff --git a/target/arm/arm-semi.c b/target/arm/arm-semi.c > >> index 9e5a414dd89..253c66b172a 100644 > >> --- a/target/arm/arm-semi.c > >> +++ b/target/arm/arm-semi.c > >> @@ -27,6 +27,7 @@ > >> > >> #include "cpu.h" > >> #include "hw/semihosting/semihost.h" > >> +#include "hw/semihosting/console.h" > >> #ifdef CONFIG_USER_ONLY > >> #include "qemu.h" > >> > >> @@ -314,32 +315,12 @@ target_ulong do_arm_semihosting(CPUARMState *env= ) > >> return set_swi_errno(ts, close(arg0)); > >> } > >> case TARGET_SYS_WRITEC: > >> - { > >> - char c; > >> - > >> - if (get_user_u8(c, args)) > >> - /* FIXME - should this error code be -TARGET_EFAULT ? *= / > >> - return (uint32_t)-1; > >> - /* Write to debug console. stderr is near enough. */ > >> - if (use_gdb_syscalls()) { > >> - return arm_gdb_syscall(cpu, arm_semi_cb, "write,2,%x,= 1", > >> args); > >> - } else { > >> - return write(STDERR_FILENO, &c, 1); > >> - } > >> - } > >> + { > >> + qemu_semihosting_console_out(env, args, 1); > >> + return 0xdeadbeef; > >> + } > >> case TARGET_SYS_WRITE0: > >> - if (!(s =3D lock_user_string(args))) > >> - /* FIXME - should this error code be -TARGET_EFAULT ? */ > >> - return (uint32_t)-1; > >> - len =3D strlen(s); > >> - if (use_gdb_syscalls()) { > >> - return arm_gdb_syscall(cpu, arm_semi_cb, "write,2,%x,%x", > >> - args, len); > >> - } else { > >> - ret =3D write(STDERR_FILENO, s, len); > >> - } > >> - unlock_user(s, args, 0); > >> - return ret; > >> + return qemu_semihosting_console_out(env, args, 0); > >> case TARGET_SYS_WRITE: > >> GET_ARG(0); > >> GET_ARG(1); > >> -- > >> 2.20.1 > >> > >> > > > > Hi Alex, > > > > this patch breaks build for softmmu target when CONFIG_SEMIHOSTING is n= ot > > enabled as qemu_semihosting_console_out > > is not defined in such case - neither linux-user/arm/semihost.c nor > > hw/semihosting/console.c compiled and function > > is not in stubs/semihost.c >=20 > How do you do that? I tried ../../configure --without-default-devices > and that still builds for me. It's usual RHEL way - use --without-default-devices and use specific list of enabled devices (this mean disable CONFIG_SEMIHOSTING in default_config/* file). >=20 > But I suspect what's needed is to change: >=20 > #ifndef CONFIG_USER_ONLY >=20 > to >=20 > #ifdef CONFIG_SEMIHOSTING >=20 > to the relevant headers and helper bits. Yeah, have to find out what are relevant pieces. Mirek >=20 > > > > Mirek >=20 >=20 > -- > Alex Benn=C3=A9e >=20 --=20 Miroslav Rezanina Software Engineer - Virtualization Team Maintainer