diff for duplicates of <YXrYo9mtueDT0bnu@google.com> diff --git a/a/1.txt b/N1/1.txt index 4982451..76d99ee 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -25,7 +25,7 @@ On Thu, Oct 28, 2021, Maxim Levitsky wrote: > > + * is in the guest. If the vCPU is not in the guest, hardware will > > + * automatically process AVIC interrupts at VMRUN. > > + */ -> > + if (vcpu->mode == IN_GUEST_MODE) { +> > + if (vcpu->mode = IN_GUEST_MODE) { > > int cpu = READ_ONCE(vcpu->cpu); > > > > /* @@ -54,7 +54,7 @@ On Thu, Oct 28, 2021, Maxim Levitsky wrote: IIUC, you're asking if we should do something like: - if (vcpu->mode == IN_GUEST_MODE) { + if (vcpu->mode = IN_GUEST_MODE) { <signal doorbell> } else if (!is_vcpu_loaded(vcpu)) { kvm_vcpu_wake_up(); @@ -76,7 +76,7 @@ is modified to avoid the rcu lock/unlock. TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization would be to make it less expensive. -[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com +[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com > Before this patch, the avic_vcpu_is_running would only be false when the vCPU > is scheduled out (e.g when vcpu_put was done on it) diff --git a/a/content_digest b/N1/content_digest index 132ebbf..99f6f2c 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,9 +2,42 @@ "ref\020211009021236.4122790-36-seanjc@google.com\0" "ref\0b2ba4c4e6a9083f3fa0b9af4504f9f54e72ca24c.camel@redhat.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" - "Subject\0[PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" + "Subject\0Re: [PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" "Date\0Thu, 28 Oct 2021 17:06:43 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Maxim Levitsky <mlevitsk@redhat.com>\0" + "Cc\0Marc Zyngier <maz@kernel.org>" + Huacai Chen <chenhuacai@kernel.org> + Aleksandar Markovic <aleksandar.qemu.devel@gmail.com> + Paul Mackerras <paulus@ozlabs.org> + Anup Patel <anup.patel@wdc.com> + Paul Walmsley <paul.walmsley@sifive.com> + Palmer Dabbelt <palmer@dabbelt.com> + Albert Ou <aou@eecs.berkeley.edu> + Christian Borntraeger <borntraeger@de.ibm.com> + Janosch Frank <frankja@linux.ibm.com> + Paolo Bonzini <pbonzini@redhat.com> + James Morse <james.morse@arm.com> + Alexandru Elisei <alexandru.elisei@arm.com> + Suzuki K Poulose <suzuki.poulose@arm.com> + Atish Patra <atish.patra@wdc.com> + David Hildenbrand <david@redhat.com> + Cornelia Huck <cohuck@redhat.com> + Claudio Imbrenda <imbrenda@linux.ibm.com> + Vitaly Kuznetsov <vkuznets@redhat.com> + Wanpeng Li <wanpengli@tencent.com> + Jim Mattson <jmattson@google.com> + Joerg Roedel <joro@8bytes.org> + linux-arm-kernel@lists.infradead.org + kvmarm@lists.cs.columbia.edu + linux-mips@vger.kernel.org + kvm@vger.kernel.org + kvm-ppc@vger.kernel.org + kvm-riscv@lists.infradead.org + linux-riscv@lists.infradead.org + linux-kernel@vger.kernel.org + David Matlack <dmatlack@google.com> + Oliver Upton <oupton@google.com> + " Jing Zhang <jingzhangos@google.com>\0" "\00:1\0" "b\0" "On Thu, Oct 28, 2021, Maxim Levitsky wrote:\n" @@ -34,7 +67,7 @@ "> > +\t * is in the guest. If the vCPU is not in the guest, hardware will\n" "> > +\t * automatically process AVIC interrupts at VMRUN.\n" "> > +\t */\n" - "> > +\tif (vcpu->mode == IN_GUEST_MODE) {\n" + "> > +\tif (vcpu->mode = IN_GUEST_MODE) {\n" "> > \t\tint cpu = READ_ONCE(vcpu->cpu);\n" "> > \n" "> > \t\t/*\n" @@ -63,7 +96,7 @@ "\n" "IIUC, you're asking if we should do something like:\n" "\n" - "\tif (vcpu->mode == IN_GUEST_MODE) {\n" + "\tif (vcpu->mode = IN_GUEST_MODE) {\n" "\t\t<signal doorbell>\n" "\t} else if (!is_vcpu_loaded(vcpu)) {\n" "\t\tkvm_vcpu_wake_up();\n" @@ -85,7 +118,7 @@ "TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization\n" "would be to make it less expensive.\n" "\n" - "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com\n" + "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com\n" " \n" "> Before this patch, the avic_vcpu_is_running would only be false when the vCPU\n" "> is scheduled out (e.g when vcpu_put was done on it)\n" @@ -94,4 +127,4 @@ "> \tMaxim Levitsky\n" > -90eb2d628550312b0b4dea2c6b5a8869d8773d3546f678fd7ec609333b7459ed +3e5be8943f9ae6b6d2b1fd8bfcbd3b4eaa4ae768425b52f296b4c41525a55567
diff --git a/a/1.txt b/N2/1.txt index 4982451..a54abe7 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -76,11 +76,15 @@ is modified to avoid the rcu lock/unlock. TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization would be to make it less expensive. -[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com +[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com > Before this patch, the avic_vcpu_is_running would only be false when the vCPU > is scheduled out (e.g when vcpu_put was done on it) > > Best regards, > Maxim Levitsky -> +> +_______________________________________________ +kvmarm mailing list +kvmarm@lists.cs.columbia.edu +https://lists.cs.columbia.edu/mailman/listinfo/kvmarm diff --git a/a/content_digest b/N2/content_digest index 132ebbf..6ed161a 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -2,9 +2,37 @@ "ref\020211009021236.4122790-36-seanjc@google.com\0" "ref\0b2ba4c4e6a9083f3fa0b9af4504f9f54e72ca24c.camel@redhat.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" - "Subject\0[PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" + "Subject\0Re: [PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" "Date\0Thu, 28 Oct 2021 17:06:43 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Maxim Levitsky <mlevitsk@redhat.com>\0" + "Cc\0Cornelia Huck <cohuck@redhat.com>" + Wanpeng Li <wanpengli@tencent.com> + kvm@vger.kernel.org + David Hildenbrand <david@redhat.com> + linux-kernel@vger.kernel.org + Paul Mackerras <paulus@ozlabs.org> + Atish Patra <atish.patra@wdc.com> + linux-riscv@lists.infradead.org + Claudio Imbrenda <imbrenda@linux.ibm.com> + kvmarm@lists.cs.columbia.edu + Janosch Frank <frankja@linux.ibm.com> + Marc Zyngier <maz@kernel.org> + Joerg Roedel <joro@8bytes.org> + Huacai Chen <chenhuacai@kernel.org> + Christian Borntraeger <borntraeger@de.ibm.com> + Aleksandar Markovic <aleksandar.qemu.devel@gmail.com> + Albert Ou <aou@eecs.berkeley.edu> + kvm-ppc@vger.kernel.org + Paul Walmsley <paul.walmsley@sifive.com> + David Matlack <dmatlack@google.com> + linux-arm-kernel@lists.infradead.org + Jim Mattson <jmattson@google.com> + Anup Patel <anup.patel@wdc.com> + linux-mips@vger.kernel.org + Palmer Dabbelt <palmer@dabbelt.com> + kvm-riscv@lists.infradead.org + Paolo Bonzini <pbonzini@redhat.com> + " Vitaly Kuznetsov <vkuznets@redhat.com>\0" "\00:1\0" "b\0" "On Thu, Oct 28, 2021, Maxim Levitsky wrote:\n" @@ -85,13 +113,17 @@ "TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization\n" "would be to make it less expensive.\n" "\n" - "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com\n" + "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com\n" " \n" "> Before this patch, the avic_vcpu_is_running would only be false when the vCPU\n" "> is scheduled out (e.g when vcpu_put was done on it)\n" "> \n" "> Best regards,\n" "> \tMaxim Levitsky\n" - > + "> \n" + "_______________________________________________\n" + "kvmarm mailing list\n" + "kvmarm@lists.cs.columbia.edu\n" + https://lists.cs.columbia.edu/mailman/listinfo/kvmarm -90eb2d628550312b0b4dea2c6b5a8869d8773d3546f678fd7ec609333b7459ed +7aec980069b3a791791d00263a805dd8921e0c42d3fc191913735c504a4b37ca
diff --git a/a/1.txt b/N3/1.txt index 4982451..f00f60a 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -76,7 +76,7 @@ is modified to avoid the rcu lock/unlock. TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization would be to make it less expensive. -[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com +[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com > Before this patch, the avic_vcpu_is_running would only be false when the vCPU > is scheduled out (e.g when vcpu_put was done on it) diff --git a/a/content_digest b/N3/content_digest index 132ebbf..e339a48 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -2,9 +2,42 @@ "ref\020211009021236.4122790-36-seanjc@google.com\0" "ref\0b2ba4c4e6a9083f3fa0b9af4504f9f54e72ca24c.camel@redhat.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" - "Subject\0[PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" + "Subject\0Re: [PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" "Date\0Thu, 28 Oct 2021 17:06:43 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Maxim Levitsky <mlevitsk@redhat.com>\0" + "Cc\0Marc Zyngier <maz@kernel.org>" + Huacai Chen <chenhuacai@kernel.org> + Aleksandar Markovic <aleksandar.qemu.devel@gmail.com> + Paul Mackerras <paulus@ozlabs.org> + Anup Patel <anup.patel@wdc.com> + Paul Walmsley <paul.walmsley@sifive.com> + Palmer Dabbelt <palmer@dabbelt.com> + Albert Ou <aou@eecs.berkeley.edu> + Christian Borntraeger <borntraeger@de.ibm.com> + Janosch Frank <frankja@linux.ibm.com> + Paolo Bonzini <pbonzini@redhat.com> + James Morse <james.morse@arm.com> + Alexandru Elisei <alexandru.elisei@arm.com> + Suzuki K Poulose <suzuki.poulose@arm.com> + Atish Patra <atish.patra@wdc.com> + David Hildenbrand <david@redhat.com> + Cornelia Huck <cohuck@redhat.com> + Claudio Imbrenda <imbrenda@linux.ibm.com> + Vitaly Kuznetsov <vkuznets@redhat.com> + Wanpeng Li <wanpengli@tencent.com> + Jim Mattson <jmattson@google.com> + Joerg Roedel <joro@8bytes.org> + linux-arm-kernel@lists.infradead.org + kvmarm@lists.cs.columbia.edu + linux-mips@vger.kernel.org + kvm@vger.kernel.org + kvm-ppc@vger.kernel.org + kvm-riscv@lists.infradead.org + linux-riscv@lists.infradead.org + linux-kernel@vger.kernel.org + David Matlack <dmatlack@google.com> + Oliver Upton <oupton@google.com> + " Jing Zhang <jingzhangos@google.com>\0" "\00:1\0" "b\0" "On Thu, Oct 28, 2021, Maxim Levitsky wrote:\n" @@ -85,7 +118,7 @@ "TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization\n" "would be to make it less expensive.\n" "\n" - "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com\n" + "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com\n" " \n" "> Before this patch, the avic_vcpu_is_running would only be false when the vCPU\n" "> is scheduled out (e.g when vcpu_put was done on it)\n" @@ -94,4 +127,4 @@ "> \tMaxim Levitsky\n" > -90eb2d628550312b0b4dea2c6b5a8869d8773d3546f678fd7ec609333b7459ed +9f0f259a754417ddad435f0be9ac17b01d9c52b35dbbea620a79b6da46e17ca6
diff --git a/a/1.txt b/N4/1.txt index 4982451..87dd624 100644 --- a/a/1.txt +++ b/N4/1.txt @@ -76,11 +76,16 @@ is modified to avoid the rcu lock/unlock. TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization would be to make it less expensive. -[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com +[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com > Before this patch, the avic_vcpu_is_running would only be false when the vCPU > is scheduled out (e.g when vcpu_put was done on it) > > Best regards, > Maxim Levitsky -> +> + +_______________________________________________ +linux-riscv mailing list +linux-riscv@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-riscv diff --git a/a/content_digest b/N4/content_digest index 132ebbf..3c3117c 100644 --- a/a/content_digest +++ b/N4/content_digest @@ -2,9 +2,42 @@ "ref\020211009021236.4122790-36-seanjc@google.com\0" "ref\0b2ba4c4e6a9083f3fa0b9af4504f9f54e72ca24c.camel@redhat.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" - "Subject\0[PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" + "Subject\0Re: [PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" "Date\0Thu, 28 Oct 2021 17:06:43 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Maxim Levitsky <mlevitsk@redhat.com>\0" + "Cc\0Marc Zyngier <maz@kernel.org>" + Huacai Chen <chenhuacai@kernel.org> + Aleksandar Markovic <aleksandar.qemu.devel@gmail.com> + Paul Mackerras <paulus@ozlabs.org> + Anup Patel <anup.patel@wdc.com> + Paul Walmsley <paul.walmsley@sifive.com> + Palmer Dabbelt <palmer@dabbelt.com> + Albert Ou <aou@eecs.berkeley.edu> + Christian Borntraeger <borntraeger@de.ibm.com> + Janosch Frank <frankja@linux.ibm.com> + Paolo Bonzini <pbonzini@redhat.com> + James Morse <james.morse@arm.com> + Alexandru Elisei <alexandru.elisei@arm.com> + Suzuki K Poulose <suzuki.poulose@arm.com> + Atish Patra <atish.patra@wdc.com> + David Hildenbrand <david@redhat.com> + Cornelia Huck <cohuck@redhat.com> + Claudio Imbrenda <imbrenda@linux.ibm.com> + Vitaly Kuznetsov <vkuznets@redhat.com> + Wanpeng Li <wanpengli@tencent.com> + Jim Mattson <jmattson@google.com> + Joerg Roedel <joro@8bytes.org> + linux-arm-kernel@lists.infradead.org + kvmarm@lists.cs.columbia.edu + linux-mips@vger.kernel.org + kvm@vger.kernel.org + kvm-ppc@vger.kernel.org + kvm-riscv@lists.infradead.org + linux-riscv@lists.infradead.org + linux-kernel@vger.kernel.org + David Matlack <dmatlack@google.com> + Oliver Upton <oupton@google.com> + " Jing Zhang <jingzhangos@google.com>\0" "\00:1\0" "b\0" "On Thu, Oct 28, 2021, Maxim Levitsky wrote:\n" @@ -85,13 +118,18 @@ "TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization\n" "would be to make it less expensive.\n" "\n" - "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com\n" + "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com\n" " \n" "> Before this patch, the avic_vcpu_is_running would only be false when the vCPU\n" "> is scheduled out (e.g when vcpu_put was done on it)\n" "> \n" "> Best regards,\n" "> \tMaxim Levitsky\n" - > + "> \n" + "\n" + "_______________________________________________\n" + "linux-riscv mailing list\n" + "linux-riscv@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-riscv -90eb2d628550312b0b4dea2c6b5a8869d8773d3546f678fd7ec609333b7459ed +4955d881fe6ed9c870957b220954a6228aa3f0b37821c269ee4f952c3876009b
diff --git a/a/1.txt b/N5/1.txt index 4982451..5e7a9d4 100644 --- a/a/1.txt +++ b/N5/1.txt @@ -76,11 +76,16 @@ is modified to avoid the rcu lock/unlock. TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization would be to make it less expensive. -[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com +[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com > Before this patch, the avic_vcpu_is_running would only be false when the vCPU > is scheduled out (e.g when vcpu_put was done on it) > > Best regards, > Maxim Levitsky -> +> + +_______________________________________________ +linux-arm-kernel mailing list +linux-arm-kernel@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N5/content_digest index 132ebbf..61d582e 100644 --- a/a/content_digest +++ b/N5/content_digest @@ -2,9 +2,42 @@ "ref\020211009021236.4122790-36-seanjc@google.com\0" "ref\0b2ba4c4e6a9083f3fa0b9af4504f9f54e72ca24c.camel@redhat.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" - "Subject\0[PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" + "Subject\0Re: [PATCH v2 35/43] KVM: SVM: Signal AVIC doorbell iff vCPU is in guest mode\0" "Date\0Thu, 28 Oct 2021 17:06:43 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Maxim Levitsky <mlevitsk@redhat.com>\0" + "Cc\0Marc Zyngier <maz@kernel.org>" + Huacai Chen <chenhuacai@kernel.org> + Aleksandar Markovic <aleksandar.qemu.devel@gmail.com> + Paul Mackerras <paulus@ozlabs.org> + Anup Patel <anup.patel@wdc.com> + Paul Walmsley <paul.walmsley@sifive.com> + Palmer Dabbelt <palmer@dabbelt.com> + Albert Ou <aou@eecs.berkeley.edu> + Christian Borntraeger <borntraeger@de.ibm.com> + Janosch Frank <frankja@linux.ibm.com> + Paolo Bonzini <pbonzini@redhat.com> + James Morse <james.morse@arm.com> + Alexandru Elisei <alexandru.elisei@arm.com> + Suzuki K Poulose <suzuki.poulose@arm.com> + Atish Patra <atish.patra@wdc.com> + David Hildenbrand <david@redhat.com> + Cornelia Huck <cohuck@redhat.com> + Claudio Imbrenda <imbrenda@linux.ibm.com> + Vitaly Kuznetsov <vkuznets@redhat.com> + Wanpeng Li <wanpengli@tencent.com> + Jim Mattson <jmattson@google.com> + Joerg Roedel <joro@8bytes.org> + linux-arm-kernel@lists.infradead.org + kvmarm@lists.cs.columbia.edu + linux-mips@vger.kernel.org + kvm@vger.kernel.org + kvm-ppc@vger.kernel.org + kvm-riscv@lists.infradead.org + linux-riscv@lists.infradead.org + linux-kernel@vger.kernel.org + David Matlack <dmatlack@google.com> + Oliver Upton <oupton@google.com> + " Jing Zhang <jingzhangos@google.com>\0" "\00:1\0" "b\0" "On Thu, Oct 28, 2021, Maxim Levitsky wrote:\n" @@ -85,13 +118,18 @@ "TL;DR: rcuwait_wake_up() is cheap, and if it's too expensive, a better optimization\n" "would be to make it less expensive.\n" "\n" - "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini at redhat.com\n" + "[*] https://lkml.kernel.org/r/20211020110638.797389-1-pbonzini@redhat.com\n" " \n" "> Before this patch, the avic_vcpu_is_running would only be false when the vCPU\n" "> is scheduled out (e.g when vcpu_put was done on it)\n" "> \n" "> Best regards,\n" "> \tMaxim Levitsky\n" - > + "> \n" + "\n" + "_______________________________________________\n" + "linux-arm-kernel mailing list\n" + "linux-arm-kernel@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -90eb2d628550312b0b4dea2c6b5a8869d8773d3546f678fd7ec609333b7459ed +7116929e3599f2bb586f1ae08bbd39a3e092a84a7473bc920e7dd9f4a66531e7
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.