From nobody Fri Oct 31 04:01:21 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org ARC-Seal: i=1; a=rsa-sha256; t=1756425272; cv=none; d=zohomail.com; s=zohoarc; b=HK9208Q9Ifyi7dVtdDGciPYZDQCfg9VWL5U4gMiqIcXRN7E5MoJzgmbO/Erf3AT6g+D1nMmIpNHgirtKZE+uE3wVHs0Os7diXOxpju6ZZPIXJfqB2EsGekTqhRr1sRUpevw6UvpCr87PTzu4VQK0FCTgr6+rh/t+hQ90SwSlKXE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1756425272; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=kV2+nmwE+JHTTYsZOf5tG0vjn+zAyV6DLLw3E85gpnk=; b=Yysxc3TVt8oCknpl5n9joNr27uV3ZXubgSME67NCKs6qI71N2RRbu01rY0W1HjkTN93T+fvGobtxz5KX3MyjbyzmTT7xjb25sY43UzgKwQimGcSCq47+iNAsbhN0dtMtDAwuuTRD/40tLPvAmlDo2KyzHOLlatXsYhnl5ytC/PQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1756425272797136.5798151486539; Thu, 28 Aug 2025 16:54:32 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.1100175.1453757 (Exim 4.92) (envelope-from ) id 1urmRe-0004Zo-2V; Thu, 28 Aug 2025 23:54:18 +0000 Received: by outflank-mailman (output) from mailman id 1100175.1453757; Thu, 28 Aug 2025 23:54:18 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1urmRd-0004Yb-Tq; Thu, 28 Aug 2025 23:54:17 +0000 Received: by outflank-mailman (input) for mailman id 1100175; Thu, 28 Aug 2025 23:54:17 +0000 Received: from mail.xenproject.org ([104.130.215.37]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1urmRd-0004H2-3G for xen-devel@lists.xenproject.org; Thu, 28 Aug 2025 23:54:17 +0000 Received: from xenbits.xenproject.org ([104.239.192.120]) by mail.xenproject.org with esmtp (Exim 4.96) (envelope-from ) id 1urmRc-00B9sx-0m; Thu, 28 Aug 2025 23:54:16 +0000 Received: from [19.12.91.86] (helo=localhost) by xenbits.xenproject.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1urmRc-006h8P-0j; Thu, 28 Aug 2025 23:54:16 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=xen.org; s=20200302mail; h=Content-Transfer-Encoding:MIME-Version:References: In-Reply-To:Message-ID:Date:Subject:Cc:To:From; bh=kV2+nmwE+JHTTYsZOf5tG0vjn+zAyV6DLLw3E85gpnk=; b=qEtFkzRj0xEqsJKafxfYmU8out eIt5cQVo010pHmfUyC8XSgWQAywRt5O/0DcWkzPL6Sf8vn0xmNXcYGn2SWc8eY2+FR+ZboZXbcXVH fu2K44bLHptgicWK5LZagP+pPQS68A+eUGe8MaExbJp7Npx9jfDPdMPADgjfyG7eNAmA=; From: dmukhin@xen.org To: xen-devel@lists.xenproject.org Cc: andrew.cooper3@citrix.com, anthony.perard@vates.tech, jbeulich@suse.com, julien@xen.org, michal.orzel@amd.com, roger.pau@citrix.com, sstabellini@kernel.org, dmukhin@ford.com Subject: [PATCH v5 03/15] emul/ns16x50: implement emulator stub Date: Thu, 28 Aug 2025 16:53:57 -0700 Message-ID: <20250828235409.2835815-4-dmukhin@ford.com> X-Mailer: git-send-email 2.51.0 In-Reply-To: <20250828235409.2835815-1-dmukhin@ford.com> References: <20250828235409.2835815-1-dmukhin@ford.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @xen.org) X-ZM-MESSAGEID: 1756425276446124100 Content-Type: text/plain; charset="utf-8" From: Denis Mukhin =20 The change is the first on the way on introducing minimally functional NS16550-compatible UART emulator. Define UART state and a set of emulated registers. Implement alloc/free vUART hooks. Stub out I/O port handler. Add initialization of the NS16x50-compatible UART emulator state machine. Plumb debug logging. Signed-off-by: Denis Mukhin --- Changes since v4: - new patch=20 --- xen/arch/x86/hvm/hvm.c | 20 ++ xen/common/emul/vuart/Kconfig | 18 ++ xen/common/emul/vuart/Makefile | 1 + xen/common/emul/vuart/ns16x50.c | 362 ++++++++++++++++++++++++++++++++ xen/include/xen/sched.h | 4 + 5 files changed, 405 insertions(+) create mode 100644 xen/common/emul/vuart/ns16x50.c diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c index 23bd7f078a1d..26760cf995df 100644 --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include #include @@ -689,6 +690,21 @@ int hvm_domain_initialise(struct domain *d, if ( rc !=3D 0 ) goto fail1; =20 + if ( IS_ENABLED(CONFIG_VUART_NS16X50) ) + { + struct vuart_info info =3D { + .name =3D "COM2", + .compatible =3D "ns16550", + .base_addr =3D 0x2f8, + .size =3D 8, + .irq =3D 3, + }; + + rc =3D vuart_init(d, &info); + if ( rc ) + goto out_vioapic_deinit; + } + stdvga_init(d); =20 rtc_init(d); @@ -712,6 +728,8 @@ int hvm_domain_initialise(struct domain *d, return 0; =20 fail2: + vuart_deinit(d); + out_vioapic_deinit: vioapic_deinit(d); fail1: if ( is_hardware_domain(d) ) @@ -774,6 +792,8 @@ void hvm_domain_destroy(struct domain *d) if ( hvm_funcs.domain_destroy ) alternative_vcall(hvm_funcs.domain_destroy, d); =20 + vuart_deinit(d); + vioapic_deinit(d); =20 XFREE(d->arch.hvm.pl_time); diff --git a/xen/common/emul/vuart/Kconfig b/xen/common/emul/vuart/Kconfig index ce1b976b7da7..539e6d5e4fc7 100644 --- a/xen/common/emul/vuart/Kconfig +++ b/xen/common/emul/vuart/Kconfig @@ -3,4 +3,22 @@ config VUART_FRAMEWORK =20 menu "UART Emulation" =20 +config VUART_NS16X50 + bool "NS16550-compatible UART Emulator" if EXPERT + depends on X86 && HVM + select VUART_FRAMEWORK + help + In-hypervisor NS16x50 UART emulation. + + Only legacy PC COM2 port is emulated. + + This is strictly for testing purposes (such as early HVM guest console), + and not appropriate for use in production. + +config VUART_NS16X50_DEBUG + bool "NS16550-compatible UART Emulator Debugging" + depends on VUART_NS16X50 && DEBUG + help + Enable development debugging. + endmenu diff --git a/xen/common/emul/vuart/Makefile b/xen/common/emul/vuart/Makefile index 97f792dc6641..fe904f6cb65d 100644 --- a/xen/common/emul/vuart/Makefile +++ b/xen/common/emul/vuart/Makefile @@ -1 +1,2 @@ obj-y +=3D vuart.o +obj-$(CONFIG_VUART_NS16X50) +=3D ns16x50.o diff --git a/xen/common/emul/vuart/ns16x50.c b/xen/common/emul/vuart/ns16x5= 0.c new file mode 100644 index 000000000000..f0479e1022fb --- /dev/null +++ b/xen/common/emul/vuart/ns16x50.c @@ -0,0 +1,362 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ +/* + * NS16550-compatible UART Emulator. + * + * See: + * - Serial and UART Tutorial: + * https://download.freebsd.org/doc/en/articles/serial-uart/serial-uar= t_en.pdf + * - UART w/ 16 byte FIFO: + * https://www.ti.com/lit/ds/symlink/tl16c550c.pdf + * - UART w/ 64 byte FIFO: + * https://www.ti.com/lit/ds/symlink/tl16c750.pdf + * + * Limitations: + * - Only x86; + * - Only Xen console as a backend, no inter-domain communication (similar= to + * vpl011 on Arm); + * - Only 8n1 emulation (8-bit data, no parity, 1 stop bit); + * - No baud rate emulation (reports 115200 baud to the guest OS); + * - No FIFO-less mode emulation; + * - No RX FIFO interrupt moderation (FCR) emulation; + * - No integration w/ VM snapshotting (HVM_REGISTER_SAVE_RESTORE() and + * friends); + * - No ISA IRQ sharing allowed; + * - No MMIO-based UART emulation. + */ + +#define pr_prefix "ns16x50" +#define pr_fmt(fmt) pr_prefix ": " fmt + +#ifdef CONFIG_VUART_NS16X50_DEBUG +#define guest_prefix "FROM GUEST " +#define ns16x50_log_level 2 +#else +#define guest_prefix "" +#define ns16x50_log_level 0 +#endif + +#include +#include +#include +#include +#include +#include + +#include + +#define ns16x50_log(n, lvl, vdev, fmt, args...) do { \ + if ( ns16x50_log_level >=3D n) \ + gprintk(lvl, pr_fmt("%s: " fmt), (vdev)->name, ## args); \ +} while (0) + +#define ns16x50_err(vdev, fmt, args...) \ + ns16x50_log(0, KERN_ERR, vdev, fmt, ## args) +#define ns16x50_warn(vdev, fmt, args...) \ + ns16x50_log(1, KERN_WARNING, vdev, fmt, ## args) +#define ns16x50_info(vdev, fmt, args...) \ + ns16x50_log(2, KERN_INFO, vdev, fmt, ## args) +#define ns16x50_debug(vdev, fmt, args...) \ + ns16x50_log(3, KERN_DEBUG, vdev, fmt, ## args) + +/* + * Number of supported registers in the UART. + */ +#define NS16X50_REGS_NUM (UART_SCR + 1) + +/* + * Number of emulated registers. + * + * - Emulated registers [0..NS16X50_REGS_NUM] are R/W registers for DLAB= =3D0. + * - DLAB=3D1, R/W, DLL =3D NS16X50_REGS_NUM + 0 + * - DLAB=3D1, R/W, DLM =3D NS16X50_REGS_NUM + 1 + * - R/O, IIR (IIR_THR) =3D NS16X50_REGS_NUM + 2 + */ +#define NS16X50_EMU_REGS_NUM (NS16X50_REGS_NUM + 3) + +/* + * Virtual ns16x50 device state. + */ +struct vuart_ns16x50 { + struct xencons_interface cons; /* Emulated RX/TX FIFOs */ + uint8_t regs[NS16X50_EMU_REGS_NUM]; /* Emulated registers */ + const char *name; /* Device name */ + struct domain *owner; /* Owner domain */ + const struct vuart_info *info; /* UART description */ + spinlock_t lock; /* Protection */ +}; + +/* + * Emulate 8-bit write access to ns16x50 register. + */ +static int ns16x50_io_write8( + struct vuart_ns16x50 *vdev, uint32_t reg, uint8_t *data) +{ + int rc =3D 0; + + return rc; +} + +/* + * Emulate 16-bit write access to ns16x50 register. + * NB: some guest OSes use outw() to access UART_DLL. + */ +static int ns16x50_io_write16( + struct vuart_ns16x50 *vdev, uint32_t reg, uint16_t *data) +{ + int rc =3D -EINVAL; + + return rc; +} + +/* + * Emulate write access to ns16x50 register. + */ +static int ns16x50_io_write( + struct vuart_ns16x50 *vdev, uint8_t reg, uint32_t size, uint32_t *data) +{ + int rc; + + switch ( size ) + { + case 1: + rc =3D ns16x50_io_write8(vdev, reg, (uint8_t *)data); + break; + + case 2: + rc =3D ns16x50_io_write16(vdev, reg, (uint16_t *)data); + break; + + default: + rc =3D -EINVAL; + break; + } + + return rc; +} + +/* + * Emulate 8-bit read access to ns16x50 register. + */ +static int ns16x50_io_read8( + struct vuart_ns16x50 *vdev, uint32_t reg, uint8_t *data) +{ + uint8_t val =3D 0xff; + int rc =3D 0; + + *data =3D val; + + return rc; +} + +/* + * Emulate 16-bit read access to ns16x50 register. + */ +static int ns16x50_io_read16( + struct vuart_ns16x50 *vdev, uint32_t reg, uint16_t *data) +{ + uint16_t val =3D 0xffff; + int rc =3D -EINVAL; + + *data =3D val; + + return rc; +} + +/* + * Emulate read access to ns16x50 register. + */ +static int ns16x50_io_read( + struct vuart_ns16x50 *vdev, uint8_t reg, uint32_t size, uint32_t *data) +{ + int rc; + + switch ( size ) + { + case 1: + rc =3D ns16x50_io_read8(vdev, reg, (uint8_t *)data); + break; + + case 2: + rc =3D ns16x50_io_read16(vdev, reg, (uint16_t *)data); + break; + + default: + *data =3D 0xffffffff; + rc =3D -EINVAL; + break; + } + + return rc; +} + +/* + * Emulate I/O access to ns16x50 register. + * Note, emulation always returns X86EMUL_OKAY, once I/O port trap is enab= led. + */ +static int cf_check ns16x50_io_handle( + int dir, unsigned int addr, unsigned int size, uint32_t *data) +{ +#define op(dir) (((dir) =3D=3D IOREQ_WRITE) ? 'W' : 'R') + struct domain *d =3D rcu_lock_current_domain(); + struct vuart *vuart =3D vuart_find_by_io_range(d, addr, size); + struct vuart_ns16x50 *vdev; + const struct domain *owner; + const struct vuart_info *info; + uint32_t reg; + unsigned dlab; + int rc; + + if ( !vuart || !vuart->vdev ) + { + printk(KERN_ERR "%c io 0x%04x %d: not initialized\n", + op(dir), addr, size); + + ASSERT_UNREACHABLE(); + goto out; + } + vdev =3D vuart->vdev; + + owner =3D vuart->owner; + ASSERT(owner); + if ( d !=3D owner ) + { + ns16x50_err(vdev, "%c io 0x%04x %d: does not match current domain = %pv\n", + op(dir), addr, size, d); + + ASSERT_UNREACHABLE(); + goto out; + } + + info =3D vuart->info; + ASSERT(info); + reg =3D addr - info->base_addr; + if ( !IS_ALIGNED(reg, size) ) + { + ns16x50_err(vdev, "%c 0x%04x %d: unaligned access\n", + op(dir), addr, size); + goto out; + } + + dlab =3D 0; + if ( reg >=3D NS16X50_REGS_NUM ) + { + ns16x50_err(vdev, "%c io 0x%04x %d: DLAB=3D%d %02"PRIx32" 0x%08"PR= Ix32": not implemented\n", + op(dir), addr, size, dlab, reg, *data); + goto out; + } + + spin_lock(&vdev->lock); + + if ( dir =3D=3D IOREQ_WRITE ) + { + ns16x50_debug(vdev, "%c 0x%04x %d: DLAB=3D%d %02"PRIx32" 0x%08"PRI= x32"\n", + op(dir), addr, size, dlab, reg, *data); + rc =3D ns16x50_io_write(vdev, reg, size, data); + } + else + { + rc =3D ns16x50_io_read(vdev, reg, size, data); + ns16x50_debug(vdev, "%c 0x%04x %d: DLAB=3D%d %02"PRIx32" 0x%08"PRI= x32"\n", + op(dir), addr, size, dlab, reg, *data); + } + if ( rc < 0 ) + ns16x50_err(vdev, "%c 0x%04x %d: DLAB=3D%d %02"PRIx32" 0x%08"PRIx3= 2": unsupported access\n", + op(dir), addr, size, dlab, reg, *data); + + spin_unlock(&vdev->lock); + +out: + rcu_unlock_domain(d); + + return X86EMUL_OKAY; +#undef op +} + +static int ns16x50_init(void *arg) +{ + struct vuart_ns16x50 *vdev =3D arg; + const struct vuart_info *info =3D vdev->info; + struct domain *d =3D vdev->owner; + + ASSERT(vdev); + + register_portio_handler(d, info->base_addr, info->size, ns16x50_io_han= dle); + + return 0; +} + +static void cf_check ns16x50_deinit(void *arg) +{ + struct vuart_ns16x50 *vdev =3D arg; + + ASSERT(vdev); +} + +static void * cf_check ns16x50_alloc(struct domain *d, const struct vuart_= info *info) +{ + struct vuart_ns16x50 *vdev; + int rc; + + if ( !info ) + return ERR_PTR(-EINVAL); + + if ( vuart_find_by_io_range(d, info->base_addr, info->size) ) + { + ns16x50_err(info, "already registered\n"); + return ERR_PTR(-EBUSY); + } + + if ( !is_hvm_domain(d) ) + { + ns16x50_err(info, "not an HVM domain\n"); + return ERR_PTR(-ENOSYS); + } + + vdev =3D xvzalloc(typeof(*vdev)); + if ( !vdev ) + { + ns16x50_err(info, "failed to allocate memory\n"); + return ERR_PTR(-ENOMEM); + } + + /* Save convenience pointer. */ + vdev->name =3D info->name; + vdev->owner =3D d; + vdev->info =3D info; + + rc =3D ns16x50_init(vdev); + if ( rc ) + return ERR_PTR(rc); + + return vdev; +} + +static void cf_check ns16x50_free(void *arg) +{ + struct vuart_ns16x50 *vdev =3D arg; + + if ( vdev ) + ns16x50_deinit(vdev); + + XVFREE(vdev); +} + +#define ns16x50_emulator \ +{ \ + .compatible =3D "ns16550", \ + .alloc =3D ns16x50_alloc, \ + .free =3D ns16x50_free, \ + .dump_state =3D NULL, \ + .put_rx =3D NULL, \ +} + +VUART_REGISTER(ns16x50, ns16x50_emulator); + +/* + * Local variables: + * mode: C + * c-file-style: "BSD" + * c-basic-offset: 4 + * indent-tabs-mode: nil + * End: + */ diff --git a/xen/include/xen/sched.h b/xen/include/xen/sched.h index 02bdc256ce37..613f4596e33d 100644 --- a/xen/include/xen/sched.h +++ b/xen/include/xen/sched.h @@ -23,6 +23,7 @@ #include #include #include +#include #include #include #include @@ -660,6 +661,9 @@ struct domain struct { /* Permission to take ownership of the physical console input. */ bool input_allowed; +#ifdef CONFIG_VUART_FRAMEWORK + struct vuart *vuart; +#endif } console; } __aligned(PAGE_SIZE); =20 --=20 2.51.0