Messages in this thread | | | Date | Thu, 17 Nov 2011 01:50:21 -0800 | From | Joel Becker <> | Subject | Re: [PATCH] ocfs2: send correct UUID to cleancache initialization |
| |
On Mon, Oct 17, 2011 at 04:47:17PM -0700, Dan Magenheimer wrote: > ocfs2: Fix cleancache initialization call to correctly pass uuid > > As reported by Steven Whitehouse in https://lkml.org/lkml/2011/5/27/221 > the ocfs2 volume UUID is incorrectly passed to cleancache. > As a result, shared-ephemeral tmem pools will not actually > be created; instead they will be private (unshared) which > misses out on a major benefit of tmem. > > Reported-by: Steven Whitehouse <swhiteho@redhat.com> > Signed-off-by: Dan Magenheimer <dan.magenheimer@oracle.com>
This patch is now in the 'fixes' branch of ocfs2.git.
Joel
> > diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c > --- a/fs/ocfs2/super.c > +++ b/fs/ocfs2/super.c > @@ -2374,7 +2374,7 @@ static int ocfs2_initialize_super(struct super_block *sb, > mlog_errno(status); > goto bail; > } > - cleancache_init_shared_fs((char *)&uuid_net_key, sb); > + cleancache_init_shared_fs((char *)&di->id2.i_super.s_uuid, sb); > > bail: > return status;
--
"Well-timed silence hath more eloquence than speech." - Martin Fraquhar Tupper
http://www.jlbec.org/ jlbec@evilplan.org
| |