From nobody Thu Apr 25 14:05:50 2024 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; 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=1584611828; cv=none; d=zohomail.com; s=zohoarc; b=c+5vljELvCwnPUC0ts6k5VAVdqVGJp1yTlhRuo8QDj0sNGOhqRHMPx42QpHDJ38DB63tTyj6BA/T5lpmwRK53UYVK1CJUiWUXhjDGZhtiZvWMECSqv1yRCWxTKc7fcZ/bIbYA6wRiQsLzO7Z/sr668h+oPRQwH1sHyb9Wghoy8k= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1584611828; 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; bh=7xn6Z+AYxJkCgSluRkTkvis4GNJurUrhe8LbDCjcN88=; b=TAyhmrZDLnSMrH1LCtDjaQ+ZDP/PBrO5BhfphtM4rSJvCZuz29ZzbZx7iTjCcKOUUMIgZRQpwt1sCvWrQqoJ26PrlU8aZxJkp9/KHTGakIqJz7Fz1Q5hY/9+4si+LaukOGarVVMQhzBDoM1dOVk/2d4B9TvNNE/7DDTw3+lN2Ec= ARC-Authentication-Results: i=1; mx.zohomail.com; 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 1584611828787768.2739374692778; Thu, 19 Mar 2020 02:57:08 -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 1jEruP-0007xW-5z; Thu, 19 Mar 2020 09:56:13 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1jEruO-0007xG-Iw for xen-devel@lists.xenproject.org; Thu, 19 Mar 2020 09:56:12 +0000 Received: from mx2.suse.de (unknown [195.135.220.15]) by us1-rack-iad1.inumbo.com (Halon) with ESMTPS id dcbc682c-69c7-11ea-b34e-bc764e2007e4; Thu, 19 Mar 2020 09:56:11 +0000 (UTC) Received: from relay2.suse.de (unknown [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id C6909AE59; Thu, 19 Mar 2020 09:56:10 +0000 (UTC) X-Inumbo-ID: dcbc682c-69c7-11ea-b34e-bc764e2007e4 X-Virus-Scanned: by amavisd-new at test-mx.suse.de From: Miroslav Benes To: boris.ostrovsky@oracle.com, jgross@suse.com, sstabellini@kernel.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, hpa@zytor.com, jpoimboe@redhat.com Date: Thu, 19 Mar 2020 10:56:05 +0100 Message-Id: <20200319095606.23627-2-mbenes@suse.cz> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20200319095606.23627-1-mbenes@suse.cz> References: <20200319095606.23627-1-mbenes@suse.cz> MIME-Version: 1.0 Subject: [Xen-devel] [PATCH v2 1/2] x86/xen: Make the boot CPU idle task reliable 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: andrew.cooper3@citrix.com, x86@kernel.org, linux-kernel@vger.kernel.org, live-patching@vger.kernel.org, xen-devel@lists.xenproject.org, Miroslav Benes , jslaby@suse.cz Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" The unwinder reports the boot CPU idle task's stack on XEN PV as unreliable, which affects at least live patching. There are two reasons for this. First, the task does not follow the x86 convention that its stack starts at the offset right below saved pt_regs. It allows the unwinder to easily detect the end of the stack and verify it. Second, startup_xen() function does not store the return address before jumping to xen_start_kernel() which confuses the unwinder. Amend both issues by moving the starting point of initial stack in startup_xen() and storing the return address before the jump, which is exactly what call instruction does. Signed-off-by: Miroslav Benes --- arch/x86/xen/xen-head.S | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/arch/x86/xen/xen-head.S b/arch/x86/xen/xen-head.S index 1d0cee3163e4..edc776af0e0a 100644 --- a/arch/x86/xen/xen-head.S +++ b/arch/x86/xen/xen-head.S @@ -35,7 +35,11 @@ SYM_CODE_START(startup_xen) rep __ASM_SIZE(stos) =20 mov %_ASM_SI, xen_start_info - mov $init_thread_union+THREAD_SIZE, %_ASM_SP +#ifdef CONFIG_X86_64 + mov initial_stack(%rip), %_ASM_SP +#else + mov pa(initial_stack), %_ASM_SP +#endif =20 #ifdef CONFIG_X86_64 /* Set up %gs. @@ -51,7 +55,7 @@ SYM_CODE_START(startup_xen) wrmsr #endif =20 - jmp xen_start_kernel + call xen_start_kernel SYM_CODE_END(startup_xen) __FINIT #endif --=20 2.25.1 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel From nobody Thu Apr 25 14:05:50 2024 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; 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=1584611829; cv=none; d=zohomail.com; s=zohoarc; b=I2iW0TG4o9KahPcDW4N4dR0pKDK+tMIDRuC368iOZ8QHuh4aDm1RtWOdZ5YjF0abYinRpfSPP4EfanBwdlDd+HXvxE8fXVJOVljFY+lDmNanQJo8+860dsDMIEptziFG8AX06WQlppGfFGwxpP0XHTuCrSfy+qNMN9vgT47jUlI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1584611829; 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; bh=bTTfFTMaG3yaXd19WcsZ3MrU4Gk8O/jBnykBbk8w3Ug=; b=cxxaSNrP4t4or/0QTKgjPdTq5/EBNHC7pmQlZzbOu63KduOpE5s5I2my9OyzQh4QMaqSnEDTXrtRam/xowIb5ztp403QH53LwdtFUJn9k4CWZBSxo8mhs3FUSOWOKmOmt3M2TveXwRq4fq2lEDfnbvdFIjmyOwO281KQxn0PXiY= ARC-Authentication-Results: i=1; mx.zohomail.com; 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 1584611829926898.747672118296; Thu, 19 Mar 2020 02:57:09 -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 1jEruU-0007zk-Dl; Thu, 19 Mar 2020 09:56:18 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1jEruT-0007zX-Et for xen-devel@lists.xenproject.org; Thu, 19 Mar 2020 09:56:17 +0000 Received: from mx2.suse.de (unknown [195.135.220.15]) by us1-rack-iad1.inumbo.com (Halon) with ESMTPS id dd1cf138-69c7-11ea-92cf-bc764e2007e4; Thu, 19 Mar 2020 09:56:12 +0000 (UTC) Received: from relay2.suse.de (unknown [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 6CB58AF85; Thu, 19 Mar 2020 09:56:11 +0000 (UTC) X-Inumbo-ID: dd1cf138-69c7-11ea-92cf-bc764e2007e4 X-Virus-Scanned: by amavisd-new at test-mx.suse.de From: Miroslav Benes To: boris.ostrovsky@oracle.com, jgross@suse.com, sstabellini@kernel.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, hpa@zytor.com, jpoimboe@redhat.com Date: Thu, 19 Mar 2020 10:56:06 +0100 Message-Id: <20200319095606.23627-3-mbenes@suse.cz> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20200319095606.23627-1-mbenes@suse.cz> References: <20200319095606.23627-1-mbenes@suse.cz> MIME-Version: 1.0 Subject: [Xen-devel] [PATCH v2 2/2] x86/xen: Make the secondary CPU idle tasks reliable 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: andrew.cooper3@citrix.com, x86@kernel.org, linux-kernel@vger.kernel.org, live-patching@vger.kernel.org, xen-devel@lists.xenproject.org, Miroslav Benes , jslaby@suse.cz Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" The unwinder reports the secondary CPU idle tasks' stack on XEN PV as unreliable, which affects at least live patching. cpu_initialize_context() sets up the context of the CPU through VCPUOP_initialise hypercall. After it is woken up, the idle task starts in cpu_bringup_and_idle() function and its stack starts at the offset right below pt_regs. The unwinder correctly detects the end of stack there but it is confused by NULL return address in the last frame. Introduce a wrapper in assembly, which just calls cpu_bringup_and_idle(). The return address is thus pushed on the stack and the wrapper contains the annotation hint for the unwinder regarding the stack state. Signed-off-by: Miroslav Benes --- arch/x86/xen/smp_pv.c | 3 ++- arch/x86/xen/xen-head.S | 8 ++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/arch/x86/xen/smp_pv.c b/arch/x86/xen/smp_pv.c index 802ee5bba66c..6b88cdcbef8f 100644 --- a/arch/x86/xen/smp_pv.c +++ b/arch/x86/xen/smp_pv.c @@ -53,6 +53,7 @@ static DEFINE_PER_CPU(struct xen_common_irq, xen_irq_work= ) =3D { .irq =3D -1 }; static DEFINE_PER_CPU(struct xen_common_irq, xen_pmu_irq) =3D { .irq =3D -= 1 }; =20 static irqreturn_t xen_irq_work_interrupt(int irq, void *dev_id); +extern unsigned char asm_cpu_bringup_and_idle[]; =20 static void cpu_bringup(void) { @@ -309,7 +310,7 @@ cpu_initialize_context(unsigned int cpu, struct task_st= ruct *idle) * pointing just below where pt_regs would be if it were a normal * kernel entry. */ - ctxt->user_regs.eip =3D (unsigned long)cpu_bringup_and_idle; + ctxt->user_regs.eip =3D (unsigned long)asm_cpu_bringup_and_idle; ctxt->flags =3D VGCF_IN_KERNEL; ctxt->user_regs.eflags =3D 0x1000; /* IOPL_RING1 */ ctxt->user_regs.ds =3D __USER_DS; diff --git a/arch/x86/xen/xen-head.S b/arch/x86/xen/xen-head.S index edc776af0e0a..9dc6f9a420a8 100644 --- a/arch/x86/xen/xen-head.S +++ b/arch/x86/xen/xen-head.S @@ -58,6 +58,14 @@ SYM_CODE_START(startup_xen) call xen_start_kernel SYM_CODE_END(startup_xen) __FINIT + +.pushsection .text +SYM_CODE_START(asm_cpu_bringup_and_idle) + UNWIND_HINT_EMPTY + + call cpu_bringup_and_idle +SYM_CODE_END(asm_cpu_bringup_and_idle) +.popsection #endif =20 .pushsection .text --=20 2.25.1 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel