All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhouping Liu <zliu@redhat.com>
To: Jan Stancek <jstancek@redhat.com>
Cc: LTP List <ltp-list@lists.sourceforge.net>
Subject: Re: [LTP] [PATCH resend 5/6] mm: use new numa_helper
Date: Fri, 10 Aug 2012 05:54:38 -0400 (EDT)	[thread overview]
Message-ID: <23193633.36569632.1344592478934.JavaMail.root@redhat.com> (raw)
In-Reply-To: <2014107378.3027931.1344587065940.JavaMail.root@redhat.com>



----- Original Message -----
> From: "Jan Stancek" <jstancek@redhat.com>
> To: "Zhouping Liu" <zliu@redhat.com>
> Cc: "LTP List" <ltp-list@lists.sourceforge.net>, "Caspar Zhang" <caspar@casparzhang.com>
> Sent: Friday, August 10, 2012 4:24:25 PM
> Subject: Re: [LTP] [PATCH resend 5/6] mm: use new numa_helper
> 
> 
> 
> ----- Original Message -----
> > From: "Zhouping Liu" <zliu@redhat.com>
> > To: "Caspar Zhang" <caspar@casparzhang.com>
> > Cc: "LTP List" <ltp-list@lists.sourceforge.net>
> > Sent: Friday, 10 August, 2012 10:02:01 AM
> > Subject: Re: [LTP] [PATCH resend 5/6] mm: use new numa_helper
> > 
> 
> > > -» return·nnodes;
> > > +» ret·=·get_allowed_nodes(NH_MEMS|NH_CPUS,·1,·&nd1);
> > 
> > Caspar, I found get_a_numa_node always return the first node id,
> > which
> > will make all cases called the function
> > only run in the first node, I don't think it's a good way, it
> > decrease
> > the coverage.
> 
> We had a discussion with Caspar on this topic:
> http://article.gmane.org/gmane.linux.ltp/16495
> 
> Can you explain why a specific node matters here?

yeah, I didn't carefully read the discussion before,
and discussed with Caspar, I agree with the current solution.
so Reviewed-by: Zhouping Liu <zliu@redhat.com>

but always I'm worrying it maybe has difference between different nodes,
especially in old version, such as RHEL5, which didn't improve the
performance of scheduling in NUMA system, so that more process will choose
the first node(I guess), and in a large NUMA system, which have 2+ available
NUMA nodes(contain memory and CPUs), I think the second node represent more
universality than first one, that's all my supposing, so I will investigating
the supposing, actually if it exists the difference between different nodes,
I will do a new patch.

Thanks,
Zhouping 

> 
> Regards,
> Jan
> 
> > 
> > we should prefer to run cases on the second node, we can chose the
> > first
> > only if it just only has
> > one node with NH_MEMS|NH_CPUS. do you think so?
> > 
> > and others is good for me.
> > 
> > Thanks,
> > Zhouping
> 
>

------------------------------------------------------------------------------
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

  reply	other threads:[~2012-08-10  9:54 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-09  6:15 [LTP] [PATCH resend 0/6] NUMA and follow-up fix Caspar Zhang
2012-08-09  6:15 ` [LTP] [PATCH resend 1/6] numa_helper: add flag to get_allowed_nodes() Caspar Zhang
2012-08-10  7:43   ` Zhouping Liu
2012-08-09  6:15 ` [LTP] [PATCH resend 2/6] update testcases to use flag for get_allowed_nodes() Caspar Zhang
2012-08-10  7:45   ` Zhouping Liu
2012-08-09  6:15 ` [LTP] [PATCH resend 3/6] syscalls/numa: check syscall availability Caspar Zhang
2012-08-09  9:22   ` Jan Stancek
2012-08-09 10:35     ` Caspar Zhang
2012-08-10  7:46   ` Zhouping Liu
2012-08-10  7:54     ` Wanlong Gao
2012-08-10  8:13       ` Zhouping Liu
2012-08-09  6:15 ` [LTP] [PATCH resend 4/6] numa_helper: move to libkerntest Caspar Zhang
2012-08-10  3:26   ` [LTP] [PATCH resend 4/6 v2] " Caspar Zhang
2012-08-10  7:51     ` Zhouping Liu
2012-08-09  6:15 ` [LTP] [PATCH resend 5/6] mm: use new numa_helper Caspar Zhang
2012-08-10  8:02   ` Zhouping Liu
2012-08-10  8:22     ` Caspar Zhang
2012-08-10  8:24     ` Jan Stancek
2012-08-10  9:54       ` Zhouping Liu [this message]
2012-08-09  6:15 ` [LTP] [PATCH resend 6/6] mm: add common Makefile include: libmem.mk Caspar Zhang
2012-08-09 10:15 ` [LTP] [PATCH resend 0/6] NUMA and follow-up fix Jan Stancek
2012-08-09 10:31 ` [LTP] [PATCH resend v2 3/6] syscalls/numa: check syscall availability Caspar Zhang
2012-08-10  8:10   ` Zhouping Liu
2012-08-10  3:12 ` [LTP] [PATCH resend 0/6] NUMA and follow-up fix Zhouping Liu
2012-08-10  3:28   ` Caspar Zhang
2012-08-10  3:57     ` Zhouping Liu
2012-08-10  9:22 ` Wanlong Gao

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=23193633.36569632.1344592478934.JavaMail.root@redhat.com \
    --to=zliu@redhat.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 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.