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 42E1BC369CB for ; Thu, 24 Apr 2025 02:51:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 545B56B0008; Wed, 23 Apr 2025 22:51:33 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4F2EE6B000A; Wed, 23 Apr 2025 22:51:33 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3BBEA6B000E; Wed, 23 Apr 2025 22:51:33 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 1B2456B0008 for ; Wed, 23 Apr 2025 22:51:33 -0400 (EDT) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id CDDBD1CDF8B for ; Thu, 24 Apr 2025 02:51:33 +0000 (UTC) X-FDA: 83367411666.05.2904F0D Received: from sea.source.kernel.org (sea.source.kernel.org [172.234.252.31]) by imf26.hostedemail.com (Postfix) with ESMTP id 02435140003 for ; Thu, 24 Apr 2025 02:51:31 +0000 (UTC) Authentication-Results: imf26.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=aLGctXxc; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf26.hostedemail.com: domain of sj@kernel.org designates 172.234.252.31 as permitted sender) smtp.mailfrom=sj@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1745463092; a=rsa-sha256; cv=none; b=NSFDzPY2HQcFFZ5tUBfm6EIdkulqLduxySU0JiltbRC4RnMw5z4ct3sspRpjePv9gW/FVE 022MyElloqPKbJizGxsZh0tlGqMesFaTusJXSYn/yBuJ1V9X4tNxlHXwPEFD39FdJbyu6D s1bdwPe/TPMIYLw/YlxLhzy4/Qs7g7E= ARC-Authentication-Results: i=1; imf26.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=aLGctXxc; dmarc=pass (policy=quarantine) header.from=kernel.org; spf=pass (imf26.hostedemail.com: domain of sj@kernel.org designates 172.234.252.31 as permitted sender) smtp.mailfrom=sj@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1745463092; 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=jaj2tGfQOj7QVJ3Zmshc2AVm39mUYNn5m9Cxvcm+rCA=; b=Qcj2gvV1HvHr5KW+E0RalEH2jgKj2DdhDL2lGzeF2jiaxGNBxm6C8nc1koWuArxjQEsY6Y hYQTFeFn+90g2UNgr36wtZyiePP5zg5topz46PjwyGAPdWjQfL/kFkmugr3Dye8ZBGfoSs LZY7M8YZy3wl95fDpSrHmzwfOCXWixY= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 17BDC4A472; Thu, 24 Apr 2025 02:51:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4B2D7C4CEE2; Thu, 24 Apr 2025 02:51:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1745463090; bh=5eLimYjAs9IUgV4p8sHo53fodXOGP+VyuSVvK8Jm/hM=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=aLGctXxcQyYMWbhmm0iRzI0ZJj/r44OWLqzwD/UgzAgwNANRjm3bqG3y/spF0ccR0 KxojT4Gsfxa1GbJA0y7SKKlQTnP+MLldDot9Ikcy6s2lzNnLVPI0ySHWiFtAlgnqSj wmnRa33KIQ9YsGjHF9BtKFDow6WADGnnB3EhdnJD/0NvJGdyRyatqkG+gJmwqCks55 LfJUjz40QFYR02eYAt+hrSqq1ltKAYEsYJdLgbbAO5wtv7kIi9py2UJUUa8vKh1ckS lJsFhqrXZjdUj2L1fbPQgorZK9m5Df9KsyGjUW6PoB21RM2n7artmLLHJVpaCrqJ98 edXvZm48o2Gmw== From: SeongJae Park To: Ye Liu Cc: SeongJae Park , 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 Subject: Re: [PATCH v3] tools/mm: Add script to display page state for a given PID and VADDR Date: Wed, 23 Apr 2025 19:51:28 -0700 Message-Id: <20250424025128.48038-1-sj@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <758b1d83-f1b3-43d2-8c37-324cc89c78f7@linux.dev> References: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rspamd-Server: rspam10 X-Rspamd-Queue-Id: 02435140003 X-Stat-Signature: qt73jn6xzyaab5osykqic6m6im474ebq X-Rspam-User: X-HE-Tag: 1745463091-770296 X-HE-Meta: U2FsdGVkX1/laL/93KMckg8AyviCK2l2yzxwszHsr6wIKPphGz64U6Hcu8d6PyTjz8EPJBthDifVaxKzTpU1marEJKFR2zGPYbt8UkU16fPyEy4JBrKBwSwYhdZF1j2TKhj1WCd+3rtvuRElrr9xVgNeS1NvPe+xcVuZCyt6fFdKGj0v6Xkh1p09ViEHUW80nWgNc0ezoGgVupUzrXpzTmVPEpO+rUYMjmY0XQW3deX4oWyro8nE/jPuJRETazQ/Tpj4kteAjyv+XlK0nwq7grlbg+lIMtIRFDPqUB1mjlIemSmWmEQlXcvuEHRStm+Nj7s/mzynl5b6ST0GijFrFbA+kiGtPjGwQggckHWZct6RDiV1etwYFHwzxgW1t2lvU700ffUk/CQ4eD4AsTVrZV9JEs9k23Wm00aHMBxNPDxaiqV4YS8oa+XEdajkI0VDxsRooyAxwdpkpQEcqJMpLdm/EX2X7EzNhvjXChE1iWDQlFNQ2pdDBvsTaNkO5s76xEclNiQ0vTJV1cwjvIpVpx1r+euw8qpDM4VLcVACYa02dFEM01oKc6Yk3xNXcuZwW3U72055CTSsbpVuQ3mvXrcC7GWaP2EmcdjO7zfvqYzzJxh4LrFVlcdR62EQYN6MH251xYue3esBVGLGBnhF11DmG9dhEc5JGN0oHHrX8rFUK2Nn7nF4L15sZ0f1VRQB6GZiJawcAnBpy86sj8Gmxi2ZYFc9H0rXJYFQPhgTmMBFvfPz0Pq2ZpRPbvqn6jS62dFoTJDJxjnb2JdGiHlQg5urp4lucuhJruiaKV8JGd3DESgq1kZBUF7wozwwQ4tBzItcOkYZJVwB4vHi0X1bCPNO2KpNuZYHlan88rVrp2moWvDUuCVWersGY5ZQkjHoM9DLS0rtf9qX9sG2KZ021eFiYG0TiSK4t/Saw8hn3fw61SsmwY37BQc4h3QPgBbmrb8wBmhAyAEUYdyKJ/O Onr1/6b9 EI7hMtqyz0UIq7o93UzPsKHJEX5sICwfH7/IHJ59Ujq5Tk4ZXd5xuPjkwtBso+ovANDo+NFJIkTRl6rdE0AFcTagI21E/tVrZ/qoSWuqX0+zo1LquCMyPu12ZnDRdp/RznaXy2CDmldCKs/Bh+Lh3NvSUn/HOmnqR5N5TkGQA0vJDycqXEWhhKhBTWNBH/d7evXrEFRG0AJTYoCsdDlWF1EQK/GxHWItIZNjRPgVH2sXt7XzJ41eag5QehTHtJwYEOuUAZw/6nppxrLjbhgnE5c2+KLd2ZsHFTd72oi8u97bxSIaLlmxIRh3qG8RAI3EDhnvEWH2fw9yQFTuo1hc6FBaU/zQxJet/0j6WQsrq6GRPde85AJowllx4WQ== 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: 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 > 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 [...]