All of lore.kernel.org
 help / color / mirror / Atom feed
* NILFS crash on 2.6.37
@ 2011-01-21  2:49 Wakko Warner
  2011-01-21  7:37   ` Ryusuke Konishi
  0 siblings, 1 reply; 10+ messages in thread
From: Wakko Warner @ 2011-01-21  2:49 UTC (permalink / raw)
  To: linux-kernel

At one time I had created a filesystem with nilfs on it.  I don't know if
it's nilfs or nilfs2.  Attempting to mount it with nilfs2 on 2.6.37 (also
happens on 2.6.36.2), the kernel oopses.  Once the kernel oopses, any
further attempts to mount it hangs the process.  I have also noticed that
attempting to umount other filesystems also hangs the process.

[ 1202.579525] NILFS version 2 loaded
[ 1202.581501] NILFS warning: broken superblock. using spare superblock.
[ 1202.582093] NILFS warning: broken superblock. using spare superblock.
[ 1202.821896] segctord starting. Construction interval = 5 seconds, CP frequency < 30 seconds
[ 1202.824419] BUG: unable to handle kernel NULL pointer dereference at   (null)
[ 1202.824561] IP: [<c03196bc>] memcpy+0xc/0x1b
[ 1202.824667] *pdpt = 0000000033b94001 *pde = 0000000000000000 
[ 1202.824797] Oops: 0002 [#1] SMP 
[ 1202.824922] last sysfs file: /sys/devices/virtual/block/dm-9/dm/name
[ 1202.824978] Modules linked in: nilfs2 crc32 sha256_generic nfsd exportfs ipv6 nfs lockd nfs_acl auth_rpcgss sunrpc ext2 8250_pnp sr_mod cdrom ata_piix e1000 floppy aic7xxx libata sg 8250 evdev serial_core button processor thermal_sys
[ 1202.825136] 
[ 1202.825136] Pid: 3529, comm: mount.nilfs2 Not tainted 2.6.37 #1 /
[ 1202.825136] EIP: 0060:[<c03196bc>] EFLAGS: 00010202 CPU: 3
[ 1202.825136] EIP is at memcpy+0xc/0x1b
[ 1202.825136] EAX: 00000000 EBX: 00000100 ECX: 00000040 EDX: f7b55400
[ 1202.825136] ESI: f7b55400 EDI: 00000000 EBP: f3f45b20 ESP: f3b35e74
[ 1202.825136]  DS: 007b ES: 007b FS: 00d8 GS: 0033 SS: 0068
[ 1202.825136] Process mount.nilfs2 (pid: 3529, ti=f3b34000 task=f3870d40 task.ti=f3b34000)
[ 1202.825136] Stack:
[ 1202.825136]  f3d77cc0 f3f45b00 f7b55400 f953720e f51f0c38 f95369e9 00000000 00000000
[ 1202.825136]  f3f45b08 00000000 f9537a08 00000001 f3b35ee8 f5006584 00000002 00000003
[ 1202.825136]  f51f0c00 f4c23800 f3d77cc0 f3f45b00 00000000 392d6d64 00000000 f50dbde0
[ 1202.825136] Call Trace:
[ 1202.825136]  [<f953720e>] ? nilfs_setup_super+0x6c/0xa5 [nilfs2]
[ 1202.825136]  [<f95369e9>] ? nilfs_get_root_dentry+0x81/0xcb [nilfs2]
[ 1202.825136]  [<f9537a08>] ? nilfs_mount+0x4f9/0x62c [nilfs2]
[ 1202.825136]  [<c02745cf>] ? kstrdup+0x36/0x3f
[ 1202.825136]  [<f953750f>] ? nilfs_mount+0x0/0x62c [nilfs2]
[ 1202.825136]  [<c0293940>] ? vfs_kern_mount+0x4d/0x12c
[ 1202.825136]  [<c02a5100>] ? get_fs_type+0x76/0x8f
[ 1202.825136]  [<c0293a68>] ? do_kern_mount+0x33/0xbf
[ 1202.825136]  [<c02a784a>] ? do_mount+0x2ed/0x714
[ 1202.825136]  [<c02a6171>] ? copy_mount_options+0x28/0xfc
[ 1202.825136]  [<c02a7ce3>] ? sys_mount+0x72/0xaf
[ 1202.825136]  [<c0473085>] ? syscall_call+0x7/0xb
[ 1202.825136] Code: 43 4c 04 83 fe 01 19 c0 24 fc 04 08 88 43 4d 5b 5e 5f c3 90 90 90 90 90 90 90 90 90 90 90 90 57 56 53 89 cb c1 e9 02 89 c7 89 d6 <f3> a5 89 d9 83 e1 03 74 02 f3 a4 5b 5e 5f c3 55 57 56 53 89 c5 
[ 1202.825136] EIP: [<c03196bc>] memcpy+0xc/0x1b SS:ESP 0068:f3b35e74
[ 1202.825136] CR2: 0000000000000000
[ 1202.831111] ---[ end trace e5a2d0918d598efa ]---

-- 
 Microsoft has beaten Volkswagen's world record.  Volkswagen only created 22
 million bugs.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-21  7:37   ` Ryusuke Konishi
  0 siblings, 0 replies; 10+ messages in thread
