lkml.org 
[lkml]   [2021]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: Folios for 5.15 request - Was: re: Folio discussion recap -
    On Wed, Sep 22, 2021 at 04:56:16PM +0000, Chris Mason wrote:
    >
    > > On Sep 22, 2021, at 12:26 PM, Matthew Wilcox <willy@infradead.org> wrote:
    > >
    > > On Wed, Sep 22, 2021 at 11:46:04AM -0400, Kent Overstreet wrote:
    > >> On Wed, Sep 22, 2021 at 11:08:58AM -0400, Johannes Weiner wrote:
    > >>> On Tue, Sep 21, 2021 at 05:22:54PM -0400, Kent Overstreet wrote:
    > >>>> - it's become apparent that there haven't been any real objections to the code
    > >>>> that was queued up for 5.15. There _are_ very real discussions and points of
    > >>>> contention still to be decided and resolved for the work beyond file backed
    > >>>> pages, but those discussions were what derailed the more modest, and more
    > >>>> badly needed, work that affects everyone in filesystem land
    > >>>
    > >>> Unfortunately, I think this is a result of me wanting to discuss a way
    > >>> forward rather than a way back.
    > >>>
    > >>> To clarify: I do very much object to the code as currently queued up,
    > >>> and not just to a vague future direction.
    > >>>
    > >>> The patches add and convert a lot of complicated code to provision for
    > >>> a future we do not agree on. The indirections it adds, and the hybrid
    > >>> state it leaves the tree in, make it directly more difficult to work
    > >>> with and understand the MM code base. Stuff that isn't needed for
    > >>> exposing folios to the filesystems.
    > >>>
    > >>> As Willy has repeatedly expressed a take-it-or-leave-it attitude in
    > >>> response to my feedback, I'm not excited about merging this now and
    > >>> potentially leaving quite a bit of cleanup work to others if the
    > >>> downstream discussion don't go to his liking.
    > >
    > > We're at a take-it-or-leave-it point for this pull request. The time
    > > for discussion was *MONTHS* ago.
    > >
    >
    > I’ll admit I’m not impartial, but my fundamental goal is moving the patches forward. Given folios will need long term maintenance, engagement, and iteration throughout mm/, take-it-or-leave-it pulls seem like a recipe for future conflict, and more importantly, bugs.
    >
    > I’d much rather work it out now.

    That's the nature of a pull request. It's binary -- either it's pulled or
    it's rejected. Well, except that Linus has opted for silence, leaving
    me in limbo. I have no idea what he's thinking. I don't know if he
    agrees with Johannes. I don't know what needs to change for Linus to
    like this series enough to pull it (either now or in the 5.16 merge
    window). And that makes me frustrated. This is over a year of work
    from me and others, and it's being held up over concerns which seem to
    me to be entirely insubstantial (the name "folio"? really? and even
    my change to use "pageset" was met with silence from Linus.)

    I agree with Kent & Johannes that struct page is a mess. I agree that
    cleaning it up will bring many benefits. I've even started a design
    document here:

    https://kernelnewbies.org/MemoryTypes

    I do see some advantages to splitting out anon memory descriptors from
    file memory descriptors, but there is also plenty of code which handles
    both types in the same way. I see the requests to continue to use
    struct page to mean a "memory descriptor which is either anon or file",
    but I really think that's the wrong approach. A struct page should
    represent /a page/ of memory. Otherwise we're just confusing people.
    I know it's a confusion we've had since compound pages were introduced,
    what, 25+ years ago, but that expediency has overstayed its welcome.

    The continued silence from Linus is really driving me to despair.
    I'm sorry I've been so curt with some of the requests. I really am
    willing to change things; I wasn't planning on doing anything with slab
    until Kent prodded me to do it. But equally, I strongly believe that
    everything I've done here is a step towards the things that everybody
    wants, and I'm frustrated that it's being perceived as a step away,
    or even to the side of what people want.

    So ... if any of you have Linus' ear. Maybe you're at a conference with
    him later this week. Please, just get him to tell me what I need to do
    to make him happy with this patchset.

    \
     
     \ /
      Last update: 2021-09-22 21:57    [W:4.050 / U:0.984 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site