From mboxrd@z Thu Jan 1 00:00:00 1970 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D9572C433EF for ; Tue, 8 Mar 2022 12:12:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346716AbiCHMNL (ORCPT ); Tue, 8 Mar 2022 07:13:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51714 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1344736AbiCHMNK (ORCPT ); Tue, 8 Mar 2022 07:13:10 -0500 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 650103F30B for ; Tue, 8 Mar 2022 04:12:14 -0800 (PST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 36071139F; Tue, 8 Mar 2022 04:12:13 -0800 (PST) Received: from monolith.localdoman (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8DFC13FA5D; Tue, 8 Mar 2022 04:12:12 -0800 (PST) Date: Tue, 8 Mar 2022 12:12:38 +0000 From: Alexandru Elisei To: Dongli Si Cc: kvm@vger.kernel.org Subject: Re: [PATCH kvmtool] x86: Fixed Unable to execute init process since glibc version 2.33 Message-ID: References: <20220226060048.3-1-sidongli1997@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220226060048.3-1-sidongli1997@gmail.com> Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org Hi, Hi, I was unable to apply the patch as is because your base is missing commit 20b93be583f6 ("x86: Set the correct APIC ID"). Changing it is so it applies was trivial though. I tested the patch with a VM image using glibc 2.36 and Linux v5.17-rc7, the glibc error went away and I was able to boot to userspace; as an added bonus, this kernel message went away: [ 0.000000] CPU: vendor_id 'LKVMLKVMLKVM' unknown, using generic init. [ 0.000000] CPU: Your system may be unstable. But this warning is new: [ 0.000000] [Firmware Bug]: TSC doesn't count with P0 frequency! I don't know what causes it (kvmtool's bios implementation is not feature complete with regards to my CPU features?), and doesn't look like it's something introduced by this patch, so as far as I'm concerned the patch is working as intended: Tested-by: Alexandru Elisei Thanks, Alex On Sat, Feb 26, 2022 at 02:00:48PM +0800, Dongli Si wrote: > From: Dongli Si > > glibc detected invalid CPU Vendor name will cause an error: > > [ 0.450127] Run /sbin/init as init process > /lib64/libc.so.6: CPU ISA level is lower than required > [ 0.451931] Kernel panic - not syncing: Attempted to kill init! exitcode=0x00007f00 > [ 0.452117] CPU: 0 PID: 1 Comm: init Not tainted 5.17.0-rc1 #72 > > Signed-off-by: Dongli Si > --- > x86/cpuid.c | 14 +++++++++----- > 1 file changed, 9 insertions(+), 5 deletions(-) > > diff --git a/x86/cpuid.c b/x86/cpuid.c > index c3b67d9..d58a027 100644 > --- a/x86/cpuid.c > +++ b/x86/cpuid.c > @@ -2,6 +2,7 @@ > > #include "kvm/kvm.h" > #include "kvm/util.h" > +#include "kvm/cpufeature.h" > > #include > #include > @@ -10,7 +11,7 @@ > > static void filter_cpuid(struct kvm_cpuid2 *kvm_cpuid) > { > - unsigned int signature[3]; > + struct cpuid_regs regs; > unsigned int i; > > /* > @@ -22,10 +23,13 @@ static void filter_cpuid(struct kvm_cpuid2 *kvm_cpuid) > switch (entry->function) { > case 0: > /* Vendor name */ > - memcpy(signature, "LKVMLKVMLKVM", 12); > - entry->ebx = signature[0]; > - entry->ecx = signature[1]; > - entry->edx = signature[2]; > + regs = (struct cpuid_regs) { > + .eax = 0x00, > + }; > + host_cpuid(®s); > + entry->ebx = regs.ebx; > + entry->ecx = regs.ecx; > + entry->edx = regs.edx; > break; > case 1: > /* Set X86_FEATURE_HYPERVISOR */ > -- > 2.32.0 >