From: Ryusuke Konishi @ 2011-01-21  7:37 UTC (permalink / raw)
  To: wakko; +Cc: linux-nilfs, linux-kernel

(Cc'ed to linux-nilfs)
On Thu, 20 Jan 2011 21:49:18 -0500, Wakko Warner wrote:
> At one time I had created a filesystem with nilfs on it.  I don't know if
> it's nilfs or nilfs2.  Attempting to mount it with nilfs2 on 2.6.37 (also
> happens on 2.6.36.2), the kernel oopses.  Once the kernel oopses, any
> further attempts to mount it hangs the process.  I have also noticed that
> attempting to umount other filesystems also hangs the process.
> 
> [ 1202.579525] NILFS version 2 loaded
> [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> [ 1202.821896] segctord starting. Construction interval = 5 seconds, CP frequency < 30 seconds
> [ 1202.824419] BUG: unable to handle kernel NULL pointer dereference at   (null)
> [ 1202.824561] IP: [<c03196bc>] memcpy+0xc/0x1b
> [ 1202.824667] *pdpt = 0000000033b94001 *pde = 0000000000000000 
> [ 1202.824797] Oops: 0002 [#1] SMP 
<snip>
> [ 1202.825136] Call Trace:
> [ 1202.825136]  [<f953720e>] ? nilfs_setup_super+0x6c/0xa5 [nilfs2]
> [ 1202.825136]  [<f95369e9>] ? nilfs_get_root_dentry+0x81/0xcb [nilfs2]
> [ 1202.825136]  [<f9537a08>] ? nilfs_mount+0x4f9/0x62c [nilfs2]
> [ 1202.825136]  [<c02745cf>] ? kstrdup+0x36/0x3f
> [ 1202.825136]  [<f953750f>] ? nilfs_mount+0x0/0x62c [nilfs2]
> [ 1202.825136]  [<c0293940>] ? vfs_kern_mount+0x4d/0x12c
> [ 1202.825136]  [<c02a5100>] ? get_fs_type+0x76/0x8f
> [ 1202.825136]  [<c0293a68>] ? do_kern_mount+0x33/0xbf
> [ 1202.825136]  [<c02a784a>] ? do_mount+0x2ed/0x714
> [ 1202.825136]  [<c02a6171>] ? copy_mount_options+0x28/0xfc
> [ 1202.825136]  [<c02a7ce3>] ? sys_mount+0x72/0xaf
> [ 1202.825136]  [<c0473085>] ? syscall_call+0x7/0xb
> [ 1202.825136] Code: 43 4c 04 83 fe 01 19 c0 24 fc 04 08 88 43 4d 5b 5e 5f c3 90 90 90 90 90 90 90 90 90 90 90 90 57 56 53 89 cb c1 e9 02 89 c7 89 d6 <f3> a5 89 d9 83 e1 03 74 02 f3 a4 5b 5e 5f c3 55 57 56 53 89 c5 
> [ 1202.825136] EIP: [<c03196bc>] memcpy+0xc/0x1b SS:ESP 0068:f3b35e74
> [ 1202.825136] CR2: 0000000000000000
> [ 1202.831111] ---[ end trace e5a2d0918d598efa ]---

Thanks for the report.

This seems to be a regression on 2.6.36 which could happen when one of
two superblocks is broken.

I'll fix it.

Ryusuke Konishi

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-21  7:37   ` Ryusuke Konishi
  0 siblings, 0 replies; 10+ messages in thread
From: Ryusuke Konishi @ 2011-01-21  7:37 UTC (permalink / raw)
  To: wakko-WZdgCHJQUilg9hUCZPvPmw
  Cc: linux-nilfs-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA

(Cc'ed to linux-nilfs)
On Thu, 20 Jan 2011 21:49:18 -0500, Wakko Warner wrote:
> At one time I had created a filesystem with nilfs on it.  I don't know if
> it's nilfs or nilfs2.  Attempting to mount it with nilfs2 on 2.6.37 (also
> happens on 2.6.36.2), the kernel oopses.  Once the kernel oopses, any
> further attempts to mount it hangs the process.  I have also noticed that
> attempting to umount other filesystems also hangs the process.
> 
> [ 1202.579525] NILFS version 2 loaded
> [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> [ 1202.821896] segctord starting. Construction interval = 5 seconds, CP frequency < 30 seconds
> [ 1202.824419] BUG: unable to handle kernel NULL pointer dereference at   (null)
> [ 1202.824561] IP: [<c03196bc>] memcpy+0xc/0x1b
> [ 1202.824667] *pdpt = 0000000033b94001 *pde = 0000000000000000 
> [ 1202.824797] Oops: 0002 [#1] SMP 
<snip>
> [ 1202.825136] Call Trace:
> [ 1202.825136]  [<f953720e>] ? nilfs_setup_super+0x6c/0xa5 [nilfs2]
> [ 1202.825136]  [<f95369e9>] ? nilfs_get_root_dentry+0x81/0xcb [nilfs2]
> [ 1202.825136]  [<f9537a08>] ? nilfs_mount+0x4f9/0x62c [nilfs2]
> [ 1202.825136]  [<c02745cf>] ? kstrdup+0x36/0x3f
> [ 1202.825136]  [<f953750f>] ? nilfs_mount+0x0/0x62c [nilfs2]
> [ 1202.825136]  [<c0293940>] ? vfs_kern_mount+0x4d/0x12c
> [ 1202.825136]  [<c02a5100>] ? get_fs_type+0x76/0x8f
> [ 1202.825136]  [<c0293a68>] ? do_kern_mount+0x33/0xbf
> [ 1202.825136]  [<c02a784a>] ? do_mount+0x2ed/0x714
> [ 1202.825136]  [<c02a6171>] ? copy_mount_options+0x28/0xfc
> [ 1202.825136]  [<c02a7ce3>] ? sys_mount+0x72/0xaf
> [ 1202.825136]  [<c0473085>] ? syscall_call+0x7/0xb
> [ 1202.825136] Code: 43 4c 04 83 fe 01 19 c0 24 fc 04 08 88 43 4d 5b 5e 5f c3 90 90 90 90 90 90 90 90 90 90 90 90 57 56 53 89 cb c1 e9 02 89 c7 89 d6 <f3> a5 89 d9 83 e1 03 74 02 f3 a4 5b 5e 5f c3 55 57 56 53 89 c5 
> [ 1202.825136] EIP: [<c03196bc>] memcpy+0xc/0x1b SS:ESP 0068:f3b35e74
> [ 1202.825136] CR2: 0000000000000000
> [ 1202.831111] ---[ end trace e5a2d0918d598efa ]---

Thanks for the report.

This seems to be a regression on 2.6.36 which could happen when one of
two superblocks is broken.

I'll fix it.

Ryusuke Konishi
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-21  9:18     ` Ryusuke Konishi
  0 siblings, 0 replies; 10+ messages in thread
From: Ryusuke Konishi @ 2011-01-21  9:18 UTC (permalink / raw)
  To: wakko; +Cc: linux-nilfs, linux-kernel

On Fri, 21 Jan 2011 16:37:38 +0900 (JST), Ryusuke Konishi wrote:
> (Cc'ed to linux-nilfs)
> On Thu, 20 Jan 2011 21:49:18 -0500, Wakko Warner wrote:
> > At one time I had created a filesystem with nilfs on it.  I don't know if
> > it's nilfs or nilfs2.  Attempting to mount it with nilfs2 on 2.6.37 (also
> > happens on 2.6.36.2), the kernel oopses.  Once the kernel oopses, any
> > further attempts to mount it hangs the process.  I have also noticed that
> > attempting to umount other filesystems also hangs the process.
> > 
> > [ 1202.579525] NILFS version 2 loaded
> > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> > [ 1202.821896] segctord starting. Construction interval = 5 seconds, CP frequency < 30 seconds
> > [ 1202.824419] BUG: unable to handle kernel NULL pointer dereference at   (null)
> > [ 1202.824561] IP: [<c03196bc>] memcpy+0xc/0x1b
> > [ 1202.824667] *pdpt = 0000000033b94001 *pde = 0000000000000000 
> > [ 1202.824797] Oops: 0002 [#1] SMP 
> <snip>
> > [ 1202.825136] Call Trace:
> > [ 1202.825136]  [<f953720e>] ? nilfs_setup_super+0x6c/0xa5 [nilfs2]
> > [ 1202.825136]  [<f95369e9>] ? nilfs_get_root_dentry+0x81/0xcb [nilfs2]
> > [ 1202.825136]  [<f9537a08>] ? nilfs_mount+0x4f9/0x62c [nilfs2]
> > [ 1202.825136]  [<c02745cf>] ? kstrdup+0x36/0x3f
> > [ 1202.825136]  [<f953750f>] ? nilfs_mount+0x0/0x62c [nilfs2]
> > [ 1202.825136]  [<c0293940>] ? vfs_kern_mount+0x4d/0x12c
> > [ 1202.825136]  [<c02a5100>] ? get_fs_type+0x76/0x8f
> > [ 1202.825136]  [<c0293a68>] ? do_kern_mount+0x33/0xbf
> > [ 1202.825136]  [<c02a784a>] ? do_mount+0x2ed/0x714
> > [ 1202.825136]  [<c02a6171>] ? copy_mount_options+0x28/0xfc
> > [ 1202.825136]  [<c02a7ce3>] ? sys_mount+0x72/0xaf
> > [ 1202.825136]  [<c0473085>] ? syscall_call+0x7/0xb
> > [ 1202.825136] Code: 43 4c 04 83 fe 01 19 c0 24 fc 04 08 88 43 4d 5b 5e 5f c3 90 90 90 90 90 90 90 90 90 90 90 90 57 56 53 89 cb c1 e9 02 89 c7 89 d6 <f3> a5 89 d9 83 e1 03 74 02 f3 a4 5b 5e 5f c3 55 57 56 53 89 c5 
> > [ 1202.825136] EIP: [<c03196bc>] memcpy+0xc/0x1b SS:ESP 0068:f3b35e74
> > [ 1202.825136] CR2: 0000000000000000
> > [ 1202.831111] ---[ end trace e5a2d0918d598efa ]---
> 
> Thanks for the report.
> 
> This seems to be a regression on 2.6.36 which could happen when one of
> two superblocks is broken.
> 
> I'll fix it.
> 
> Ryusuke Konishi

Does the following patch fix your problem?

Ryusuke Konishi
---
diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c
index 0994f6a7..58fd707 100644
--- a/fs/nilfs2/super.c
+++ b/fs/nilfs2/super.c
@@ -704,7 +704,8 @@ skip_mount_setup:
 	sbp[0]->s_state =
 		cpu_to_le16(le16_to_cpu(sbp[0]->s_state) & ~NILFS_VALID_FS);
 	/* synchronize sbp[1] with sbp[0] */
-	memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
+	if (sbp[1])
+		memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
 	return nilfs_commit_super(sbi, NILFS_SB_COMMIT_ALL);
 }
 

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-21  9:18     ` Ryusuke Konishi
  0 siblings, 0 replies; 10+ messages in thread
From: Ryusuke Konishi @ 2011-01-21  9:18 UTC (permalink / raw)
  To: wakko-WZdgCHJQUilg9hUCZPvPmw
  Cc: linux-nilfs-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA

On Fri, 21 Jan 2011 16:37:38 +0900 (JST), Ryusuke Konishi wrote:
> (Cc'ed to linux-nilfs)
> On Thu, 20 Jan 2011 21:49:18 -0500, Wakko Warner wrote:
> > At one time I had created a filesystem with nilfs on it.  I don't know if
> > it's nilfs or nilfs2.  Attempting to mount it with nilfs2 on 2.6.37 (also
> > happens on 2.6.36.2), the kernel oopses.  Once the kernel oopses, any
> > further attempts to mount it hangs the process.  I have also noticed that
> > attempting to umount other filesystems also hangs the process.
> > 
> > [ 1202.579525] NILFS version 2 loaded
> > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> > [ 1202.821896] segctord starting. Construction interval = 5 seconds, CP frequency < 30 seconds
> > [ 1202.824419] BUG: unable to handle kernel NULL pointer dereference at   (null)
> > [ 1202.824561] IP: [<c03196bc>] memcpy+0xc/0x1b
> > [ 1202.824667] *pdpt = 0000000033b94001 *pde = 0000000000000000 
> > [ 1202.824797] Oops: 0002 [#1] SMP 
> <snip>
> > [ 1202.825136] Call Trace:
> > [ 1202.825136]  [<f953720e>] ? nilfs_setup_super+0x6c/0xa5 [nilfs2]
> > [ 1202.825136]  [<f95369e9>] ? nilfs_get_root_dentry+0x81/0xcb [nilfs2]
> > [ 1202.825136]  [<f9537a08>] ? nilfs_mount+0x4f9/0x62c [nilfs2]
> > [ 1202.825136]  [<c02745cf>] ? kstrdup+0x36/0x3f
> > [ 1202.825136]  [<f953750f>] ? nilfs_mount+0x0/0x62c [nilfs2]
> > [ 1202.825136]  [<c0293940>] ? vfs_kern_mount+0x4d/0x12c
> > [ 1202.825136]  [<c02a5100>] ? get_fs_type+0x76/0x8f
> > [ 1202.825136]  [<c0293a68>] ? do_kern_mount+0x33/0xbf
> > [ 1202.825136]  [<c02a784a>] ? do_mount+0x2ed/0x714
> > [ 1202.825136]  [<c02a6171>] ? copy_mount_options+0x28/0xfc
> > [ 1202.825136]  [<c02a7ce3>] ? sys_mount+0x72/0xaf
> > [ 1202.825136]  [<c0473085>] ? syscall_call+0x7/0xb
> > [ 1202.825136] Code: 43 4c 04 83 fe 01 19 c0 24 fc 04 08 88 43 4d 5b 5e 5f c3 90 90 90 90 90 90 90 90 90 90 90 90 57 56 53 89 cb c1 e9 02 89 c7 89 d6 <f3> a5 89 d9 83 e1 03 74 02 f3 a4 5b 5e 5f c3 55 57 56 53 89 c5 
> > [ 1202.825136] EIP: [<c03196bc>] memcpy+0xc/0x1b SS:ESP 0068:f3b35e74
> > [ 1202.825136] CR2: 0000000000000000
> > [ 1202.831111] ---[ end trace e5a2d0918d598efa ]---
> 
> Thanks for the report.
> 
> This seems to be a regression on 2.6.36 which could happen when one of
> two superblocks is broken.
> 
> I'll fix it.
> 
> Ryusuke Konishi

Does the following patch fix your problem?

Ryusuke Konishi
---
diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c
index 0994f6a7..58fd707 100644
--- a/fs/nilfs2/super.c
+++ b/fs/nilfs2/super.c
@@ -704,7 +704,8 @@ skip_mount_setup:
 	sbp[0]->s_state =
 		cpu_to_le16(le16_to_cpu(sbp[0]->s_state) & ~NILFS_VALID_FS);
 	/* synchronize sbp[1] with sbp[0] */
-	memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
+	if (sbp[1])
+		memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
 	return nilfs_commit_super(sbi, NILFS_SB_COMMIT_ALL);
 }
 
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
  2011-01-21  9:18     ` Ryusuke Konishi
  (?)
@ 2011-01-22  1:57     ` Wakko Warner
  2011-01-22 14:09         ` Ryusuke Konishi
  -1 siblings, 1 reply; 10+ messages in thread
From: Wakko Warner @ 2011-01-22  1:57 UTC (permalink / raw)
  To: Ryusuke Konishi; +Cc: linux-nilfs, linux-kernel

Ryusuke Konishi wrote:
> On Fri, 21 Jan 2011 16:37:38 +0900 (JST), Ryusuke Konishi wrote:
> > (Cc'ed to linux-nilfs)
> > > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.

Why does it say this 2 times?  Just curious.

> > Thanks for the report.
> > 
> > This seems to be a regression on 2.6.36 which could happen when one of
> > two superblocks is broken.
> > 
> > I'll fix it.
> 
> Does the following patch fix your problem?

> diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c
> index 0994f6a7..58fd707 100644
> --- a/fs/nilfs2/super.c
> +++ b/fs/nilfs2/super.c
> @@ -704,7 +704,8 @@ skip_mount_setup:
>  	sbp[0]->s_state =
>  		cpu_to_le16(le16_to_cpu(sbp[0]->s_state) & ~NILFS_VALID_FS);
>  	/* synchronize sbp[1] with sbp[0] */
> -	memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
> +	if (sbp[1])
> +		memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
>  	return nilfs_commit_super(sbi, NILFS_SB_COMMIT_ALL);
>  }

That patch fix it.  Just a note, the 2.6.37 vanilla source was about 20
lines up from 704

-- 
 Microsoft has beaten Volkswagen's world record.  Volkswagen only created 22
 million bugs.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-22 14:09         ` Ryusuke Konishi
  0 siblings, 0 replies; 10+ messages in thread
From: Ryusuke Konishi @ 2011-01-22 14:09 UTC (permalink / raw)
  To: wakko; +Cc: linux-nilfs, linux-kernel

On Fri, 21 Jan 2011 20:57:48 -0500, Wakko Warner wrote:
> Ryusuke Konishi wrote:
> > On Fri, 21 Jan 2011 16:37:38 +0900 (JST), Ryusuke Konishi wrote:
> > > (Cc'ed to linux-nilfs)
> > Does the following patch fix your problem?
> 
> > diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c
> > index 0994f6a7..58fd707 100644
> > --- a/fs/nilfs2/super.c
> > +++ b/fs/nilfs2/super.c
> > @@ -704,7 +704,8 @@ skip_mount_setup:
> >  	sbp[0]->s_state =
> >  		cpu_to_le16(le16_to_cpu(sbp[0]->s_state) & ~NILFS_VALID_FS);
> >  	/* synchronize sbp[1] with sbp[0] */
> > -	memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
> > +	if (sbp[1])
> > +		memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
> >  	return nilfs_commit_super(sbi, NILFS_SB_COMMIT_ALL);
> >  }
> 
> That patch fix it.  Just a note, the 2.6.37 vanilla source was about 20
> lines up from 704

Thanks for testing.  Will send upstream and to stable.


> > > > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > > > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> 
> Why does it say this 2 times?  Just curious.

Because each super block is read twice in different block sizes.

Well, indeed it looks confusing.  I'll try to do something.


Ryusuke Konishi

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-22 14:09         ` Ryusuke Konishi
  0 siblings, 0 replies; 10+ messages in thread
From: Ryusuke Konishi @ 2011-01-22 14:09 UTC (permalink / raw)
  To: wakko-WZdgCHJQUilg9hUCZPvPmw
  Cc: linux-nilfs-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA

On Fri, 21 Jan 2011 20:57:48 -0500, Wakko Warner wrote:
> Ryusuke Konishi wrote:
> > On Fri, 21 Jan 2011 16:37:38 +0900 (JST), Ryusuke Konishi wrote:
> > > (Cc'ed to linux-nilfs)
> > Does the following patch fix your problem?
> 
> > diff --git a/fs/nilfs2/super.c b/fs/nilfs2/super.c
> > index 0994f6a7..58fd707 100644
> > --- a/fs/nilfs2/super.c
> > +++ b/fs/nilfs2/super.c
> > @@ -704,7 +704,8 @@ skip_mount_setup:
> >  	sbp[0]->s_state =
> >  		cpu_to_le16(le16_to_cpu(sbp[0]->s_state) & ~NILFS_VALID_FS);
> >  	/* synchronize sbp[1] with sbp[0] */
> > -	memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
> > +	if (sbp[1])
> > +		memcpy(sbp[1], sbp[0], nilfs->ns_sbsize);
> >  	return nilfs_commit_super(sbi, NILFS_SB_COMMIT_ALL);
> >  }
> 
> That patch fix it.  Just a note, the 2.6.37 vanilla source was about 20
> lines up from 704

Thanks for testing.  Will send upstream and to stable.


> > > > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > > > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> 
> Why does it say this 2 times?  Just curious.

Because each super block is read twice in different block sizes.

Well, indeed it looks confusing.  I'll try to do something.


Ryusuke Konishi
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-22 16:09           ` Wakko Warner
  0 siblings, 0 replies; 10+ messages in thread
From: Wakko Warner @ 2011-01-22 16:09 UTC (permalink / raw)
  To: Ryusuke Konishi; +Cc: linux-nilfs, linux-kernel

Ryusuke Konishi wrote:
> On Fri, 21 Jan 2011 20:57:48 -0500, Wakko Warner wrote:
> > That patch fix it.  Just a note, the 2.6.37 vanilla source was about 20
> > lines up from 704
> 
> Thanks for testing.  Will send upstream and to stable.

No problem.

> > > > > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > > > > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> > 
> > Why does it say this 2 times?  Just curious.
> 
> Because each super block is read twice in different block sizes.
> 
> Well, indeed it looks confusing.  I'll try to do something.

Ah, I see.

-- 
 Microsoft has beaten Volkswagen's world record.  Volkswagen only created 22
 million bugs.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: NILFS crash on 2.6.37
@ 2011-01-22 16:09           ` Wakko Warner
  0 siblings, 0 replies; 10+ messages in thread
From: Wakko Warner @ 2011-01-22 16:09 UTC (permalink / raw)
  To: Ryusuke Konishi
  Cc: linux-nilfs-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA

Ryusuke Konishi wrote:
> On Fri, 21 Jan 2011 20:57:48 -0500, Wakko Warner wrote:
> > That patch fix it.  Just a note, the 2.6.37 vanilla source was about 20
> > lines up from 704
> 
> Thanks for testing.  Will send upstream and to stable.

No problem.

> > > > > [ 1202.581501] NILFS warning: broken superblock. using spare superblock.
> > > > > [ 1202.582093] NILFS warning: broken superblock. using spare superblock.
> > 
> > Why does it say this 2 times?  Just curious.
> 
> Because each super block is read twice in different block sizes.
> 
> Well, indeed it looks confusing.  I'll try to do something.

Ah, I see.

-- 
 Microsoft has beaten Volkswagen's world record.  Volkswagen only created 22
 million bugs.
--
To unsubscribe from this list: send the line "unsubscribe linux-nilfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2011-01-22 16:09 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-21  2:49 NILFS crash on 2.6.37 Wakko Warner
2011-01-21  7:37 ` Ryusuke Konishi
2011-01-21  7:37   ` Ryusuke Konishi
2011-01-21  9:18   ` Ryusuke Konishi
2011-01-21  9:18     ` Ryusuke Konishi
2011-01-22  1:57     ` Wakko Warner
2011-01-22 14:09       ` Ryusuke Konishi
2011-01-22 14:09         ` Ryusuke Konishi
2011-01-22 16:09         ` Wakko Warner
2011-01-22 16:09           ` Wakko Warner

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.