diff for duplicates of <YXl5anv0Lyjx1cws@google.com> diff --git a/a/1.txt b/N1/1.txt index 534dca0..822f624 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,7 +1,7 @@ On Wed, Oct 27, 2021, Paolo Bonzini wrote: > On 27/10/21 17:06, Sean Christopherson wrote: > > > Does this still need to check the "running" flag? That should be a strict -> > > superset of vcpu->mode == IN_GUEST_MODE. +> > > superset of vcpu->mode = IN_GUEST_MODE. > > > > No. Signalling the doorbell when "running" is set but the vCPU is not in the > > guest is just an expensive nop. So even if KVM were to rework its handling of @@ -25,5 +25,5 @@ I completely misunderstanding what your referring to? + * 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); diff --git a/a/content_digest b/N1/content_digest index 883fe77..c0e66f5 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -4,15 +4,47 @@ "ref\0YXlrEWmBohaDXmqL@google.com\0" "ref\0185502d7-861e-fa5c-b225-419710fe77ed@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\0Wed, 27 Oct 2021 16:08:10 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Paolo Bonzini <pbonzini@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> + 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 Wed, Oct 27, 2021, Paolo Bonzini wrote:\n" "> On 27/10/21 17:06, Sean Christopherson wrote:\n" "> > > Does this still need to check the \"running\" flag? That should be a strict\n" - "> > > superset of vcpu->mode == IN_GUEST_MODE.\n" + "> > > superset of vcpu->mode = IN_GUEST_MODE.\n" "> > \n" "> > No. Signalling the doorbell when \"running\" is set but the vCPU is not in the\n" "> > guest is just an expensive nop. So even if KVM were to rework its handling of\n" @@ -36,7 +68,7 @@ "+ * is in the guest. If the vCPU is not in the guest, hardware will\n" "+ * automatically process AVIC interrupts at VMRUN.\n" "+ */\n" - "+ if (vcpu->mode == IN_GUEST_MODE) {\n" + "+ if (vcpu->mode = IN_GUEST_MODE) {\n" int cpu = READ_ONCE(vcpu->cpu); -183f77a2dce0b6df4ba2c3d75663a9e30c27bb101e1d0c3a160987445d593a5a +b4b2beff54d382db3f151fc24d794764cbadfa1c3a6a600167159e3f45229e06
diff --git a/a/1.txt b/N2/1.txt index 534dca0..bd45596 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -27,3 +27,7 @@ I completely misunderstanding what your referring to? + */ + if (vcpu->mode == IN_GUEST_MODE) { int cpu = READ_ONCE(vcpu->cpu); +_______________________________________________ +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 883fe77..420e1de 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -4,9 +4,36 @@ "ref\0YXlrEWmBohaDXmqL@google.com\0" "ref\0185502d7-861e-fa5c-b225-419710fe77ed@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\0Wed, 27 Oct 2021 16:08:10 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Paolo Bonzini <pbonzini@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 + " Vitaly Kuznetsov <vkuznets@redhat.com>\0" "\00:1\0" "b\0" "On Wed, Oct 27, 2021, Paolo Bonzini wrote:\n" @@ -37,6 +64,10 @@ "+ * automatically process AVIC interrupts at VMRUN.\n" "+ */\n" "+ if (vcpu->mode == IN_GUEST_MODE) {\n" - int cpu = READ_ONCE(vcpu->cpu); + " int cpu = READ_ONCE(vcpu->cpu);\n" + "_______________________________________________\n" + "kvmarm mailing list\n" + "kvmarm@lists.cs.columbia.edu\n" + https://lists.cs.columbia.edu/mailman/listinfo/kvmarm -183f77a2dce0b6df4ba2c3d75663a9e30c27bb101e1d0c3a160987445d593a5a +5c6f61169ecbc715a580b451e452d4939e4d8928d463b3dc2ba69356c5a753a7
diff --git a/a/content_digest b/N3/content_digest index 883fe77..f148ced 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -4,9 +4,41 @@ "ref\0YXlrEWmBohaDXmqL@google.com\0" "ref\0185502d7-861e-fa5c-b225-419710fe77ed@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\0Wed, 27 Oct 2021 16:08:10 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Paolo Bonzini <pbonzini@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> + 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 Wed, Oct 27, 2021, Paolo Bonzini wrote:\n" @@ -39,4 +71,4 @@ "+ if (vcpu->mode == IN_GUEST_MODE) {\n" int cpu = READ_ONCE(vcpu->cpu); -183f77a2dce0b6df4ba2c3d75663a9e30c27bb101e1d0c3a160987445d593a5a +6f848b7bade1ed1d0941ffd5066a84521dfb5e2845fb3daf873fa419ba0d1aea
diff --git a/a/1.txt b/N4/1.txt index 534dca0..826e373 100644 --- a/a/1.txt +++ b/N4/1.txt @@ -27,3 +27,8 @@ I completely misunderstanding what your referring to? + */ + if (vcpu->mode == IN_GUEST_MODE) { int cpu = READ_ONCE(vcpu->cpu); + +_______________________________________________ +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 883fe77..a3f4656 100644 --- a/a/content_digest +++ b/N4/content_digest @@ -4,9 +4,41 @@ "ref\0YXlrEWmBohaDXmqL@google.com\0" "ref\0185502d7-861e-fa5c-b225-419710fe77ed@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\0Wed, 27 Oct 2021 16:08:10 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Paolo Bonzini <pbonzini@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> + 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 Wed, Oct 27, 2021, Paolo Bonzini wrote:\n" @@ -37,6 +69,11 @@ "+ * automatically process AVIC interrupts at VMRUN.\n" "+ */\n" "+ if (vcpu->mode == IN_GUEST_MODE) {\n" - int cpu = READ_ONCE(vcpu->cpu); + " int cpu = READ_ONCE(vcpu->cpu);\n" + "\n" + "_______________________________________________\n" + "linux-riscv mailing list\n" + "linux-riscv@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-riscv -183f77a2dce0b6df4ba2c3d75663a9e30c27bb101e1d0c3a160987445d593a5a +6dba0ec4c91e6f1298020ad087e010d7a52bd9a61d383b32d26fd5106c5e6660
diff --git a/a/1.txt b/N5/1.txt index 534dca0..b62c5fc 100644 --- a/a/1.txt +++ b/N5/1.txt @@ -27,3 +27,8 @@ I completely misunderstanding what your referring to? + */ + if (vcpu->mode == IN_GUEST_MODE) { int cpu = READ_ONCE(vcpu->cpu); + +_______________________________________________ +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 883fe77..56569f2 100644 --- a/a/content_digest +++ b/N5/content_digest @@ -4,9 +4,41 @@ "ref\0YXlrEWmBohaDXmqL@google.com\0" "ref\0185502d7-861e-fa5c-b225-419710fe77ed@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\0Wed, 27 Oct 2021 16:08:10 +0000\0" - "To\0kvm-riscv@lists.infradead.org\0" + "To\0Paolo Bonzini <pbonzini@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> + 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 Wed, Oct 27, 2021, Paolo Bonzini wrote:\n" @@ -37,6 +69,11 @@ "+ * automatically process AVIC interrupts at VMRUN.\n" "+ */\n" "+ if (vcpu->mode == IN_GUEST_MODE) {\n" - int cpu = READ_ONCE(vcpu->cpu); + " int cpu = READ_ONCE(vcpu->cpu);\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 -183f77a2dce0b6df4ba2c3d75663a9e30c27bb101e1d0c3a160987445d593a5a +2574922ea41db0ec01bdaf4dfe70b603621e7cc1ea1b497327090e0e4a360adb
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.