[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <43def405-a55c-4bea-9caf-220408ddac40@redhat.com>
Date: Tue, 16 Apr 2024 10:08:35 +0200
From: David Hildenbrand <david@...hat.com>
To: Barry Song <21cnbao@...il.com>
Cc: akpm@...ux-foundation.org, linux-mm@...ck.org,
cerasuolodomenico@...il.com, chrisl@...nel.org, kasong@...cent.com,
linux-kernel@...r.kernel.org, peterx@...hat.com, ryan.roberts@....com,
surenb@...gle.com, v-songbaohua@...o.com, willy@...radead.org,
yosryahmed@...gle.com, yuzhao@...gle.com, corbet@....net
Subject: Re: [PATCH v6 0/4] mm: add per-order mTHP alloc and swpout counters
On 12.04.24 15:16, Barry Song wrote:
> On Sat, Apr 13, 2024 at 12:54 AM David Hildenbrand <david@...hat.com> wrote:
>>
>> On 12.04.24 13:48, Barry Song wrote:
>>> From: Barry Song <v-songbaohua@...o.com>
>>>
>>> The patchset introduces a framework to facilitate mTHP counters, starting
>>> with the allocation and swap-out counters. Currently, only four new nodes
>>> are appended to the stats directory for each mTHP size.
>>>
>>> /sys/kernel/mm/transparent_hugepage/hugepages-<size>/stats
>>> anon_fault_alloc
>>> anon_fault_fallback
>>> anon_fault_fallback_charge
>>> anon_swpout
>>> anon_swpout_fallback
>>>
>>> These nodes are crucial for us to monitor the fragmentation levels of
>>> both the buddy system and the swap partitions. In the future, we may
>>> consider adding additional nodes for further insights.
>>>
>>> -v6:
>>> * collect reviewed-by tags for patch2/4, 3/4, 4/4, Ryan;
>>> * move back to static array by using MAX_PTRS_PER_PTE, Ryan;
>>> * move to for_each_possible_cpu to handle cpu hotplug, Ryan;
>>> * other minor cleanups according to Ryan;
>>
>> Please *really* not multiple versions of the same patch set on one a
>> single day.
>
> Ok. I will leave more time for you to review the older versions before moving
> to a new version.
Yes please. There is not anything gained from sending out stuff to fast,
besides mixing discussions, same questions/comments ... and effectively
more work for reviewers.
--
Cheers,
David / dhildenb
Powered by blists - more mailing lists