On Thu, Apr 07, 2022 at 09:03:50PM +0800, Miaohe Lin wrote: > PageSwapCache is only reliable when PageAnon is true because PG_swapcache > serves as PG_owner_priv_1 which can be used by fs if it's pagecache page. > So we should test PageAnon to distinguish pagecache page from swapcache > page to avoid false-postive PageSwapCache test. > > Signed-off-by: Miaohe Lin <linmiaohe@xxxxxxxxxx> > --- > mm/memory-failure.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/memory-failure.c b/mm/memory-failure.c > index ef402b490663..2e97302d62e4 100644 > --- a/mm/memory-failure.c > +++ b/mm/memory-failure.c > @@ -2262,7 +2262,7 @@ static int __soft_offline_page(struct page *page) > return 0; > } > > - if (!PageHuge(page) && PageLRU(page) && !PageSwapCache(page)) > + if (!PageHuge(page) && PageLRU(page) && !PageAnon(page)) > /* > * Try to invalidate first. This should work for > * non dirty unmapped page cache pages. > -- I foudn that with this change the following VM_BUG_ON_FOLIO() is triggered when calling soft-offline for a swapcache. Maybe we need check both of PageAnon and PageSwapCache instead of either? Thanks, Naoya Horiguchi [ 41.232172] page:0000000033d8a20c refcount:0 mapcount:0 mapping:00000000bc103d88 index:0x36d pfn:0x14359b [ 41.234931] memcg:ffff8c2f84d72000 [ 41.235850] aops:swap_aops [ 41.236576] flags: 0x57ffffc0080415(locked|uptodate|lru|owner_priv_1|swapbacked|node=1|zone=2|lastcpupid=0x1fffff) [ 41.239221] raw: 0057ffffc0080415 ffffef2c050eda48 ffffef2c050dbe08 0000000000000000 [ 41.241216] raw: 000000000000036d 000000000000036e 00000000ffffffff ffff8c2f84d72000 [ 41.243184] page dumped because: VM_BUG_ON_FOLIO(folio_test_lru(folio)) [ 41.244872] ------------[ cut here ]------------ [ 41.246074] kernel BUG at mm/memcontrol.c:7062! [ 41.247248] invalid opcode: 0000 [#1] PREEMPT SMP PTI [ 41.248539] CPU: 5 PID: 1036 Comm: bash Tainted: G E 5.18.0-rc1-v5.18-rc1-220408-2310-012-gf501f+ #11 [ 41.251844] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.15.0-1.fc35 04/01/2014 [ 41.254087] RIP: 0010:mem_cgroup_swapout+0x181/0x2c0 [ 41.255399] Code: 03 0f 85 37 01 00 00 65 48 ff 08 48 83 c4 08 5b 5d 41 5c 41 5d 41 5e 41 5f e9 8b a3 e0 ff 48 c7 c6 88 b7 5a aa e8 3f 26 f7 ff <0f> 0b 48 c7 c6 d0 ec 5a aa e8 31 26 f7 ff 0f 0b 66 90 8b 45 5c 48 [ 41.260408] RSP: 0018:ffffa9340218fce0 EFLAGS: 00010082 [ 41.261780] RAX: 000000000000003b RBX: ffff8c2fc180a000 RCX: 0000000000000000 [ 41.263604] RDX: 0000000000000002 RSI: ffffffffaa599561 RDI: 00000000ffffffff [ 41.265435] RBP: ffffef2c050d66c0 R08: 0000000000000000 R09: 00000000ffffdfff [ 41.267266] R10: ffffa9340218fad0 R11: ffffffffaa940d08 R12: 000000000000036e [ 41.269094] R13: 0000000000000000 R14: 0000000000000000 R15: ffff8c2fc180a008 [ 41.270911] FS: 00007f00259d3740(0000) GS:ffff8c30bbc80000(0000) knlGS:0000000000000000 [ 41.272975] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 41.274422] CR2: 0000561d1325a973 CR3: 00000001452c2001 CR4: 0000000000170ee0 [ 41.276253] Call Trace: [ 41.277013] <TASK> [ 41.277561] __remove_mapping+0xce/0x300 [ 41.278604] remove_mapping+0x12/0xe0 [ 41.279571] soft_offline_page+0x834/0x8b0 [ 41.280972] soft_offline_page_store+0x43/0x70 [ 41.282171] kernfs_fop_write_iter+0x11c/0x1b0 [ 41.283292] new_sync_write+0xf9/0x160 [ 41.284310] vfs_write+0x209/0x290 [ 41.285174] ksys_write+0x4f/0xc0 [ 41.286049] do_syscall_64+0x3b/0x90 [ 41.286991] entry_SYSCALL_64_after_hwframe+0x44/0xae [ 41.288328] RIP: 0033:0x7f00257018b7 [ 41.289232] Code: 0f 00 f7 d8 64 89 02 48 c7 c0 ff ff ff ff eb b7 0f 1f 00 f3 0f 1e fa 64 8b 04 25 18 00 00 00 85 c0 75 10 b8 01 00 00 00 0f 05 <48> 3d 00 f0 ff ff 77 51 c3 48 83 ec 28 48 89 54 24 18 48 89 74 24 [ 41.294001] RSP: 002b:00007fff3dc50748 EFLAGS: 00000246 ORIG_RAX: 0000000000000001 [ 41.295937] RAX: ffffffffffffffda RBX: 000000000000000c RCX: 00007f00257018b7 [ 41.297767] RDX: 000000000000000c RSI: 0000561d1325a970 RDI: 0000000000000001 [ 41.299600] RBP: 0000561d1325a970 R08: 0000000000000000 R09: 00007f00257b64e0 [ 41.301418] R10: 00007f00257b63e0 R11: 0000000000000246 R12: 000000000000000c [ 41.303305] R13: 00007f00257fb5a0 R14: 000000000000000c R15: 00007f00257fb7a0 [ 41.305179] </TASK>
