lkml.org 
[lkml]   [2002]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH] page walker bugfix (was: 2.5.49-mm2)
    Hi Andrew,
    hi list readers,

    On Wed, Nov 27, 2002 at 01:11:38AM -0800, Andrew Morton wrote:
    > .. Some code from Ingo Oeser to start using the expanded and cleaned up
    > user pagetable walker code. This affects the st and sg drivers; I'm
    > not sure of the testing status of this?

    The testing status is: None, but it compiles.

    The sg-driver maintainer has already said he does some testing
    and the author of the previous code in st.c was positive about
    using these features. That's why I've choosen these as my "victims".

    I also found a locking bug in walk_user_pages() in case of OOM or
    SIGBUS. Fixed by the attached patch.

    The attached patch also exports some functions to enable modular
    builds for testing.

    And it also deletes the redundant check of the pfn_valid(), which
    is done in follow_page() already.

    Regards

    Ingo Oeser
    --
    Science is what we can tell a computer. Art is everything else. --- D.E.Knuth
    diff -Naur linux-2.5.49-mm2/mm/page_walk.c linux-2.5.49-mm2-ioe/mm/page_walk.c
    --- linux-2.5.49-mm2/mm/page_walk.c Wed Nov 27 14:48:33 2002
    +++ linux-2.5.49-mm2-ioe/mm/page_walk.c Wed Nov 27 17:05:35 2002
    @@ -116,6 +116,7 @@
    struct vm_area_struct *vma = pw->vma;
    int write = pw->vm_flags & (VM_MAYWRITE|VM_WRITE);

    + /* follow_page also does all the checking of ptes for us. */
    while (!(page= follow_page(vma->vm_mm, pw->virt, write))) {
    int fault;

    @@ -138,16 +139,7 @@
    }
    spin_lock(&vma->vm_mm->page_table_lock);
    }
    - /*
    - * Given a physical address, is there a useful struct page pointing to
    - * it? This may become more complex in the future if we start dealing
    - * with IO-aperture pages for direct-IO.
    - */
    - if (pfn_valid(page_to_pfn(page)))
    - return page;
    -
    - spin_unlock(&vma->vm_mm->page_table_lock);
    - return ERR_PTR(-EFAULT);
    + return page;
    }

    /* Setup page walking for the simple and common case.
    @@ -291,7 +283,7 @@
    pw->page = single_page_walk(pw);
    if (IS_ERR(pw->page)) {
    ret = PTR_ERR(pw->page);
    - break;
    + goto out_unlocked;
    }
    ret = pw->walker(pw->data);

    @@ -304,7 +296,8 @@
    } while (pw->virt < vma->vm_end);
    spin_unlock(&pw->mm->page_table_lock);
    } while (ret == 0);
    -
    +
    +out_unlocked:
    /* cleanup after errors */
    if (ret < 0 && pw->cleaner)
    pw->cleaner(pw->data);
    @@ -393,3 +386,12 @@
    EXPORT_SYMBOL(get_user_pages);
    EXPORT_SYMBOL(get_one_user_page);
    EXPORT_SYMBOL_GPL(walk_user_pages);
    +EXPORT_SYMBOL_GPL(setup_simple_page_walk);
    +EXPORT_SYMBOL_GPL(setup_sgl_page_walk);
    +EXPORT_SYMBOL_GPL(fixup_sgl_walk);
    +EXPORT_SYMBOL_GPL(gup_add_pages);
    +EXPORT_SYMBOL_GPL(gup_cleanup_pages);
    +EXPORT_SYMBOL_GPL(gup_cleanup_pages_kfree);
    +EXPORT_SYMBOL_GPL(gup_add_sgls);
    +EXPORT_SYMBOL_GPL(gup_cleanup_sgls);
    +EXPORT_SYMBOL_GPL(gup_cleanup_sgls_kfree);
    \
     
     \ /
      Last update: 2005-03-22 13:31    [W:0.028 / U:0.576 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site