From: Gabriel Krisman Bertazi <krisman@suse.de>
To: Eugen Hristev <eugen.hristev@collabora.com>
Cc: tytso@mit.edu, adilger.kernel@dilger.ca,
linux-ext4@vger.kernel.org, jaegeuk@kernel.org,
chao@kernel.org, linux-f2fs-devel@lists.sourceforge.net,
linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
kernel@collabora.com, viro@zeniv.linux.org.uk,
brauner@kernel.org, jack@suse.cz,
Gabriel Krisman Bertazi <krisman@collabora.com>
Subject: Re: [PATCH v13 2/9] f2fs: Simplify the handling of cached insensitive names
Date: Wed, 13 Mar 2024 19:36:25 -0400 [thread overview]
Message-ID: <87edcdk8li.fsf@mailhost.krisman.be> (raw)
In-Reply-To: <20240305101608.67943-3-eugen.hristev@collabora.com> (Eugen Hristev's message of "Tue, 5 Mar 2024 12:16:01 +0200")
Eugen Hristev <eugen.hristev@collabora.com> writes:
> +void f2fs_free_casefolded_name(struct f2fs_filename *fname)
> +{
> + unsigned char *buf = (unsigned char *)fname->cf_name.name;
> +
> + kmem_cache_free(f2fs_cf_name_slab, buf);
> + fname->cf_name.name = NULL;
In my previous review, I mentioned you could drop the "if (buf)" check
here *if and only if* you used kfree. By doing an unchecked kmem_cache_free
like this, you will immediately hit an Oops in the first lookup (see below).
Please, make sure you actually stress test this patchset with fstests
against both f2fs and ext4 before sending each new version.
Thanks,
[ 74.202044] F2FS-fs (loop0): Using encoding defined by superblock: utf8-12.1.0 with flags 0x0
[ 74.206592] F2FS-fs (loop0): Found nat_bits in checkpoint
[ 74.221467] F2FS-fs (loop0): Mounted with checkpoint version = 3e684111
FSTYP -- f2fs
PLATFORM -- Linux/x86_64 sle15sp5 6.7.0-gf27274eae416 #8 SMP PREEMPT_DYNAMIC Thu Mar 14 00:22:47 CET 2024
MKFS_OPTIONS -- -O encrypt /dev/loop1
MOUNT_OPTIONS -- -o acl,user_xattr /dev/loop1 /root/work/scratch
[ 75.038385] F2FS-fs (loop1): Found nat_bits in checkpoint
[ 75.054311] F2FS-fs (loop1): Mounted with checkpoint version = 6b9fbccb
[ 75.176328] F2FS-fs (loop0): Using encoding defined by superblock: utf8-12.1.0 with flags 0x0
[ 75.179261] F2FS-fs (loop0): Found nat_bits in checkpoint
[ 75.194264] F2FS-fs (loop0): Mounted with checkpoint version = 3e684114
f2fs/001 1s ... [ 75.570867] run fstests f2fs/001 at 2024-03-14 00:24:33
[ 75.753604] BUG: unable to handle page fault for address: fffff14ad2000008
[ 75.754209] #PF: supervisor read access in kernel mode
[ 75.754647] #PF: error_code(0x0000) - not-present page
[ 75.755077] PGD 0 P4D 0
[ 75.755300] Oops: 0000 [#1] PREEMPT SMP NOPTI
[ 75.755683] CPU: 0 PID: 2740 Comm: xfs_io Not tainted 6.7.0-gf27274eae416 #8
[ 75.756266] Hardware name: QEMU Standard PC (Q35 + ICH9, 2009), BIOS unknown 2/2/2022
[ 75.756911] RIP: 0010:kmem_cache_free+0x6a/0x320
[ 75.757309] Code: 80 48 01 d8 0f 82 b4 02 00 00 48 c7 c2 00 00 00 80 48 2b 15 f8 c2 18 01 48 01 d0 48 c1 e8 0c 48 c1 e0 06 48 03 05 d6 c2 18 01 <48> 8b 50 08 49 89 c6 f6 c2 01 0f 85 ea 01 00 00 0f 1f 44 00 00 49
[ 75.758834] RSP: 0018:ffffa59bc231bb10 EFLAGS: 00010286
[ 75.759270] RAX: fffff14ad2000000 RBX: 0000000000000000 RCX: 0000000000000000
[ 75.759860] RDX: 0000620400000000 RSI: 0000000000000000 RDI: ffff9dfc80043600
[ 75.760450] RBP: ffffa59bc231bb30 R08: ffffa59bc231b9a0 R09: 00000000000003fa
[ 75.761037] R10: 00000000000fd024 R11: 0000000000000107 R12: ffff9dfc80043600
[ 75.761626] R13: ffffffff8404dc7a R14: 0000000000000000 R15: ffff9dfc8f1aa000
[ 75.762221] FS: 00007f9601efb780(0000) GS:ffff9dfcfbc00000(0000) knlGS:0000000000000000
[ 75.762888] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 75.763372] CR2: fffff14ad2000008 CR3: 0000000111750000 CR4: 0000000000750ef0
[ 75.763962] PKRU: 55555554
[ 75.764194] Call Trace:
[ 75.764435] <TASK>
[ 75.764677] ? __die_body+0x1a/0x60
[ 75.764982] ? page_fault_oops+0x154/0x440
[ 75.765335] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.765760] ? search_module_extables+0x46/0x70
[ 75.766149] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.766548] ? fixup_exception+0x22/0x300
[ 75.766892] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.767292] ? exc_page_fault+0xa6/0x140
[ 75.767633] ? asm_exc_page_fault+0x22/0x30
[ 75.767995] ? f2fs_free_filename+0x2a/0x40
[ 75.768362] ? kmem_cache_free+0x6a/0x320
[ 75.768703] ? f2fs_free_filename+0x2a/0x40
[ 75.769061] f2fs_free_filename+0x2a/0x40
[ 75.769403] f2fs_lookup+0x19f/0x380
[ 75.769712] __lookup_slow+0x8b/0x130
[ 75.770034] walk_component+0xfc/0x170
[ 75.770353] path_lookupat+0x69/0x140
[ 75.770664] filename_lookup+0xe1/0x1c0
[ 75.770991] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.771393] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.771792] ? do_wp_page+0x3f6/0xbf0
[ 75.772109] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.772523] ? preempt_count_add+0x70/0xa0
[ 75.772902] ? vfs_statx+0x89/0x180
[ 75.773224] vfs_statx+0x89/0x180
[ 75.773530] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.773939] vfs_fstatat+0x80/0xa0
[ 75.774237] __do_sys_newfstatat+0x26/0x60
[ 75.774595] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.775021] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.775448] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.775878] ? do_user_addr_fault+0x563/0x7c0
[ 75.776273] ? srso_alias_return_thunk+0x5/0xfbef5
[ 75.776699] do_syscall_64+0x50/0x110
[ 75.777028] entry_SYSCALL_64_after_hwframe+0x6e/0x76
[ 75.777479] RIP: 0033:0x7f9601b07aea
[ 75.777793] Code: ff c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 40 00 90 90 90 90 90 90 90 90 90 90 90 90 90 90 90 90 41 89 ca b8 06 01 00 00 0f 05 <3d> 00 f0 ff ff 77 07 31 c0 c3 0f 1f 40 00 48 8b 15 01 23 0e 00 f7
[ 75.779391] RSP: 002b:00007ffc160eaae8 EFLAGS: 00000246 ORIG_RAX: 0000000000000106
[ 75.780050] RAX: ffffffffffffffda RBX: 0000000000000042 RCX: 00007f9601b07aea
[ 75.780663] RDX: 00007ffc160eab80 RSI: 00007ffc160ecb88 RDI: 00000000ffffff9c
[ 75.781278] RBP: 00007ffc160ead20 R08: 00007ffc160ead20 R09: 0000000000000000
[ 75.781902] R10: 0000000000000000 R11: 0000000000000246 R12: 00007ffc160eae70
[ 75.782532] R13: 00007ffc160ecb88 R14: 00007ffc160eae70 R15: 0000000000000020
[ 75.783150] </TASK>
[ 75.783349] Modules linked in:
[ 75.783628] CR2: fffff14ad2000008
[ 75.783918] ---[ end trace 0000000000000000 ]---
[ 75.784315] RIP: 0010:kmem_cache_free+0x6a/0x320
[ 75.784718] Code: 80 48 01 d8 0f 82 b4 02 00 00 48 c7 c2 00 00 00 80 48 2b 15 f8 c2 18 01 48 01 d0 48 c1 e8 0c 48 c1 e0 06 48 03 05 d6 c2 18 01 <48> 8b 50 08 49 89 c6 f6 c2 01 0f 85 ea 01 00 00 0f 1f 44 00 00 49
[ 75.786294] RSP: 0018:ffffa59bc231bb10 EFLAGS: 00010286
[ 75.786747] RAX: fffff14ad2000000 RBX: 0000000000000000 RCX: 0000000000000000
[ 75.787369] RDX: 0000620400000000 RSI: 0000000000000000 RDI: ffff9dfc80043600
[ 75.788016] RBP: ffffa59bc231bb30 R08: ffffa59bc231b9a0 R09: 00000000000003fa
[ 75.788672] R10: 00000000000fd024 R11: 0000000000000107 R12: ffff9dfc80043600
[ 75.789296] R13: ffffffff8404dc7a R14: 0000000000000000 R15: ffff9dfc8f1aa000
[ 75.789938] FS: 00007f9601efb780(0000) GS:ffff9dfcfbc00000(0000) knlGS:0000000000000000
[ 75.790677] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 75.791212] CR2: fffff14ad2000008 CR3: 0000000111750000 CR4: 0000000000750ef0
[ 75.791862] PKRU: 55555554
[ 75.792112] Kernel panic - not syncing: Fatal exception
[ 75.792797] Kernel Offset: 0x2a00000 from 0xffffffff81000000 (relocation range: 0xffffffff80000000-0xffffffffbfffffff)
--
Gabriel Krisman Bertazi
next prev parent reply other threads:[~2024-03-13 23:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-05 10:15 [PATCH v13 0/9] Cache insensitive cleanup for ext4/f2fs Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 1/9] ext4: Simplify the handling of cached insensitive names Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 2/9] f2fs: " Eugen Hristev
2024-03-13 23:36 ` Gabriel Krisman Bertazi [this message]
2024-03-14 8:44 ` Eugen Hristev
2024-03-14 14:41 ` Gabriel Krisman Bertazi
2024-03-18 9:53 ` Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 3/9] libfs: Introduce case-insensitive string comparison helper Eugen Hristev
2024-03-13 23:16 ` Gabriel Krisman Bertazi
2024-03-05 10:16 ` [PATCH v13 4/9] ext4: Reuse generic_ci_match for ci comparisons Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 5/9] f2fs: " Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 6/9] ext4: Log error when lookup of encoded dentry fails Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 7/9] f2fs: " Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 8/9] ext4: Move CONFIG_UNICODE defguards into the code flow Eugen Hristev
2024-03-05 10:16 ` [PATCH v13 9/9] f2fs: " Eugen Hristev
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87edcdk8li.fsf@mailhost.krisman.be \
--to=krisman@suse.de \
--cc=adilger.kernel@dilger.ca \
--cc=brauner@kernel.org \
--cc=chao@kernel.org \
--cc=eugen.hristev@collabora.com \
--cc=jack@suse.cz \
--cc=jaegeuk@kernel.org \
--cc=kernel@collabora.com \
--cc=krisman@collabora.com \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).