__get_user_pages_fast为什么不用锁

littlenewer

__get_user_pages_fast会遍历当前进程的页表，如果两个线程共享内存空间，不就会引起冲突了吗。为何此处不用加锁呢？另外，如果这种访问时安全的，那为什么还要在遍历页表前关中断呢？这样做的目的是什么？一直没想通这两个问题。

int __get_user_pages_fast(unsigned long start, int nr_pages, int write,
                          struct page **pages)
{
        struct mm_struct *mm = current->mm;
        unsigned long addr, len, end;
        unsigned long next;
        unsigned long flags;
        pgd_t *pgdp;
        int nr = 0;

        start &= PAGE_MASK;
        addr = start;
        len = (unsigned long) nr_pages << PAGE_SHIFT;
        end = start + len;
        if (unlikely(!access_ok(write ? VERIFY_WRITE : VERIFY_READ,
                                        (void __user *)start, len)))
                return 0;

        /*
         * XXX: batch / limit 'nr', to avoid large irq off latency
         * needs some instrumenting to determine the common sizes used by
         * important workloads (eg. DB2), and whether limiting the batch size
         * will decrease performance.
         *
         * It seems like we're in the clear for the moment. Direct-IO is
         * the main guy that batches up lots of get_user_pages, and even
         * they are limited to 64-at-a-time which is not so many.
         */
        /*
         * This doesn't prevent pagetable teardown, but does prevent
         * the pagetables and pages from being freed on x86.
         *
         * So long as we atomically load page table pointers versus teardown
         * (which we do on x86, with the above PAE exception), we can follow the
         * address down to the the page and take a ref on it.
         */
        local_irq_save(flags);
        pgdp = pgd_offset(mm, addr);
        do {
                pgd_t pgd = *pgdp;

                next = pgd_addr_end(addr, end);
                if (pgd_none(pgd))
                        break;
                if (!gup_pud_range(pgd, addr, next, write, pages, &nr))
                        break;
        } while (pgdp++, addr = next, addr != end);
        local_irq_restore(flags);

        return nr;
}

littlenewer

兄台的分析很有道理。明白了，谢谢