From: Vlastimil Babka <vbabka@suse.cz> To: linux-mm@kvack.org Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@linux-foundation.org>, Joonsoo Kim <iamjoonsoo.kim@lge.com>, Minchan Kim <minchan@kernel.org>, Sasha Levin <sasha.levin@oracle.com>, "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>, Mel Gorman <mgorman@suse.de>, Michal Hocko <mhocko@suse.cz>, Vlastimil Babka <vbabka@suse.cz>, Rasmus Villemoes <linux@rasmusvillemoes.dk> Subject: [PATCH 2/2] mm, page_owner: provide symbolic page flags and gfp_flags Date: Mon, 30 Nov 2015 17:10:21 +0100 [thread overview] Message-ID: <1448899821-9671-2-git-send-email-vbabka@suse.cz> (raw) In-Reply-To: <1448899821-9671-1-git-send-email-vbabka@suse.cz> With the new format strings for flags, we can now provide symbolic page and gfp flags in the /sys/kernel/debug/page_owner file. This replaces the positional printing of page flags as single letters, which might have looked nicer, but was limited to a subset of flags, and required the user to remember the letters. Example of the adjusted format: Page allocated via order 0, mask 0x24213ca(GFP_HIGHUSER_MOVABLE|GFP_COLD|GFP_NOWARN|GFP_NORETRY) PFN 674308 type Movable Block 1317 type Movable Flags 0x1fffff80010068(uptodate|lru|active|mappedtodisk) [<ffffffff81164e9a>] __alloc_pages_nodemask+0x15a/0xa30 [<ffffffff811ab938>] alloc_pages_current+0x88/0x120 [<ffffffff8115bc46>] __page_cache_alloc+0xe6/0x120 [<ffffffff81168b9b>] __do_page_cache_readahead+0xdb/0x200 [<ffffffff81168df5>] ondemand_readahead+0x135/0x260 [<ffffffff81168f8c>] page_cache_async_readahead+0x6c/0x70 [<ffffffff8115d5f8>] generic_file_read_iter+0x378/0x590 [<ffffffff811d12a7>] __vfs_read+0xa7/0xd0 Page has been migrated, last migrate reason: compaction Signed-off-by: Vlastimil Babka <vbabka@suse.cz> --- mm/page_owner.c | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/mm/page_owner.c b/mm/page_owner.c index ff862b6d12da..421765a53c68 100644 --- a/mm/page_owner.c +++ b/mm/page_owner.c @@ -135,8 +135,8 @@ print_page_owner(char __user *buf, size_t count, unsigned long pfn, return -ENOMEM; ret = snprintf(kbuf, count, - "Page allocated via order %u, mask 0x%x\n", - page_ext->order, page_ext->gfp_mask); + "Page allocated via order %u, mask %pgg\n", + page_ext->order, &page_ext->gfp_mask); if (ret >= count) goto err; @@ -145,23 +145,12 @@ print_page_owner(char __user *buf, size_t count, unsigned long pfn, pageblock_mt = get_pfnblock_migratetype(page, pfn); page_mt = gfpflags_to_migratetype(page_ext->gfp_mask); ret += snprintf(kbuf + ret, count - ret, - "PFN %lu type %s Block %lu type %s Flags %s%s%s%s%s%s%s%s%s%s%s%s\n", + "PFN %lu type %s Block %lu type %s Flags %pgp\n", pfn, migratetype_names[page_mt], pfn >> pageblock_order, migratetype_names[pageblock_mt], - PageLocked(page) ? "K" : " ", - PageError(page) ? "E" : " ", - PageReferenced(page) ? "R" : " ", - PageUptodate(page) ? "U" : " ", - PageDirty(page) ? "D" : " ", - PageLRU(page) ? "L" : " ", - PageActive(page) ? "A" : " ", - PageSlab(page) ? "S" : " ", - PageWriteback(page) ? "W" : " ", - PageCompound(page) ? "C" : " ", - PageSwapCache(page) ? "B" : " ", - PageMappedToDisk(page) ? "M" : " "); + &page->flags); if (ret >= count) goto err; -- 2.6.3
WARNING: multiple messages have this Message-ID (diff)
From: Vlastimil Babka <vbabka@suse.cz> To: linux-mm@kvack.org Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@linux-foundation.org>, Joonsoo Kim <iamjoonsoo.kim@lge.com>, Minchan Kim <minchan@kernel.org>, Sasha Levin <sasha.levin@oracle.com>, "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>, Mel Gorman <mgorman@suse.de>, Michal Hocko <mhocko@suse.cz>, Vlastimil Babka <vbabka@suse.cz>, Rasmus Villemoes <linux@rasmusvillemoes.dk> Subject: [PATCH 2/2] mm, page_owner: provide symbolic page flags and gfp_flags Date: Mon, 30 Nov 2015 17:10:21 +0100 [thread overview] Message-ID: <1448899821-9671-2-git-send-email-vbabka@suse.cz> (raw) In-Reply-To: <1448899821-9671-1-git-send-email-vbabka@suse.cz> With the new format strings for flags, we can now provide symbolic page and gfp flags in the /sys/kernel/debug/page_owner file. This replaces the positional printing of page flags as single letters, which might have looked nicer, but was limited to a subset of flags, and required the user to remember the letters. Example of the adjusted format: Page allocated via order 0, mask 0x24213ca(GFP_HIGHUSER_MOVABLE|GFP_COLD|GFP_NOWARN|GFP_NORETRY) PFN 674308 type Movable Block 1317 type Movable Flags 0x1fffff80010068(uptodate|lru|active|mappedtodisk) [<ffffffff81164e9a>] __alloc_pages_nodemask+0x15a/0xa30 [<ffffffff811ab938>] alloc_pages_current+0x88/0x120 [<ffffffff8115bc46>] __page_cache_alloc+0xe6/0x120 [<ffffffff81168b9b>] __do_page_cache_readahead+0xdb/0x200 [<ffffffff81168df5>] ondemand_readahead+0x135/0x260 [<ffffffff81168f8c>] page_cache_async_readahead+0x6c/0x70 [<ffffffff8115d5f8>] generic_file_read_iter+0x378/0x590 [<ffffffff811d12a7>] __vfs_read+0xa7/0xd0 Page has been migrated, last migrate reason: compaction Signed-off-by: Vlastimil Babka <vbabka@suse.cz> --- mm/page_owner.c | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/mm/page_owner.c b/mm/page_owner.c index ff862b6d12da..421765a53c68 100644 --- a/mm/page_owner.c +++ b/mm/page_owner.c @@ -135,8 +135,8 @@ print_page_owner(char __user *buf, size_t count, unsigned long pfn, return -ENOMEM; ret = snprintf(kbuf, count, - "Page allocated via order %u, mask 0x%x\n", - page_ext->order, page_ext->gfp_mask); + "Page allocated via order %u, mask %pgg\n", + page_ext->order, &page_ext->gfp_mask); if (ret >= count) goto err; @@ -145,23 +145,12 @@ print_page_owner(char __user *buf, size_t count, unsigned long pfn, pageblock_mt = get_pfnblock_migratetype(page, pfn); page_mt = gfpflags_to_migratetype(page_ext->gfp_mask); ret += snprintf(kbuf + ret, count - ret, - "PFN %lu type %s Block %lu type %s Flags %s%s%s%s%s%s%s%s%s%s%s%s\n", + "PFN %lu type %s Block %lu type %s Flags %pgp\n", pfn, migratetype_names[page_mt], pfn >> pageblock_order, migratetype_names[pageblock_mt], - PageLocked(page) ? "K" : " ", - PageError(page) ? "E" : " ", - PageReferenced(page) ? "R" : " ", - PageUptodate(page) ? "U" : " ", - PageDirty(page) ? "D" : " ", - PageLRU(page) ? "L" : " ", - PageActive(page) ? "A" : " ", - PageSlab(page) ? "S" : " ", - PageWriteback(page) ? "W" : " ", - PageCompound(page) ? "C" : " ", - PageSwapCache(page) ? "B" : " ", - PageMappedToDisk(page) ? "M" : " "); + &page->flags); if (ret >= count) goto err; -- 2.6.3 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2015-11-30 16:11 UTC|newest] Thread overview: 115+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-24 12:36 [PATCH v2 0/9] page_owner improvements for debugging Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-24 12:36 ` [PATCH v2 1/9] mm, debug: fix wrongly filtered flags in dump_vma() Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-27 9:52 ` Vlastimil Babka 2015-11-27 9:52 ` Vlastimil Babka 2015-11-24 12:36 ` [PATCH v2 2/9] mm, page_owner: print symbolic migratetype of both page and pageblock Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 8:11 ` Joonsoo Kim 2015-11-25 8:11 ` Joonsoo Kim 2015-11-24 12:36 ` [PATCH v2 3/9] mm, page_owner: convert page_owner_inited to static key Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 14:52 ` Michal Hocko 2015-11-25 14:52 ` Michal Hocko 2015-11-25 15:08 ` Vlastimil Babka 2015-11-25 15:08 ` Vlastimil Babka 2015-11-25 15:25 ` Peter Zijlstra 2015-11-25 15:25 ` Peter Zijlstra 2015-11-25 15:46 ` Michal Hocko 2015-11-25 15:46 ` Michal Hocko 2015-11-24 12:36 ` [PATCH v2 4/9] mm, page_owner: copy page owner info during migration Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-24 12:36 ` [PATCH v2 5/9] mm, page_owner: track and print last migrate reason Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 8:13 ` Joonsoo Kim 2015-11-25 8:13 ` Joonsoo Kim 2015-11-26 10:39 ` Vlastimil Babka 2015-11-26 10:39 ` Vlastimil Babka 2015-11-24 12:36 ` [PATCH v2 6/9] mm, debug: introduce dump_gfpflag_names() for symbolic printing of gfp_flags Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 8:16 ` Joonsoo Kim 2015-11-25 8:16 ` Joonsoo Kim 2015-11-25 10:28 ` Vlastimil Babka 2015-11-25 10:28 ` Vlastimil Babka 2015-11-27 3:40 ` yalin wang 2015-11-27 3:40 ` yalin wang 2015-11-24 12:36 ` [PATCH v2 7/9] mm, page_owner: dump page owner info from dump_page() Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 14:58 ` Michal Hocko 2015-11-25 14:58 ` Michal Hocko 2015-11-26 10:43 ` Vlastimil Babka 2015-11-26 10:43 ` Vlastimil Babka 2015-11-24 12:36 ` [PATCH v2 8/9] mm, page_alloc: print symbolic gfp_flags on allocation failure Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 14:33 ` Michal Hocko 2015-11-25 14:33 ` Michal Hocko 2015-11-24 12:36 ` [PATCH v2 9/9] mm, oom: print symbolic gfp_flags in oom warning Vlastimil Babka 2015-11-24 12:36 ` Vlastimil Babka 2015-11-25 14:31 ` Michal Hocko 2015-11-25 14:31 ` Michal Hocko 2016-01-07 21:29 ` David Rientjes 2016-01-07 21:29 ` David Rientjes 2016-01-08 11:34 ` Vlastimil Babka 2016-01-08 11:34 ` Vlastimil Babka 2015-11-25 14:30 ` [PATCH v2 0/9] page_owner improvements for debugging Michal Hocko 2015-11-25 14:30 ` Michal Hocko 2015-11-30 16:10 ` [PATCH 1/2] mm, printk: introduce new format string for flags Vlastimil Babka 2015-11-30 16:10 ` Vlastimil Babka 2015-11-30 16:10 ` Vlastimil Babka [this message] 2015-11-30 16:10 ` [PATCH 2/2] mm, page_owner: provide symbolic page flags and gfp_flags Vlastimil Babka 2015-12-02 11:01 ` [PATCH 1/2] mm, printk: introduce new format string for flags Rasmus Villemoes 2015-12-02 11:01 ` Rasmus Villemoes 2015-12-02 20:34 ` Vlastimil Babka 2015-12-02 20:34 ` Vlastimil Babka 2015-12-03 12:37 ` Rasmus Villemoes 2015-12-03 12:37 ` Rasmus Villemoes 2015-12-03 13:46 ` Vlastimil Babka 2015-12-03 13:46 ` Vlastimil Babka 2015-12-04 15:16 ` [PATCH v2 1/3] " Vlastimil Babka 2015-12-04 15:16 ` Vlastimil Babka 2015-12-04 15:16 ` [PATCH v2 2/3] mm, page_owner: provide symbolic page flags and gfp_flags Vlastimil Babka 2015-12-04 15:16 ` Vlastimil Babka 2015-12-04 15:16 ` [PATCH v2 3/3] mm, debug: move bad flags printing to bad_page() Vlastimil Babka 2015-12-04 15:16 ` Vlastimil Babka 2015-12-05 20:00 ` [PATCH v2 1/3] mm, printk: introduce new format string for flags Rasmus Villemoes 2015-12-05 20:00 ` Rasmus Villemoes 2015-12-09 11:29 ` Arnd Bergmann 2015-12-09 11:29 ` Arnd Bergmann 2015-12-09 20:48 ` Vlastimil Babka 2015-12-09 20:48 ` Vlastimil Babka 2015-12-10 12:26 ` James Hogan 2015-12-10 12:26 ` James Hogan 2015-12-10 12:26 ` James Hogan 2015-12-10 2:59 ` Joonsoo Kim 2015-12-10 2:59 ` Joonsoo Kim 2015-12-10 4:04 ` Steven Rostedt 2015-12-10 4:04 ` Steven Rostedt 2015-12-10 4:12 ` Joonsoo Kim 2015-12-10 4:12 ` Joonsoo Kim 2015-12-10 8:41 ` Rasmus Villemoes 2015-12-10 8:41 ` Rasmus Villemoes 2015-12-10 10:03 ` Vlastimil Babka 2015-12-10 10:03 ` Vlastimil Babka 2015-12-14 3:03 ` Joonsoo Kim 2015-12-14 3:03 ` Joonsoo Kim 2015-12-10 3:51 ` Steven Rostedt 2015-12-10 3:51 ` Steven Rostedt 2015-12-10 9:51 ` Vlastimil Babka 2015-12-10 9:51 ` Vlastimil Babka 2015-12-02 17:40 ` [PATCH 1/2] " yalin wang 2015-12-02 17:40 ` yalin wang 2015-12-02 21:04 ` Vlastimil Babka 2015-12-02 21:04 ` Vlastimil Babka 2015-12-03 0:11 ` yalin wang 2015-12-03 0:11 ` yalin wang 2015-12-03 8:03 ` Rasmus Villemoes 2015-12-03 8:03 ` Rasmus Villemoes 2015-12-03 18:38 ` yalin wang 2015-12-03 18:38 ` yalin wang 2015-12-04 1:04 ` yalin wang 2015-12-04 1:04 ` yalin wang 2015-12-04 14:15 ` Vlastimil Babka 2015-12-04 14:15 ` Vlastimil Babka 2015-12-10 4:03 ` Steven Rostedt 2015-12-10 4:03 ` Steven Rostedt
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1448899821-9671-2-git-send-email-vbabka@suse.cz \ --to=vbabka@suse.cz \ --cc=akpm@linux-foundation.org \ --cc=iamjoonsoo.kim@lge.com \ --cc=kirill.shutemov@linux.intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=linux@rasmusvillemoes.dk \ --cc=mgorman@suse.de \ --cc=mhocko@suse.cz \ --cc=minchan@kernel.org \ --cc=sasha.levin@oracle.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.