Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <e518246c-4ffa-14af-e568-798572f75bee@quicinc.com>
Date:   Mon, 25 Jul 2022 16:39:03 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.0.3
Subject: Re: [PATCH] mm/page_owner.c: allow page_owner with given
 start_pfn/count
Content-Language: en-US
To:     Matthew Wilcox <willy@infradead.org>
CC:     <akpm@linux-foundation.org>, <minchan@kernel.org>,
        <iamjoonsoo.kim@lge.com>, <linux-kernel@vger.kernel.org>,
        <linux-mm@kvack.org>, <quic_guptap@quicinc.com>
References: <20220722150810.27740-1-quic_yingangl@quicinc.com>
 <YtrEXB6o+VROImdg@casper.infradead.org>
From:   Kassey Li <quic_yingangl@quicinc.com>
In-Reply-To: <YtrEXB6o+VROImdg@casper.infradead.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

hi, Matthew:
	sorry for the delay, I just started to learn how to upstream patch, and 
setup my Thunderbird with plain text only.
	you are right, two users will cause problem here.
	the uses case is dump CMA area to understand the page usage in a given 
cma pool. 2nd, dump whole memory page owner is very time cost, mostly 
our android device has 8G memory now.
	I will research and check again, if you have more idea on this , please 
kindly to share.

BR
Kassey

On 7/22/2022 11:38 PM, Matthew Wilcox wrote:
> On Fri, Jul 22, 2022 at 11:08:10PM +0800, Kassey Li wrote:
>> by default, page_owner iterates all page from min_low_pfn to
>> max_pfn, this cost too much time if we want an alternative pfn range.
>>
>> with this patch it allows user to set pfn range to dump the page_onwer.
> 
> This is a really bad UI.  If two users try to do different ranges at the
> same time, it'll go wrong.  What use cases are you actually trying to
> solve?