On Thu, Oct 21, 2021 at 05:37:41PM -0400, Johannes Weiner wrote:
> Here is my summary of the discussion, and my conclusion:
Thank you for this. It's the clearest, most useful post on this thread,
including my own. It really highlights the substantial points that
should be discussed.
> The premise of the folio was initially to simply be a type that says:
> I'm the headpage for one or more pages. Never a tailpage. Cool.
>
> However, after we talked about what that actually means, we seem to
> have some consensus on the following:
>
> 1) If folio is to be a generic headpage, it'll be the new
> dumping ground for slab, network, drivers etc. Nobody is
> psyched about this, hence the idea to split the page into
> subtypes which already resulted in the struct slab patches.
>
> 2) If higher-order allocations are going to be the norm, it's
> wasteful to statically allocate full descriptors at a 4k
> granularity. Hence the push to eliminate overloading and do
> on-demand allocation of necessary descriptor space.
>
> I think that's accurate, but for the record: is there anybody who
> disagrees with this and insists that struct folio should continue to
> be the dumping ground for all kinds of memory types?
I think there's a useful distinction to be drawn between "where we're
going with this patchset", "where we're going in the next six-twelve
months" and "where we're going eventually". I think we have minor
differences of opinion on the answers to those questions, and they can
be resolved as we go, instead of up-front.
My answer to that question is that, while this full conversion is not
part of this patch, struct folio is logically:
struct folio {
... almost everything that's currently in struct page ...
};
struct page {
unsigned long flags;
unsigned long compound_head;
union {
struct { /* First tail page only */
unsigned char compound_dtor;
unsigned char compound_order;
atomic_t compound_mapcount;
unsigned int compound_nr;
};
struct { /* Second tail page only */
atomic_t hpage_pinned_refcount;
struct list_head deferred_list;
};
unsigned long padding1[4];
};
unsigned int padding2[2];
#ifdef CONFIG_MEMCG
unsigned long padding3;
#endif
#ifdef WANT_PAGE_VIRTUAL
void *virtual;
#endif
#ifdef LAST_CPUPID_NOT_IN_PAGE_FLAGS
int _last_cpupid;
#endif
};
(I'm open to being told I have some of that wrong, eg maybe _last_cpupid
is actually part of struct folio and isn't a per-page property at all)
I'd like to get there in the next year. I think dynamically allocating
memory descriptors is more than a year out.
Now, as far as struct folio being a dumping group, I would like to
split other things out from struct folio. Let me address that below.
> Let's assume the answer is "no" for now and move on.
>
> If folios are NOT the common headpage type, it begs two questions:
>
> 1) What subtype(s) of page SHOULD it represent?
>
> This is somewhat unclear at this time. Some say file+anon.
> It's also been suggested everything userspace-mappable, but
> that would again bring back major type punning. Who knows?
>
> Vocal proponents of the folio type have made conflicting
> statements on this, which certainly gives me pause.
>
> 2) What IS the common type used for attributes and code shared
> between subtypes?
>
> For example: if a folio is anon+file, then the code that
> maps memory to userspace needs a generic type in order to
> map both folios and network pages. Same as the page table
> walkers, and things like GUP.
>
> Will this common type be struct page? Something new? Are we
> going to duplicate the implementation for each subtype?
>
> Another example: GUP can return tailpages. I don't see how
> it could return folio with even its most generic definition
> of "headpage".
>
> (But bottomline, it's not clear how folio can be the universal
> headpage type and simultaneously avoid being the type dumping ground
> that the page was. Maybe I'm not creative enough?)
This whole section is predicated on "If it is NOT the headpage type",
but I think this is a great list of why it _should_ be the generic
headpage type.
To answer a questions in here; GUP should continue to return precise
pages because that's what its callers expect. But we should have a
better interface than GUP which returns a rather more compressed list
(something like today's biovec).
> Anyway. I can even be convinved that we can figure out the exact fault
> lines along which we split the page down the road.
>
> My worry is more about 2). A shared type and generic code is likely to
> emerge regardless of how we split it. Think about it, the only world
> in which that isn't true would be one in which either
>
> a) page subtypes are all the same, or
> b) the subtypes have nothing in common
>
> and both are clearly bogus.
Amen!
I'm convinced that pgtable, slab and zsmalloc uses of struct page can all
be split out into their own types instead of being folios. They have
little-to-nothing in common with anon+file; they can't be mapped into
userspace and they can't be on the LRU. The only situation you can find
them in is something like compaction which walks PFNs.
I don't think we can split out ZONE_DEVICE and netpool into their own
types. While they can't be on the LRU, they can be mapped to userspace,
like random device drivers. So they can be found by GUP, and we want
(need) to be able to go to folio from there in order to get, lock and
set a folio as dirty. Also, they have a mapcount as well as a refcount.
The real question, I think, is whether it's worth splitting anon & file
pages out from generic pages. I can see arguments for it, but I can also
see arguments against it (whether it's two types: lru_mem and folio,
three types: anon_mem, file_mem and folio or even four types: ksm_mem,
anon_mem and file_mem). I don't think a compelling argument has been
made either way.
Perhaps you could comment on how you'd see separate anon_mem and
file_mem types working for the memcg code? Would you want to have
separate lock_anon_memcg() and lock_file_memcg(), or would you want
them to be cast to a common type like lock_folio_memcg()?
P.S. One variant we haven't explored is separating type specialisation
from finding the head page. eg, instead of having
struct slab *slab = page_slab(page);
we could have:
struct slab *slab = folio_slab(page_folio(page));
I don't think it's particularly worth doing, but Kent mused about it
at one point.
