All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/9] page_owner improvements for debugging
@ 2015-11-24 12:36 ` Vlastimil Babka
  0 siblings, 0 replies; 115+ messages in thread
From: Vlastimil Babka @ 2015-11-24 12:36 UTC (permalink / raw)
  To: linux-mm
  Cc: linux-kernel, Andrew Morton, Joonsoo Kim, Minchan Kim,
	Sasha Levin, Kirill A. Shutemov, Mel Gorman, Michal Hocko,
	Vlastimil Babka

This is the second version of patchset which originally aimed to improve the
page_owner functionality. Thanks to feedback from v1 and some bugs I
discovered along the way, it is now larger in scope and number of patches.
It's based on next-20151124.

For page_owner, the main changes are
o Use static key to further reduce overhead when compiled in but not enabled.
o Improve output wrt. page and pageblock migratetypes
o Transfer the info on page migrations and track last migration reason.
o Dump the info as part of dump_page() to hopefully help debugging.

For the last point, Kirill requested a human readable printing of gfp_mask and
migratetype after v1. At that point it probably makes a lot of sense to do the
same for page alloc failure and OOM warnings. The flags have been undergoing
revisions recently, and we might be getting reports from various kernel
versions that differ. The ./scripts/gfp-translate tool needs to be pointed at
the corresponding sources to be accurate.  The downside is potentially breaking
scripts that grep these warnings, but it's not a first change done there over
the years.

Note I'm not entirely happy about the dump_gfpflag_names() implementation, due
to usage of pr_cont() unreliable on SMP (and I've seen spurious newlines in
dmesg output, while being correct on serial console or /var/log/messages).
It also doesn't allow plugging the gfp_mask translation into
/sys/kernel/debug/page_owner where it also could make sense. Maybe a new
*printf formatting flag? Too specialized maybe? Or just prepare the string in
a buffer on stack with strscpy?

Other changes since v1:
o Change placement of page owner migration calls to cover missing cases (Hugh)
o Move dump_page_owner() call up from dump_page_badflags(), so the latter can
  be used for adding debugging prints without page owner info (Kirill)

Vlastimil Babka (9):
  mm, debug: fix wrongly filtered flags in dump_vma()
  mm, page_owner: print symbolic migratetype of both page and pageblock
  mm, page_owner: convert page_owner_inited to static key
  mm, page_owner: copy page owner info during migration
  mm, page_owner: track and print last migrate reason
  mm, debug: introduce dump_gfpflag_names() for symbolic printing of
    gfp_flags
  mm, page_owner: dump page owner info from dump_page()
  mm, page_alloc: print symbolic gfp_flags on allocation failure
  mm, oom: print symbolic gfp_flags in oom warning

 Documentation/vm/page_owner.txt |  9 +++--
 include/linux/migrate.h         |  6 ++-
 include/linux/mmdebug.h         |  1 +
 include/linux/mmzone.h          |  3 ++
 include/linux/page_ext.h        |  1 +
 include/linux/page_owner.h      | 50 ++++++++++++++++++-------
 include/trace/events/gfpflags.h | 14 +++++--
 mm/debug.c                      | 44 ++++++++++++++++------
 mm/migrate.c                    | 23 ++++++++++--
 mm/oom_kill.c                   | 10 +++--
 mm/page_alloc.c                 | 18 ++++++++-
 mm/page_owner.c                 | 82 +++++++++++++++++++++++++++++++++++++----
 mm/vmstat.c                     | 15 +-------
 13 files changed, 213 insertions(+), 63 deletions(-)

-- 
2.6.3


^ permalink raw reply	[flat|nested] 115+ messages in thread

end of thread, other threads:[~2016-01-08 11:34 UTC | newest]

Thread overview: 115+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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     ` [PATCH 2/2] mm, page_owner: provide symbolic page flags and gfp_flags Vlastimil Babka
2015-11-30 16:10       ` 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

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.