[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4BF255F3.9040002@linux.intel.com>
Date: Tue, 18 May 2010 10:55:15 +0200
From: Andi Kleen <ak@...ux.intel.com>
To: Dave Hansen <dave@...ux.vnet.ibm.com>
CC: Shaohui Zheng <shaohui.zheng@...el.com>, Greg KH <gregkh@...e.de>,
akpm@...ux-foundation.org, linux-mm@...ck.org,
Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...hat.com>,
"H. Peter Anvin" <hpa@...or.com>, x86@...nel.org,
Hidetoshi Seto <seto.hidetoshi@...fujitsu.com>,
Wu Fengguang <fengguang.wu@...el.com>,
Heiko Carstens <heiko.carstens@...ibm.com>,
linux-kernel@...r.kernel.org, haicheng.li@...ux.intel.com,
shaohui.zheng@...ux.intel.com
Subject: Re: [RFC, 6/7] NUMA hotplug emulator
>
> Maybe configfs isn't the way to go. I just think extending the 'probe'
> file is a bad idea, especially in the way your patch did it. I'm open
> to other alternatives. Since this is only for testing, perhaps debugfs
> applies better. What other alternatives have you explored? How about a
> Systemtap set to do it? :)
First this is a debugging interface. It doesn't need to have the
most pretty interface in the world, because it will be only used for
QA by a few people.
Requiring setting parameters in two different file systems doesn't
sound that appealing to me.
systemtap for configuration also doesn't seem right.
I liked Dave's earlier proposal to do a command line parameter like interface
for "probe". Perhaps that can be done. It shouldn't need a lot of code.
In fact there are already two different parser libraries for this:
lib/parser.c and lib/params.c. One could chose the one that one likes
better :-)
Anything that needs a lot of code is a bad idea for this I think.
A simple parser using one of the existing libraries should be simple
enough though.
Again it's just a QA interface, not the next generation of POSIX.
-Andi
--
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