From nobody Sun Feb 8 21:47:35 2026 Delivered-To: importer@patchew.org Received-SPF: none (zoho.com: 192.237.175.120 is neither permitted nor denied by domain of lists.xenproject.org) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=none (zoho.com: 192.237.175.120 is neither permitted nor denied by domain of lists.xenproject.org) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1564661219; cv=none; d=zoho.com; s=zohoarc; b=OOZMQc68JIIRKgeO/q5FN3PPXC9uDYs+6+0vXIQxETa5sxc/a5HCJUd5WuvpXOvNlx9H55z9R2N2BDfACqBnSHZuqyuKPDEl5H5LZ0SQUvCWB8tN79EVYjbZ0j1z3vmPD2Z6rjRaJ+J28O2aAHjT3saMNy5XcgPmRKf4SNCgG40= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1564661219; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=Ed0UebafeV3JwMIEEVWBn+KA/ooMEZahDPuwJBawgZM=; b=U1RUtmPCaNfVb4Kwc84ovb2aVpIgDpM7j/ACC/Qe5mfZhk5AGwcge+4AWCejgdPxTOPl6FOBMY1dQaseC5kvSDM0RWMV6UUIomgZ3nziUMDH7fNCOFsAC+WyMT/7/+eRG/utvzV9OO7quuAmoegluIdSBrtMiqWG07d4KlEhuU0= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=fail; spf=none (zoho.com: 192.237.175.120 is neither permitted nor denied by domain of lists.xenproject.org) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1564661219519939.4800220137122; Thu, 1 Aug 2019 05:06:59 -0700 (PDT) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1ht9pz-0002LF-03; Thu, 01 Aug 2019 12:05:38 +0000 Received: from us1-rack-dfw2.inumbo.com ([104.130.134.6]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1ht9py-0002L7-16 for xen-devel@lists.xenproject.org; Thu, 01 Aug 2019 12:05:38 +0000 Received: from mail-lj1-x241.google.com (unknown [2a00:1450:4864:20::241]) by us1-rack-dfw2.inumbo.com (Halon) with ESMTPS id ab7185bd-b454-11e9-8980-bc764e045a96; Thu, 01 Aug 2019 12:05:36 +0000 (UTC) Received: by mail-lj1-x241.google.com with SMTP id v18so69065334ljh.6 for ; Thu, 01 Aug 2019 05:05:36 -0700 (PDT) Received: from 3489.kyiv.epam.com (ll-22.209.223.85.sovam.net.ua. [85.223.209.22]) by smtp.gmail.com with ESMTPSA id u18sm12147150lfe.65.2019.08.01.05.05.34 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Thu, 01 Aug 2019 05:05:34 -0700 (PDT) X-Inumbo-ID: ab7185bd-b454-11e9-8980-bc764e045a96 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=HuOjIzz1M8CEnJZd66BCEfSyypiUYI4PIF63pd0WSRw=; b=cfZ/g1qG4Qr0oprxD0SbN1DvE2YhkFbAqSzhpVXhwS98Dfmqn/3NtwPdkd+S71BWPr fCjWJ+qMGbLjg9vSeJswf3OiMJCxRw9u5YefHIT7uwUaY3TGqlttkzxDlLF0e8y+9FdQ jzu8abcFF+vm2G2IqyVgJrDEiphdxzvyBvr06Lq1ftK/CXG+Fy0w+o0emRO7W9Psk+RE EEoM66GEIcXoEsfdWJzrxqQCrmfNq3LUJGkmH4LYNGjLPF5diQQ1+sEWmuEAJqyDPFjY z8oxi1Gny4IlyMDIm1Bq0muK8K1Wwi+DmfxiwybQ6xejwMekfAguzfhkuHyjAMU53WsH /HpA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=HuOjIzz1M8CEnJZd66BCEfSyypiUYI4PIF63pd0WSRw=; b=nQcrdT2W0+7sBSXnOHfjH8/9dF1TMbYhzqphAXC+NtfsDWT7D0GGYRGv3DkEnZUy06 z1rH2S5g241WW4nPSEm21XMfD/iVpmEQzvtVxhAqoNgapzF5z88Y4k4WZdzhrFBSQI7g QOT84UvtvLdUfRBvZdm7Mlz53vTGv9prSLtikOTpsFyd39zszOnqbgTEW3whnnIULc8y F97tW2Fm9HhdW+xhWKfh/Yo9L4qjdXlC419vZrSLKoP6kWYUr4vYMhDWfA4wQo+gCrQx E5j7nCQS0XrnWc9tzplpxYL0slewckckg2cjj8wA9wdSFxiBj5J79AO/abCIuXMsLBuI FonQ== X-Gm-Message-State: APjAAAUNKt0/V0XBsPYuUbC2ihAqyulh+/049JbIEJPpWV9sSYl0tN99 AeR2yWP2M+3/zOSHAMkb2wBQpo7LXGU= X-Google-Smtp-Source: APXvYqyYcDYPRlVZEYU0m8x2hQZ3JSqNxDClQgL9fg9y94Hhf3Ib59jX/Hw9ihPDg2YT7TenLRnRpA== X-Received: by 2002:a2e:8ed2:: with SMTP id e18mr26301962ljl.235.1564661135137; Thu, 01 Aug 2019 05:05:35 -0700 (PDT) From: Viktor Mitin To: xen-devel@lists.xenproject.org Date: Thu, 1 Aug 2019 15:04:52 +0300 Message-Id: <20190801120452.6814-3-viktor.mitin.19@gmail.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190801120452.6814-1-viktor.mitin.19@gmail.com> References: <20190801120452.6814-1-viktor.mitin.19@gmail.com> Subject: [Xen-devel] [PATCH v5 2/2] xen/arm: consolidate make_timer_node and make_timer_domU_node X-BeenThere: xen-devel@lists.xenproject.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Cc: Volodymyr Babchuk , Julien Grall , Stefano Stabellini , Viktor Mitin , Viktor Mitin MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Functions make_timer_node and make_timer_domU_node are quite similar. So it is better to consolidate them to avoid discrepancy. The main difference between the functions is the timer interrupts used. Keep the domU version for the compatible as it is simpler. Mean do not copy platform's 'compatible' property into hwdom device tree, instead set either arm,armv7-timer or arm,armv8-timer, depending on the platform type. Keep the hw version for the clock as it is relevant for the both cases. The new function has changed prototype due to fdt_property_interrupts make_timer_node(const struct kernel_info *kinfo) Suggested-by: Julien Grall Signed-off-by: Viktor Mitin --- v4 updates: updated "Keep the domU version for the compatible as it is simpler" v5 updates: - changed 'kept' to 'keep', etc. - removed empty line - updated indentation of parameters in functions calls - fixed NITs - updated commit message --- xen/arch/arm/domain_build.c | 106 +++++++++++++----------------------- 1 file changed, 39 insertions(+), 67 deletions(-) diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c index bc7d17dd2c..58542130ca 100644 --- a/xen/arch/arm/domain_build.c +++ b/xen/arch/arm/domain_build.c @@ -973,10 +973,8 @@ static int __init make_timer_node(const struct kernel_= info *kinfo) { /* sentinel */ }, }; struct dt_device_node *dev; - u32 len; - const void *compatible; int res; - unsigned int irq; + unsigned int irq[MAX_TIMER_PPI]; gic_interrupt_t intrs[3]; u32 clock_frequency; bool clock_valid; @@ -990,35 +988,49 @@ static int __init make_timer_node(const struct kernel= _info *kinfo) return -FDT_ERR_XEN(ENOENT); } =20 - compatible =3D dt_get_property(dev, "compatible", &len); - if ( !compatible ) - { - dprintk(XENLOG_ERR, "Can't find compatible property for timer node= \n"); - return -FDT_ERR_XEN(ENOENT); - } - res =3D fdt_begin_node(fdt, "timer"); if ( res ) return res; =20 - res =3D fdt_property(fdt, "compatible", compatible, len); - if ( res ) - return res; - - /* The timer IRQ is emulated by Xen. It always exposes an active-low - * level-sensitive interrupt */ - - irq =3D timer_get_irq(TIMER_PHYS_SECURE_PPI); - dt_dprintk(" Secure interrupt %u\n", irq); - set_interrupt(intrs[0], irq, 0xf, DT_IRQ_TYPE_LEVEL_LOW); + if ( !is_64bit_domain(kinfo->d) ) + { + res =3D fdt_property_string(fdt, "compatible", "arm,armv7-timer"); + if ( res ) + return res; + } + else + { + res =3D fdt_property_string(fdt, "compatible", "arm,armv8-timer"); + if ( res ) + return res; + } =20 - irq =3D timer_get_irq(TIMER_PHYS_NONSECURE_PPI); - dt_dprintk(" Non secure interrupt %u\n", irq); - set_interrupt(intrs[1], irq, 0xf, DT_IRQ_TYPE_LEVEL_LOW); + /* + * The timer IRQ is emulated by Xen. + * It always exposes an active-low level-sensitive interrupt + */ =20 - irq =3D timer_get_irq(TIMER_VIRT_PPI); - dt_dprintk(" Virt interrupt %u\n", irq); - set_interrupt(intrs[2], irq, 0xf, DT_IRQ_TYPE_LEVEL_LOW); + if ( is_hardware_domain(kinfo->d) ) + { + irq[TIMER_PHYS_SECURE_PPI] =3D timer_get_irq(TIMER_PHYS_SECURE_PPI= ); + irq[TIMER_PHYS_NONSECURE_PPI] =3D + timer_get_irq(TIMER_PHYS_NONSECURE_PPI= ); + irq[TIMER_VIRT_PPI] =3D timer_get_irq(TIMER_VIRT_PPI); + } + else + { + irq[TIMER_PHYS_SECURE_PPI] =3D GUEST_TIMER_PHYS_S_PPI; + irq[TIMER_PHYS_NONSECURE_PPI] =3D GUEST_TIMER_PHYS_NS_PPI; + irq[TIMER_VIRT_PPI] =3D GUEST_TIMER_VIRT_PPI; + } + dt_dprintk(" Secure interrupt %u\n", irq[TIMER_PHYS_SECURE_PPI]); + set_interrupt(intrs[0], irq[TIMER_PHYS_SECURE_PPI], + 0xf, DT_IRQ_TYPE_LEVEL_LOW); + dt_dprintk(" Non secure interrupt %u\n", irq[TIMER_PHYS_NONSECURE_PPI= ]); + set_interrupt(intrs[1], irq[TIMER_PHYS_NONSECURE_PPI], + 0xf, DT_IRQ_TYPE_LEVEL_LOW); + dt_dprintk(" Virt interrupt %u\n", irq[TIMER_VIRT_PPI]); + set_interrupt(intrs[2], irq[TIMER_VIRT_PPI], 0xf, DT_IRQ_TYPE_LEVEL_LO= W); =20 res =3D fdt_property_interrupts(kinfo, intrs, 3); if ( res ) @@ -1603,46 +1615,6 @@ static int __init make_gic_domU_node(const struct do= main *d, void *fdt) } } =20 -static int __init make_timer_domU_node(const struct domain *d, void *fdt) -{ - int res; - gic_interrupt_t intrs[3]; - - res =3D fdt_begin_node(fdt, "timer"); - if ( res ) - return res; - - if ( !is_64bit_domain(d) ) - { - res =3D fdt_property_string(fdt, "compatible", "arm,armv7-timer"); - if ( res ) - return res; - } - else - { - res =3D fdt_property_string(fdt, "compatible", "arm,armv8-timer"); - if ( res ) - return res; - } - - set_interrupt(intrs[0], GUEST_TIMER_PHYS_S_PPI, 0xf, DT_IRQ_TYPE_LEVEL= _LOW); - set_interrupt(intrs[1], GUEST_TIMER_PHYS_NS_PPI, 0xf, DT_IRQ_TYPE_LEVE= L_LOW); - set_interrupt(intrs[2], GUEST_TIMER_VIRT_PPI, 0xf, DT_IRQ_TYPE_LEVEL_L= OW); - - res =3D fdt_property(fdt, "interrupts", intrs, sizeof (intrs[0]) * 3); - if ( res ) - return res; - - res =3D fdt_property_cell(fdt, "interrupt-parent", - GUEST_PHANDLE_GIC); - if (res) - return res; - - res =3D fdt_end_node(fdt); - - return res; -} - #ifdef CONFIG_SBSA_VUART_CONSOLE static int __init make_vpl011_uart_node(const struct domain *d, void *fdt) { @@ -1748,7 +1720,7 @@ static int __init prepare_dtb_domU(struct domain *d, = struct kernel_info *kinfo) if ( ret ) goto err; =20 - ret =3D make_timer_domU_node(d, kinfo->fdt); + ret =3D make_timer_node(kinfo); if ( ret ) goto err; =20 --=20 2.17.1 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel