* [PATCH v2] Remove duplicate setting of the B field in tlbie
@ 2016-09-16 7:25 Balbir Singh
2016-09-27 5:46 ` Paul Mackerras
0 siblings, 1 reply; 2+ messages in thread
From: Balbir Singh @ 2016-09-16 7:25 UTC (permalink / raw)
To: Aneesh Kumar K.V, Paul Mackerras, Michael Ellerman
Cc: kvm-ppc, LINUX FOR POWERPC
Remove duplicate setting of the the "B" field when doing a tlbie(l).
In compute_tlbie_rb(), the "B" field is set again just before
returning the rb value to be used for tlbie(l).
Signed-off-by: Balbir Singh <bsingharora@gmail.com>
---
Changelog - Leave the more readable version around
arch/powerpc/include/asm/kvm_book3s_64.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/arch/powerpc/include/asm/kvm_book3s_64.h b/arch/powerpc/include/asm/kvm_book3s_64.h
index 88d17b4..aa7d4fd 100644
--- a/arch/powerpc/include/asm/kvm_book3s_64.h
+++ b/arch/powerpc/include/asm/kvm_book3s_64.h
@@ -159,7 +159,6 @@ static inline unsigned long compute_tlbie_rb(unsigned long v, unsigned long r,
/* This covers 14..54 bits of va*/
rb = (v & ~0x7fUL) << 16; /* AVA field */
- rb |= (v >> HPTE_V_SSIZE_SHIFT) << 8; /* B field */
/*
* AVA in v had cleared lower 23 bits. We need to derive
* that from pteg index
@@ -211,7 +210,7 @@ static inline unsigned long compute_tlbie_rb(unsigned long v, unsigned long r,
break;
}
}
- rb |= (v >> 54) & 0x300; /* B field */
+ rb |= (v >> HPTE_V_SSIZE_SHIFT) << 8; /* B field */
return rb;
}
--
2.5.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] Remove duplicate setting of the B field in tlbie
2016-09-16 7:25 [PATCH v2] Remove duplicate setting of the B field in tlbie Balbir Singh
@ 2016-09-27 5:46 ` Paul Mackerras
0 siblings, 0 replies; 2+ messages in thread
From: Paul Mackerras @ 2016-09-27 5:46 UTC (permalink / raw)
To: Balbir Singh
Cc: Aneesh Kumar K.V, Michael Ellerman, kvm-ppc, LINUX FOR POWERPC
On Fri, Sep 16, 2016 at 05:25:50PM +1000, Balbir Singh wrote:
>
> Remove duplicate setting of the the "B" field when doing a tlbie(l).
> In compute_tlbie_rb(), the "B" field is set again just before
> returning the rb value to be used for tlbie(l).
>
> Signed-off-by: Balbir Singh <bsingharora@gmail.com>
Thanks, applied to kvm-ppc-next.
Paul.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-09-27 5:47 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-16 7:25 [PATCH v2] Remove duplicate setting of the B field in tlbie Balbir Singh
2016-09-27 5:46 ` Paul Mackerras
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).