linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Ben Gamari <bgamari.foss@gmail.com>
To: Minchan Kim <minchan.kim@gmail.com>
Cc: linux-mm@kvack.org
Subject: Re: [PATCH v4 0/7] f/madivse(DONTNEED) support
Date: Mon, 06 Dec 2010 20:52:52 -0500	[thread overview]
Message-ID: <87mxoi9wnf.fsf@gmail.com> (raw)
In-Reply-To: <AANLkTim71krrCcmhTTCZTzxeUDkvOdBTOkeYQu6EXt32@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1584 bytes --]

On Tue, 7 Dec 2010 09:24:54 +0900, Minchan Kim <minchan.kim@gmail.com> wrote:
> Sorry missing you in Cc.
> 
> Ben. Could you test this series?
> You don't need to apply patch all so just apply 2,3,4,5.
> This patches are based on mmotm-12-02 so if you suffered this patches
> from applying your kernel, I can help you. :)
> 
I am very sorry for my lack of responsiveness. It's the last week of the
semester so things have been a bit busy. Nevertheless, I did do some
testing on v3 of the patch last week. Unfortunately, the results weren't
so promising, although this very well could be due to problems with my
test. For both patched and unpatched rsyncs and kernels I did roughly
the following,

  $ rm -Rf $DEST
  $ cat /proc/vmstat > vmstat-pre
  $ time rsync -a $SRC $DEST
  $ cat /proc/vmstat > vmstat-post
  $ time rsync -a $SRC $DEST
  $ cat /proc/vmstat > vmstat-post-warm

Where $DEST and $SRC both reside on local a SATA drive (hdparm reports
read speeds of about 100MByte/sec). I ran this (test.sh) three times on
both a patched kernel (2.6.37-rc3) and an unpatched kernel
(2.6.37-rc3-mm1). The results can be found in the attached tarball.

Judging by the results, something is horribly wrong. The "drop" (patched
rsync) runtimes are generally 3x longer than the "nodrop" (unpatched
rsync) runtimes in the case of the patched kernel. This suggests that
rsync is doing something I did not anticipate.

I'll redo the test tonight with v4 of the patch and will
investigate the source of the performance drop as soon as the
school-related workload subsides.

Cheers,

- Ben



[-- Attachment #2: patchv3-benchmark.tar.gz --]
[-- Type: application/octet-stream, Size: 14605 bytes --]

  parent reply	other threads:[~2010-12-07  1:52 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-05 17:29 [PATCH v4 0/7] f/madivse(DONTNEED) support Minchan Kim
2010-12-05 17:29 ` [PATCH v4 1/7] Fix checkpatch's report in swap.c Minchan Kim
2010-12-06  1:47   ` Rik van Riel
2010-12-07 14:37   ` Johannes Weiner
2010-12-05 17:29 ` [PATCH v4 2/7] deactivate invalidated pages Minchan Kim
2010-12-06 14:53   ` Mel Gorman
2010-12-07 14:49   ` Johannes Weiner
2010-12-07 15:07     ` Minchan Kim
2010-12-07 15:19       ` Johannes Weiner
2010-12-07 15:26         ` Minchan Kim
2010-12-07 15:56           ` Johannes Weiner
2010-12-07 22:51             ` Minchan Kim
2010-12-08  0:56               ` KAMEZAWA Hiroyuki
2010-12-08  1:43                 ` Minchan Kim
2010-12-08  1:56                   ` KAMEZAWA Hiroyuki
2010-12-08  2:15                     ` Minchan Kim
2010-12-08  6:56                       ` Balbir Singh
2010-12-09  0:19                         ` Minchan Kim
2010-12-05 17:29 ` [PATCH v4 3/7] move memcg reclaimable page into tail of inactive list Minchan Kim
2010-12-06  0:04   ` KAMEZAWA Hiroyuki
2010-12-06  3:04   ` Rik van Riel
2010-12-07  0:17     ` Minchan Kim
2010-12-06  3:34   ` Balbir Singh
2010-12-07  0:20     ` Minchan Kim
2010-12-07 14:52   ` Johannes Weiner
2010-12-08  8:08   ` KOSAKI Motohiro
2010-12-05 17:29 ` [PATCH v4 4/7] Reclaim invalidated page ASAP Minchan Kim
2010-12-07 15:05   ` Johannes Weiner
2010-12-07 15:21     ` Minchan Kim
2010-12-08  8:04   ` KOSAKI Motohiro
2010-12-08  8:16     ` Minchan Kim
2010-12-08 13:01       ` Ben Gamari
2010-12-08 23:10         ` Minchan Kim
2010-12-13 15:31           ` Minchan Kim
2010-12-13 20:06             ` Ben Gamari
2010-12-14  2:36               ` Minchan Kim
2011-07-25  3:08                 ` Ben Gamari
2011-07-25  3:47                   ` Minchan Kim
2010-12-14  2:07             ` KAMEZAWA Hiroyuki
2010-12-14  2:34               ` Minchan Kim
2010-12-05 17:29 ` [PATCH v4 5/7] add profile information for invalidated page reclaim Minchan Kim
2010-12-06  3:24   ` Rik van Riel
2010-12-08  8:02   ` KOSAKI Motohiro
2010-12-08  8:13     ` Minchan Kim
2010-12-08  8:36       ` KOSAKI Motohiro
2010-12-05 17:29 ` [PATCH v4 6/7] Remove zap_details NULL dependency Minchan Kim
2010-12-06  3:25   ` Rik van Riel
2010-12-07  4:26   ` Hugh Dickins
2010-12-07  5:30     ` Minchan Kim
2010-12-05 17:29 ` [PATCH v4 7/7] Prevent activation of page in madvise_dontneed Minchan Kim
2010-12-07  4:48   ` Hugh Dickins
2010-12-07  5:44     ` Minchan Kim
2010-12-08  7:26       ` Hugh Dickins
2010-12-08  7:55         ` Minchan Kim
     [not found] ` <AANLkTim71krrCcmhTTCZTzxeUDkvOdBTOkeYQu6EXt32@mail.gmail.com>
2010-12-07  1:52   ` Ben Gamari [this message]
2010-12-07  2:16     ` [PATCH v4 0/7] f/madivse(DONTNEED) support Minchan Kim

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=87mxoi9wnf.fsf@gmail.com \
    --to=bgamari.foss@gmail.com \
    --cc=linux-mm@kvack.org \
    --cc=minchan.kim@gmail.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;
as well as URLs for NNTP newsgroup(s).