From: Zhenzhong Duan <zhenzhong.duan@oracle.com>
To: linux-kernel@vger.kernel.org, tglx@linutronix.de, pbonzini@redhat.com
Cc: mingo@redhat.com, bp@alien8.de, x86@kernel.org,
rkrcmar@redhat.com, sean.j.christopherson@intel.com,
vkuznets@redhat.com, wanpengli@tencent.com, jmattson@google.com,
joro@8bytes.org, boris.ostrovsky@oracle.com, jgross@suse.com,
peterz@infradead.org, will@kernel.org,
linux-hyperv@vger.kernel.org, kvm@vger.kernel.org,
mikelley@microsoft.com, kys@microsoft.com,
haiyangz@microsoft.com, sthemmin@microsoft.com,
sashal@kernel.org
Subject: Re: [PATCH v8 0/5] Add a unified parameter "nopvspin"
Date: Tue, 19 Nov 2019 10:01:53 +0800 [thread overview]
Message-ID: <1ef9b26d-3b21-f285-bb93-063ee30e546f@oracle.com> (raw)
In-Reply-To: <86263ee3-f94f-8a6b-3842-f15fb0316798@oracle.com>
Hi Maintainers,
May I get a final update on this patchset?
There is only a few days remaining before my layoff at Oracle. I can't
login the mail address after that.
No matter if you prefer to reject it, please let me know. I'd like to
get an end to this patchset.
Thanks
Zhenzhong
On 2019/10/29 9:33, Zhenzhong Duan wrote:
> Hi Baolo, Thomas
>
> This patchset is reviewed pass and keep silent for a while, will
> anyone of you
>
> consider to pick it up? Thanks
>
> Zhenzhong
>
> On 2019/10/23 19:16, Zhenzhong Duan wrote:
>> There are cases folks want to disable spinlock optimization for
>> debug/test purpose. Xen and hyperv already have parameters
>> "xen_nopvspin"
>> and "hv_nopvspin" to support that, but kvm doesn't.
>>
>> The first patch adds that feature to KVM guest with "nopvspin".
>>
>> For compatibility reason original parameters "xen_nopvspin" and
>> "hv_nopvspin" are retained and marked obsolete.
>>
>> v8:
>> PATCH2: use 'kvm-guest' instead of 'kvm_guest' [Sean
>> Christopherson]
>> PATCH3: add a comment to explain missed 'return' [Sean
>> Christopherson]
>>
>> v7:
>> PATCH3: update comment and use goto, add RB [Vitaly
>> Kuznetsov]
>>
>> v6:
>> PATCH1: add Reviewed-by [Vitaly
>> Kuznetsov]
>> PATCH2: change 'pv' to 'PV', add Reviewed-by [Vitaly
>> Kuznetsov]
>> PATCH3: refactor 'if' branch in kvm_spinlock_init() [Vitaly
>> Kuznetsov]
>>
>> v5:
>> PATCH1: new patch to revert a currently unnecessory commit,
>> code is simpler a bit after that change. [Boris
>> Ostrovsky]
>> PATCH3: fold 'if' statement,add comments on virt_spin_lock_key,
>> reorder with PATCH2 to better reflect dependency
>> PATCH4: fold 'if' statement, add Reviewed-by [Boris
>> Ostrovsky]
>> PATCH5: add Reviewed-by [Michael Kelley]
>>
>> v4:
>> PATCH1: use variable name nopvspin instead of pvspin and
>> defined it as __initdata, changed print message,
>> updated patch description [Sean
>> Christopherson]
>> PATCH2: remove Suggested-by, use "kvm-guest:" prefix [Sean
>> Christopherson]
>> PATCH3: make variable nopvsin and xen_pvspin coexist
>> remove Reviewed-by due to code change [Sean
>> Christopherson]
>> PATCH4: make variable nopvsin and hv_pvspin coexist [Sean
>> Christopherson]
>>
>> v3:
>> PATCH2: Fix indentation
>>
>> v2:
>> PATCH1: pick the print code change into separate PATCH2,
>> updated patch description [Vitaly Kuznetsov]
>> PATCH2: new patch with print code change [Vitaly Kuznetsov]
>> PATCH3: add Reviewed-by [Juergen Gross]
>>
>> Zhenzhong Duan (5):
>> Revert "KVM: X86: Fix setup the virt_spin_lock_key before static key
>> get initialized"
>> x86/kvm: Change print code to use pr_*() format
>> x86/kvm: Add "nopvspin" parameter to disable PV spinlocks
>> xen: Mark "xen_nopvspin" parameter obsolete
>> x86/hyperv: Mark "hv_nopvspin" parameter obsolete
>>
>> Documentation/admin-guide/kernel-parameters.txt | 14 ++++-
>> arch/x86/hyperv/hv_spinlock.c | 4 ++
>> arch/x86/include/asm/qspinlock.h | 1 +
>> arch/x86/kernel/kvm.c | 79
>> ++++++++++++++++---------
>> arch/x86/xen/spinlock.c | 4 +-
>> kernel/locking/qspinlock.c | 7 +++
>> 6 files changed, 76 insertions(+), 33 deletions(-)
>>
next prev parent reply other threads:[~2019-11-19 2:05 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-23 11:16 [PATCH v8 0/5] Add a unified parameter "nopvspin" Zhenzhong Duan
2019-10-23 11:16 ` [PATCH v8 1/5] Revert "KVM: X86: Fix setup the virt_spin_lock_key before static key get initialized" Zhenzhong Duan
2019-11-19 7:00 ` Wanpeng Li
2019-10-23 11:16 ` [PATCH v8 2/5] x86/kvm: Change print code to use pr_*() format Zhenzhong Duan
2019-10-23 15:38 ` Sean Christopherson
2019-11-19 7:01 ` Wanpeng Li
2019-10-23 11:16 ` [PATCH v8 3/5] x86/kvm: Add "nopvspin" parameter to disable PV spinlocks Zhenzhong Duan
2019-10-23 15:40 ` Sean Christopherson
2019-11-19 7:01 ` Wanpeng Li
2019-10-23 11:16 ` [PATCH v8 4/5] xen: Mark "xen_nopvspin" parameter obsolete Zhenzhong Duan
2019-10-23 11:16 ` [PATCH v8 5/5] x86/hyperv: Mark "hv_nopvspin" " Zhenzhong Duan
2019-10-29 1:33 ` [PATCH v8 0/5] Add a unified parameter "nopvspin" Zhenzhong Duan
2019-11-19 2:01 ` Zhenzhong Duan [this message]
2020-03-14 11:14 ` Paolo Bonzini
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=1ef9b26d-3b21-f285-bb93-063ee30e546f@oracle.com \
--to=zhenzhong.duan@oracle.com \
--cc=boris.ostrovsky@oracle.com \
--cc=bp@alien8.de \
--cc=haiyangz@microsoft.com \
--cc=jgross@suse.com \
--cc=jmattson@google.com \
--cc=joro@8bytes.org \
--cc=kvm@vger.kernel.org \
--cc=kys@microsoft.com \
--cc=linux-hyperv@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mikelley@microsoft.com \
--cc=mingo@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peterz@infradead.org \
--cc=rkrcmar@redhat.com \
--cc=sashal@kernel.org \
--cc=sean.j.christopherson@intel.com \
--cc=sthemmin@microsoft.com \
--cc=tglx@linutronix.de \
--cc=vkuznets@redhat.com \
--cc=wanpengli@tencent.com \
--cc=will@kernel.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).