Hi,
On 18/07/17 21:07, Stefano Stabellini wrote:
On Mon, 17 Jul 2017, Bhupinder Thakur wrote:
This patch finally adds the support for vuart console. It adds
two new fields in the console initialization:
- optional
- prefer_gnttab
optional flag tells whether the console is optional.
prefer_gnttab tells whether the ring buffer should be allocated using
grant table.
Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org>
---
CC: Ian Jackson <ian.jack...@eu.citrix.com>
CC: Wei Liu <wei.l...@citrix.com>
CC: Stefano Stabellini <sstabell...@kernel.org>
CC: Julien Grall <julien.gr...@arm.com>
Changes since v4:
- Renamed VUART_CFLAGS- to CFLAGS_vuart- in the Makefile as per the convention.
config/arm32.mk | 1 +
config/arm64.mk | 1 +
tools/console/Makefile | 3 ++-
tools/console/daemon/io.c | 29 ++++++++++++++++++++++++++++-
4 files changed, 32 insertions(+), 2 deletions(-)
diff --git a/config/arm32.mk b/config/arm32.mk
index f95228e..b9f23fe 100644
--- a/config/arm32.mk
+++ b/config/arm32.mk
@@ -1,5 +1,6 @@
CONFIG_ARM := y
CONFIG_ARM_32 := y
+CONFIG_VUART_CONSOLE := y
CONFIG_ARM_$(XEN_OS) := y
CONFIG_XEN_INSTALL_SUFFIX :=
What about leaving this off for ARM32 by default?
Why? This will only disable xenconsole changes and not the hypervisor.
The changes are quite tiny, so I would even be in favor of enabling for
all architectures.
Or are you suggesting to disable the VPL011 emulation in the hypervisor?
But I don't see the emulation AArch64 specific, and a user could disable
it if he doesn't want it...
Cheers,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
