linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Weiner <jweiner@redhat.com>
To: Dan Magenheimer <dan.magenheimer@oracle.com>
Cc: Pekka Enberg <penberg@kernel.org>,
	Cyclonus J <cyclonusj@gmail.com>,
	Sasha Levin <levinsasha928@gmail.com>,
	Christoph Hellwig <hch@infradead.org>,
	David Rientjes <rientjes@google.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	linux-mm@kvack.org, LKML <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Konrad Wilk <konrad.wilk@oracle.com>,
	Jeremy Fitzhardinge <jeremy@goop.org>,
	Seth Jennings <sjenning@linux.vnet.ibm.com>,
	ngupta@vflare.org, Chris Mason <chris.mason@oracle.com>,
	JBeulich@novell.com, Dave Hansen <dave@linux.vnet.ibm.com>,
	Jonathan Corbet <corbet@lwn.net>
Subject: Re: [GIT PULL] mm: frontswap (for 3.2 window)
Date: Sun, 30 Oct 2011 22:47:48 +0100	[thread overview]
Message-ID: <20111030214748.GB3650@redhat.com> (raw)
In-Reply-To: <b86860d2-3aac-4edd-b460-bd95cb1103e6@default>

On Fri, Oct 28, 2011 at 10:07:12AM -0700, Dan Magenheimer wrote:
> 
> > From: Johannes Weiner [mailto:jweiner@redhat.com]
> > Subject: Re: [GIT PULL] mm: frontswap (for 3.2 window)
> > 
> > On Fri, Oct 28, 2011 at 06:36:03PM +0300, Pekka Enberg wrote:
> > > On Fri, Oct 28, 2011 at 6:21 PM, Dan Magenheimer
> > > <dan.magenheimer@oracle.com> wrote:
> > > Looking at your patches, there's no trace that anyone outside your own
> > > development team even looked at the patches. Why do you feel that it's
> > > OK to ask Linus to pull them?
> > 
> > People did look at it.
> > 
> > In my case, the handwavy benefits did not convince me.  The handwavy
> > 'this is useful' from just more people of the same company does not
> > help, either.
> > 
> > I want to see a usecase that tangibly gains from this, not just more
> > marketing material.  Then we can talk about boring infrastructure and
> > adding hooks to the VM.
> > 
> > Convincing the development community of the problem you are trying to
> > solve is the undocumented part of the process you fail to follow.
> 
> Hi Johannes --
> 
> First, there are several companies and several unaffiliated kernel
> developers contributing here, building on top of frontswap.  I happen
> to be spearheading it, and my company is backing me up.  (It
> might be more appropriate to note that much of the resistance comes
> from people of your company... but please let's keep our open-source
> developer hats on and have a technical discussion rather than one
> which pleases our respective corporate overlords.)

I didn't mean to start a mud fight about this, I only mentioned the
part about your company because I already assume it sees value in tmem
- it probably wouldn't fund its development otherwise.  I just tend to
not care too much about Acks from the same company as the patch itself
and I believe other people do the same.

> Second, have you read http://lwn.net/Articles/454795/ ?
> If not, please do.  If yes, please explain what you don't
> see as convincing or tangible or documented.  All of this
> exists today as working publicly available code... it's
> not marketing material.

I remember answering this to you in private already some time ago when
discussing frontswap.

You keep proposing a bridge and I keep asking for proof that this is
not a bridge to nowhere.  Unless that question is answered, I am not
interested in discussing the bridge's design.

According to the LWN article, there are the following backends:

1. Zcache: allow swapping into compressed memory

This sets aside a portion of memory which the kernel will swap
compressed pages into upon pressure.  Now, obviously, reserving memory
from the system for this increases the pressure in the first place,
eating away on what space we have for anonymous memory and page cache.

Do you auto-size that region depending on workload?

If so, how?  If not, is it documented how to size it manually?

Where are the performance numbers for various workloads, including
both those that benefit from every bit of page cache and those that
would fit into memory without zcache occupying space?

However, looking at the zcache code, it seems it wants to allocate
storage pages only when already trying to swap out.  Are you sure this
works in reality?

2. RAMster: allow swapping between machines in a cluster

Are there people using it?  It, too, sounds like a good idea but I
don't see any proof it actually works as intended.

3. Xen: allow guests to swap into the host.

