diff for duplicates of <YA8PXCEVukW0UzC5@google.com> diff --git a/a/1.txt b/N1/1.txt index 321a85e..406380e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -215,7 +215,7 @@ Combine these into a single statement? I think the result is easier to read? > * count is also read inside the mmu_lock critical section. > */ > kvm->mmu_notifier_count++; -> + if (likely(kvm->mmu_notifier_count = 1)) { +> + if (likely(kvm->mmu_notifier_count == 1)) { > + kvm->mmu_notifier_range_start = range->start; > + kvm->mmu_notifier_range_end = range->end; > + } else { @@ -266,7 +266,7 @@ Newline here. > + if (hva) > + *hva = addr; > -> if (addr = KVM_HVA_ERR_RO_BAD) { +> if (addr == KVM_HVA_ERR_RO_BAD) { > if (writable) > @@ -2041,19 +2055,19 @@ kvm_pfn_t gfn_to_pfn_prot(struct kvm *kvm, gfn_t gfn, bool write_fault, > bool *writable) @@ -293,4 +293,8 @@ Newline here. > > -- > 2.30.0.280.ga3ce27912f-goog -> +> +_______________________________________________ +kvmarm mailing list +kvmarm@lists.cs.columbia.edu +https://lists.cs.columbia.edu/mailman/listinfo/kvmarm diff --git a/a/content_digest b/N1/content_digest index 1e5239a..426e184 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,31 +1,28 @@ "ref\020210125064234.2078146-1-stevensd@google.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" "Subject\0Re: [PATCH] KVM: x86/mmu: consider the hva in mmu_notifer retry\0" - "Date\0Mon, 25 Jan 2021 18:35:08 +0000\0" + "Date\0Mon, 25 Jan 2021 10:35:08 -0800\0" "To\0David Stevens <stevensd@chromium.org>\0" - "Cc\0Paolo Bonzini <pbonzini@redhat.com>" - Vitaly Kuznetsov <vkuznets@redhat.com> - Wanpeng Li <wanpengli@tencent.com> - Jim Mattson <jmattson@google.com> - Joerg Roedel <joro@8bytes.org> + "Cc\0Wanpeng Li <wanpengli@tencent.com>" kvm@vger.kernel.org - linux-kernel@vger.kernel.org - Marc Zyngier <maz@kernel.org> - James Morse <james.morse@arm.com> - Julien Thierry <julien.thierry.kdev@gmail.com> - Suzuki K Poulose <suzuki.poulose@arm.com> - linux-arm-kernel@lists.infradead.org + David Hildenbrand <david@redhat.com> + linux-mips@vger.kernel.org + Paul Mackerras <paulus@ozlabs.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> - linux-mips@vger.kernel.org - Paul Mackerras <paulus@ozlabs.org> kvm-ppc@vger.kernel.org - Christian Borntraeger <borntraeger@de.ibm.com> - Janosch Frank <frankja@linux.ibm.com> - David Hildenbrand <david@redhat.com> + linux-arm-kernel@lists.infradead.org + Jim Mattson <jmattson@google.com> Cornelia Huck <cohuck@redhat.com> - " Claudio Imbrenda <imbrenda@linux.ibm.com>\0" + linux-kernel@vger.kernel.org + Paolo Bonzini <pbonzini@redhat.com> + " Vitaly Kuznetsov <vkuznets@redhat.com>\0" "\00:1\0" "b\0" "+Cc the other architectures, I'm guessing this would be a helpful optimization\n" @@ -245,7 +242,7 @@ "> \t * count is also read inside the mmu_lock critical section.\n" "> \t */\n" "> \tkvm->mmu_notifier_count++;\n" - "> +\tif (likely(kvm->mmu_notifier_count = 1)) {\n" + "> +\tif (likely(kvm->mmu_notifier_count == 1)) {\n" "> +\t\tkvm->mmu_notifier_range_start = range->start;\n" "> +\t\tkvm->mmu_notifier_range_end = range->end;\n" "> +\t} else {\n" @@ -296,7 +293,7 @@ "> +\tif (hva)\n" "> +\t\t*hva = addr;\n" "> \n" - "> \tif (addr = KVM_HVA_ERR_RO_BAD) {\n" + "> \tif (addr == KVM_HVA_ERR_RO_BAD) {\n" "> \t\tif (writable)\n" "> @@ -2041,19 +2055,19 @@ kvm_pfn_t gfn_to_pfn_prot(struct kvm *kvm, gfn_t gfn, bool write_fault,\n" "> \t\t bool *writable)\n" @@ -323,6 +320,10 @@ "> \n" "> -- \n" "> 2.30.0.280.ga3ce27912f-goog\n" - > + "> \n" + "_______________________________________________\n" + "kvmarm mailing list\n" + "kvmarm@lists.cs.columbia.edu\n" + https://lists.cs.columbia.edu/mailman/listinfo/kvmarm -ce4843de00cac95f77de09a324116470f87d5ae6c8bffc5d9afb568924066741 +72ed5b821c6906068f0293e321a74577270624010192ca0261797ad64bd25133
diff --git a/a/1.txt b/N2/1.txt index 321a85e..f331e72 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -215,7 +215,7 @@ Combine these into a single statement? I think the result is easier to read? > * count is also read inside the mmu_lock critical section. > */ > kvm->mmu_notifier_count++; -> + if (likely(kvm->mmu_notifier_count = 1)) { +> + if (likely(kvm->mmu_notifier_count == 1)) { > + kvm->mmu_notifier_range_start = range->start; > + kvm->mmu_notifier_range_end = range->end; > + } else { @@ -266,7 +266,7 @@ Newline here. > + if (hva) > + *hva = addr; > -> if (addr = KVM_HVA_ERR_RO_BAD) { +> if (addr == KVM_HVA_ERR_RO_BAD) { > if (writable) > @@ -2041,19 +2055,19 @@ kvm_pfn_t gfn_to_pfn_prot(struct kvm *kvm, gfn_t gfn, bool write_fault, > bool *writable) diff --git a/a/content_digest b/N2/content_digest index 1e5239a..db3902c 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,7 +1,7 @@ "ref\020210125064234.2078146-1-stevensd@google.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" "Subject\0Re: [PATCH] KVM: x86/mmu: consider the hva in mmu_notifer retry\0" - "Date\0Mon, 25 Jan 2021 18:35:08 +0000\0" + "Date\0Mon, 25 Jan 2021 10:35:08 -0800\0" "To\0David Stevens <stevensd@chromium.org>\0" "Cc\0Paolo Bonzini <pbonzini@redhat.com>" Vitaly Kuznetsov <vkuznets@redhat.com> @@ -245,7 +245,7 @@ "> \t * count is also read inside the mmu_lock critical section.\n" "> \t */\n" "> \tkvm->mmu_notifier_count++;\n" - "> +\tif (likely(kvm->mmu_notifier_count = 1)) {\n" + "> +\tif (likely(kvm->mmu_notifier_count == 1)) {\n" "> +\t\tkvm->mmu_notifier_range_start = range->start;\n" "> +\t\tkvm->mmu_notifier_range_end = range->end;\n" "> +\t} else {\n" @@ -296,7 +296,7 @@ "> +\tif (hva)\n" "> +\t\t*hva = addr;\n" "> \n" - "> \tif (addr = KVM_HVA_ERR_RO_BAD) {\n" + "> \tif (addr == KVM_HVA_ERR_RO_BAD) {\n" "> \t\tif (writable)\n" "> @@ -2041,19 +2055,19 @@ kvm_pfn_t gfn_to_pfn_prot(struct kvm *kvm, gfn_t gfn, bool write_fault,\n" "> \t\t bool *writable)\n" @@ -325,4 +325,4 @@ "> 2.30.0.280.ga3ce27912f-goog\n" > -ce4843de00cac95f77de09a324116470f87d5ae6c8bffc5d9afb568924066741 +9a0339382cdb5b0f42edd18b5558263cd92ae6f5ed11b09b47a15d0a661184c3
diff --git a/a/1.txt b/N3/1.txt index 321a85e..6d2c13b 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -215,7 +215,7 @@ Combine these into a single statement? I think the result is easier to read? > * count is also read inside the mmu_lock critical section. > */ > kvm->mmu_notifier_count++; -> + if (likely(kvm->mmu_notifier_count = 1)) { +> + if (likely(kvm->mmu_notifier_count == 1)) { > + kvm->mmu_notifier_range_start = range->start; > + kvm->mmu_notifier_range_end = range->end; > + } else { @@ -266,7 +266,7 @@ Newline here. > + if (hva) > + *hva = addr; > -> if (addr = KVM_HVA_ERR_RO_BAD) { +> if (addr == KVM_HVA_ERR_RO_BAD) { > if (writable) > @@ -2041,19 +2055,19 @@ kvm_pfn_t gfn_to_pfn_prot(struct kvm *kvm, gfn_t gfn, bool write_fault, > bool *writable) @@ -293,4 +293,9 @@ Newline here. > > -- > 2.30.0.280.ga3ce27912f-goog -> +> + +_______________________________________________ +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/N3/content_digest index 1e5239a..8292ad5 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -1,31 +1,31 @@ "ref\020210125064234.2078146-1-stevensd@google.com\0" "From\0Sean Christopherson <seanjc@google.com>\0" "Subject\0Re: [PATCH] KVM: x86/mmu: consider the hva in mmu_notifer retry\0" - "Date\0Mon, 25 Jan 2021 18:35:08 +0000\0" + "Date\0Mon, 25 Jan 2021 10:35:08 -0800\0" "To\0David Stevens <stevensd@chromium.org>\0" - "Cc\0Paolo Bonzini <pbonzini@redhat.com>" - Vitaly Kuznetsov <vkuznets@redhat.com> - Wanpeng Li <wanpengli@tencent.com> - Jim Mattson <jmattson@google.com> - Joerg Roedel <joro@8bytes.org> + "Cc\0Wanpeng Li <wanpengli@tencent.com>" kvm@vger.kernel.org - linux-kernel@vger.kernel.org - Marc Zyngier <maz@kernel.org> - James Morse <james.morse@arm.com> - Julien Thierry <julien.thierry.kdev@gmail.com> - Suzuki K Poulose <suzuki.poulose@arm.com> - linux-arm-kernel@lists.infradead.org + David Hildenbrand <david@redhat.com> + linux-mips@vger.kernel.org + Paul Mackerras <paulus@ozlabs.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> - linux-mips@vger.kernel.org - Paul Mackerras <paulus@ozlabs.org> + Julien Thierry <julien.thierry.kdev@gmail.com> + Suzuki K Poulose <suzuki.poulose@arm.com> kvm-ppc@vger.kernel.org - Christian Borntraeger <borntraeger@de.ibm.com> - Janosch Frank <frankja@linux.ibm.com> - David Hildenbrand <david@redhat.com> + linux-arm-kernel@lists.infradead.org + Jim Mattson <jmattson@google.com> Cornelia Huck <cohuck@redhat.com> - " Claudio Imbrenda <imbrenda@linux.ibm.com>\0" + linux-kernel@vger.kernel.org + James Morse <james.morse@arm.com> + Paolo Bonzini <pbonzini@redhat.com> + " Vitaly Kuznetsov <vkuznets@redhat.com>\0" "\00:1\0" "b\0" "+Cc the other architectures, I'm guessing this would be a helpful optimization\n" @@ -245,7 +245,7 @@ "> \t * count is also read inside the mmu_lock critical section.\n" "> \t */\n" "> \tkvm->mmu_notifier_count++;\n" - "> +\tif (likely(kvm->mmu_notifier_count = 1)) {\n" + "> +\tif (likely(kvm->mmu_notifier_count == 1)) {\n" "> +\t\tkvm->mmu_notifier_range_start = range->start;\n" "> +\t\tkvm->mmu_notifier_range_end = range->end;\n" "> +\t} else {\n" @@ -296,7 +296,7 @@ "> +\tif (hva)\n" "> +\t\t*hva = addr;\n" "> \n" - "> \tif (addr = KVM_HVA_ERR_RO_BAD) {\n" + "> \tif (addr == KVM_HVA_ERR_RO_BAD) {\n" "> \t\tif (writable)\n" "> @@ -2041,19 +2055,19 @@ kvm_pfn_t gfn_to_pfn_prot(struct kvm *kvm, gfn_t gfn, bool write_fault,\n" "> \t\t bool *writable)\n" @@ -323,6 +323,11 @@ "> \n" "> -- \n" "> 2.30.0.280.ga3ce27912f-goog\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 -ce4843de00cac95f77de09a324116470f87d5ae6c8bffc5d9afb568924066741 +bf24b75a77a1477768a567e620e2af9f584289da3f87d0b9136394475df3777f
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.