From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.sourceforge.net (lists.sourceforge.net [216.105.38.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 95AD6ECAAA7 for ; Mon, 29 Aug 2022 18:34:26 +0000 (UTC) Received: from [127.0.0.1] (helo=sfs-ml-1.v29.lw.sourceforge.com) by sfs-ml-1.v29.lw.sourceforge.com with esmtp (Exim 4.95) (envelope-from ) id 1oSjab-0007ya-BO; Mon, 29 Aug 2022 18:34:25 +0000 Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-1.v29.lw.sourceforge.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1oSjaZ-0007yU-Q8 for linux-f2fs-devel@lists.sourceforge.net; Mon, 29 Aug 2022 18:34:23 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sourceforge.net; s=x; h=In-Reply-To:Content-Type:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=48BYPGKXIYs/+s2Khm5OWGDYN9Nw1jMklVjc7KIOK/M=; b=Lbeb53uuwu9b26lgG3U4kCLG0a Svm7oahpqIhvtPHtmsTOhj0ctIihf+Pfm6UqB9NBtQXT/2kc3l27xS7NLGyh0ZCXlAHGUV8OoL0+u OhEWKmgr8M4cRRMlCMOipcpeUVUghydJHVQZvLx3ZJGbvWRyIHESIsObcMmwd06RVWmE=; DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sf.net; s=x ; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To :From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=48BYPGKXIYs/+s2Khm5OWGDYN9Nw1jMklVjc7KIOK/M=; b=HrpngjXVe4phB7NYqyJufPUC+m 3z78klwzBMaX6TonZCOcUFonAeYmkBQlJ9kxouZw54u4nyrpNAEFwMLiKEWh57RwOh8F8IKEMWDWM SWKkFotESqyJVz8jgD9Ibt3UL+rX4gFj6U1jAcKdfoFss2bBgoyRJdXFrXcB0vGAiQRg=; Received: from dfw.source.kernel.org ([139.178.84.217]) by sfi-mx-1.v28.lw.sourceforge.com with esmtps (TLS1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.95) id 1oSjaY-006gUW-0X for linux-f2fs-devel@lists.sourceforge.net; Mon, 29 Aug 2022 18:34:23 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 952E461343; Mon, 29 Aug 2022 18:34:16 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1970BC433B5; Mon, 29 Aug 2022 18:34:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1661798056; bh=zhm95cok2WKuPGTcOLYqozQIGbOcYROaqVASpobqVOE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=B3Y5fnNI7VcvF87gvAddr1EYic37Qv3YZ0hpDMdPiKqMuOtMe3Vo4FMvBD1KLyjpA 3GHgdvzzsVVIgOcOFyd55aFYOETHZdSVPBtc8vdqBmudW5EzSAJ1V9rYWg5aJem3He FBdbpa1Xm0PbMJvfMq4UsWJm3BYbCZZ0PsM01BKXpBj9+KzWhXhcDYR0ZWfnVh5zJl OGRdMx9/Ss8M9q91ON+rnShQXvhNhc+ciSZo/3xkM7EO+myrhp4B/XvkzFgo8eSn4Q jb8Ue1VYyAyl2TtQkcyCKqFY+DDcbghNi4jjiUY4ijBzp+Ac7bcXT4ZtN1jc76qX+L 4iqyhKiIQ+uqw== Date: Mon, 29 Aug 2022 11:34:14 -0700 From: Jaegeuk Kim To: Matthew Wilcox Message-ID: References: <000000000000d5b4fe05e7127662@google.com> <20220825183734.0b08ae10a2e9e1bd156a19cd@linux-foundation.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Headers-End: 1oSjaY-006gUW-0X Subject: Re: [f2fs-dev] [syzbot] BUG: unable to handle kernel NULL pointer dereference in set_page_dirty X-BeenThere: linux-f2fs-devel@lists.sourceforge.net X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: syzkaller-bugs@googlegroups.com, linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, linux-mm@kvack.org, syzbot , linux-fsdevel@vger.kernel.org, Andrew Morton Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net On 08/29, Matthew Wilcox wrote: > On Mon, Aug 29, 2022 at 10:52:57AM -0700, Jaegeuk Kim wrote: > > On 08/25, Andrew Morton wrote: > > > (cc fsf2 developers) > > > > > > On Thu, 25 Aug 2022 08:29:32 -0700 syzbot wrote: > > > > > > > Hello, > > > > > > > > syzbot found the following issue on: > > > > > > > > HEAD commit: a41a877bc12d Merge branch 'for-next/fixes' into for-kernelci > > > > git tree: git://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-kernelci > > > > console output: https://syzkaller.appspot.com/x/log.txt?x=175def47080000 > > > > kernel config: https://syzkaller.appspot.com/x/.config?x=5cea15779c42821c > > > > dashboard link: https://syzkaller.appspot.com/bug?extid=775a3440817f74fddb8c > > > > compiler: Debian clang version 13.0.1-++20220126092033+75e33f71c2da-1~exp1~20220126212112.63, GNU ld (GNU Binutils for Debian) 2.35.2 > > > > userspace arch: arm64 > > > > > > > > Unfortunately, I don't have any reproducer for this issue yet. > > > > > > > > IMPORTANT: if you fix the issue, please add the following tag to the commit: > > > > Reported-by: syzbot+775a3440817f74fddb8c@syzkaller.appspotmail.com > > > > > > > > Unable to handle kernel NULL pointer dereference at virtual address 0000000000000000 > > > > Mem abort info: > > > > ESR = 0x0000000086000005 > > > > EC = 0x21: IABT (current EL), IL = 32 bits > > > > SET = 0, FnV = 0 > > > > EA = 0, S1PTW = 0 > > > > FSC = 0x05: level 1 translation fault > > > > user pgtable: 4k pages, 48-bit VAs, pgdp=00000001249cc000 > > > > [0000000000000000] pgd=080000012ee65003, p4d=080000012ee65003, pud=0000000000000000 > > > > Internal error: Oops: 86000005 [#1] PREEMPT SMP > > > > Modules linked in: > > > > CPU: 0 PID: 3044 Comm: syz-executor.0 Not tainted 6.0.0-rc2-syzkaller-16455-ga41a877bc12d #0 > > > > Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 06/20/2022 > > > > pstate: 80400005 (Nzcv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--) > > > > pc : 0x0 > > > > lr : folio_mark_dirty+0xbc/0x208 mm/page-writeback.c:2748 > > > > sp : ffff800012803830 > > > > x29: ffff800012803830 x28: ffff0000d02c8000 x27: 0000000000000009 > > > > x26: 0000000000000001 x25: 0000000000000a00 x24: 0000000000000080 > > > > x23: 0000000000000000 x22: ffff0000ef276c00 x21: 05ffc00000000007 > > > > x20: ffff0000f14b83b8 x19: fffffc00036409c0 x18: fffffffffffffff5 > > > > x17: ffff80000dd7a698 x16: ffff80000dbb8658 x15: 0000000000000000 > > > > x14: 0000000000000000 x13: 0000000000000000 x12: 0000000000000000 > > > > x11: ff808000083e9814 x10: 0000000000000000 x9 : ffff8000083e9814 > > > > x8 : 0000000000000000 x7 : 0000000000000000 x6 : 0000000000000000 > > > > x5 : ffff0000d9028000 x4 : ffff0000d5c31000 x3 : ffff0000d9027f80 > > > > x2 : fffffffffffffff0 x1 : fffffc00036409c0 x0 : ffff0000f14b83b8 > > > > Call trace: > > > > 0x0 > > > > set_page_dirty+0x38/0xbc mm/folio-compat.c:62 > > > > 2363 void f2fs_update_meta_page(struct f2fs_sb_info *sbi, > > 2364 void *src, block_t blk_addr) > > 2365 { > > 2366 struct page *page = f2fs_grab_meta_page(sbi, blk_addr); > > > > --> f2fs_grab_meta_page() gives a locked page by grab_cache_page(). > > > > 2367 > > 2368 memcpy(page_address(page), src, PAGE_SIZE); > > 2369 set_page_dirty(page); > > 2370 f2fs_put_page(page, 1); > > 2371 } > > > > Is there a change in folio? > > Not directly, but there was a related change, 0af573780b0b which > requires aops->set_page_dirty to be set; is that perhaps missing? > I don't see one in the f2fs_compress_aops, for example. Do you mean dirty_folio? I think all aops have it except the compressed one that we don't make it dirty. > > The other possibiity is that it's a mapping that is missing an ->a_ops. > Is that something f2fs ever does? Hmm, no, I haven't seen this before, and we set aops when mounting the file system. Ah, if this happens on the corrupted image, yeah, maybe.. I need to check the error path in f2fs_fill_super. > > I only managed to narrow down the crash to the line: > return mapping->a_ops->dirty_folio(mapping, folio); > so either mapping->a_ops is NULL or mapping->a_ops->dirty_folio is > NULL. The reproducer was on ARM and ARM doesn't emit a 'Code:' line, > unlike x86. _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel