lkml.org 
[lkml]   [2021]   [Aug]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/5] mm: migrate: Move the page count validation to the proper place
    On Fri, Aug 06, 2021 at 11:07:18AM +0800, Baolin Wang wrote:
    > Hi Matthew,
    >
    > > On Thu, Aug 05, 2021 at 11:05:56PM +0800, Baolin Wang wrote:
    > > > We've got the expected count for anonymous page or file page by
    > > > expected_page_refs() at the beginning of migrate_page_move_mapping(),
    > > > thus we should move the page count validation a little forward to
    > > > reduce duplicated code.
    > >
    > > Please add an explanation to the changelog for why it's safe to pull
    > > this out from under the i_pages lock.
    >
    > Sure. In folio_migrate_mapping(), we are sure that the migration page was
    > isolated from lru list and locked, so I think there are no race to get the
    > page count without i_pages lock. Please correct me if I missed something
    > else. Thanks.

    Unless the page has been removed from i_pages, this isn't a correct
    explanation. Even if it has been removed from i_pages, unless an
    RCU grace period has passed, another CPU may still be able to inc the
    refcount on it (temporarily). The same is true for the page tables,
    by the way; if someone is using get_user_pages_fast(), they may still
    be able to see the page.

    \
     
     \ /
      Last update: 2021-08-07 04:04    [W:4.016 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site