lkml.org 
[lkml]   [2012]   [Jul]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [RFC/PATCH] zcache/ramster rewrite and promotion
On Tue, Jul 31, 2012 at 11:53:57PM +0300, Pekka Enberg wrote:
> On Tue, Jul 31, 2012 at 11:18 PM, Dan Magenheimer
> <dan.magenheimer@oracle.com> wrote:
> > diffstat vs 3.5:
> > drivers/staging/ramster/Kconfig | 2
> > drivers/staging/ramster/Makefile | 2
> > drivers/staging/zcache/Kconfig | 2
> > drivers/staging/zcache/Makefile | 2
> > mm/Kconfig | 2
> > mm/Makefile | 4
> > mm/tmem/Kconfig | 33
> > mm/tmem/Makefile | 5
> > mm/tmem/tmem.c | 894 +++++++++++++
> > mm/tmem/tmem.h | 259 +++
> > mm/tmem/zbud.c | 1060 +++++++++++++++
> > mm/tmem/zbud.h | 33
> > mm/tmem/zcache-main.c | 1686 +++++++++++++++++++++++++
> > mm/tmem/zcache.h | 53
> > mm/tmem/ramster.h | 59
> > mm/tmem/ramster/heartbeat.c | 462 ++++++
> > mm/tmem/ramster/heartbeat.h | 87 +
> > mm/tmem/ramster/masklog.c | 155 ++
> > mm/tmem/ramster/masklog.h | 220 +++
> > mm/tmem/ramster/nodemanager.c | 995 +++++++++++++++
> > mm/tmem/ramster/nodemanager.h | 88 +
> > mm/tmem/ramster/r2net.c | 414 ++++++
> > mm/tmem/ramster/ramster.c | 985 ++++++++++++++
> > mm/tmem/ramster/ramster.h | 161 ++
> > mm/tmem/ramster/ramster_nodemanager.h | 39
> > mm/tmem/ramster/tcp.c | 2253 ++++++++++++++++++++++++++++++++++
> > mm/tmem/ramster/tcp.h | 159 ++
> > mm/tmem/ramster/tcp_internal.h | 248 +++
> > 28 files changed, 10358 insertions(+), 4 deletions(-)
>
> So it's basically this commit, right?

Yeah, one big RFC patch.
>
> https://oss.oracle.com/git/djm/tmem.git/?p=djm/tmem.git;a=commitdiff;h=22844fe3f52d912247212408294be330a867937c
>
> Why on earth would you want to move that under the mm directory?

If you take aside that problem that it is one big patch instead
of being split up in more reasonable pieces - would you recommend
that it reside in a different directory?

Or is that it does not make sense b/c it has other components in it - such
as tcp/nodemaneger/hearbeat/etc so it should go under the refactor knife?

And if you rip out the ramster from this and just concentrate on zcache -
should that go in drivers/mm or mm/tmem/zcache?

>
> Pekka


\
 
 \ /
  Last update: 2012-07-31 23:41    [W:0.100 / U:0.180 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site