| Message ID | 20220831004454.33921-1-yuanjilin@cdjrlc.com (mailing list archive) |
|---|---|
| State | Rejected, archived |
| Headers | show |
| Series | KVM: fix repeated words in comments | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 10 jobs. |
| snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 10 jobs. |
| snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
| snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 6 jobs. |
| snowpatch_ozlabs/github-powerpc_kernel_qemu | fail | boot (ppc64le_guest_defconfig, pseries+p8+tcg, pseries+p9+tcg, qemu-system-ppc64, ppc64le-rootfs.... failed at step Run qemu-pseries+p9+tcg with fedora-34 build kernel. |
On Wed, Aug 31, 2022 at 08:44:54AM +0800, Jilin Yuan wrote: > Delete the redundant word 'that'. Nack, having two "that"s in a row is intentional. It's "make sure that" (i.e. bring about the situation described in what follows) "that many HPTEs" (i.e. the number of HPTEs referenced previously) "are invalid". > Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com> > --- > arch/powerpc/kvm/book3s_64_mmu_hv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c b/arch/powerpc/kvm/book3s_64_mmu_hv.c > index e9744b41a226..8e4b42b5f37f 100644 > --- a/arch/powerpc/kvm/book3s_64_mmu_hv.c > +++ b/arch/powerpc/kvm/book3s_64_mmu_hv.c > @@ -1601,7 +1601,7 @@ long kvm_vm_ioctl_resize_hpt_commit(struct kvm *kvm, > * is valid, it is written to the HPT as if an H_ENTER with the > * exact flag set was done. When the invalid count is non-zero > * in the header written to the stream, the kernel will make > - * sure that that many HPTEs are invalid, and invalidate them > + * sure that many HPTEs are invalid, and invalidate them > * if not. > */ Paul.
Le 31/08/2022 à 02:44, Jilin Yuan a écrit : > Delete the redundant word 'that'. > > Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com> This change was already proposed and rejected here : https://patchwork.ozlabs.org/project/linuxppc-dev/patch/20220724062920.1551-1-wangjianli@cdjrlc.com/ Please study history before submitting such low-value changes. > --- > arch/powerpc/kvm/book3s_64_mmu_hv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c b/arch/powerpc/kvm/book3s_64_mmu_hv.c > index e9744b41a226..8e4b42b5f37f 100644 > --- a/arch/powerpc/kvm/book3s_64_mmu_hv.c > +++ b/arch/powerpc/kvm/book3s_64_mmu_hv.c > @@ -1601,7 +1601,7 @@ long kvm_vm_ioctl_resize_hpt_commit(struct kvm *kvm, > * is valid, it is written to the HPT as if an H_ENTER with the > * exact flag set was done. When the invalid count is non-zero > * in the header written to the stream, the kernel will make > - * sure that that many HPTEs are invalid, and invalidate them > + * sure that many HPTEs are invalid, and invalidate them > * if not. > */ >
diff --git a/arch/powerpc/kvm/book3s_64_mmu_hv.c b/arch/powerpc/kvm/book3s_64_mmu_hv.c index e9744b41a226..8e4b42b5f37f 100644 --- a/arch/powerpc/kvm/book3s_64_mmu_hv.c +++ b/arch/powerpc/kvm/book3s_64_mmu_hv.c @@ -1601,7 +1601,7 @@ long kvm_vm_ioctl_resize_hpt_commit(struct kvm *kvm, * is valid, it is written to the HPT as if an H_ENTER with the * exact flag set was done. When the invalid count is non-zero * in the header written to the stream, the kernel will make - * sure that that many HPTEs are invalid, and invalidate them + * sure that many HPTEs are invalid, and invalidate them * if not. */
Delete the redundant word 'that'. Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com> --- arch/powerpc/kvm/book3s_64_mmu_hv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)