lkml.org 
[lkml]   [2013]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 7/11] ksm: make KSM page migration possible
    On Sat, 26 Jan 2013, Simon Jeons wrote:
    > On Fri, 2013-01-25 at 18:03 -0800, Hugh Dickins wrote:
    > > + while (!get_page_unless_zero(page)) {
    > > + /*
    > > + * Another check for page->mapping != expected_mapping would
    > > + * work here too. We have chosen the !PageSwapCache test to
    > > + * optimize the common case, when the page is or is about to
    > > + * be freed: PageSwapCache is cleared (under spin_lock_irq)
    > > + * in the freeze_refs section of __remove_mapping(); but Anon
    > > + * page->mapping reset to NULL later, in free_pages_prepare().
    > > + */
    > > + if (!PageSwapCache(page))
    > > + goto stale;
    > > + cpu_relax();
    > > + }
    > > +
    > > + if (ACCESS_ONCE(page->mapping) != expected_mapping) {
    > > put_page(page);
    > > goto stale;
    > > }
    > > +
    > > if (locked) {
    > > lock_page(page);
    > > - if (page->mapping != expected_mapping) {
    > > + if (ACCESS_ONCE(page->mapping) != expected_mapping) {
    > > unlock_page(page);
    > > put_page(page);
    > > goto stale;
    > > }
    > > }
    >
    > Could you explain why need check page->mapping twice after get page?

    Once for the !locked case, which should not return page if mapping changed.
    Once for the locked case, which should not return page if mapping changed.
    We could use "else", but that wouldn't be an improvement.


    \
     
     \ /
      Last update: 2013-01-28 00:41    [W:2.276 / U:0.072 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site