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 0C663C6FA83 for ; Mon, 12 Sep 2022 15:07:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230004AbiILPHO (ORCPT ); Mon, 12 Sep 2022 11:07:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59162 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229889AbiILPHK (ORCPT ); Mon, 12 Sep 2022 11:07:10 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 85CA51AF20; Mon, 12 Sep 2022 08:07:08 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id D0B38B80DBE; Mon, 12 Sep 2022 15:07:06 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 84992C433D6; Mon, 12 Sep 2022 15:07:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1662995225; bh=Rg7NLMuwouiEh43/lwL7/797vz3AXVpAGDi9DAV/nXk=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Qm7KKsKsfKnft9z+m8x2ba258/rcheKKGMsbvPHtcAMw1sRZkUxSTUQcW/jW4d70U l2vnJG/MyS/AqmGmo2OCqn4Am8OIE2yY7k51XtMykXJP0SeDWgUb5mBX8JVVb4NWr3 4owBowB0tbm0pEpyVMgGUBqQ2KL+fKi3lhanUx9GOwI0tjtvmPmWkTxpFeOwh6/do3 nS5fItI2EGs4FmsoASmDgL5WqmGJ88Z2JeZxeE724PoPw+2sa/Ndqu9BlFjkmcjIav nyqzwmHW2/bWFgNFR+fvG6xfweXYHyiYxyG4p8s7CkcEpGS1Z+zdlmCj/zJPzylPqy 2yCAnpIJ/W+xQ== Received: from [89.101.193.71] (helo=wait-a-minute.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1oXl1b-009ic4-7f; Mon, 12 Sep 2022 16:07:03 +0100 Date: Mon, 12 Sep 2022 16:07:02 +0100 Message-ID: <877d28ocmh.wl-maz@kernel.org> From: Marc Zyngier To: isaku.yamahata@intel.com Cc: linux-kernel@vger.kernel.org, kvm@vger.kernel.org, Paolo Bonzini , Sean Christopherson , Will Deacon , Yuan Yao , isaku.yamahata@gmail.com, Kai Huang , Chao Gao , Atish Patra , Shaokun Zhang , Qi Liu , John Garry , Daniel Lezcano , Huang Ying , Huacai Chen Subject: Re: [PATCH v4 06/26] KVM: arm64: Simplify the CPUHP logic In-Reply-To: <2b74ba0aa514e3e86c4513d12614664002996067.1662679124.git.isaku.yamahata@intel.com> References: <2b74ba0aa514e3e86c4513d12614664002996067.1662679124.git.isaku.yamahata@intel.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/27.1 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-SA-Exim-Connect-IP: 89.101.193.71 X-SA-Exim-Rcpt-To: isaku.yamahata@intel.com, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, pbonzini@redhat.com, seanjc@google.com, will@kernel.org, yuan.yao@linux.intel.com, isaku.yamahata@gmail.com, kai.huang@intel.com, chao.gao@intel.com, atishp@atishpatra.org, zhangshaokun@hisilicon.com, liuqi115@huawei.com, john.garry@huawei.com, daniel.lezcano@linaro.org, ying.huang@intel.com, chenhuacai@kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 09 Sep 2022 00:25:22 +0100, isaku.yamahata@intel.com wrote: > > From: Marc Zyngier > > For a number of historical reasons, the KVM/arm64 hotplug setup is pretty > complicated, and we have two extra CPUHP notifiers for vGIC and timers. > > It looks pretty pointless, and gets in the way of further changes. > So let's just expose some helpers that can be called from the core > CPUHP callback, and get rid of everything else. > > This gives us the opportunity to drop a useless notifier entry, > as well as tidy-up the timer enable/disable, which was a bit odd. > > Signed-off-by: Marc Zyngier > Signed-off-by: Chao Gao > Link: https://lore.kernel.org/r/20220216031528.92558-5-chao.gao@intel.com > Signed-off-by: Isaku Yamahata Since this patch is substantially different from the version pointed to on the list, please drop Chao's SoB and the Link: tag, as they are not relevant anymore. Thanks, M. -- Without deviation from the norm, progress is not possible.