Re: [PATCH] block: add lock for safe nrpages access in invalidate_bdev()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Yuan Tan <tanyuan@xxxxxxxxxxx>
Subject: Re: [PATCH] block: add lock for safe nrpages access in invalidate_bdev()
From: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Date: Mon, 10 Mar 2025 17:11:22 +0000
Cc: axboe@xxxxxxxxx, syzbot+f2aaf773187f5cae54f3@xxxxxxxxxxxxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, syzkaller-bugs@xxxxxxxxxxxxxxxx, falcon@xxxxxxxxxxx
In-reply-to: <20250310165400.3166618-1-tanyuan@tinylab.org>

On Mon, Mar 10, 2025 at 09:54:00AM -0700, Yuan Tan wrote:
> Syzbot reported a data-race in __filemap_add_folio / invalidate_bdev[1]
> due to concurrent access to mapping->nrpages.
> Adds a lock around the access to nrpages.

Did you even read my analysis?  This is a grossly inappropriate fix.
NAK.

References:
- [syzbot] [mm?] [fs?] KCSAN: data-race in __filemap_add_folio / invalidate_bdev (8)
  - From: syzbot
- [PATCH] block: add lock for safe nrpages access in invalidate_bdev()
  - From: Yuan Tan

Prev by Date: [PATCH] block: add lock for safe nrpages access in invalidate_bdev()
Next by Date: Re: [PATCH v4 05/12] iomap: Support SW-based atomic writes
Previous by thread: [PATCH] block: add lock for safe nrpages access in invalidate_bdev()
Next by thread: [PATCH v4 00/11] afs, rxrpc: Clean up refcounting on afs_cell and afs_server records
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]

Powered by Linux