[12/11] hugetlb: fix lack of radix tree locking

From: William Lee Irwin III (wli@holomorphy.com)
Date: Tue Nov 12 2002 - 04:16:40 EST


This adds a spinlock to each individual key, initializes it and the
atomic counter, and uses it when manipulating the radix tree.
As pointed out by akpm.

 hugetlbpage.c | 8 ++++++++
 1 files changed, 8 insertions(+)

diff -urpN htlb-2.5.47-11/arch/i386/mm/hugetlbpage.c htlb-2.5.47-12/arch/i386/mm/hugetlbpage.c
--- htlb-2.5.47-11/arch/i386/mm/hugetlbpage.c 2002-11-11 23:27:18.000000000 -0800
+++ htlb-2.5.47-12/arch/i386/mm/hugetlbpage.c 2002-11-12 00:35:02.000000000 -0800
@@ -32,6 +32,7 @@ static spinlock_t htlbpage_lock = SPIN_L
 struct hugetlb_key {
         struct radix_tree_root tree;
         atomic_t count;
+ spinlock_t lock;
         int key;
         int busy;
         uid_t uid;
@@ -392,6 +393,7 @@ static int prefault_key(struct hugetlb_k
         BUG_ON(vma->vm_end & ~HPAGE_MASK);
 
         spin_lock(&mm->page_table_lock);
+ spin_lock(&key->lock);
         for (addr = vma->vm_start; addr < vma->vm_end; addr += HPAGE_SIZE) {
                 unsigned long idx;
                 pte_t *pte = huge_pte_alloc(mm, addr);
@@ -410,6 +412,7 @@ static int prefault_key(struct hugetlb_k
                 if (!page) {
                         page = alloc_hugetlb_page();
                         if (!page) {
+ spin_unlock(&key->lock);
                                 ret = -ENOMEM;
                                 goto out;
                         }
@@ -418,6 +421,7 @@ static int prefault_key(struct hugetlb_k
                 }
                 set_huge_pte(mm, vma, page, pte, vma->vm_flags & VM_WRITE);
         }
+ spin_unlock(&key->lock);
 out:
         spin_unlock(&mm->page_table_lock);
         return ret;
@@ -625,6 +629,10 @@ static int __init hugetlb_init(void)
         }
         htlbpage_max = htlbpagemem = htlbzone_pages = i;
         printk("Total HugeTLB memory allocated, %ld\n", htlbpagemem);
+ for (i = 0; i < MAX_ID; ++i) {
+ atomic_init(&htlbpagek[i].count);
+ spinlock_init(&htlbpagek[i].lock);
+ }
         return 0;
 }
 module_init(hugetlb_init);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Nov 15 2002 - 22:00:25 EST