summaryrefslogtreecommitdiffstats
path: root/arch
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2005-10-21 13:41:19 +1000
committerLinus Torvalds <torvalds@g5.osdl.org>2005-10-21 12:24:41 -0700
commit3078fcc1d18c7235b034dc889642c5300959fa20 (patch)
tree7d4a495e007a8edc59901fc40aaeeec448fd5883 /arch
parent2c86c83bf4e7fa1ef0d625e297607dc5d0d94dd4 (diff)
downloadop-kernel-dev-3078fcc1d18c7235b034dc889642c5300959fa20.zip
op-kernel-dev-3078fcc1d18c7235b034dc889642c5300959fa20.tar.gz
[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>
Diffstat (limited to 'arch')
-rw-r--r--arch/ppc64/kernel/iSeries_htab.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ppc64/kernel/iSeries_htab.c b/arch/ppc64/kernel/iSeries_htab.c
index 2192055..073b766 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,
}
if (slot < 0) { /* MSB set means secondary group */
- vflags |= HPTE_V_VALID;
+ vflags |= HPTE_V_SECONDARY;
secondary = 1;
slot &= 0x7fffffffffffffff;
}
OpenPOWER on IntegriCloud