linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dan Magenheimer <dan.magenheimer@oracle.com>
To: Ian Campbell <Ian.Campbell@eu.citrix.com>
Cc: Chris Mason <chris.mason@oracle.com>,
	viro@zeniv.linux.org.uk, akpm@linux-foundation.org,
	adilger.kernel@dilger.ca, tytso@mit.edu, mfasheh@suse.com,
	jlbec@evilplan.org, matthew@wil.cx, linux-btrfs@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	linux-ext4@vger.kernel.org, ocfs2-devel@oss.oracle.com,
	linux-mm@kvack.org, hch@infradead.org, ngupta@vflare.org,
	jeremy@goop.org, JBeulich@novell.com,
	Kurt Hackel <kurt.hackel@oracle.com>,
	npiggin@kernel.dk, Dave Mccracken <dave.mccracken@oracle.com>,
	riel@redhat.com, avi@redhat.com,
	Konrad Wilk <konrad.wilk@oracle.com>,
	mel@csn.ul.ie, yinghan@google.com, gthelen@google.com,
	torvalds@linux-foundation.org
Subject: RE: [PATCH] xen: cleancache shim to Xen Transcendent Memory
Date: Mon, 18 Apr 2011 07:12:46 -0700 (PDT)	[thread overview]
Message-ID: <276f7410-ff4d-4a3b-ab9c-fd1b5fe8c952@default> (raw)
In-Reply-To: <1303116441.5997.107.camel@zakaz.uk.xensource.com>

> From: Ian Campbell [mailto:Ian.Campbell@eu.citrix.com]
> > >
> > > On Thu, 2011-04-14 at 14:20 -0700, Dan Magenheimer wrote:
> > >
> > > There's no need to build this into a kernel which doesn't have
> > > cleancache (or one of the other frontends), is there? I think there
> > > should be a Kconfig option (even if its not a user visible one)
> with
> > > the appropriate depends/selects.
> >
> > Yes, you're right.  It should eventually depend on
> >
> > CONFIG_CLEANCACHE || CONFIG_FRONTSWAP
> >
> > though there's no sense merging this xen cleancache
> > shim at all unless/until Linus merges cleancache
> > (and hopefully later some evolution of frontswap).
> 
> Cleancache isn't in already? I thought I saw references to it in
> drivers/staging?

Linus said he would review it after 2.6.39-rc1 was released,
but has neither given thumbs up nor thumbs down so I'm
assuming he didn't have time and it will be reconsidered
for 2.6.40.  This latest patchset (V8) is updated in linux-next.

Yes, zcache is in driver/staging and has references to it.
I guess that proves the chicken comes before the egg...
or was it vice versa? :-)
 
> > And once cleancache (and/or frontswap) is merged,
> > there's very little reason NOT to enable one or
> > both on a Xen guest kernel.
> 
> There are software knobs to allow the host- and guest-admin to opt in
> or out as they desire though, right?

Definitely.  Both Xen and a Linux guest have runtime
options, which currently default to off.

Dan

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

  reply	other threads:[~2011-04-18 14:12 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20110414212002.GA27846@ca-server1.us.oracle.com>
2011-04-15 22:02 ` [PATCH] xen: cleancache shim to Xen Transcendent Memory Ian Campbell
2011-04-15 22:53   ` Dan Magenheimer
2011-04-18  8:47     ` Ian Campbell
2011-04-18 14:12       ` Dan Magenheimer [this message]
2011-04-18 16:03         ` Ian Campbell
2011-04-14 21:20 Dan Magenheimer

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=276f7410-ff4d-4a3b-ab9c-fd1b5fe8c952@default \
    --to=dan.magenheimer@oracle.com \
    --cc=Ian.Campbell@eu.citrix.com \
    --cc=JBeulich@novell.com \
    --cc=adilger.kernel@dilger.ca \
    --cc=akpm@linux-foundation.org \
    --cc=avi@redhat.com \
    --cc=chris.mason@oracle.com \
    --cc=dave.mccracken@oracle.com \
    --cc=gthelen@google.com \
    --cc=hch@infradead.org \
    --cc=jeremy@goop.org \
    --cc=jlbec@evilplan.org \
    --cc=konrad.wilk@oracle.com \
    --cc=kurt.hackel@oracle.com \
    --cc=linux-btrfs@vger.kernel.org \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=matthew@wil.cx \
    --cc=mel@csn.ul.ie \
    --cc=mfasheh@suse.com \
    --cc=ngupta@vflare.org \
    --cc=npiggin@kernel.dk \
    --cc=ocfs2-devel@oss.oracle.com \
    --cc=riel@redhat.com \
    --cc=torvalds@linux-foundation.org \
    --cc=tytso@mit.edu \
    --cc=viro@zeniv.linux.org.uk \
    --cc=yinghan@google.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).