The article mentions that there is code to put the guests under
pressure and let them swap to host memory when the pressure is too
high.  This sounds useful.

Where is the code that controls the amount of pressure put on the
guests?

Where are the performance numbers?  Surely you can construct a case
where the initial machine sizes are not quite right and then collect
data that demonstrates the machines are rebalancing as expected?

4. kvm: same as Xen

Apart from the questions that already apply to Xen, I remember KVM
people in particular complaining about the synchroneous single-page
interface that results in a hypercall per swapped page.  What happened
to this concern?

---

I would really appreciate if you could pick one of those backends and
present them as a real and practical solution to real and practical
problems.  With documentation on configuration and performance data of
real workloads.  We can discuss implementation details like how memory
is exchanged between source and destination when we come to it.

I am not asking for just more code that uses your interface, I want to
know the real value for real people of the combination of all that
stuff.  With proof, not just explanations of how it's supposed to
work.

Until you can accept that, please include

	Nacked-by: Johannes Weiner <hannes@cmpxchg.org>

on all further stand-alone submissions of tmem core code and/or hooks
in the VM.  Thanks.

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  parent reply	other threads:[~2011-10-30 21:48 UTC|newest]

Thread overview: 87+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-27 18:52 [GIT PULL] mm: frontswap (for 3.2 window) Dan Magenheimer
     [not found] ` <alpine.DEB.2.00.1110271318220.7639@chino.kir.corp.google.com20111027211157.GA1199@infradead.org>
2011-10-27 19:30 ` Kurt Hackel
2011-10-27 20:18 ` David Rientjes
2011-10-27 21:11   ` Christoph Hellwig
2011-10-27 21:49     ` Dan Magenheimer
2011-10-27 21:52       ` Christoph Hellwig
2011-10-27 22:21         ` Dan Magenheimer
2011-10-28  7:12         ` Sasha Levin
     [not found]           ` <CAOzbF4fnD=CGR-nizZoBxmFSuAjFC3uAHf3wDj5RLneJvJhrOQ@mail.gmail.comCAOJsxLGOTw7rtFnqeHvzFxifA0QgPVDHZzrEo=-uB2Gkrvp=JQ@mail.gmail.com>
     [not found]             ` <552d2067-474d-4aef-a9a4-89e5fd8ef84f@default20111031181651.GF3466@redhat.com>
     [not found]               ` <60592afd-97aa-4eaf-b86b-f6695d31c7f1@default20111031223717.GI3466@redhat.com>
     [not found]                 ` <1b2e4f74-7058-4712-85a7-84198723e3ee@default20111101012017.GJ3466@redhat.com>
     [not found]                   ` <6a9db6d9-6f13-4855-b026-ba668c29ddfa@default20111101180702.GL3466@redhat.com>
     [not found]                     ` <b8a0ca71-a31b-488a-9a92-2502d4a6e9bf@default20111102013122.GA18879@redhat.com>
2011-10-28  7:30           ` Cyclonus J
2011-10-28 14:26             ` Pekka Enberg
2011-10-28 15:21               ` Dan Magenheimer
     [not found]                 ` <CAOJsxLEE-qf9me1SAZLFiEVhHVnDh7BDrSx1+abe9R4mfkhD=g@mail.gmail.com20111028163053.GC1319@redhat.com>
