lkml.org 
[lkml]   [2020]   [Sep]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 1/4] mm: Trial do_wp_page() simplification
    > I botched the last version of the patch, here is something a bit
    > better.

    So I'd like to understand why this problem happens.

    Myt argument to Hugh a few weeks ago was that page pinning should take
    care of all this:

    (a) if the pinner is going to change the page, it will have to get
    the pin with FOLL_WRITE in addition to FOLL_PIN

    (b) that means it will do the COW and mark the page writable and dirty

    (c) the whole _point_ of the FOLL_PIN is that subsequent operations
    shouldn't make it non-writable any more (ie it can't be unmapped, and
    we should synchronize on fork etc)

    So I get the strong feeling that this whole approach to "fix" COW is
    really papering over the real problem.

    To me, the whole point of pinning is to avoid issues like this. If it
    didn't fix this issue, then why did we go to all the effort of
    treating pinned pages differently?

    Your patch may avoid the problem, but I think it implies things are
    horribly horribly broken in pinning land.

    I also note that the _only_ user of page_maybe_dma_pinned() right now
    is a debug check in gup_benchmark. I think your patch may _work_, and
    I think that thanks to page_maybe_dma_pinned() it might even avoid the
    horrible case, but I feel we should just truly fix pinning to be
    meaningful instead.

    IOW, in addition to keeping the page mapped, it should keep the page writable.

    Why isn't it?

    Linus

    \
     
     \ /
      Last update: 2020-09-17 20:12    [W:4.164 / U:0.788 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site