linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Balbir Singh <balbir@linux.vnet.ibm.com>
To: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Dave Hansen <dave@linux.vnet.ibm.com>,
	Nick Piggin <nickpiggin@yahoo.com.au>,
	hugh@veritas.com, menage@google.com, xemul@openvz.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [RFC][PATCH] Remove cgroup member from struct page (v3)
Date: Thu, 18 Sep 2008 16:56:17 -0700	[thread overview]
Message-ID: <48D2EAA1.1000301@linux.vnet.ibm.com> (raw)
In-Reply-To: <20080918200116.06b41fa7.kamezawa.hiroyu@jp.fujitsu.com>

KAMEZAWA Hiroyuki wrote:
> On Wed, 17 Sep 2008 21:58:08 -0700
> Balbir Singh <balbir@linux.vnet.ibm.com> wrote:
>>> BTW, I already have lazy-lru-by-pagevec protocol on my patch(hash version) and
>>> seems to work well. I'm now testing it and will post today if I'm enough lucky.
>> cool! Please do post what numbers you see as well. I would appreciate if you can
>> try this version and see what sort of performance issues you see.
>>
> 
> This is the result on 8cpu box. I think I have to reduce footprint of fastpath of
> my patch ;)
> 
> Test result of your patch is (2).
> ==
> Xeon 8cpu/2socket/1-node equips 48GB of memory.
> run shell/exec benchmark 3 times just after boot.
> 
> lps ... loops per sec.
> lpm ... loops per min.
> (*) Shell tests somtimes fail because of division by zero, etc...
> 
> (1). rc6-mm1(2008/9/13 version)
> ==
> Run                                       == 1st ==  == 2nd ==  ==3rd==
> Execl Throughput                           2425.2     2534.5     2465.8  (lps)
> C Compiler Throughput                      1438.3     1476.3     1459.1  (lpm)
> Shell Scripts (1 concurrent)               9360.3     9368.3     9360.0  (lpm)
> Shell Scripts (8 concurrent)               3868.0     3870.0     3868.0  (lpm)
> Shell Scripts (16 concurrent)              2207.0     2204.0     2201.0  (lpm)
> Dc: sqrt(2) to 99 decimal places         101644.3   102184.5   102118.5  (lpm)
> 
> (2). (1) +remove-page-cgroup-pointer-v3 (radix-tree + dynamic allocation)
> ==
> Run                                       == 1st ==  == 2nd ==  == 3rd ==
> Execl Throughput                           2514.1      2548.9    2648.7  (lps)
> C Compiler Throughput                      1353.9      1324.6    1324.7  (lpm)
> Shell Scripts (1 concurrent)               8866.7      8871.0    8856.0  (lpm)
> Shell Scripts (8 concurrent)               3674.3      3680.0    3677.7  (lpm)
> Shell Scripts (16 concurrent)              failed.     failed    2094.3  (lpm)
> Dc: sqrt(2) to 99 decimal places          98837.0     98206.9   98250.6  (lpm)
> 
> (3). (1) + pre-allocation by "vmalloc" + hash + misc(atomic flags etc..)
> ==
> Run                                       == 1st ==  == 2nd ==  == 3rd ==
> Execl Throughput                           2385.4      2579.2    2361.5  (lps)
> C Compiler Throughput                      1424.3      1436.3    1430.6  (lpm)
> Shell Scripts (1 concurrent)               9222.0      9234.0    9246.7  (lpm)
> Shell Scripts (8 concurrent)               3787.7      3799.3    failed  (lpm)
> Shell Scripts (16 concurrent)              2165.7      2166.7    failed  (lpm)
> Dc: sqrt(2) to 99 decimal places         102228.9    102658.5   104049.8 (lpm)
> 
> (4). (3) + get/put page charge/uncharge + lazy lru handling
> Run                                       == 1st ==  == 2nd ==  == 3rd ==
> Execl Throughput                           2349.4      2335.7    2338.9  (lps)
> C Compiler Throughput                      1430.8      1445.0    1435.3  (lpm)
> Shell Scripts (1 concurrent)               9250.3      9262.0    9265.0  (lpm)
> Shell Scripts (8 concurrent)               3831.0      3834.4    3833.3  (lpm)
> Shell Scripts (16 concurrent)              2193.3      2195.3    2196.0  (lpm)
> Dc: sqrt(2) to 99 decimal places         102956.8    102886.9   101884.6 (lpm)
> 
> 
> It seems "execl" test is affected by footprint and cache hit rate than other
> tests. I need some more efforts for reducing overhead in (4).
> 
> Note:
> (1)'s struct page is 64 bytes.
> (2)(3)(4)'s struct page is 56 bytes.

Thanks, Kame! I'll look at the lazy lru patches and see if I can find anything.
Do you have a unified patch anywhere, I seem to get confused with the patches, I
see 10/9, 11/9 and 12/9. I'll do some analysis when I find some free time, I am
currently at plumbers conference.

