From patchwork Wed Feb 26 22:47:36 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Denis Mukhin X-Patchwork-Id: 13993282 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C6169C021B8 for ; Wed, 26 Feb 2025 22:48:23 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.897288.1305984 (Exim 4.92) (envelope-from ) id 1tnQC4-0000k8-Kf; Wed, 26 Feb 2025 22:47:56 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 897288.1305984; Wed, 26 Feb 2025 22:47:56 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1tnQC4-0000k1-Hj; Wed, 26 Feb 2025 22:47:56 +0000 Received: by outflank-mailman (input) for mailman id 897288; Wed, 26 Feb 2025 22:47:55 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1tnQC2-0000jv-7z for xen-devel@lists.xenproject.org; Wed, 26 Feb 2025 22:47:55 +0000 Received: from mail-4316.protonmail.ch (mail-4316.protonmail.ch [185.70.43.16]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id affe6a39-f493-11ef-9897-31a8f345e629; Wed, 26 Feb 2025 23:47:42 +0100 (CET) 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" X-Inumbo-ID: affe6a39-f493-11ef-9897-31a8f345e629 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=proton.me; s=protonmail; t=1740610061; x=1740869261; bh=pOKqC6MekEMjRRkDkhrbf7qKIV1RtZ44qMkcVKOSt3E=; h=Date:To:From:Cc:Subject:Message-ID:Feedback-ID:From:To:Cc:Date: Subject:Reply-To:Feedback-ID:Message-ID:BIMI-Selector: List-Unsubscribe:List-Unsubscribe-Post; b=nC5XeyTXEWjZW0oVZxNksL4CSO4iF8pGWBU2XAwH0yKk4wQjgxry93iLP4+jnaOJ4 UOO/nQv9/CYhLq871+ePKAcVLTyX/P5ggZl55JUaOU7TaGdxmDZEHO9Sp94fJysiEV g8GuinfGgUw7t9o84ygiPRt1oTUyEZTmDgoPvmeCOWYmVQgk/zqNp9lPEXi35bxBpc IM1cQJzVMVS0hrmYGkSTdQCyN1zqDcBhsLyf9ut0D1uXhl2+/35F0CvKA0ZMMO6IUe jozqUwbP7W9JmDcBDw/qao2MAfpHIYOd0Z3Q97sfxnn85+Ovw9Ia7kGr7lwp8TdQfH JHnShsHeYDNbg== Date: Wed, 26 Feb 2025 22:47:36 +0000 To: xen-devel@lists.xenproject.org From: dmkhn@proton.me 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 v4] xen/consoled: clean up console handling for PV shim Message-ID: <20250226224642.728198-1-dmukhin@ford.com> Feedback-ID: 123220910:user:proton X-Pm-Message-ID: 7d5b074154935d305eb1e4f7dab233ae2b0e1895 MIME-Version: 1.0 There are few places which check pv_shim console under CONFIG_PV_SHIM or CONFIG_X86 in xen console driver. Instead of inconsistent #ifdef-ing, introduce and use consoled_is_enabled() in switch_serial_input() and __serial_rx(). PV shim case is fixed in __serial_rx() - should be under 'pv_shim && pv_console' check. Signature of consoled_guest_{rx,tx} has changed so the errors can be logged on the callsites. Also, move get_initial_domain_id() to arch-independent header since it is now required by console driver. Lastly, add missing SPDX-License-Identifier to xen/consoled.h Signed-off-by: Denis Mukhin --- Changes since v3: - drop consoled_guest_rx() stub under !CONFIG_PV_SHIM --- xen/arch/x86/include/asm/pv/shim.h | 5 +++-- xen/arch/x86/pv/shim.c | 8 ++------ xen/common/domain.c | 9 +++++++++ xen/drivers/char/console.c | 13 +++++-------- xen/drivers/char/consoled.c | 17 +++++++++++++---- xen/include/xen/consoled.h | 20 ++++++++++++++++++-- xen/include/xen/domain.h | 2 ++ 7 files changed, 52 insertions(+), 22 deletions(-) diff --git a/xen/arch/x86/include/asm/pv/shim.h b/xen/arch/x86/include/asm/pv/shim.h index 6153e27005..27053d4f6f 100644 --- a/xen/arch/x86/include/asm/pv/shim.h +++ b/xen/arch/x86/include/asm/pv/shim.h @@ -31,7 +31,7 @@ long cf_check pv_shim_cpu_up(void *data); long cf_check pv_shim_cpu_down(void *data); void pv_shim_online_memory(unsigned int nr, unsigned int order); void pv_shim_offline_memory(unsigned int nr, unsigned int order); -domid_t get_initial_domain_id(void); +domid_t pv_shim_get_initial_domain_id(void); uint64_t pv_shim_mem(uint64_t avail); void pv_shim_fixup_e820(void); const struct platform_bad_page *pv_shim_reserved_pages(unsigned int *size); @@ -76,8 +76,9 @@ static inline void pv_shim_offline_memory(unsigned int nr, unsigned int order) { ASSERT_UNREACHABLE(); } -static inline domid_t get_initial_domain_id(void) +static inline domid_t pv_shim_get_initial_domain_id(void) { + ASSERT_UNREACHABLE(); return 0; } static inline uint64_t pv_shim_mem(uint64_t avail) diff --git a/xen/arch/x86/pv/shim.c b/xen/arch/x86/pv/shim.c index b9c034ccff..62b6a92392 100644 --- a/xen/arch/x86/pv/shim.c +++ b/xen/arch/x86/pv/shim.c @@ -605,8 +605,7 @@ long pv_shim_event_channel_op(int cmd, XEN_GUEST_HANDLE_PARAM(void) arg) if ( pv_console && send.port == pv_console_evtchn() ) { - consoled_guest_rx(); - rc = 0; + rc = consoled_guest_rx(); } else rc = xen_hypercall_event_channel_op(EVTCHNOP_send, &send); @@ -1018,13 +1017,10 @@ void pv_shim_offline_memory(unsigned int nr, unsigned int order) } } -domid_t get_initial_domain_id(void) +domid_t pv_shim_get_initial_domain_id(void) { uint32_t eax, ebx, ecx, edx; - if ( !pv_shim ) - return 0; - cpuid(xen_cpuid_base + 4, &eax, &ebx, &ecx, &edx); return (eax & XEN_HVM_CPUID_DOMID_PRESENT) ? ecx : 1; diff --git a/xen/common/domain.c b/xen/common/domain.c index 49d4cb8221..a3c4d7d27d 100644 --- a/xen/common/domain.c +++ b/xen/common/domain.c @@ -2261,6 +2261,15 @@ int continue_hypercall_on_cpu( return 0; } +domid_t get_initial_domain_id(void) +{ +#ifdef CONFIG_X86 + if ( pv_shim ) + return pv_shim_get_initial_domain_id(); +#endif + return hardware_domid; +} + /* * Local variables: * mode: C diff --git a/xen/drivers/char/console.c b/xen/drivers/char/console.c index a3735310e3..beef62a2c5 100644 --- a/xen/drivers/char/console.c +++ b/xen/drivers/char/console.c @@ -32,9 +32,9 @@ #include #include #include +#include #ifdef CONFIG_X86 -#include #include #endif #ifdef CONFIG_SBSA_VUART_CONSOLE @@ -507,11 +507,9 @@ static void switch_serial_input(void) break; } -#ifdef CONFIG_PV_SHIM - if ( next_rx == 1 ) + if ( consoled_is_enabled() && next_rx == 1 ) domid = get_initial_domain_id(); else -#endif domid = next_rx - 1; d = rcu_lock_domain_by_id(domid); if ( d ) @@ -562,10 +560,9 @@ static void __serial_rx(char c) rc = vpl011_rx_char_xen(d, c); #endif -#ifdef CONFIG_X86 - if ( pv_shim && pv_console ) - consoled_guest_tx(c); -#endif + if ( consoled_is_enabled() ) + /* Deliver input to the PV shim console. */ + rc = consoled_guest_tx(c); if ( rc ) guest_printk(d, XENLOG_G_WARNING diff --git a/xen/drivers/char/consoled.c b/xen/drivers/char/consoled.c index b415b632ce..8704ec251e 100644 --- a/xen/drivers/char/consoled.c +++ b/xen/drivers/char/consoled.c @@ -43,13 +43,13 @@ struct xencons_interface *consoled_get_ring_addr(void) static char buf[BUF_SZ + 1]; /* Receives characters from a domain's PV console */ -void consoled_guest_rx(void) +int consoled_guest_rx(void) { size_t idx = 0; XENCONS_RING_IDX cons, prod; if ( !cons_ring ) - return; + return -ENODEV; spin_lock(&rx_lock); @@ -91,15 +91,17 @@ void consoled_guest_rx(void) out: spin_unlock(&rx_lock); + + return 0; } /* Sends a character into a domain's PV console */ -void consoled_guest_tx(char c) +int consoled_guest_tx(char c) { XENCONS_RING_IDX cons, prod; if ( !cons_ring ) - return; + return -ENODEV; cons = ACCESS_ONCE(cons_ring->in_cons); prod = cons_ring->in_prod; @@ -125,6 +127,13 @@ void consoled_guest_tx(char c) notify: /* Always notify the guest: prevents receive path from getting stuck. */ pv_shim_inject_evtchn(pv_console_evtchn()); + + return 0; +} + +bool consoled_is_enabled(void) +{ + return pv_shim && pv_console; } /* diff --git a/xen/include/xen/consoled.h b/xen/include/xen/consoled.h index bd7ab6329e..c9c1556003 100644 --- a/xen/include/xen/consoled.h +++ b/xen/include/xen/consoled.h @@ -1,12 +1,28 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ #ifndef __XEN_CONSOLED_H__ #define __XEN_CONSOLED_H__ #include +#ifdef CONFIG_PV_SHIM + void consoled_set_ring_addr(struct xencons_interface *ring); struct xencons_interface *consoled_get_ring_addr(void); -void consoled_guest_rx(void); -void consoled_guest_tx(char c); +int consoled_guest_rx(void); +int consoled_guest_tx(char c); +bool consoled_is_enabled(void); + +#else + +static inline int consoled_guest_tx(char c) +{ + ASSERT_UNREACHABLE(); + return -ENODEV; +} + +#define consoled_is_enabled() (false) + +#endif /* CONFIG_PV_SHIM */ #endif /* __XEN_CONSOLED_H__ */ /* diff --git a/xen/include/xen/domain.h b/xen/include/xen/domain.h index 3de5635291..83069de501 100644 --- a/xen/include/xen/domain.h +++ b/xen/include/xen/domain.h @@ -35,6 +35,8 @@ void getdomaininfo(struct domain *d, struct xen_domctl_getdomaininfo *info); void arch_get_domain_info(const struct domain *d, struct xen_domctl_getdomaininfo *info); +domid_t get_initial_domain_id(void); + /* CDF_* constant. Internal flags for domain creation. */ /* Is this a privileged domain? */ #define CDF_privileged (1U << 0)