Messages in this thread Patch in this message | | | Date | Tue, 12 Nov 2002 01:31:50 -0800 | From | William Lee Irwin III <> | Subject | Re: [12/11] hugetlb: fix lack of radix tree locking |
| |
On Tue, Nov 12, 2002 at 01:16:40AM -0800, William Lee Irwin III wrote: > + for (i = 0; i < MAX_ID; ++i) { > + atomic_init(&htlbpagek[i].count); > + spinlock_init(&htlbpagek[i].lock); > + }
erm, wtf?
This adds a spinlock to each individual key, initializes it and the atomic counter, and uses it when manipulating the radix tree.
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:52:31.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_set(&htlbpagek[i].count, 0); + spin_lock_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/
| |