On 05/24/2013 04:25 AM, Gu Zheng wrote: > Hi Vahram, > I saw the same issue in the bugzilla: > https://bugzilla.kernel.org/show_bug.cgi?id=53331, > and I sent out a patch this issue, but I've get any feedback.
Sorry I missed that bug. I just realized that bugzilla.kernel.org has been sending my email to my old IBM address. I've got some catching up to do. > In fact, I think it's the right way to fix this issue, > can you help to test it? I might go with your patch for now, as it is an improvement, but I don't really like the way that lmLogShutdown is skipped altogether during a "nointegrity" unmount. If we skip it during a "nointegrity" freeze, we should also skip lmLogInit on "nointegrity" unfreeze. I'd like to keep the nointegrity logic in jfs_logmgr.c, but it would be easiest to fix in the freeze and unfreeze functions. A code cleanup may be better than the easy fix. I'll have to work on that. It also looks like freeze/unfreeze doesn't take into account that two or more mounted file systems can share the same journal. I don't know how often, if ever, this is done in practice, but it looks like it would be a problem. Two "nointegrity" mounts may have problem too. That should be easy enough to verify. Thanks, Shaggy > > Thanks, > Gu > > > On 05/24/2013 04:57 PM, Vahram Martirosyan wrote: > >> In function jfs_freeze() the log is shut down through lmLogShutdown() call. >> When the "nointegrity" mount option is enabled, the log is actually not >> initialized. As a result the freeze operation in that case brings to a >> kernel OOPS. >> >> The solution is to check if the "nointegrity" option is enabled and if it is >> not >> then shut the log down. >> >> May be this is not the best solution, but at least it fixes the OOPS. >> >> Found by Linux File System Verification project (linuxtesting.org) >> >> Signed-off-by: Vahram Martirosyan <vahram.martiros...@linuxtesting.org> >> --- >> fs/jfs/super.c | 10 ++++++---- >> 1 file changed, 6 insertions(+), 4 deletions(-) >> >> diff --git a/fs/jfs/super.c b/fs/jfs/super.c >> index a3d424d..9788970 100644 >> --- a/fs/jfs/super.c >> +++ b/fs/jfs/super.c >> @@ -615,10 +615,12 @@ static int jfs_freeze(struct super_block *sb) >> >> if (!(sb->s_flags & MS_RDONLY)) { >> txQuiesce(sb); >> - rc = lmLogShutdown(log); >> - if (rc != 0) { >> - jfs_err("lmLogShutdown failed with return code %d", rc); >> - return rc; >> + if (!log->no_integrity) { >> + rc = lmLogShutdown(log); >> + if (rc != 0) { >> + jfs_err("lmLogShutdown failed with return code >> %d", rc); >> + return rc; >> + } >> } >> rc = updateSuper(sb, FM_CLEAN); >> if (rc != 0) { > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/