Chao Yu e2374015f2 f2fs: fix to propagate return value of scan_nat_page() ... As Anatoly Trosinenko reported in bugzilla: How to reproduce: 1. Compile the 73fcb1a370 version of the kernel using the config attached 2. Unpack and mount the attached filesystem image as F2FS 3. The kernel will BUG() on mount (BUGs are explicitly enabled in config) [ 2.233612] F2FS-fs (sda): Found nat_bits in checkpoint [ 2.248422] ------------[ cut here ]------------ [ 2.248857] kernel BUG at fs/f2fs/node.c:1967! [ 2.249760] invalid opcode: 0000 [#1] SMP NOPTI [ 2.250219] Modules linked in: [ 2.251848] CPU: 0 PID: 944 Comm: mount Not tainted 4.17.0-rc5+ #1 [ 2.252331] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.10.2-1ubuntu1 04/01/2014 [ 2.253305] RIP: 0010:build_free_nids+0x337/0x3f0 [ 2.253672] RSP: 0018:ffffae7fc0857c50 EFLAGS: 00000246 [ 2.254080] RAX: 00000000ffffffff RBX: 0000000000000123 RCX: 0000000000000001 [ 2.254638] RDX: ffff9aa7063d5c00 RSI: 0000000000000122 RDI: ffff9aa705852e00 [ 2.255190] RBP: ffff9aa705852e00 R08: 0000000000000001 R09: ffff9aa7059090c0 [ 2.255719] R10: 0000000000000000 R11: 0000000000000000 R12: ffff9aa705852e00 [ 2.256242] R13: ffff9aa7063ad000 R14: ffff9aa705919000 R15: 0000000000000123 [ 2.256809] FS: 00000000023078c0(0000) GS:ffff9aa707800000(0000) knlGS:0000000000000000 [ 2.258654] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 2.259153] CR2: 00000000005511ae CR3: 0000000005872000 CR4: 00000000000006f0 [ 2.259801] Call Trace: [ 2.260583] build_node_manager+0x5cd/0x600 [ 2.260963] f2fs_fill_super+0x66a/0x17c0 [ 2.261300] ? f2fs_commit_super+0xe0/0xe0 [ 2.261622] mount_bdev+0x16e/0x1a0 [ 2.261899] mount_fs+0x30/0x150 [ 2.262398] vfs_kern_mount.part.28+0x4f/0xf0 [ 2.262743] do_mount+0x5d0/0xc60 [ 2.263010] ? _copy_from_user+0x37/0x60 [ 2.263313] ? memdup_user+0x39/0x60 [ 2.263692] ksys_mount+0x7b/0xd0 [ 2.263960] __x64_sys_mount+0x1c/0x20 [ 2.264268] do_syscall_64+0x43/0xf0 [ 2.264560] entry_SYSCALL_64_after_hwframe+0x44/0xa9 [ 2.265095] RIP: 0033:0x48d31a [ 2.265502] RSP: 002b:00007ffc6fe60a08 EFLAGS: 00000246 ORIG_RAX: 00000000000000a5 [ 2.266089] RAX: ffffffffffffffda RBX: 0000000000008000 RCX: 000000000048d31a [ 2.266607] RDX: 00007ffc6fe62fa5 RSI: 00007ffc6fe62f9d RDI: 00007ffc6fe62f94 [ 2.267130] RBP: 00000000023078a0 R08: 0000000000000000 R09: 0000000000000000 [ 2.267670] R10: 0000000000008000 R11: 0000000000000246 R12: 0000000000000000 [ 2.268192] R13: 0000000000000000 R14: 00007ffc6fe60c78 R15: 0000000000000000 [ 2.268767] Code: e8 5f c3 ff ff 83 c3 01 41 83 c7 01 81 fb c7 01 00 00 74 48 44 39 7d 04 76 42 48 63 c3 48 8d 04 c0 41 8b 44 06 05 83 f8 ff 75 c1 <0f> 0b 49 8b 45 50 48 8d b8 b0 00 00 00 e8 37 59 69 00 b9 01 00 [ 2.270434] RIP: build_free_nids+0x337/0x3f0 RSP: ffffae7fc0857c50 [ 2.271426] ---[ end trace ab20c06cd3c8fde4 ]--- During loading NAT entries, we will do sanity check, once the entry info is corrupted, it will cause BUG_ON directly to protect user data from being overwrited. In this case, it will be better to just return failure on mount() instead of panic, so that user can get hint from kmsg and try fsck for recovery immediately rather than after an abnormal reboot. https://bugzilla.kernel.org/show_bug.cgi?id=199769 Reported-by: Anatoly Trosinenko <anatoly.trosinenko@gmail.com> Signed-off-by: Chao Yu <yuchao0@huawei.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>		2018-07-27 18:03:59 +09:00
..
acl.c	posix_acl: convert posix_acl.a_refcount from atomic_t to refcount_t	2018-01-02 19:27:28 -08:00
acl.h	f2fs: remove dead code f2fs_check_acl	2016-09-14 16:52:36 -07:00
checkpoint.c	f2fs: don't acquire orphan ino during recovery	2018-07-27 18:03:59 +09:00
data.c	f2fs: allow wrong configured dio to buffered write	2018-07-15 12:14:04 +09:00
debug.c	f2fs: avoid stucking GC due to atomic write	2018-05-31 11:31:51 -07:00
dir.c	f2fs: clean up symbol namespace	2018-05-31 11:31:53 -07:00
extent_cache.c	f2fs: clean up symbol namespace	2018-05-31 11:31:53 -07:00
f2fs.h	f2fs: fix to propagate return value of scan_nat_page()	2018-07-27 18:03:59 +09:00
file.c	f2fs: indicate shutdown f2fs to allow unmount successfully	2018-07-27 18:03:56 +09:00
gc.c	f2fs: let sync node IO interrupt async one	2018-06-04 14:33:20 -07:00
gc.h	f2fs: introduce sbi->gc_mode to determine the policy	2018-05-31 11:31:51 -07:00
hash.c	f2fs: check entire encrypted bigname when finding a dentry	2017-05-04 11:44:35 -04:00
inline.c	f2fs: clean up symbol namespace	2018-05-31 11:31:53 -07:00
inode.c	f2fs: support in-memory inode checksum when checking consistency	2018-07-27 18:03:59 +09:00
Kconfig	fs/*/Kconfig: drop links to 404-compliant http://acl.bestbits.at	2018-01-01 12:45:37 -07:00
Makefile	License cleanup: add SPDX GPL-2.0 license identifier to files with no license	2017-11-02 11:10:55 +01:00
namei.c	vfs/y2038: inode timestamps conversion to timespec64	2018-06-15 07:31:07 +09:00
node.c	f2fs: fix to propagate return value of scan_nat_page()	2018-07-27 18:03:59 +09:00
node.h	f2fs: support in-memory inode checksum when checking consistency	2018-07-27 18:03:59 +09:00
recovery.c	f2fs: clean up symbol namespace	2018-05-31 11:31:53 -07:00
segment.c	treewide: Use array_size in f2fs_kvzalloc()	2018-06-12 16:19:22 -07:00
segment.h	f2fs: fix to update mtime correctly	2018-06-04 14:31:11 -07:00
shrinker.c	f2fs: clean up symbol namespace	2018-05-31 11:31:53 -07:00
super.c	f2fs: fix error path of fill_super	2018-07-27 18:03:59 +09:00
sysfs.c	f2fs: avoid potential deadlock in f2fs_sbi_store	2018-07-27 18:03:59 +09:00
trace.c	f2fs: fix potential hangtask in f2fs_trace_pid	2018-01-02 19:27:30 -08:00
trace.h	f2fs: add sbi and page pointer in f2fs_io_info	2015-05-28 15:41:32 -07:00
xattr.c	f2fs: clean up symbol namespace	2018-05-31 11:31:53 -07:00
xattr.h	f2fs: guard macro variables with braces	2017-04-10 19:48:10 -07:00