Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) client-ip=23.128.96.38;
Message-ID: <9f91b2be-dfb2-8bc3-91f8-6bd0715cc89d@suse.cz>
Date:   Tue, 17 Oct 2023 10:13:53 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.1
Subject: Re: [PATCH 5/5] tools/mm: Update the usage output to be more
 organized
Content-Language: en-US
To:     Audra Mitchell <audra@redhat.com>, linux-mm@kvack.org
Cc:     raquini@redhat.com, akpm@linux-foundation.org, djakov@kernel.org,
        linux-kernel@vger.kernel.org
References: <20231013190350.579407-1-audra@redhat.com>
 <20231013190350.579407-6-audra@redhat.com>
From:   Vlastimil Babka <vbabka@suse.cz>
In-Reply-To: <20231013190350.579407-6-audra@redhat.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 10/13/23 21:03, Audra Mitchell wrote:
> Organize the usage options alphabetically and improve the description
> of some options. Also separate the more complicated cull options from
> the single use compare options.
> 
> Signed-off-by: Audra Mitchell <audra@redhat.com>

Acked-by: Vlastimil Babka <vbabka@suse.cz>

Thanks a lot for making the tool useful again!

> ---
>  tools/mm/page_owner_sort.c | 33 ++++++++++++++++++++-------------
>  1 file changed, 20 insertions(+), 13 deletions(-)
> 
> diff --git a/tools/mm/page_owner_sort.c b/tools/mm/page_owner_sort.c
> index 5a260096ebaa..e1f264444342 100644
> --- a/tools/mm/page_owner_sort.c
> +++ b/tools/mm/page_owner_sort.c
> @@ -634,19 +634,26 @@ static void print_allocator(FILE *out, int allocator)
>  static void usage(void)
>  {
>  	printf("Usage: ./page_owner_sort [OPTIONS] <input> <output>\n"
> -		"-m\t\tSort by total memory.\n"
> -		"-s\t\tSort by the stack trace.\n"
> -		"-t\t\tSort by times (default).\n"
> -		"-p\t\tSort by pid.\n"
> -		"-P\t\tSort by tgid.\n"
> -		"-n\t\tSort by task command name.\n"
> -		"-a\t\tSort by memory allocate time.\n"
> -		"-d\t\tPrint debug information.\n"
> -		"--pid <pidlist>\tSelect by pid. This selects the information of blocks whose process ID numbers appear in <pidlist>.\n"
> -		"--tgid <tgidlist>\tSelect by tgid. This selects the information of blocks whose Thread Group ID numbers appear in <tgidlist>.\n"
> -		"--name <cmdlist>\n\t\tSelect by command name. This selects the information of blocks whose command name appears in <cmdlist>.\n"
> -		"--cull <rules>\tCull by user-defined rules.<rules> is a single argument in the form of a comma-separated list with some common fields predefined\n"
> -		"--sort <order>\tSpecify sort order as: [+|-]key[,[+|-]key[,...]]\n"
> +		"-a\t\t\tSort by memory allocation time.\n"
> +		"-m\t\t\tSort by total memory.\n"
> +		"-n\t\t\tSort by task command name.\n"
> +		"-p\t\t\tSort by pid.\n"
> +		"-P\t\t\tSort by tgid.\n"
> +		"-s\t\t\tSort by the stacktrace.\n"
> +		"-t\t\t\tSort by number of times record is seen (default).\n\n"
> +		"--pid <pidlist>\t\tSelect by pid. This selects the information"
> +		" of\n\t\t\tblocks whose process ID numbers appear in <pidlist>.\n"
> +		"--tgid <tgidlist>\tSelect by tgid. This selects the information"
> +		" of\n\t\t\tblocks whose Thread Group ID numbers appear in "
> +		"<tgidlist>.\n"
> +		"--name <cmdlist>\tSelect by command name. This selects the"
> +		" information\n\t\t\tof blocks whose command name appears in"
> +		" <cmdlist>.\n"
> +		"--cull <rules>\t\tCull by user-defined rules. <rules> is a "
> +		"single\n\t\t\targument in the form of a comma-separated list "
> +		"with some\n\t\t\tcommon fields predefined (pid, tgid, comm, "
> +		"stacktrace, allocator)\n"
> +		"--sort <order>\t\tSpecify sort order as: [+|-]key[,[+|-]key[,...]]\n"
>  	);
>  }
>