[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090508114742.GB17129@elte.hu>
Date: Fri, 8 May 2009 13:47:42 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Wu Fengguang <fengguang.wu@...el.com>,
Frédéric Weisbecker <fweisbec@...il.com>,
Steven Rostedt <rostedt@...dmis.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Li Zefan <lizf@...fujitsu.com>
Cc: Andrew Morton <akpm@...ux-foundation.org>,
LKML <linux-kernel@...r.kernel.org>,
KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>,
Andi Kleen <andi@...stfloor.org>,
Matt Mackall <mpm@...enic.com>,
Alexey Dobriyan <adobriyan@...il.com>, linux-mm@...ck.org
Subject: Re: [PATCH 4/8] proc: export more page flags in /proc/kpageflags
* Wu Fengguang <fengguang.wu@...el.com> wrote:
> Export all page flags faithfully in /proc/kpageflags.
Ongoing objection and NAK against extended haphazard exporting of
kernel internals via an ad-hoc ABI via ad-hoc, privatized
instrumentation that only helps the MM code and nothing else. It was
a mistake to introduce the /proc/kpageflags hack a year ago, and it
even more wrong today to expand on it.
/proc/kpageflags should be done via the proper methods outlined in
the previous mails i wrote on this topic: for example by using the
'object collections' abstraction i suggested. Clean enumeration of
all pages (files, tasks, etc.) and the definition of histograms over
it via free-form filter expressions is the right way to do this. It
would not only help other subsystems, it would also be far more
capable.
So this should be done in cooperation with instrumentation folks,
while improving _all_ of Linux instrumentation in general. Or, if
you dont have the time/interest to work with us on that, it should
not be done at all. Not having the resources/interest to do
something properly is not a license to introduce further
instrumentation crap into Linux.
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists