public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
To: Rik van Riel <riel@redhat.com>
Cc: linux-kernel@vger.kernel.org, peterz@infradead.org,
	mingo@kernel.org, mgorman@suse.de
Subject: Re: [PATCH] sched,numa: document and fix numa_preferred_nid setting
Date: Mon, 22 Jun 2015 21:34:37 +0530	[thread overview]
Message-ID: <20150622160437.GD16576@linux.vnet.ibm.com> (raw)
In-Reply-To: <558456DB.3040108@redhat.com>


> Would you happen to have 2 instance and 4 instance SPECjbb
> numbers, too?  The single instance numbers seem to be within
> the margin of error, but I would expect multi-instance numbers
> to show more dramatic changes, due to changes in how workloads
> converge...
>
> Those behave very differently from single instance, especially
> with the "always set the preferred_nid, even if we moved the
> task to a node we do NOT prefer" patch...
>
> It would be good to understand the behaviour of these patches
> under more circumstances.

Here are specjbb2005 numbers with 1 JVM per System, 2 JVMs per System
and 4 JVMs per System.

Plain 4.1.0-rc7-tip (i)
tip + Rik's ++ (ii)
tip + Srikar's ++ (iii)
tip + Srikar's + Modified Rik's patch (iv)

(i) = Plain 4.1.0-rc7-tip = tip = 4.1.0-rc7 (b7ca96b)

(ii) =  tip + only Rik's suggested patches =  (i) + Rik's fix numa_preferred_nid setting
	+ Srikar's numa hotness + correct nid for evaluating task weight

(iii)  = tip + Srikar's ++ (iii) = (i) + Srikar's  numa hotness + correct nid for evaluating
	 task weight + numa_has_capacity fix +  always update preferred node

(iv) =  tip + Srikar's ++ (iv) = (i) + Srikar's  numa hotness + correct nid for evaluating
	 task weight + numa_has_capacity fix + Rik's modified patch.
	(Rik's modified patch == I removed node_isset check before setting
	nid as the preferred node)

jbb2005_1JVMperSYSTEM
Plain 4.1.0-rc7-tip (i)
		  Metric:         Min         Max         Avg      StdDev     %Change
	      bopsperJVM:   265519.00   272466.00   269377.80     2391.04

tip + Rik's ++ (ii)
	      bopsperJVM:   264298.00   271236.00   266818.20     2579.62      -0.94%

tip + Srikar's ++ (iii)
	      bopsperJVM:   266774.00   272434.00   269839.60     2083.19       0.17%

tip + Srikar's + Rik's (iv)
	      bopsperJVM:   265037.00   274419.00   269280.00     3146.74      -0.04%



jbb2005_2JVMperSYSTEM
Plain 4.1.0-rc7-tip (i)
		  Metric:         Min         Max         Avg      StdDev     %Change
	      bopsperJVM:   269575.00   288495.00   279910.80     6151.49

tip + Srikar's ++ (iii)
	      bopsperJVM:   278810.00   287706.00   282514.00     2946.37      0.90%

tip + Rik's ++ (ii)
	      bopsperJVM:   286785.00   289515.00   288311.80     1206.66      2.90%

tip + Srikar's + Rik's (iv)
	      bopsperJVM:   283295.00   293466.00   287848.80     3427.06      2.70%


jbb2005_4JVMperSYSTEM
Plain 4.1.0-rc7-tip (i)
		  Metric:         Min         Max         Avg      StdDev     %Change
	      bopsperJVM:   248392.00   263826.00   257263.20     5946.44

tip + Rik's ++ (ii)
	      bopsperJVM:   257057.00   260303.00   258819.00     1234.46      0.60%

tip + Srikar's ++ (iii)
	      bopsperJVM:   252968.00   262006.00   257321.80     3131.00      0.02%

tip + Srikar's + Rik's (iv)
	      bopsperJVM:   257063.00   266196.00   262547.80     3099.57      1.99%


Summary:
While Rik's suggested patchset performs the best in 2 JVM case and
numa01. A modified version of his patch, provides good performance in 2
JVM, 4 JVM cases and numa01. However these two patchsets dont regress in
numa02 (probably a little less with modified patch)


-- 
Thanks and Regards
Srikar Dronamraju

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

  reply	other threads:[~2015-06-22 16:04 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-16 19:54 [PATCH] sched,numa: document and fix numa_preferred_nid setting Rik van Riel
2015-06-18 15:55 ` Srikar Dronamraju
2015-06-18 16:06   ` Rik van Riel
2015-06-18 16:41     ` Srikar Dronamraju
2015-06-18 17:00       ` Rik van Riel
2015-06-18 17:11         ` Srikar Dronamraju
2015-06-19 17:16         ` Srikar Dronamraju
2015-06-19 17:52           ` Rik van Riel
2015-06-22 16:04             ` Srikar Dronamraju [this message]
2015-06-22 16:48           ` Srikar Dronamraju
2015-06-18 16:12   ` Ingo Molnar
2015-06-18 18:16     ` Rik van Riel
2015-06-22 16:13 ` Srikar Dronamraju
2015-06-22 22:28   ` Rik van Riel

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=20150622160437.GD16576@linux.vnet.ibm.com \
    --to=srikar@linux.vnet.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mgorman@suse.de \
    --cc=mingo@kernel.org \
    --cc=peterz@infradead.org \
    --cc=riel@redhat.com \
    /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