From: Rusty Russell <rusty@rustcorp.com.au>
To: Ingo Molnar <mingo@kernel.org>,
Madhavan Srinivasan <maddy@linux.vnet.ibm.com>
Cc: linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
linux-mm@kvack.org, linux-arch@vger.kernel.org, x86@kernel.org,
benh@kernel.crashing.org, paulus@samba.org,
kirill.shutemov@linux.intel.com, akpm@linux-foundation.org,
riel@redhat.com, mgorman@suse.de, ak@linux.intel.com,
peterz@infradead.org, dave.hansen@intel.com,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [PATCH V3 2/2] powerpc/pseries: init fault_around_order for pseries
Date: Wed, 30 Apr 2014 16:34:11 +0930 [thread overview]
Message-ID: <87d2fz47tg.fsf@rustcorp.com.au> (raw)
In-Reply-To: <20140429070632.GB27951@gmail.com>
Ingo Molnar <mingo@kernel.org> writes:
> * Madhavan Srinivasan <maddy@linux.vnet.ibm.com> wrote:
>
>> Performance data for different FAULT_AROUND_ORDER values from 4 socket
>> Power7 system (128 Threads and 128GB memory). perf stat with repeat of 5
>> is used to get the stddev values. Test ran in v3.14 kernel (Baseline) and
>> v3.15-rc1 for different fault around order values.
>>
>> FAULT_AROUND_ORDER Baseline 1 3 4 5 8
>>
>> Linux build (make -j64)
>> minor-faults 47,437,359 35,279,286 25,425,347 23,461,275 22,002,189 21,435,836
>> times in seconds 347.302528420 344.061588460 340.974022391 348.193508116 348.673900158 350.986543618
>> stddev for time ( +- 1.50% ) ( +- 0.73% ) ( +- 1.13% ) ( +- 1.01% ) ( +- 1.89% ) ( +- 1.55% )
>> %chg time to baseline -0.9% -1.8% 0.2% 0.39% 1.06%
>
> Probably too noisy.
A little, but 3 still looks like the winner.
>> Linux rebuild (make -j64)
>> minor-faults 941,552 718,319 486,625 440,124 410,510 397,416
>> times in seconds 30.569834718 31.219637539 31.319370649 31.434285472 31.972367174 31.443043580
>> stddev for time ( +- 1.07% ) ( +- 0.13% ) ( +- 0.43% ) ( +- 0.18% ) ( +- 0.95% ) ( +- 0.58% )
>> %chg time to baseline 2.1% 2.4% 2.8% 4.58% 2.85%
>
> Here it looks like a speedup. Optimal value: 5+.
No, lower time is better. Baseline (no faultaround) wins.
etc.
It's not a huge surprise that a 64k page arch wants a smaller value than
a 4k system. But I agree: I don't see much upside for FAO > 0, but I do
see downside.
Most extreme results:
Order 1: 2% loss on recompile. 10% win 4% loss on seq. 9% loss random.
Order 3: 2% loss on recompile. 6% win 5% loss on seq. 14% loss on random.
Order 4: 2.8% loss on recompile. 10% win 7% loss on seq. 9% loss on random.
> I'm starting to suspect that maybe workloads ought to be given a
> choice in this matter, via madvise() or such.
I really don't think they'll be able to use it; it'll change far too
much with machine and kernel updates. I think we should apply patch #1
(with fixes) to make it a variable, then set it to 0 for PPC.
Cheers,
Rusty.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
WARNING: multiple messages have this Message-ID (diff)
From: Rusty Russell <rusty@rustcorp.com.au>
To: Ingo Molnar <mingo@kernel.org>,
Madhavan Srinivasan <maddy@linux.vnet.ibm.com>
Cc: linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,
linux-mm@kvack.org, linux-arch@vger.kernel.org, x86@kernel.org,
benh@kernel.crashing.org, paulus@samba.org,
kirill.shutemov@linux.intel.com, akpm@linux-foundation.org,
riel@redhat.com, mgorman@suse.de, ak@linux.intel.com,
peterz@infradead.org, dave.hansen@intel.com,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [PATCH V3 2/2] powerpc/pseries: init fault_around_order for pseries
Date: Wed, 30 Apr 2014 16:34:11 +0930 [thread overview]
Message-ID: <87d2fz47tg.fsf@rustcorp.com.au> (raw)
Message-ID: <20140430070411.ApcZsuIawv1HwH8TKfryIzHROcBLX0j37fLAm3i7mAo@z> (raw)
In-Reply-To: <20140429070632.GB27951@gmail.com>
Ingo Molnar <mingo@kernel.org> writes:
> * Madhavan Srinivasan <maddy@linux.vnet.ibm.com> wrote:
>
>> Performance data for different FAULT_AROUND_ORDER values from 4 socket
>> Power7 system (128 Threads and 128GB memory). perf stat with repeat of 5
>> is used to get the stddev values. Test ran in v3.14 kernel (Baseline) and
>> v3.15-rc1 for different fault around order values.
>>
>> FAULT_AROUND_ORDER Baseline 1 3 4 5 8
>>
>> Linux build (make -j64)
>> minor-faults 47,437,359 35,279,286 25,425,347 23,461,275 22,002,189 21,435,836
>> times in seconds 347.302528420 344.061588460 340.974022391 348.193508116 348.673900158 350.986543618
>> stddev for time ( +- 1.50% ) ( +- 0.73% ) ( +- 1.13% ) ( +- 1.01% ) ( +- 1.89% ) ( +- 1.55% )
>> %chg time to baseline -0.9% -1.8% 0.2% 0.39% 1.06%
>
> Probably too noisy.
A little, but 3 still looks like the winner.
>> Linux rebuild (make -j64)
>> minor-faults 941,552 718,319 486,625 440,124 410,510 397,416
>> times in seconds 30.569834718 31.219637539 31.319370649 31.434285472 31.972367174 31.443043580
>> stddev for time ( +- 1.07% ) ( +- 0.13% ) ( +- 0.43% ) ( +- 0.18% ) ( +- 0.95% ) ( +- 0.58% )
>> %chg time to baseline 2.1% 2.4% 2.8% 4.58% 2.85%
>
> Here it looks like a speedup. Optimal value: 5+.
No, lower time is better. Baseline (no faultaround) wins.
etc.
It's not a huge surprise that a 64k page arch wants a smaller value than
a 4k system. But I agree: I don't see much upside for FAO > 0, but I do
see downside.
Most extreme results:
Order 1: 2% loss on recompile. 10% win 4% loss on seq. 9% loss random.
Order 3: 2% loss on recompile. 6% win 5% loss on seq. 14% loss on random.
Order 4: 2.8% loss on recompile. 10% win 7% loss on seq. 9% loss on random.
> I'm starting to suspect that maybe workloads ought to be given a
> choice in this matter, via madvise() or such.
I really don't think they'll be able to use it; it'll change far too
much with machine and kernel updates. I think we should apply patch #1
(with fixes) to make it a variable, then set it to 0 for PPC.
Cheers,
Rusty.
WARNING: multiple messages have this Message-ID (diff)
From: Rusty Russell <rusty@rustcorp.com.au>
To: Ingo Molnar <mingo@kernel.org>,
Madhavan Srinivasan <maddy@linux.vnet.ibm.com>
Cc: linux-arch@vger.kernel.org, riel@redhat.com, ak@linux.intel.com,
dave.hansen@intel.com, peterz@infradead.org, x86@kernel.org,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
paulus@samba.org, mgorman@suse.de,
Linus Torvalds <torvalds@linux-foundation.org>,
akpm@linux-foundation.org, linuxppc-dev@lists.ozlabs.org,
kirill.shutemov@linux.intel.com
Subject: Re: [PATCH V3 2/2] powerpc/pseries: init fault_around_order for pseries
Date: Wed, 30 Apr 2014 16:34:11 +0930 [thread overview]
Message-ID: <87d2fz47tg.fsf@rustcorp.com.au> (raw)
In-Reply-To: <20140429070632.GB27951@gmail.com>
Ingo Molnar <mingo@kernel.org> writes:
> * Madhavan Srinivasan <maddy@linux.vnet.ibm.com> wrote:
>
>> Performance data for different FAULT_AROUND_ORDER values from 4 socket
>> Power7 system (128 Threads and 128GB memory). perf stat with repeat of 5
>> is used to get the stddev values. Test ran in v3.14 kernel (Baseline) and
>> v3.15-rc1 for different fault around order values.
>>
>> FAULT_AROUND_ORDER Baseline 1 3 4 5 8
>>
>> Linux build (make -j64)
>> minor-faults 47,437,359 35,279,286 25,425,347 23,461,275 22,002,189 21,435,836
>> times in seconds 347.302528420 344.061588460 340.974022391 348.193508116 348.673900158 350.986543618
>> stddev for time ( +- 1.50% ) ( +- 0.73% ) ( +- 1.13% ) ( +- 1.01% ) ( +- 1.89% ) ( +- 1.55% )
>> %chg time to baseline -0.9% -1.8% 0.2% 0.39% 1.06%
>
> Probably too noisy.
A little, but 3 still looks like the winner.
>> Linux rebuild (make -j64)
>> minor-faults 941,552 718,319 486,625 440,124 410,510 397,416
>> times in seconds 30.569834718 31.219637539 31.319370649 31.434285472 31.972367174 31.443043580
>> stddev for time ( +- 1.07% ) ( +- 0.13% ) ( +- 0.43% ) ( +- 0.18% ) ( +- 0.95% ) ( +- 0.58% )
>> %chg time to baseline 2.1% 2.4% 2.8% 4.58% 2.85%
>
> Here it looks like a speedup. Optimal value: 5+.
No, lower time is better. Baseline (no faultaround) wins.
etc.
It's not a huge surprise that a 64k page arch wants a smaller value than
a 4k system. But I agree: I don't see much upside for FAO > 0, but I do
see downside.
Most extreme results:
Order 1: 2% loss on recompile. 10% win 4% loss on seq. 9% loss random.
Order 3: 2% loss on recompile. 6% win 5% loss on seq. 14% loss on random.
Order 4: 2.8% loss on recompile. 10% win 7% loss on seq. 9% loss on random.
> I'm starting to suspect that maybe workloads ought to be given a
> choice in this matter, via madvise() or such.
I really don't think they'll be able to use it; it'll change far too
much with machine and kernel updates. I think we should apply patch #1
(with fixes) to make it a variable, then set it to 0 for PPC.
Cheers,
Rusty.
next prev parent reply other threads:[~2014-04-30 7:04 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-04-28 9:01 [PATCH V3 0/2] mm: FAULT_AROUND_ORDER patchset performance data for powerpc Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-28 9:01 ` [PATCH V3 1/2] mm: move FAULT_AROUND_ORDER to arch/ Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-28 9:06 ` Peter Zijlstra
2014-04-28 9:06 ` Peter Zijlstra
2014-04-28 9:06 ` Peter Zijlstra
2014-04-28 9:06 ` Peter Zijlstra
2014-04-29 9:33 ` Madhavan Srinivasan
2014-04-29 9:33 ` Madhavan Srinivasan
2014-04-29 9:33 ` Madhavan Srinivasan
2014-04-28 9:36 ` Kirill A. Shutemov
2014-04-28 9:36 ` Kirill A. Shutemov
2014-04-28 9:36 ` Kirill A. Shutemov
2014-04-28 9:36 ` Kirill A. Shutemov
2014-04-28 9:36 ` Kirill A. Shutemov
2014-04-29 9:35 ` Madhavan Srinivasan
2014-04-29 9:35 ` Madhavan Srinivasan
2014-04-29 9:35 ` Madhavan Srinivasan
2014-04-28 9:01 ` [PATCH V3 2/2] powerpc/pseries: init fault_around_order for pseries Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-28 9:01 ` Madhavan Srinivasan
2014-04-29 2:18 ` Rusty Russell
2014-04-29 2:18 ` Rusty Russell
2014-04-29 2:18 ` Rusty Russell
2014-04-29 2:18 ` Rusty Russell
2014-04-29 2:18 ` Rusty Russell
2014-04-29 9:36 ` Madhavan Srinivasan
2014-04-29 9:36 ` Madhavan Srinivasan
2014-04-29 9:36 ` Madhavan Srinivasan
2014-04-29 7:06 ` Ingo Molnar
2014-04-29 7:06 ` Ingo Molnar
2014-04-29 7:06 ` Ingo Molnar
2014-04-29 10:35 ` Madhavan Srinivasan
2014-04-29 10:35 ` Madhavan Srinivasan
2014-04-29 10:35 ` Madhavan Srinivasan
2014-04-30 7:04 ` Rusty Russell [this message]
2014-04-30 7:04 ` Rusty Russell
2014-04-30 7:04 ` Rusty Russell
2014-04-30 8:15 ` Madhavan Srinivasan
2014-04-30 8:15 ` Madhavan Srinivasan
2014-04-30 8:15 ` Madhavan Srinivasan
2014-05-06 11:29 ` Ingo Molnar
2014-05-06 11:29 ` Ingo Molnar
2014-05-06 11:29 ` Ingo Molnar
2014-05-06 11:29 ` Ingo Molnar
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=87d2fz47tg.fsf@rustcorp.com.au \
--to=rusty@rustcorp.com.au \
--cc=ak@linux.intel.com \
--cc=akpm@linux-foundation.org \
--cc=benh@kernel.crashing.org \
--cc=dave.hansen@intel.com \
--cc=kirill.shutemov@linux.intel.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=maddy@linux.vnet.ibm.com \
--cc=mgorman@suse.de \
--cc=mingo@kernel.org \
--cc=paulus@samba.org \
--cc=peterz@infradead.org \
--cc=riel@redhat.com \
--cc=torvalds@linux-foundation.org \
--cc=x86@kernel.org \
/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.