From: Caspar Zhang <caspar@casparzhang.com>
To: Jan Stancek <jstancek@redhat.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH 0/2] numa_helper: get nodes with mem/cpu/both
Date: Fri, 20 Jul 2012 19:02:47 +0800 [thread overview]
Message-ID: <50093AD7.9040602@casparzhang.com> (raw)
In-Reply-To: <cover.1342777340.git.jstancek@redhat.com>
On 07/20/2012 06:19 PM, Jan Stancek wrote:
> numa_helper has been used for syscalls testcases, which
> care about nodes with memory. Based on discussion here:
> http://article.gmane.org/gmane.linux.ltp/16325
> it was deemed, that 'flag' extension would be useful,
> to allow tests to ask for nodes with memory/cpu/both.
>
> This patch series is introducing flag to get_allowed_nodes().
>
> For example, a config like this one:
> # numactl -H
> available: 3 nodes (2,4-5)
> node 2 cpus: 0
> node 2 size: 511 MB
> node 2 free: 88 MB
> node 4 cpus:
> node 4 size: 256 MB
> node 4 free: 34 MB
> node 5 cpus:
> node 5 size: 255 MB
> node 5 free: 59 MB
>
> would return:
> nodes (flag=0): 2 4 5 (any)
> nodes (flag=1): 2 4 5 (mem)
> nodes (flag=2): 2 (cpus)
> nodes (flag=3): 2 (mem and cpus)
>
> Tested with:
> numactl-devel-0.9.8-7.el5
> numactl-devel-2.0.3-9.el6
> numactl-devel-2.0.7-6.el7
>
> numa_node_to_cpus() is broken in older libnuma < 1, so cpumap
> is used instead.
>
> Next step (not part of this series) is to move it outside
> syscalls area, so it can be used by other testcases.
>
> Jan Stancek (2):
> numa_helper: add flag to get_allowed_nodes()
> update testcases to use flag for get_allowed_nodes()
>
> .../syscalls/get_mempolicy/get_mempolicy01.c | 5 +-
> testcases/kernel/syscalls/mbind/mbind01.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages02.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages03.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages04.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages05.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages06.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages07.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages08.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages09.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages10.c | 5 +-
> .../kernel/syscalls/move_pages/move_pages11.c | 5 +-
> .../syscalls/move_pages/move_pages_support.c | 8 +-
> testcases/kernel/syscalls/numa/lib/numa_helper.c | 211 +++++++++++++++-----
> testcases/kernel/syscalls/numa/lib/numa_helper.h | 8 +-
> 15 files changed, 203 insertions(+), 84 deletions(-)
>
>
Thanks Jan for your patches! I want to test the patchset on all arches
and on major releases, so would you mind if I hold on pushing until I
finish the testing next week?
Also during that period, any comments from anyone is welcomed.
Thanks,
Caspar
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2012-07-20 11:03 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-20 10:19 [LTP] [PATCH 0/2] numa_helper: get nodes with mem/cpu/both Jan Stancek
2012-07-20 10:19 ` [LTP] [PATCH 1/2] numa_helper: add flag to get_allowed_nodes() Jan Stancek
2012-07-20 10:19 ` [LTP] [PATCH 2/2] update testcases to use flag for get_allowed_nodes() Jan Stancek
2012-07-20 11:02 ` Caspar Zhang [this message]
2012-07-20 11:28 ` [LTP] [PATCH 0/2] numa_helper: get nodes with mem/cpu/both Jan Stancek
2012-07-26 10:14 ` Jan Stancek
2012-07-26 11:21 ` Caspar Zhang
2012-07-31 2:53 ` [LTP] [PATCH] syscalls/numa: check syscall availability Caspar Zhang
2012-07-31 3:57 ` Garrett Cooper
2012-08-08 9:29 ` Caspar Zhang
2012-08-08 9:46 ` Garrett Cooper
2012-08-08 14:11 ` Caspar Zhang
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=50093AD7.9040602@casparzhang.com \
--to=caspar@casparzhang.com \
--cc=jstancek@redhat.com \
--cc=ltp-list@lists.sourceforge.net \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox