Hello,
syzbot found the following issue on:
HEAD commit: 6dc544b66971 Add linux-next specific files for 20240528
git tree: linux-next
console+strace: https://syzkaller.appspot.com/x/log.txt?x=146d3f84980000
kernel config: https://syzkaller.appspot.com/x/.config?x=6a363b35598e573d
dashboard link: https://syzkaller.appspot.com/bug?extid=17416257cb95200cba44
compiler: Debian clang version 15.0.6, GNU ld (GNU Binutils for Debian) 2.40
syz repro: https://syzkaller.appspot.com/x/repro.syz?x=11e41b72980000
C reproducer: https://syzkaller.appspot.com/x/repro.c?x=15f3d2d2980000
Downloadable assets:
disk image: https://storage.googleapis.com/syzbot-assets/334699ab67f8/disk-6dc544b6.raw.xz
vmlinux: https://storage.googleapis.com/syzbot-assets/4ca32b2218ce/vmlinux-6dc544b6.xz
kernel image: https://storage.googleapis.com/syzbot-assets/400bc5f019b3/bzImage-6dc544b6.xz
IMPORTANT: if you fix the issue, please add the following tag to the commit:
Reported-by: [email protected]
------------[ cut here ]------------
UBSAN: shift-out-of-bounds in mm/vmscan.c:4715:21
shift exponent -1 is negative
CPU: 1 PID: 5094 Comm: syz-executor177 Not tainted 6.10.0-rc1-next-20240528-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 04/02/2024
Call Trace:
<TASK>
__dump_stack lib/dump_stack.c:88 [inline]
dump_stack_lvl+0x241/0x360 lib/dump_stack.c:114
ubsan_epilogue lib/ubsan.c:231 [inline]
__ubsan_handle_shift_out_of_bounds+0x3c8/0x420 lib/ubsan.c:468
get_nr_to_scan mm/vmscan.c:4715 [inline]
try_to_shrink_lruvec+0xa99/0xbb0 mm/vmscan.c:4761
lru_gen_shrink_lruvec mm/vmscan.c:4908 [inline]
shrink_lruvec+0x554/0x3070 mm/vmscan.c:5685
shrink_node_memcgs mm/vmscan.c:5921 [inline]
shrink_node+0xb17/0x4150 mm/vmscan.c:5961
shrink_zones mm/vmscan.c:6205 [inline]
do_try_to_free_pages+0x789/0x1cb0 mm/vmscan.c:6267
try_to_free_mem_cgroup_pages+0x48f/0xb10 mm/vmscan.c:6598
try_charge_memcg+0x704/0x1850 mm/memcontrol.c:2946
try_charge mm/memcontrol.c:3094 [inline]
charge_memcg+0xa2/0x160 mm/memcontrol.c:7532
__mem_cgroup_charge+0x27/0x80 mm/memcontrol.c:7547
mem_cgroup_charge include/linux/memcontrol.h:686 [inline]
shmem_alloc_and_add_folio+0x44b/0xbd0 mm/shmem.c:1663
shmem_get_folio_gfp+0x82d/0x1f50 mm/shmem.c:2041
shmem_read_folio_gfp+0xd6/0x170 mm/shmem.c:4944
drm_gem_get_pages+0x21b/0xe50 drivers/gpu/drm/drm_gem.c:568
drm_gem_shmem_get_pages+0xdd/0x290 drivers/gpu/drm/drm_gem_shmem_helper.c:177
drm_gem_shmem_vmap+0x2ba/0x630 drivers/gpu/drm/drm_gem_shmem_helper.c:335
drm_gem_vmap drivers/gpu/drm/drm_gem.c:1205 [inline]
drm_gem_vmap_unlocked+0x103/0x1d0 drivers/gpu/drm/drm_gem.c:1247
drm_gem_fb_vmap+0xa6/0x810 drivers/gpu/drm/drm_gem_framebuffer_helper.c:365
drm_atomic_helper_prepare_planes+0x2b2/0xb50 drivers/gpu/drm/drm_atomic_helper.c:2601
drm_atomic_helper_commit+0x18b/0x9f0 drivers/gpu/drm/drm_atomic_helper.c:2029
drm_atomic_commit+0x2ac/0x310 drivers/gpu/drm/drm_atomic.c:1514
drm_atomic_helper_update_plane+0x238/0x3a0 drivers/gpu/drm/drm_atomic_helper.c:3190
drm_mode_cursor_universal drivers/gpu/drm/drm_plane.c:1221 [inline]
drm_mode_cursor_common+0xd68/0x1570 drivers/gpu/drm/drm_plane.c:1280
drm_mode_cursor_ioctl+0xe1/0x160 drivers/gpu/drm/drm_plane.c:1330
drm_ioctl_kernel+0x33a/0x440 drivers/gpu/drm/drm_ioctl.c:744
drm_ioctl+0x611/0xad0 drivers/gpu/drm/drm_ioctl.c:841
vfs_ioctl fs/ioctl.c:51 [inline]
__do_sys_ioctl fs/ioctl.c:907 [inline]
__se_sys_ioctl+0xfc/0x170 fs/ioctl.c:893
do_syscall_x64 arch/x86/entry/common.c:52 [inline]
do_syscall_64+0xf3/0x230 arch/x86/entry/common.c:83
entry_SYSCALL_64_after_hwframe+0x77/0x7f
RIP: 0033:0x7f06b4225999
Code: 28 00 00 00 75 05 48 83 c4 28 c3 e8 61 18 00 00 90 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 c7 c1 b8 ff ff ff f7 d8 64 89 01 48
RSP: 002b:00007fff29e89738 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
RAX: ffffffffffffffda RBX: 0000000000000003 RCX: 00007f06b4225999
RDX: 0000000020000340 RSI: 00000000c01c64a3 RDI: 0000000000000003
RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
R10: 0023647261632f69 R11: 0000000000000246 R12: 00007f06b426f036
R13: 00007fff29e89770 R14: 00007fff29e897b0 R15: 0000000000000000
</TASK>
---[ end trace ]---
---
This report is generated by a bot. It may contain errors.
See https://goo.gl/tpsmEJ for more information about syzbot.
syzbot engineers can be reached at [email protected].
syzbot will keep track of this issue. See:
https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
If the report is already addressed, let syzbot know by replying with:
#syz fix: exact-commit-title
If you want syzbot to run the reproducer, reply with:
#syz test: git://repo/address.git branch-or-commit-hash
If you attach or paste a git patch, syzbot will apply it before testing.
If you want to overwrite report's subsystems, reply with:
#syz set subsystems: new-subsystem
(See the list of subsystem names on the web dashboard)
If the report is a duplicate of another one, reply with:
#syz dup: exact-subject-of-another-report
If you want to undo deduplication, reply with:
#syz undup
syzbot has bisected this issue to:
commit 6be5e186fd655df4b3ba267054de2eaaadc71340
Author: Johannes Weiner <[email protected]>
Date: Tue May 14 20:26:41 2024 +0000
mm: vmscan: restore incremental cgroup iteration
bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=179315c8980000
start commit: 6dc544b66971 Add linux-next specific files for 20240528
git tree: linux-next
final oops: https://syzkaller.appspot.com/x/report.txt?x=145315c8980000
console output: https://syzkaller.appspot.com/x/log.txt?x=105315c8980000
kernel config: https://syzkaller.appspot.com/x/.config?x=6a363b35598e573d
dashboard link: https://syzkaller.appspot.com/bug?extid=17416257cb95200cba44
syz repro: https://syzkaller.appspot.com/x/repro.syz?x=11e41b72980000
C reproducer: https://syzkaller.appspot.com/x/repro.c?x=15f3d2d2980000
Reported-by: [email protected]
Fixes: 6be5e186fd65 ("mm: vmscan: restore incremental cgroup iteration")
For information about bisection process see: https://goo.gl/tpsmEJ#bisection
On Tue, May 28, 2024 at 09:49:20AM GMT, syzbot wrote:
> Hello,
>
> syzbot found the following issue on:
>
> HEAD commit: 6dc544b66971 Add linux-next specific files for 20240528
> git tree: linux-next
> console+strace: https://syzkaller.appspot.com/x/log.txt?x=146d3f84980000
> kernel config: https://syzkaller.appspot.com/x/.config?x=6a363b35598e573d
> dashboard link: https://syzkaller.appspot.com/bug?extid=17416257cb95200cba44
> compiler: Debian clang version 15.0.6, GNU ld (GNU Binutils for Debian) 2.40
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=11e41b72980000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=15f3d2d2980000
>
> Downloadable assets:
> disk image: https://storage.googleapis.com/syzbot-assets/334699ab67f8/disk-6dc544b6.raw.xz
> vmlinux: https://storage.googleapis.com/syzbot-assets/4ca32b2218ce/vmlinux-6dc544b6.xz
> kernel image: https://storage.googleapis.com/syzbot-assets/400bc5f019b3/bzImage-6dc544b6.xz
>
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: [email protected]
>
> ------------[ cut here ]------------
> UBSAN: shift-out-of-bounds in mm/vmscan.c:4715:21
> shift exponent -1 is negative
> CPU: 1 PID: 5094 Comm: syz-executor177 Not tainted 6.10.0-rc1-next-20240528-syzkaller #0
> Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 04/02/2024
> Call Trace:
> <TASK>
> __dump_stack lib/dump_stack.c:88 [inline]
> dump_stack_lvl+0x241/0x360 lib/dump_stack.c:114
> ubsan_epilogue lib/ubsan.c:231 [inline]
> __ubsan_handle_shift_out_of_bounds+0x3c8/0x420 lib/ubsan.c:468
> get_nr_to_scan mm/vmscan.c:4715 [inline]
> try_to_shrink_lruvec+0xa99/0xbb0 mm/vmscan.c:4761
> lru_gen_shrink_lruvec mm/vmscan.c:4908 [inline]
> shrink_lruvec+0x554/0x3070 mm/vmscan.c:5685
> shrink_node_memcgs mm/vmscan.c:5921 [inline]
> shrink_node+0xb17/0x4150 mm/vmscan.c:5961
> shrink_zones mm/vmscan.c:6205 [inline]
> do_try_to_free_pages+0x789/0x1cb0 mm/vmscan.c:6267
> try_to_free_mem_cgroup_pages+0x48f/0xb10 mm/vmscan.c:6598
> try_charge_memcg+0x704/0x1850 mm/memcontrol.c:2946
> try_charge mm/memcontrol.c:3094 [inline]
> charge_memcg+0xa2/0x160 mm/memcontrol.c:7532
> __mem_cgroup_charge+0x27/0x80 mm/memcontrol.c:7547
> mem_cgroup_charge include/linux/memcontrol.h:686 [inline]
> shmem_alloc_and_add_folio+0x44b/0xbd0 mm/shmem.c:1663
> shmem_get_folio_gfp+0x82d/0x1f50 mm/shmem.c:2041
> shmem_read_folio_gfp+0xd6/0x170 mm/shmem.c:4944
> drm_gem_get_pages+0x21b/0xe50 drivers/gpu/drm/drm_gem.c:568
> drm_gem_shmem_get_pages+0xdd/0x290 drivers/gpu/drm/drm_gem_shmem_helper.c:177
> drm_gem_shmem_vmap+0x2ba/0x630 drivers/gpu/drm/drm_gem_shmem_helper.c:335
> drm_gem_vmap drivers/gpu/drm/drm_gem.c:1205 [inline]
> drm_gem_vmap_unlocked+0x103/0x1d0 drivers/gpu/drm/drm_gem.c:1247
> drm_gem_fb_vmap+0xa6/0x810 drivers/gpu/drm/drm_gem_framebuffer_helper.c:365
> drm_atomic_helper_prepare_planes+0x2b2/0xb50 drivers/gpu/drm/drm_atomic_helper.c:2601
> drm_atomic_helper_commit+0x18b/0x9f0 drivers/gpu/drm/drm_atomic_helper.c:2029
> drm_atomic_commit+0x2ac/0x310 drivers/gpu/drm/drm_atomic.c:1514
> drm_atomic_helper_update_plane+0x238/0x3a0 drivers/gpu/drm/drm_atomic_helper.c:3190
> drm_mode_cursor_universal drivers/gpu/drm/drm_plane.c:1221 [inline]
> drm_mode_cursor_common+0xd68/0x1570 drivers/gpu/drm/drm_plane.c:1280
> drm_mode_cursor_ioctl+0xe1/0x160 drivers/gpu/drm/drm_plane.c:1330
> drm_ioctl_kernel+0x33a/0x440 drivers/gpu/drm/drm_ioctl.c:744
> drm_ioctl+0x611/0xad0 drivers/gpu/drm/drm_ioctl.c:841
> vfs_ioctl fs/ioctl.c:51 [inline]
> __do_sys_ioctl fs/ioctl.c:907 [inline]
> __se_sys_ioctl+0xfc/0x170 fs/ioctl.c:893
> do_syscall_x64 arch/x86/entry/common.c:52 [inline]
> do_syscall_64+0xf3/0x230 arch/x86/entry/common.c:83
> entry_SYSCALL_64_after_hwframe+0x77/0x7f
> RIP: 0033:0x7f06b4225999
> Code: 28 00 00 00 75 05 48 83 c4 28 c3 e8 61 18 00 00 90 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 c7 c1 b8 ff ff ff f7 d8 64 89 01 48
> RSP: 002b:00007fff29e89738 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
> RAX: ffffffffffffffda RBX: 0000000000000003 RCX: 00007f06b4225999
> RDX: 0000000020000340 RSI: 00000000c01c64a3 RDI: 0000000000000003
> RBP: 0000000000000000 R08: 0000000000000000 R09: 0000000000000000
> R10: 0023647261632f69 R11: 0000000000000246 R12: 00007f06b426f036
> R13: 00007fff29e89770 R14: 00007fff29e897b0 R15: 0000000000000000
> </TASK>
> ---[ end trace ]---
>
>
> ---
> This report is generated by a bot. It may contain errors.
> See https://goo.gl/tpsmEJ for more information about syzbot.
> syzbot engineers can be reached at [email protected].
>
> syzbot will keep track of this issue. See:
> https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
>
> If the report is already addressed, let syzbot know by replying with:
> #syz fix: exact-commit-title
>
> If you want syzbot to run the reproducer, reply with:
> #syz test: git://repo/address.git branch-or-commit-hash
> If you attach or paste a git patch, syzbot will apply it before testing.
>
> If you want to overwrite report's subsystems, reply with:
> #syz set subsystems: new-subsystem
> (See the list of subsystem names on the web dashboard)
>
> If the report is a duplicate of another one, reply with:
> #syz dup: exact-subject-of-another-report
>
> If you want to undo deduplication, reply with:
> #syz undup
#syz test
From 47251f8cdafd8a5e6e242b8c9785fb38f130e088 Mon Sep 17 00:00:00 2001
From: Shakeel Butt <[email protected]>
Date: Wed, 29 May 2024 07:58:50 -0700
Subject: [PATCH] mm: vmscan: reset sc->priority on retry
The commit 6be5e186fd65 ("mm: vmscan: restore incremental cgroup
iteration") added a retry reclaim heuristic to iterate all the cgroups
before returning an unsuccessful reclaim retry but missed to reset the
sc->priority. Let's fix it.
Reported-by: [email protected]
Fixes: 6be5e186fd65 ("mm: vmscan: restore incremental cgroup iteration")
Signed-off-by: Shakeel Butt <[email protected]>
---
mm/vmscan.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/mm/vmscan.c b/mm/vmscan.c
index b9170f767353..731b009a142b 100644
--- a/mm/vmscan.c
+++ b/mm/vmscan.c
@@ -6317,6 +6317,7 @@ static unsigned long do_try_to_free_pages(struct zonelist *zonelist,
* meaningful forward progress. Avoid false OOMs in this case.
*/
if (!sc->memcg_full_walk) {
+ sc->priority = initial_priority;
sc->memcg_full_walk = 1;
goto retry;
}
--
2.43.0
Hello,
syzbot has tested the proposed patch and the reproducer did not trigger any issue:
Reported-and-tested-by: [email protected]
Tested on:
commit: 9d99040b Add linux-next specific files for 20240529
git tree: linux-next
console output: https://syzkaller.appspot.com/x/log.txt?x=1530a9d2980000
kernel config: https://syzkaller.appspot.com/x/.config?x=735e953fee00ec19
dashboard link: https://syzkaller.appspot.com/bug?extid=17416257cb95200cba44
compiler: Debian clang version 15.0.6, GNU ld (GNU Binutils for Debian) 2.40
patch: https://syzkaller.appspot.com/x/patch.diff?x=14a1650c980000
Note: testing is done by a robot and is best-effort only.