-- 
	Balbir

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

  reply	other threads:[~2008-09-18 23:55 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-31 17:47 [RFC][PATCH] Remove cgroup member from struct page Balbir Singh
2008-09-01  0:01 ` KAMEZAWA Hiroyuki
2008-09-01  3:28   ` Balbir Singh
2008-09-01  4:03     ` KAMEZAWA Hiroyuki
2008-09-01  5:17       ` KAMEZAWA Hiroyuki
2008-09-01  6:16         ` Balbir Singh
2008-09-01  6:09       ` Balbir Singh
2008-09-01  6:24         ` KAMEZAWA Hiroyuki
2008-09-01  6:25           ` Balbir Singh
2008-09-01  6:59             ` KAMEZAWA Hiroyuki
2008-09-01  6:56   ` Nick Piggin
2008-09-01  7:17     ` Balbir Singh
2008-09-01  7:19     ` KAMEZAWA Hiroyuki
2008-09-01  7:43       ` Nick Piggin
2008-09-02  9:24         ` Balbir Singh
2008-09-02 10:02           ` KAMEZAWA Hiroyuki
2008-09-02  9:58             ` Balbir Singh
2008-09-02 10:07               ` KAMEZAWA Hiroyuki
2008-09-02 10:12                 ` Balbir Singh
2008-09-02 10:57                   ` KAMEZAWA Hiroyuki
2008-09-02 12:37                     ` Balbir Singh
2008-09-03  3:33                       ` KAMEZAWA Hiroyuki
2008-09-03  7:31                         ` Balbir Singh
2008-09-08 15:28                         ` Balbir Singh
2008-09-09  3:57                           ` KAMEZAWA Hiroyuki
2008-09-09  3:58                             ` Nick Piggin
2008-09-09  4:53                               ` KAMEZAWA Hiroyuki
2008-09-09  5:00                                 ` Nick Piggin
2008-09-09  5:12                                   ` KAMEZAWA Hiroyuki
2008-09-09 12:24                                     ` Balbir Singh
2008-09-09 12:28                                       ` Nick Piggin
2008-09-09 12:30                                     ` kamezawa.hiroyu
2008-09-09 12:34                                       ` Balbir Singh
2008-09-10  1:20                                       ` [Approach #2] " Balbir Singh
2008-09-10  1:49                                         ` KAMEZAWA Hiroyuki
2008-09-10  2:11                                           ` Balbir Singh
2008-09-10  2:35                                             ` KAMEZAWA Hiroyuki
2008-09-10 20:44                                           ` Nick Piggin
2008-09-10 11:03                                             ` KAMEZAWA Hiroyuki
2008-09-10 21:02                                               ` Nick Piggin
2008-09-10 11:27                                                 ` KAMEZAWA Hiroyuki
2008-09-10 14:34                                                   ` Balbir Singh
2008-09-10 22:21                                         ` Dave Hansen
2008-09-10 22:31                                           ` David Miller, Dave Hansen
2008-09-10 22:36                                           ` Balbir Singh
2008-09-10 22:56                                             ` Dave Hansen
2008-09-11  1:35                                               ` KAMEZAWA Hiroyuki
2008-09-11  1:47                                                 ` Balbir Singh
2008-09-11  1:56                                                   ` KAMEZAWA Hiroyuki
2008-09-17 23:28                                                     ` [RFC][PATCH] Remove cgroup member from struct page (v3) Balbir Singh
2008-09-18  1:40                                                       ` Andrew Morton
2008-09-18  3:57                                                         ` Balbir Singh
2008-09-18  5:00                                                           ` KAMEZAWA Hiroyuki
2008-09-18  4:26                                                         ` Hirokazu Takahashi
2008-09-18  4:50                                                           ` KAMEZAWA Hiroyuki
2008-09-18  6:13                                                             ` Hirokazu Takahashi
2008-09-18  4:43                                                         ` KAMEZAWA Hiroyuki
2008-09-18  4:58                                                           ` Balbir Singh
2008-09-18  5:15                                                             ` KAMEZAWA Hiroyuki
2008-09-18 11:01                                                             ` KAMEZAWA Hiroyuki
2008-09-18 23:56                                                               ` Balbir Singh [this message]
2008-09-19  0:37                                                                 ` KAMEZAWA Hiroyuki
2008-09-10 22:38                                           ` [Approach #2] [RFC][PATCH] Remove cgroup member from struct page Nick Piggin
2008-09-09  4:18                             ` Balbir Singh
2008-09-09  4:55                               ` KAMEZAWA Hiroyuki
2008-09-09  7:37                               ` KAMEZAWA Hiroyuki
2008-09-01  2:39 ` KAMEZAWA Hiroyuki
2008-09-01  3:42   ` Balbir Singh
2008-09-01  9:03 ` Pavel Emelyanov
2008-09-01  9:17   ` Balbir Singh
2008-09-01  9:43     ` Pavel Emelyanov
2008-09-01 13:19     ` Peter Zijlstra
2008-09-02  7:35       ` Balbir Singh

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=48D2EAA1.1000301@linux.vnet.ibm.com \
    --to=balbir@linux.vnet.ibm.com \
    --cc=akpm@linux-foundation.org \
    --cc=dave@linux.vnet.ibm.com \
    --cc=hugh@veritas.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=menage@google.com \
    --cc=nickpiggin@yahoo.com.au \
    --cc=xemul@openvz.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox