Received-SPF: pass (google.com: domain of linux-kernel+bounces-185841-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Wed, 22 May 2024 15:10:00 +0800
From: Chao Gao <chao.gao@intel.com>
To: Sean Christopherson <seanjc@google.com>
CC: Paolo Bonzini <pbonzini@redhat.com>, <kvm@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, Kai Huang <kai.huang@intel.com>
Subject: Re: [PATCH v2 2/6] KVM: Rename functions related to enabling
 virtualization hardware
Message-ID: <Zk2aSIK3sbtr0WEP@chao-email>
References: <20240522022827.1690416-1-seanjc@google.com>
 <20240522022827.1690416-3-seanjc@google.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20240522022827.1690416-3-seanjc@google.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?42XK06Mdqh5FIyg1+DvK9mvEi6GCVomyBZdq2uCuIy1yT6yldHhkYG+V0qB/?=
 =?us-ascii?Q?mwubR/t/uA0F7ldbjbrD732RB/ux9DDZRItX1C+1FeAfl8+/GrrPZiY1/NXx?=
 =?us-ascii?Q?1wdgXY7/XMnChxa5wSpGU911qhZCg/OVmmxk4z+WGpqBbUH9WquIk4bv3UyI?=
 =?us-ascii?Q?GblzuXiC26/5uYila3+wmJLuPGue3YaBcPpuAoTG/EnXmxL9wwgOEIzuK/7J?=
 =?us-ascii?Q?wGc94fA/zeN1Q1EHg8AgpKYCpVfa2VgqaaC6byFzM+Aia1sGFkpNNRJ4YmoE?=
 =?us-ascii?Q?6R+Phfc2ufi5CueHojCfXQAnzLO7j/2q5BNp71pVU/nGZIU+TObVj+LZ7HSo?=
 =?us-ascii?Q?KhNvSYQ+ToO9d7ZjNKzuvuSBXWs57BcMyxU52f+yVw/uV48DtN9PS1TmQKkA?=
 =?us-ascii?Q?PGOLZPJHkxeBtBOAu+TJNiRvn2cpJuIBwMepsQtkIDq4je9uV26VL0gWoVZc?=
 =?us-ascii?Q?w+0WYWzR81cNjK/GGHYFe55v8DDrGt7cHL0EoDNAXvn9NHSGiSkTxuSlQax+?=
 =?us-ascii?Q?vevbDQ4FP0Y4r3bdQqVOFUgb2AhtmRWT9ffhEPO/wxA4FctP4ZVt1C29YqPE?=
 =?us-ascii?Q?BYVfUUZR5zYNxjfWpwPUl/WwRunQeYAv0IPiaH6/leQlyodwgSdb3csUPF7E?=
 =?us-ascii?Q?aoMMIQ0v5fXylJ0jySys4AFcelCd1HdQEaHbw3wY/NLd2AiqQHChQEBFsByI?=
 =?us-ascii?Q?iCZkaNYxZ6Qh/diXBcQdXbB6X5Wk6xPxvVMsDzFgtuNoI5rfQfcaL9AAUls6?=
 =?us-ascii?Q?ti0tKYwW1SO5tuvqtRdMCt71Ou62SIDVLBFadITm0046yIBlVuJDMOxGAwVC?=
 =?us-ascii?Q?zlauxTkZrbbw1QxhjhXxNrEQVmJq5QjNugPhaIw1G4hxYLUs1Ku4MhnJfkvT?=
 =?us-ascii?Q?I7N63UaLljAst7IQtZa1r4yg2bDCLtfv9O2fK51DZyaei3WyFWPhdGIjjzSV?=
 =?us-ascii?Q?terluOykxn4l7z8Mx2JiyDQWD4gWPB0P3ROliraOwb2vO6qRePqEnGGoBcbC?=
 =?us-ascii?Q?qsvkwfd6pE47/g9jmTl20dJx/VCyKPdCkAhulQlMtet1pIcT8hJ1+Z4X9kwW?=
 =?us-ascii?Q?X2z4bDq+/HbGoQsZYVMwiBg+v+5p37DWQcq8xIxObMBuAMJhVUsqFueoBmJ0?=
 =?us-ascii?Q?wgFoK95NwiXDu89VDrAY69oZLuC8wwQy0of3QVzdZ0jH+2VWK/yVhFii+vAo?=
 =?us-ascii?Q?2Qs8RyKE2dAf4FtVltBfrIkkUNVtZcNsAGagObYKspZ+aIWkjrrtjFreUGpO?=
 =?us-ascii?Q?cka58addaF2pb95UhliODU7n7OB8fEnmSyzURK/DL4yJYCG3pbWHtg15fHhJ?=
 =?us-ascii?Q?KTJ0DLEBvkBccngSWQXu9fFKZTGs3fAYcOG1ApKrhdejers6ZDsc4gYS62kh?=
 =?us-ascii?Q?P6iH0PyEwa7cTY8fnj9SVvZPEpHxpa7UzdO6UDZJug+CVTogE0aMWHamSw4U?=
 =?us-ascii?Q?353WzG3KfuMxeNdYcCNJbeO2W1N1veX4puNo8KLjwUofJAR5k1nZenaMxqhr?=
 =?us-ascii?Q?H2Y/LaUIogC5WfwjSwXvGZRrsVBvcsdf6Obs7Zh8ooVZaf7NxPAkkEeFjwmZ?=
 =?us-ascii?Q?UUwaEvVXxQQXbVPwS838fYjUdC98UiHvZGfHxv56?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 9d4898fb-ee99-458f-1d4a-08dc7a2e365b
X-MS-Exchange-CrossTenant-AuthSource: CH3PR11MB8660.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 May 2024 07:10:07.9826
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: jMoh7PYddL5eVCFC3M5MwuzDLzKmKU9j5fe23M3tCHYaZspW6ULEl3EFdA4+1pp1TYdUUr60cGJOVtfBDCJKFQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB6836
X-OriginatorOrg: intel.com

On Tue, May 21, 2024 at 07:28:23PM -0700, Sean Christopherson wrote:
>Rename the various functions that enable virtualization to prepare for
>upcoming changes, and to clean up artifacts of KVM's previous behavior,
>which required manually juggling locks around kvm_usage_count.
>
>Drop the "nolock" qualifier from per-CPU functions now that there are no
>"nolock" implementations of the "all" variants, i.e. now that calling a
>non-nolock function from a nolock function isn't confusing (unlike this
>sentence).
>
>Drop "all" from the outer helpers as they no longer manually iterate
>over all CPUs, and because it might not be obvious what "all" refers to.
>Instead, use double-underscores to communicate that the per-CPU functions
>are helpers to the outer APIs.
>
>Opportunistically prepend "kvm" to all functions to help make it clear
>that they are KVM helpers, but mostly there's no reason not to.
>
>Lastly, use "virtualization" instead of "hardware", because while the
>functions do enable virtualization in hardware, there are a _lot_ of
>things that KVM enables in hardware.
>
>Signed-off-by: Sean Christopherson <seanjc@google.com>

Reviewed-by: Chao Gao <chao.gao@intel.com>