All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Stancek <jstancek@redhat.com>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH] mem/lib: Keep KSM off when child_alloc() allocate memory in single thread
Date: Tue, 21 Jun 2016 04:39:50 -0400 (EDT)	[thread overview]
Message-ID: <1126821490.620692.1466498390955.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <20160621075613.GB24636@gmail.com>



----- Original Message -----
> From: "Li Wang" <liwang@redhat.com>
> To: "Jan Stancek" <jstancek@redhat.com>
> Cc: ltp@lists.linux.it
> Sent: Tuesday, 21 June, 2016 9:56:13 AM
> Subject: Re: [PATCH] mem/lib: Keep KSM off when child_alloc() allocate memory in single thread
> > What if KSM was enabled before test started?
> 
> From what I know about KSM, if a memory area affected by function
> madvise(..., MADV_MERGEABLE), the samepages located in that merory will be
> easily merged by ksmd as long as we turn it on (echo 1
> >/sys/kernel/mm/ksm/run).
> 
> And the ".../ksm/run" default value is 0, it means if we just affect
> the memory area by madvise() function but with ksmd off, there won't
> have any samepages be merged.

True, but there's no guarantee that every distro and every user will
keep the default set to 0. As I recall IBM's PowerKVM has KSM enabled
by default.

> > I'd skip this entire else block if lite == 1. With KSM off, it's nearly
> > identical to oom(NORMAL).
> 
> Hmm, actually there still have a little bit difference from
> oom(NORMALNORMAL).
> 
> In alloc_mem() function, if testcase == KSM, the memory area will do
> madvise operation like:
> 	madvise(s, length, MADV_MERGEABLE)

That's what I meant by "nearly identical". Looking at kernel,
only ksm seems to care about that flag, and we turn KSM off:

$ grep -l MADV_MERGEABLE -r mm/
mm/Kconfig
mm/madvise.c
mm/ksm.c

Regards,
Jan

  reply	other threads:[~2016-06-21  8:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-21  3:39 [LTP] [PATCH] mem/lib: Keep KSM off when child_alloc() allocate memory in single thread Li Wang
2016-06-21  7:00 ` Jan Stancek
2016-06-21  7:56   ` Li Wang
2016-06-21  8:39     ` Jan Stancek [this message]
2016-06-21  9:07       ` Li Wang

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=1126821490.620692.1466498390955.JavaMail.zimbra@redhat.com \
    --to=jstancek@redhat.com \
    --cc=ltp@lists.linux.it \
    /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.