Received: by 2002:ab2:6203:0:b0:1f5:f2ab:c469 with SMTP id o3csp2496473lqt; Mon, 22 Apr 2024 12:21:03 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCVeJzSIIUykEjHn96QuqK+Vgc/Bum3IeFia/rP2FzQ6sIUMAJahYKQJQC+moCBScym3Dj530U3zv+RuMfT4QpjWapb6XLjY2ycuOoLPJg== X-Google-Smtp-Source: AGHT+IGyozYzxPQQ7Ej6denGQjO4GgSJbzfbyT1DtzAeQqWXY/KGZjuyl+atE1XNn2WWmKF1O81t X-Received: by 2002:a17:902:c951:b0:1e4:dcfe:848f with SMTP id i17-20020a170902c95100b001e4dcfe848fmr12742797pla.16.1713813663212; Mon, 22 Apr 2024 12:21:03 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1713813663; cv=pass; d=google.com; s=arc-20160816; b=QENgCfdAgY4Dq56Q26OGt1LHzRcKLeH4u2rHJl8avn48P1oOtIuevhTaC1iCBINg1g Cc8qIRgQrBptM9gviug0uuQRtAvZSPPxdwRgXz2khesO6pUceyu0RZ7yNDXRmi/ObKDd KpReKMrKFoeAQa2wEjGfsAsVsEk60VYQhLpay544Lud30Kb81jr848KWCjaPbVWCSTFr DYwZKsdMARZ/l+dG1CP+nJr/PhcY3FXYThRzY3tz/LGn7evPqiAloZckzY4gXVxLg3CT f+QAQ9e5yrXwGW4TkVTHoVhcnOqQVGO8ymQL/tJA/3g3hk/9bWGCRMiDxUDe9GrVbzCI 5YiA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:dkim-signature; bh=AKDhlAqoeTElANiopvVD+hgqCsfDl+GM2v6EgvlRQC0=; fh=960F75A5DqU49kkolKs17cmLVOPxhVmF9sG8CQZ/wfA=; b=hTnJ0Iqdi8bAafEH5JjccjyKF08nxsfclik+NvnNOiz2uwuegQbd39gWXqJkAfOjGC doJKDpKha5OcGwrcbtXW1MrjAf+YQTrvunc23gbDQVrCpWV+EVtkwT30L/xqt6+8JRqD 2zEvdREqvuKlBR/t5x4SdrlUTuSiAB8yFKKfuectn4Kg1sjI2qhyPkxR2Y1wl7qrjGsU UKWe4fsmr8js6zb834gjR5ZCmazaX0Drah9iRsYm0l71lAuBXCvzFADXBAMygDzBBwt7 gXwr5bDqqWzSDAzN202WEMT2d+FVNXeANFezowr/zmqKVR9lw+VeDULrrhfQY0690oRh j+yw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=b7ZSZoI2; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-153901-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-153901-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [147.75.48.161]) by mx.google.com with ESMTPS id c7-20020a170902724700b001e3fadbd18esi7953037pll.388.2024.04.22.12.21.02 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 22 Apr 2024 12:21:03 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-153901-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=b7ZSZoI2; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-153901-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-153901-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 76378B21504 for ; Mon, 22 Apr 2024 19:17:20 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 72D45155334; Mon, 22 Apr 2024 19:17:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="b7ZSZoI2" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8EA601DFFB; Mon, 22 Apr 2024 19:17:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713813430; cv=none; b=NefindJ92ura3clEBclDtn6QyWojClgySZfkWZ/Mfhet+LnYPoaBirq34Jk+BvOysWCEnZNUQMzWrrp7xUlrQJaZpbupoUScqjuv8zY8mfX+Id6nYtIb5/SzF/a5No4k5YMP79Hm/6OiOWo2UECSI2bfa7ZeakuW/qrwkj9Z7UU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713813430; c=relaxed/simple; bh=lHeD1D8H/MmEcMeeZUI4LjzVYUL80F8jZcOEKkoujmU=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=hMf0EhcTQIWEmZTv7yEqhoSLnYknfC//OI2TSzyWwxPRu2Cs/JocW+eQZ8H82RzF6X/Zw89DoxhOaWsQW5Zv+uzWW+vYjKqLvdi8Dk6Fa/w3y54BR6KKBBs1SM/IlNP3Q41FOWVSZlx6bhRPvgR75e8scOspIRjjnXCVMccbVz0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=b7ZSZoI2; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0F17AC4AF08; Mon, 22 Apr 2024 19:17:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1713813429; bh=lHeD1D8H/MmEcMeeZUI4LjzVYUL80F8jZcOEKkoujmU=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=b7ZSZoI2D2X7HToN0E2f+jHBxNlm4eQzdNjbZ10yTbZjs1FKmOK9k1iO6QPgtz2xp 8QejgxtE7KQUGIdIkS9QFuiY0V7Saxw5lrBpsl6L7mVfx3bRXjY5dzwP4fgz6FXq7U RpvoPIq+1CcoiYl6Tzv2QVAmo+EVV3Vy0dM03efQwFApmW4Vy+KD2hRKoErIGKhksF bsCKqvT/d1sOlAo9yJEqwHwQqRTjQH0woiQ7AJvYOiKKbJC/RtvbTmCtQXiXqCNm1d ZGZC1bHRjbQzp+on2e6E6QUky5rdwSx7QwYjBs0YcKt7eqGIxCpy9xGOT2uAzcvU/F c/ix7B+Whg4BQ== Received: by mail-oi1-f181.google.com with SMTP id 5614622812f47-3c74c0edd57so87520b6e.2; Mon, 22 Apr 2024 12:17:08 -0700 (PDT) X-Forwarded-Encrypted: i=1; AJvYcCXsqKhbvKp2P49PEV4d3c0iDMjtznONmHp/CWXimHDY5jkjvkXJvdiIlXadHVvQHLFGWrLY60pgz2q8XWJSOJHF64OOiioeTrtXIU9fJNeeWav6T7eZcas12hoEQC9+xHDKIkE3YLgen0pawjg4Sc2qqyLnDDK0vrYf0c7FM9ijjhk19zkS/zcnUQsUDCv4Xx3GsMxBS4tZ6rUlB4oUKg== X-Gm-Message-State: AOJu0Yw8vqlaurJaruMqH+BuXZGW9dMxfEJ64u6CkS95kfNLj3C+ilH4 Q/SmZpC+5rQlc++38MrUCVev4DbH6SjrW9o7hLgaxqjgFUbkYDb/B3p+N+otWPkMt5+RIFyL/d+ weSDbjyyF9iKdjd36h1aAhLl9/cY= X-Received: by 2002:a05:6820:e07:b0:5aa:14ff:4128 with SMTP id el7-20020a0568200e0700b005aa14ff4128mr11147467oob.1.1713813428277; Mon, 22 Apr 2024 12:17:08 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240418135412.14730-1-Jonathan.Cameron@huawei.com> In-Reply-To: From: "Rafael J. Wysocki" Date: Mon, 22 Apr 2024 21:16:56 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v7 00/16] ACPI/arm64: add support for virtual cpu hotplug To: Jonathan Cameron Cc: Thomas Gleixner , Peter Zijlstra , linux-pm@vger.kernel.org, loongarch@lists.linux.dev, linux-acpi@vger.kernel.org, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, x86@kernel.org, Russell King , "Rafael J . Wysocki" , Miguel Luis , James Morse , Salil Mehta , Jean-Philippe Brucker , Catalin Marinas , Will Deacon , Ingo Molnar , Borislav Petkov , Dave Hansen , linuxarm@huawei.com, justin.he@arm.com, jianyong.wu@arm.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, Apr 18, 2024 at 9:50=E2=80=AFPM Rafael J. Wysocki wrote: > > On Thu, Apr 18, 2024 at 3:54=E2=80=AFPM Jonathan Cameron > wrote: > > > > Whilst it is a bit quick after v6, a couple of critical issues > > were pointed out by Russell, Salil and Rafael + one build issue > > had been missed, so it seems sensible to make sure those conducting > > testing or further review have access to a fixed version. > > > > v7: > > - Fix misplaced config guard that broke bisection. > > - Greatly simplify the condition on which we call > > acpi_processor_hotadd_init(). > > - Improve teardown ordering. > > Thank you for the update! > > From a quick look, patches [01-08/16] appear to be good now, but I'll > do a more detailed review on the following days. Done now, I've sent comments on patches [4-5/16]. The other patches in the first half of the series LGTM. I can't say much about the ARM64-specific part and the last patch has been already ACKed by Thomas. Thanks!