From: Chao Gao <chao.gao@intel•com>
To: Sean Christopherson <seanjc@google•com>
Cc: x86@kernel•org, "Wanpeng Li" <wanpengli@tencent•com>,
kvm@vger•kernel.org, "David Hildenbrand" <david@redhat•com>,
"Dave Hansen" <dave.hansen@linux•intel.com>,
linux-mips@vger•kernel.org, "Atish Patra" <atish.patra@wdc•com>,
"Paul Mackerras" <paulus@samba•org>,
"H. Peter Anvin" <hpa@zytor•com>,
"Alexander Gordeev" <agordeev@linux•ibm.com>,
"Claudio Imbrenda" <imbrenda@linux•ibm.com>,
"Will Deacon" <will@kernel•org>,
"Maciej S. Szmigiero" <maciej.szmigiero@oracle•com>,
linux-s390@vger•kernel.org,
"Janosch Frank" <frankja@linux•ibm.com>,
"Marc Zyngier" <maz@kernel•org>, "Joerg Roedel" <joro@8bytes•org>,
"Huacai Chen" <chenhuacai@kernel•org>,
linux-riscv@lists•infradead.org, kvmarm@lists•cs.columbia.edu,
"Aleksandar Markovic" <aleksandar.qemu.devel@gmail•com>,
"Ingo Molnar" <mingo@redhat•com>,
"Catalin Marinas" <catalin.marinas@arm•com>,
"Palmer Dabbelt" <palmer@dabbelt•com>,
"Christian Borntraeger" <borntraeger@linux•ibm.com>,
"Ravi Bangoria" <ravi.bangoria@linux•ibm.com>,
kevin.tian@intel•com, "Albert Ou" <aou@eecs•berkeley.edu>,
"Vasily Gorbik" <gor@linux•ibm.com>,
"Suzuki K Poulose" <suzuki.poulose@arm•com>,
"Heiko Carstens" <hca@linux•ibm.com>,
"Nicholas Piggin" <npiggin@gmail•com>,
"Borislav Petkov" <bp@alien8•de>,
"Cédric Le Goater" <clg@kaod•org>,
"Paul Walmsley" <paul.walmsley@sifive•com>,
tglx@linutronix•de, "Alexandru Elisei" <alexandru.elisei@arm•com>,
linux-arm-kernel@lists•infradead.org,
"Jim Mattson" <jmattson@google•com>,
"Thomas Bogendoerfer" <tsbogend@alpha•franken.de>,
"Fabiano Rosas" <farosas@linux•ibm.com>,
"Anup Patel" <anup.patel@wdc•com>,
linux-kernel@vger•kernel.org,
"Bharata B Rao" <bharata@linux•ibm.com>,
"James Morse" <james.morse@arm•com>,
kvm-riscv@lists•infradead.org, pbonzini@redhat•com,
"Vitaly Kuznetsov" <vkuznets@redhat•com>,
linuxppc-dev@lists•ozlabs.org
Subject: Re: [PATCH 3/6] KVM: Remove opaque from kvm_arch_check_processor_compat
Date: Tue, 11 Jan 2022 11:19:34 +0800 [thread overview]
Message-ID: <20220111031933.GB2175@gao-cwp> (raw)
In-Reply-To: <Ydy8BCfE0jhJd5uE@google.com>
On Mon, Jan 10, 2022 at 11:06:44PM +0000, Sean Christopherson wrote:
>On Mon, Dec 27, 2021, Chao Gao wrote:
>> No arch implementation uses this opaque now.
>
>Except for the RISC-V part, this can be a pure revert of commit b99040853738 ("KVM:
>Pass kvm_init()'s opaque param to additional arch funcs"). I think it makes sense
>to process it as a revert, with a short blurb in the changelog to note that RISC-V
>is manually modified as RISC-V support came along in the interim.
commit b99040853738 adds opaque param to kvm_arch_hardware_setup(), which isn't
reverted in this patch. I.e., this patch is a partial revert of b99040853738
plus manual changes to RISC-V. Given that, "process it as a revert" means
clearly say in changelog that this commit contains a partial revert of commit
b99040853738 ("KVM: Pass kvm_init()'s opaque param to additional arch funcs").
Right?
next prev parent reply other threads:[~2022-01-11 3:10 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-27 8:15 [PATCH 0/6] Improve KVM's interaction with CPU hotplug Chao Gao
2021-12-27 8:15 ` [PATCH 3/6] KVM: Remove opaque from kvm_arch_check_processor_compat Chao Gao
2022-01-10 23:06 ` Sean Christopherson
2022-01-11 3:19 ` Chao Gao [this message]
2022-01-12 17:20 ` Sean Christopherson
2022-01-12 17:21 ` Sean Christopherson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220111031933.GB2175@gao-cwp \
--to=chao.gao@intel$(echo .)com \
--cc=agordeev@linux$(echo .)ibm.com \
--cc=aleksandar.qemu.devel@gmail$(echo .)com \
--cc=alexandru.elisei@arm$(echo .)com \
--cc=anup.patel@wdc$(echo .)com \
--cc=aou@eecs$(echo .)berkeley.edu \
--cc=atish.patra@wdc$(echo .)com \
--cc=bharata@linux$(echo .)ibm.com \
--cc=borntraeger@linux$(echo .)ibm.com \
--cc=bp@alien8$(echo .)de \
--cc=catalin.marinas@arm$(echo .)com \
--cc=chenhuacai@kernel$(echo .)org \
--cc=clg@kaod$(echo .)org \
--cc=dave.hansen@linux$(echo .)intel.com \
--cc=david@redhat$(echo .)com \
--cc=farosas@linux$(echo .)ibm.com \
--cc=frankja@linux$(echo .)ibm.com \
--cc=gor@linux$(echo .)ibm.com \
--cc=hca@linux$(echo .)ibm.com \
--cc=hpa@zytor$(echo .)com \
--cc=imbrenda@linux$(echo .)ibm.com \
--cc=james.morse@arm$(echo .)com \
--cc=jmattson@google$(echo .)com \
--cc=joro@8bytes$(echo .)org \
--cc=kevin.tian@intel$(echo .)com \
--cc=kvm-riscv@lists$(echo .)infradead.org \
--cc=kvm@vger$(echo .)kernel.org \
--cc=kvmarm@lists$(echo .)cs.columbia.edu \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-mips@vger$(echo .)kernel.org \
--cc=linux-riscv@lists$(echo .)infradead.org \
--cc=linux-s390@vger$(echo .)kernel.org \
--cc=linuxppc-dev@lists$(echo .)ozlabs.org \
--cc=maciej.szmigiero@oracle$(echo .)com \
--cc=maz@kernel$(echo .)org \
--cc=mingo@redhat$(echo .)com \
--cc=npiggin@gmail$(echo .)com \
--cc=palmer@dabbelt$(echo .)com \
--cc=paul.walmsley@sifive$(echo .)com \
--cc=paulus@samba$(echo .)org \
--cc=pbonzini@redhat$(echo .)com \
--cc=ravi.bangoria@linux$(echo .)ibm.com \
--cc=seanjc@google$(echo .)com \
--cc=suzuki.poulose@arm$(echo .)com \
--cc=tglx@linutronix$(echo .)de \
--cc=tsbogend@alpha$(echo .)franken.de \
--cc=vkuznets@redhat$(echo .)com \
--cc=wanpengli@tencent$(echo .)com \
--cc=will@kernel$(echo .)org \
--cc=x86@kernel$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox