Received: by 2002:a05:6358:c692:b0:131:369:b2a3 with SMTP id fe18csp1336609rwb; Fri, 28 Jul 2023 08:04:35 -0700 (PDT) X-Google-Smtp-Source: APBJJlHbgya7JdB0cC7a3Sz8OtAAFi1lNBnrKwcqvjDsDYXMWRVfk+y8lRncvDoM/0G25cPQsoxH X-Received: by 2002:a17:90a:3da4:b0:263:311f:9bcc with SMTP id i33-20020a17090a3da400b00263311f9bccmr1785912pjc.35.1690556674928; Fri, 28 Jul 2023 08:04:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690556674; cv=none; d=google.com; s=arc-20160816; b=i0ZSCykMopAJMnKsfjbno6hc8SmhnwBiATLAhvh13pJYB/8Iqv8QgdRaUZerpdsIDv aUcNC5FfMUy2lB1/IpMaUCVAnexOA263i1qNLCOBuHiXrgFJFHpf28WYft3EfiE2iszr 4Sj7jn7q5XSjH8WcRs7epsXZyfWmHat8GZRO+K7LFbISEOuyCZOrJKobRdrnpVHmBtzC wMhUtsfcV09rVOofTypxJLz7i+fVF7olymSRYKtTNfUrCgIihR5DsNDILg4kbPVkuh80 VOeahWG4z2QTSzanNl8JQEFPA72oM7xVLhf8EYTyX6aSwvbd1hAGir4tsT8TjyiTVy1l 00JA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:references :in-reply-to:subject:cc:to:dkim-signature:dkim-signature:from; bh=lC9d12Ft1vaZl1r5oaRHCKBKWx9YmTaYlRn9mDI8X2U=; fh=GYegXKRQZSi1FZ3HxnP4ilI02KuuYB4wtZPM/CUVNxw=; b=oM397PuUNp/EScFEJqOoWB0j1AUYUpitXlG8F252ijdEQpSSUcC8hMdjdD0kr0fILw HCoddB5UiqJdZqia7kldyMfxCGP8muiCyQGmw0zXVtk5dGU5OWsTmAO9HpwKeXd5q/DB 1nTduQE4rPVp01lkLTqaTEKOH0BbvBw2wAyyKtZP0V3SBP/6c9ZmV9lzxfJwXLpTOEkN M4Q+1Q2WIeMr6xucVyRNbGVRmHbwebZbV4bgxhg4TFT5jlokJnzkvcb7Oz0CjwRazM0+ 4ZYKBY1abynoV5+CYqpT+wiPEWHOAOWW4LguqRX0SAVEqLUvkU7+t9mSTz+ZGpnE7vZf sY+Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=GoKGm+ak; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=iQOrrT3P; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id m9-20020a654389000000b00563adca5169si3061639pgp.741.2023.07.28.08.04.21; Fri, 28 Jul 2023 08:04:34 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=GoKGm+ak; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e header.b=iQOrrT3P; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236169AbjG1Mze (ORCPT + 99 others); Fri, 28 Jul 2023 08:55:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59970 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235690AbjG1Mzb (ORCPT ); Fri, 28 Jul 2023 08:55:31 -0400 Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 65A5B30FA; Fri, 28 Jul 2023 05:55:30 -0700 (PDT) From: Thomas Gleixner DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1690548929; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=lC9d12Ft1vaZl1r5oaRHCKBKWx9YmTaYlRn9mDI8X2U=; b=GoKGm+akHT+hB9vRSCpMfYGpBobhIc11qZ001utqipWUHSb9C6eoGKvkxtNxt2OMbvI1Ea sSbJ4H5Xjpe9+KiXZ0xr/9ykNF8/oCyr7Nt9B8N0SyO36C31KU/Q8/cSMa9X0EfKsQ+gdR KX/H6+LPUexJqDqLLpL/3d4+P9AewlNTvzROhWI6cLfXP7qZ7Z4aYRF3T1VW8D+oTxgtSb pqrJD8ZspCvWnKQWpGskb9JaBMEuXMNJbtv02XUb4b+3ckqeZsgVaOqyAHwJutWTmH7rem xz8MKMSvrZjJDAYyHBxFfIavPxRIGpmCZ33+TaBLRS/f4cDKE6+bsTL6sBkTIA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1690548929; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=lC9d12Ft1vaZl1r5oaRHCKBKWx9YmTaYlRn9mDI8X2U=; b=iQOrrT3PGFmHIXkxPrBajQTbMYj6hUK2KHWRHTWQKkSzEaZEAZHpLGbBwmv+M18bh0AhVM CrnGCbTpjBfyX5CQ== To: Zhang Rui , peterz@infradead.org, bp@alien8.de, rafael.j.wysocki@intel.com Cc: linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org, x86@kernel.org, feng.tang@intel.com Subject: Re: [RFC PATCH] x86/acpi: Ignore invalid x2APIC entries In-Reply-To: <87jzukqjvf.ffs@tglx> References: <20230702162802.344176-1-rui.zhang@intel.com> <87jzukqjvf.ffs@tglx> Date: Fri, 28 Jul 2023 14:55:28 +0200 Message-ID: <87h6poqjpb.ffs@tglx> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 28 2023 at 14:51, Thomas Gleixner wrote: > On Mon, Jul 03 2023 at 00:28, Zhang Rui wrote: >> >> +static bool has_lapic_cpus; > > Yet another random flag. Sigh. > > I really hate this. Why not doing the obvious? > > --- a/arch/x86/kernel/apic/apic.c > +++ b/arch/x86/kernel/apic/apic.c > @@ -2452,6 +2452,9 @@ int generic_processor_info(int apicid, i > bool boot_cpu_detected = physid_isset(boot_cpu_physical_apicid, > phys_cpu_present_map); > > + if (physid_isset(apicid, phys_cpu_present_map)) > + return -EBUSY; > + > /* > * boot_cpu_physical_apicid is designed to have the apicid > * returned by read_apic_id(), i.e, the apicid of the > > As the call sites during MADT parsing ignore the return value anyway, > there is no harm and this is a proper defense against broken tables > which enumerate an APIC twice. In fact that function should not have a return value at all, but because it's not clearly separated between boot time and physical hotplug, it has to have one ...