From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1EDF5C369AB for ; Thu, 24 Apr 2025 03:27:29 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 136516B0008; Wed, 23 Apr 2025 23:27:27 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0E6436B000A; Wed, 23 Apr 2025 23:27:27 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EF0486B000C; Wed, 23 Apr 2025 23:27:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id CD1576B0008 for ; Wed, 23 Apr 2025 23:27:26 -0400 (EDT) Received: from smtpin21.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id C0EF45FAA2 for ; Thu, 24 Apr 2025 03:27:27 +0000 (UTC) X-FDA: 83367502134.21.762D16B Received: from out-173.mta0.migadu.com (out-173.mta0.migadu.com [91.218.175.173]) by imf22.hostedemail.com (Postfix) with ESMTP id E407DC0007 for ; Thu, 24 Apr 2025 03:27:25 +0000 (UTC) Authentication-Results: imf22.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=c2W6jdR3; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf22.hostedemail.com: domain of ye.liu@linux.dev designates 91.218.175.173 as permitted sender) smtp.mailfrom=ye.liu@linux.dev ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1745465246; a=rsa-sha256; cv=none; b=K4TQ6SsdrZASLX30PjE/I5mJFgDb6Qxq+DBMrkp15CbDM7RrgAsrTdbhGd8QKjmDr86J3g 2M5nqqaTLBq/oY65h/OyHEYip/aUMVxHT7atjA6pNHCtp2Zpiq4AzdjvkH0tIzCbf3+tsg 0BLXFbCXv5ilnMaSEUPgeFxse4DU2jw= ARC-Authentication-Results: i=1; imf22.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=c2W6jdR3; dmarc=pass (policy=none) header.from=linux.dev; spf=pass (imf22.hostedemail.com: domain of ye.liu@linux.dev designates 91.218.175.173 as permitted sender) smtp.mailfrom=ye.liu@linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1745465246; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=xd0gahNoT+bypY4s9KPQJobXki3DYYYU4L9ST/2/O7M=; b=J3eIY27HlThGm8Sy2lfbhf1MdDZG/my8ZJahzW1bkUVGFmTMnWR1S0L8By0bVLz4WOgV2R vpzTNgCq4qEC8an+AMJ+bZk/UOkXOXoWyO09uf6ptQIQ831tkjE0gZtERTCebGGjFir0EV XNz4HK85s1ZBqDYB9X9LDQ7sLvgOeXo= Message-ID: <91db850c-6c5d-4ea3-bca0-fc853a00a457@linux.dev> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1745465244; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xd0gahNoT+bypY4s9KPQJobXki3DYYYU4L9ST/2/O7M=; b=c2W6jdR3CXE5JKGUT7wYr+6b4tquBM/NnjD7LhPMww++9mOAdQ2wghT8kOwnNZM8Rdu7Kd kWc7gw/mMubMS5YwkbLF7zNkC3rG5p3ey4XYe4B9+AgvKcmyA6jyaFQpMutBWQGnqgKaWJ 6YDGXcs+WmTE/0VU0KBMRinxyfkstz0= Date: Thu, 24 Apr 2025 11:27:10 +0800 MIME-Version: 1.0 Subject: Re: [PATCH v3] tools/mm: Add script to display page state for a given PID and VADDR To: SeongJae Park Cc: akpm@linux-foundation.org, linux-debuggers@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-toolchains@vger.kernel.org, osandov@osandov.com, paulmck@kernel.org, sweettea-kernel@dorminy.me, liuye@kylinos.cn References: <20250424025128.48038-1-sj@kernel.org> Content-Language: en-US X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Ye Liu In-Reply-To: <20250424025128.48038-1-sj@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT X-Rspam-User: X-Rspamd-Queue-Id: E407DC0007 X-Rspamd-Server: rspam04 X-Stat-Signature: fsiho981i6ocj9ytk6au4g9us33wruey X-HE-Tag: 1745465245-919903 X-HE-Meta: U2FsdGVkX1+EZflDVe+6CEv4I/vV9MaaFW8NNqD2h6YBBKu4HhA4i+FRHCKrNZdU3R5oAAzaRu8pdJ445EbPsQwH+X0UqUSFNa9hT9rH915hkztc7GxXmtPVnSar2u+IGGO9TkamkuQrlzf+90OKsdAK9USK2Zv7aCZtY9acpluxt7GjxRbIE7EQVsyx+hYZfECVefS1wzNWONMGcz6o9fSoIBJpmIjOAN+4oBqKJvmaYqaiDVvMpUfv/hEqlLFZOhwwWjBNIqICPwCilfqUTex0SlPCRGmQ/AP+/YqkujnQRjFgc1vRgray1gok1x8hpZOolx6VjVQL83izEoii6G7DqlP32nV1BJ7lET+aSr/lQu66270BG6mkqPps6WRaWsnOy+JbxiOQgp6UqDACibhNJXSqso05lK1X6yybT4iSGNd6cBDjR66X4sF0pc3vivdqEp7Mke2OufLdX22vuF4XVj/M7+6CpyVVBN1x/GiS+4QTQ8q9Sktock4qH/Q7hAIuk+fx6FYis5TzWaeoi46a7x6O3QLMP4kNimkx93DKYOOrbnDzzJWdps2btqMA5TZPENXqbU6gAGSs+RuZIqI0UubiqX/opcvbmvVOqZkEzO30/HT78i1yI+w9s04OivqVFUNkLKuTPreJ/9m4GqwwENYz/OjDjfEKfWSIp2YkugpOjBOyN1ZlGSICdzz+SQFcENhPQzktS2YLfaox1URQuATQ2lPq5BAG7DznYVhGNOSrf7I88iASw3K0DgkB1RcJSB6fD1AyjGUJ69OWyfcx5hW/MRtlzbXxbRjz+RxfZHnPEG2nBseugFyqjmZmlRb4f4eajrNIa87LDL9n6jJOFVDfnjuTPXwDM8vV2aeXIqKxDGHqA4+JH59toFy3J/+QEzXXbWbphp2ZoOe1MMbUEv5AHu9zfBcNkLCTP55YbSuGVeD00y/73XNr2790iyiqIlVcckWiFbLhdIy T8KxS7vx w+VfhxhFiW74BYAYnys+X/PGjk+CqK2DP1WDlbMx3T+IbP3jh+0ns+0wG61zpaNdQj1nLrT4NgDlJvzo1W3n1Zh7LyhEfqtTkV+6K/DRxxEtiuQ9azJ90k2ZR9YsccnW228DIrurhaaFAYVQ7fvR0TCX9ajh+R0PC284jwaJZh170kMbYLA6i502wmQNCFGul7xTI5uALH6tkUDtNJ7aV1RhPEcZzT96UvXiCYQ+MS1xhiswbCu87Y/wt4f/JIqqqhfeRR8yYKgQPcJLzLho2GcqZpA1DyujqJGLuLuYciMruIj9ElEqcQ1R/EPEcr3UlJwv79pQxvBVthNZoFnPHWp4vgA== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: 在 2025/4/24 10:51, SeongJae Park 写道: > On Thu, 24 Apr 2025 09:55:22 +0800 Ye Liu wrote: > >> 在 2025/4/24 06:00, SeongJae Park 写道: >>> On Wed, 23 Apr 2025 09:48:50 +0800 Ye Liu wrote: >>> >>>> From: Ye Liu >>>> >>>> Introduces a new drgn script, `show_page_info.py`, which allows users >>>> to analyze the state of a page given a process ID (PID) and a virtual >>>> address (VADDR). This can help kernel developers or debuggers easily >>>> inspect page-related information in a live kernel or vmcore. >>>> >>>> The script extracts information such as the page flags, mapping, and >>>> other metadata relevant to diagnosing memory issues. >>>> >>>> Output example: >>>> sudo ./show_page_info.py 1 0x7f43df5acf00 >>>> PID: 1 Comm: systemd mm: 0xffff8881273bbc40 >>>> Raw: 0017ffffc000416c ffffea00043a4508 ffffea0004381e08 ffff88810f086a70 >>>> Raw: 0000000000000000 ffff888120c9b0c0 0000002500000007 ffff88812642c000 >>>> User Virtual Address: 0x7f43df5acf00 >>>> Page Address: 0xffffea00049a0b00 >>>> Page Flags: PG_referenced|PG_uptodate|PG_lru|PG_head|PG_active| >>>> PG_private|PG_reported >>>> Page Size: 16384 >>> Should this be called folio size? Or, could this simply removed since Compound >>> Order is given below? >> >> Page size refers to the base page size, which equals PAGESIZE. > Shouldn't 'prog["PAGE_SIZE"]' is used for what you are saying? This tool is > using drgn.helpers.linux.page_size()[1] to print this, though? > > +def show_page_state(page, addr, mm, pid, task): > + """Display detailed information about a page.""" > + print(f'PID: {pid} Comm: {task.comm.string_().decode()} mm: {hex(mm)}') > + print(format_page_data(prog.read(page.value_(), 64))) > + fields = { > + "User Virtual Address": hex(addr), > + "Page Address": hex(page.value_()), > + "Page Flags": decode_page_flags(page), > + "Page Size": page_size(page).value_(), > > [1] https://drgn.readthedocs.io/en/stable/helpers.html#drgn.helpers.linux.mm.page_size You're right — I'll update it to use prog["PAGE_SIZE"] here. >> Folio size can be calculated using the Compound Order, but of course, >> it can also be shown directly as a result. >> >>>> Page PFN: 0x12682c >>>> Page Physical: 0x12682c000 >>>> Page Virtual: 0xffff88812682c000 >>>> Page Refcount: 37 >>>> Page Mapcount: 7 >>>> Page Index: 0x0 >>>> Page Memcg Data: 0xffff88812642c000 >>>> Memcg Name: init.scope >>>> Memcg Path: /sys/fs/cgroup/memory/init.scope >>>> Page Mapping: 0xffff88810f086a70 >>>> Page Anon/File: File >>>> Page VMA: 0xffff88810e4af3b8 >>>> VMA Start: 0x7f43df5ac000 >>>> VMA End: 0x7f43df5b0000 >>>> This page is part of a compound page. >>>> This page is the head page of a compound page. >>>> Head Page: 0xffffea00049a0b00 >>>> Compound Order: 2 >>>> Number of Pages: 4 >>>> >>>> Signed-off-by: Ye Liu >>>> >>>> Changes in v3: >>>> - Adjust display style. >>>> - Link to v2:https://lore.kernel.org/all/20250421080748.114750-1-ye.liu@linux.dev/ >>>> >>>> Changes in v2: >>>> - Move the show_page_info.py file to tools/mm. >>>> - Link to v1: https://lore.kernel.org/all/20250415075024.248232-1-ye.liu@linux.dev/ >>>> --- >>>> MAINTAINERS | 5 ++ >>>> tools/mm/show_page_info.py | 120 +++++++++++++++++++++++++++++++++++++ >>>> 2 files changed, 125 insertions(+) >>>> create mode 100755 tools/mm/show_page_info.py >>>> >>>> diff --git a/MAINTAINERS b/MAINTAINERS >>>> index 17ed0b5ffdd2..85686a30dc72 100644 >>>> --- a/MAINTAINERS >>>> +++ b/MAINTAINERS >>>> @@ -18351,6 +18351,11 @@ F: Documentation/mm/page_table_check.rst >>>> F: include/linux/page_table_check.h >>>> F: mm/page_table_check.c >>>> >>>> +PAGE STATE DEBUG SCRIPT >>>> +M: Ye Liu >>>> +S: Maintained >>>> +F: tools/mm/show_page_info.py >>>> + >>>> PANASONIC LAPTOP ACPI EXTRAS DRIVER >>>> M: Kenneth Chan >>>> L: platform-driver-x86@vger.kernel.org >>>> diff --git a/tools/mm/show_page_info.py b/tools/mm/show_page_info.py >>>> new file mode 100755 >>>> index 000000000000..8622c5499dfe >>>> --- /dev/null >>>> +++ b/tools/mm/show_page_info.py >>> [...] >>>> +def main(): >>>> + """Main function to parse arguments and display page state.""" >>>> + parser = argparse.ArgumentParser(description=DESC, formatter_class=argparse.RawTextHelpFormatter) >>>> + parser.add_argument('pid', metavar='PID', type=int, help='Target process ID (PID)') >>>> + parser.add_argument('vaddr', metavar='VADDR', type=str, help='Target virtual address in hexadecimal format (e.g., 0x7fff1234abcd)') >>>> + args = parser.parse_args() >>>> + >>>> + try: >>>> + vaddr = int(args.vaddr, 16) >>>> + except ValueError: >>>> + print(f"Error: Invalid virtual address format: {args.vaddr}") >>>> + return >>>> + >>>> + task = find_task(args.pid) >>>> + mm = task.mm >>>> + page = follow_page(mm, vaddr) >>> I tried this script on my test machine and got the below error: >>> >>> $ cat ./a.c >>> #include >>> >>> int main(void) >>> { >>> int foo; >>> printf("hello\n"); >>> printf("%x\n", &foo); >> To avoid address truncation, you can use the %p format specifier >> instead of %x or %lx when printing a pointer (memory address). > Ah, you're correct, thank you. After fixing my test, the error I reported > before is disappeared. But I think the follow_page() error handling would > better to be updated to catch the exception and provide a better error message? > > Also, I'm getting below new error: > > $ sudo ./tools/mm/show_page_info.py 47657 0x7fffaf925b6c > PID: 47657 Comm: a.out mm: 0xffff959c8a022100 > Raw: 0017ffffc0020828 ffffea6b0c201408 ffffea6b0fc65648 ffff959d32bec9c1 > Raw: 00000007fffffffc 0000000000000000 0000000100000000 ffff959cba058000 > Traceback (most recent call last): > File "/usr/local/bin/drgn", line 33, in > sys.exit(load_entry_point('drgn==0.0.30+82.ge2b60e4b', 'console_scripts', 'drgn')()) > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > File "/usr/local/lib/python3.11/dist-packages/drgn-0.0.30+82.ge2b60e4b-py3.11-linux-x86_64.egg/drgn/cli.py", line 461, in _main > runpy.run_path(script, init_globals={"prog": prog}, run_name="__main__") > File "", line 291, in run_path > File "", line 98, in _run_module_code > File "", line 88, in _run_code > File "./tools/mm/show_page_info.py", line 120, in > main() > File "./tools/mm/show_page_info.py", line 115, in main > show_page_state(page, vaddr, mm, args.pid, task) > File "./tools/mm/show_page_info.py", line 63, in show_page_state > memcg_name, memcg_path = get_memcg_info(page) > ^^^^^^^^^^^^^^^^^^^^ > File "./tools/mm/show_page_info.py", line 43, in get_memcg_info > return cgroup_name(cgrp).decode(), f"/sys/fs/cgroup/memory{cgroup_path(cgrp).decode()}" > ^^^^^^^^^^^^^^^^^ > File "/usr/local/lib/python3.11/dist-packages/drgn-0.0.30+82.ge2b60e4b-py3.11-linux-x86_64.egg/drgn/helpers/linux/cgroup.py", line 71, in cgroup_name > return kernfs_name(cgrp.kn) > ^^^^^^^^^^^^^^^^^^^^ > File "/usr/local/lib/python3.11/dist-packages/drgn-0.0.30+82.ge2b60e4b-py3.11-linux-x86_64.egg/drgn/helpers/linux/kernfs.py", line 32, in kernfs_name > return kn.name.string_() if kn.parent else b"/" > ^^^^^^^^^ > AttributeError: 'struct kernfs_node' has no member 'parent'. Did you mean: '__parent'? > > Seems not entirely this script's fault but due to the recent 'struct > kernfs_node' change or my old version of drgn? But anyway, I think it is > better to provide a better error message to users. I'm also curious if you > have a plan for finding and fixing or avoiding this kind of future breakages. > > > Thanks, > SJ > > [...]