On Fri, Aug 12, 2022 at 05:33:56PM +0300, Kirill A. Shutemov wrote: > If you really need info about these pages and reference their memdesc it > is likely be 9 cache lines that scattered across memory instead of 8 cache > lines next to each other in the same page. Well, hopefully not. Most allocations should be multiple pages. That's already true for slab, netpool and file (for xfs anyway), and hopefully soon for anon. > Initially, I thought we can offset the cost by caching memdescs instead of > struct page/folio. Like page cache store memdesc, but it would require > memdesc_to_pfn() which is not possible, unless we want to store pfn > explicitly in memdesc. I think we do, at least for some memdescs. File folios definitely want to store the pfn, but I don't think getting the PFN for a slab is a common operation (although we'll still need to store the pointer to the struct page, so it's equivalent). > I don't want to be buzzkill, I like the idea a lot, but abstractions are > often costly. Getting it upstream without noticeable performance > regressions going to be a challenge. I don't think there's a way to find out whether it'll be a performance win without actually doing it. Fortunately, the steps to get to this point are mostly good cleanups anyway.
