lkml.org 
[lkml]   [2005]   [Mar]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: Page fault scalability patch V18: Drop first acquisition of ptl
    On Wed, 2 Mar 2005, Andrew Morton wrote:

    > > - if (!PageReserved(old_page))
    > > - page_cache_get(old_page);
    >
    > hm, this seems to be an unrelated change. You're saying that this page is
    > protected from munmap() by munmap()'s down_write(mmap_sem), yes? What
    > stops memory reclaim from freeing old_page?

    This is a related change discussed during V16 with Nick.

    The page is protected from munmap because of the down_read(mmap_sem) in
    the arch specific code before calling handle_mm_fault.

    > > - mark_page_accessed(page);
    > > + SetPageReferenced(page);
    >
    > Another unrelated change. IIRC, this is indeed equivalent, but I forget
    > why. Care to remind me?

    Seems that mark_page_accessed was discouraged in favor SetPageReferenced.
    We agreed that we wanted this change earlier (I believe that was in
    November?).

    > Overall, do we know which architectures are capable of using this feature?
    > Would ppc64 (and sparc64?) still have a problem with page_table_lock no
    > longer protecting their internals?

    That is up to the arch maintainers. Add something to arch/xx/Kconfig to
    allow atomic operations for an arch. Out of the box it only works for
    x86_64, ia64 and ia32.

    > I'd really like to see other architecture maintainers stand up and say
    > "yes, we need this".

    You definitely need this for machines with high SMP counts.

    > Did you consider doing the locking at the pte page level? That could be
    > neater than all those games with atomic pte operattions.

    Earlier releases back in September 2004 had some pte locking code (and
    AFAIK Nick also played around with pte locking) but that
    was less efficient than atomic operations.
    -
    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/

    \
     
     \ /
      Last update: 2005-03-22 14:10    [W:3.875 / U:0.172 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site