lkml.org 
[lkml]   [2008]   [Mar]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: [PATCH] mmu notifiers #v8 + xpmem
    Here an example of the futher orthogonal work to do on top of #v8
    during .26-rc to make the whole mmu notifier API sleep capable.

    1) Every single ptep_clear_flush_young_notify and
    ptep_clear_flush_notify must be converted like the below. The below is
    the conversion of a single one. do_wp_page has been converted by
    Christoph already but with invalidate_range (should be changed to
    invalidate_page by releasing the refcount on the page after calling
    invalidate_page). Hope it's clear why I'd rather not depend on these
    changes to be merged in .25 in order to have the mmu notifier included
    in .25.

    2) Then after all this conversion work is finished, it's trivial to
    delete ptep_clear_flush_young_notify and ptep_clear_flush_notify from
    mmu_notifier.h (they will be unused macros once the conversion is
    complete).

    3) After that the VM has to be changed to convert anon_vma lock and
    i_mmap_lock spinlocks to mutex/rwsemaphore.

    4) Then finally the mmu_notifier_unregister must be dropped to make the
    mmu notifier sleep capable with RCU in the mmu_notifier() fast path.

    It's unclear at this point if 3/4 should be switchable and happening
    under a CONFIG_XPMEM or similar or if everyone will benefit from those
    spinlock becoming mutex (the only one that is certain to appreciate
    such a change is preempt-rt, the rest of the userbase I don't know for
    sure and I'd be more confortable with a TPC number comparison before
    doing such a chance by default, but I leave the commentary on such a
    change to linux-mm in a separate thread).

    Signed-off-by: Andrea Arcangeli <andrea@qumranet.com>

    diff --git a/mm/rmap.c b/mm/rmap.c
    --- a/mm/rmap.c
    +++ b/mm/rmap.c
    @@ -274,7 +274,7 @@ static int page_referenced_one(struct pa
    unsigned long address;
    pte_t *pte;
    spinlock_t *ptl;
    - int referenced = 0;
    + int referenced = 0, clear_flush_young = 0;

    address = vma_address(page, vma);
    if (address == -EFAULT)
    @@ -287,8 +287,11 @@ static int page_referenced_one(struct pa
    if (vma->vm_flags & VM_LOCKED) {
    referenced++;
    *mapcount = 1; /* break early from loop */
    - } else if (ptep_clear_flush_young_notify(vma, address, pte))
    - referenced++;
    + } else {
    + clear_flush_young = 1;
    + if (ptep_clear_flush_young(vma, address, pte))
    + referenced++;
    + }

    /* Pretend the page is referenced if the task has the
    swap token and is in the middle of a page fault. */
    @@ -298,6 +301,11 @@ static int page_referenced_one(struct pa

    (*mapcount)--;
    pte_unmap_unlock(pte, ptl);
    +
    + if (clear_flush_young)
    + referenced += mmu_notifier_clear_flush_young(vma->vm_mm,
    + address);
    +
    out:
    return referenced;
    }


    \
     
     \ /
      Last update: 2008-03-02 17:07    [W:0.023 / U:1.232 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site