On Wed, Jul 09, 2025 at 11:30:42AM +0800, Jinliang Zheng wrote: > In addition, what I want to say is that once folio_test_uptodate() is > true, all bits in ifs->state are in the uptodate state. So there is no > need to acquire the lock and set it again. This repeated setting happens > in __iomap_write_end(). Yes, that seems fine. Can you update the commit message with some of the insights from this discussion, and with that the patch should be fine.