lkml.org 
[lkml]   [2017]   [Mar]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC 03/11] mm: remove SWAP_DIRTY in ttu
    Hi Hillf,

    On Thu, Mar 02, 2017 at 03:34:45PM +0800, Hillf Danton wrote:
    >
    > On March 02, 2017 2:39 PM Minchan Kim wrote:
    > > @@ -1424,7 +1424,8 @@ static int try_to_unmap_one(struct page *page, struct vm_area_struct *vma,
    > > } else if (!PageSwapBacked(page)) {
    > > /* dirty MADV_FREE page */
    >
    > Nit: enrich the comment please.

    I guess what you wanted is not my patch doing but one merged already
    so I just sent a small clean patch against of patch merged onto mmotm
    to make thig logic clear. You are already Cced in there so you can
    see it. Hope it well. If you want others, please tell me.
    I will do something to make it clear.

    Thanks for the review.

    > > set_pte_at(mm, address, pvmw.pte, pteval);
    > > - ret = SWAP_DIRTY;
    > > + SetPageSwapBacked(page);
    > > + ret = SWAP_FAIL;
    > > page_vma_mapped_walk_done(&pvmw);
    > > break;
    > > }
    >
    > --
    > To unsubscribe, send a message with 'unsubscribe linux-mm' in
    > the body to majordomo@kvack.org. For more info on Linux MM,
    > see: http://www.linux-mm.org/ .
    > Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

    \
     
     \ /
      Last update: 2017-03-03 03:57    [W:5.715 / U:0.048 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site