diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2005-10-20 23:41:19 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-10-21 15:24:41 -0400 |
commit | 3078fcc1d18c7235b034dc889642c5300959fa20 (patch) | |
tree | 7d4a495e007a8edc59901fc40aaeeec448fd5883 | |
parent | 2c86c83bf4e7fa1ef0d625e297607dc5d0d94dd4 (diff) |
[PATCH] ppc64: Fix typo bug in iSeries hash code
This fixes a stupid typo bug in the iSeries hash table code.
When we place a hash PTE in the secondary bucket, instead of setting the
SECONDARY flag bit, as we should, we (redundantly) set the VALID flag.
This was introduced with the patch abolishing bitfields from the hash
table code. Mea culpa, oops. It hasn't been noticed until now because
in practice we don't hit the secondary bucket terribly often.
Signed-off-by: David Gibson <dwg@au1.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | arch/ppc64/kernel/iSeries_htab.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ppc64/kernel/iSeries_htab.c b/arch/ppc64/kernel/iSeries_htab.c index 2192055a90a..073b7666174 100644 --- a/arch/ppc64/kernel/iSeries_htab.c +++ b/arch/ppc64/kernel/iSeries_htab.c | |||
@@ -66,7 +66,7 @@ static long iSeries_hpte_insert(unsigned long hpte_group, unsigned long va, | |||
66 | } | 66 | } |
67 | 67 | ||
68 | if (slot < 0) { /* MSB set means secondary group */ | 68 | if (slot < 0) { /* MSB set means secondary group */ |
69 | vflags |= HPTE_V_VALID; | 69 | vflags |= HPTE_V_SECONDARY; |
70 | secondary = 1; | 70 | secondary = 1; |
71 | slot &= 0x7fffffffffffffff; | 71 | slot &= 0x7fffffffffffffff; |
72 | } | 72 | } |