2011-10-28 15:36                 ` Pekka Enberg
2011-10-28 16:30                   ` Johannes Weiner
2011-10-28 17:01                     ` Pekka Enberg
2011-10-28 17:07                     ` Dan Magenheimer
2011-10-28 18:28                       ` John Stoffel
2011-10-28 20:19                         ` Dan Magenheimer
2011-10-28 20:52                           ` John Stoffel
2011-10-30 19:18                             ` Dan Magenheimer
2011-10-30 20:06                               ` Dave Hansen
2011-10-30 21:50                                 ` Dan Magenheimer
2011-11-02 19:45                                 ` Rik van Riel
2011-11-02 20:45                                   ` Dan Magenheimer
2011-11-06 22:32                             ` Valdis.Kletnieks
2011-11-08 12:15                               ` Ed Tomlinson
2011-10-31  8:12                           ` James Bottomley
2011-10-31 15:39                             ` Dan Magenheimer
2011-11-01 10:13                               ` James Bottomley
2011-11-01 18:10                                 ` Dan Magenheimer
2011-11-01 18:48                                   ` Dave Hansen
2011-11-01 21:32                                     ` Dan Magenheimer
2011-11-02  7:44                                   ` James Bottomley
2011-11-02 19:39                                     ` Dan Magenheimer
2011-10-31 18:44                         ` Andrea Arcangeli
2011-10-30 21:47                       ` Johannes Weiner [this message]
2011-10-30 23:19                         ` Dan Magenheimer
2011-10-31 18:34                       ` Andrea Arcangeli
2011-10-31 21:45                         ` Dan Magenheimer
2011-10-28 16:37                   ` Dan Magenheimer
2011-10-28 16:59                     ` Pekka Enberg
2011-10-28 17:20                       ` Dan Magenheimer
2011-10-31 18:16                 ` Andrea Arcangeli
2011-10-31 20:58                   ` Dan Magenheimer
2011-10-31 22:37                     ` Andrea Arcangeli
2011-10-31 23:36                       ` Dan Magenheimer
2011-11-01  1:20                         ` Andrea Arcangeli
2011-11-01 16:41                           ` Dan Magenheimer
2011-11-01 18:07                             ` Andrea Arcangeli
2011-11-01 21:00                               ` Dan Magenheimer
2011-11-02  1:31                                 ` Andrea Arcangeli
2011-11-02 19:06                                   ` Dan Magenheimer
2011-11-03  0:32                                     ` Andrea Arcangeli
2011-11-03 22:29                                       ` Dan Magenheimer
2011-11-02 20:51                         ` Rik van Riel
2011-11-02 21:14                           ` Dan Magenheimer
2011-11-15 16:29                             ` Rik van Riel
2011-11-15 17:33                               ` Jeremy Fitzhardinge
2011-11-16 14:49                                 ` Konrad Rzeszutek Wilk
2011-11-01 10:16                   ` James Bottomley
2011-11-01 18:21                     ` Dan Magenheimer
2011-11-02  8:14                       ` James Bottomley
2011-11-02 20:08                         ` Dan Magenheimer
2011-11-03 10:30                           ` Theodore Tso
2011-11-03 14:59                             ` Dan Magenheimer
2011-11-02 15:44                     ` Avi Kivity
2011-11-02 16:02                       ` Andrea Arcangeli
2011-11-02 16:13                         ` Avi Kivity
2011-11-02 20:27                           ` Dan Magenheimer
2011-11-02 20:19                       ` Dan Magenheimer
2011-10-27 21:44 ` Avi Miller
2011-10-27 22:33 ` Brian King
2011-10-28  5:17 ` Nitin Gupta
2011-10-29 13:43 ` Ed Tomlinson
2011-10-31  8:13 ` KAMEZAWA Hiroyuki
2011-10-31 16:38   ` Dan Magenheimer
2011-11-01  0:50     ` KAMEZAWA Hiroyuki
2011-11-01 15:25       ` Dan Magenheimer
2011-11-01 21:43         ` Andrew Morton
2011-11-01 22:25           ` Dan Magenheimer
2011-11-02 21:03           ` Rik van Riel
2011-11-02 21:42             ` Dan Magenheimer
2011-11-02  1:14         ` KAMEZAWA Hiroyuki
2011-11-02 15:12           ` Dan Magenheimer
2011-11-04  4:19             ` KAMEZAWA Hiroyuki
2011-11-03 16:49 ` Jan Beulich
2011-11-04  0:54   ` Andrew Morton
2011-11-04  8:49     ` Jan Beulich

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=20111030214748.GB3650@redhat.com \
    --to=jweiner@redhat.com \
    --cc=JBeulich@novell.com \
    --cc=akpm@linux-foundation.org \
    --cc=chris.mason@oracle.com \
    --cc=corbet@lwn.net \
    --cc=cyclonusj@gmail.com \
    --cc=dan.magenheimer@oracle.com \
    --cc=dave@linux.vnet.ibm.com \
    --cc=hch@infradead.org \
    --cc=jeremy@goop.org \
    --cc=konrad.wilk@oracle.com \
    --cc=levinsasha928@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ngupta@vflare.org \
    --cc=penberg@kernel.org \
    --cc=rientjes@google.com \
    --cc=sjenning@linux.vnet.ibm.com \
    --cc=torvalds@linux-foundation.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;
as well as URLs for NNTP newsgroup(s).