On Thu, Dec 15, 2022 at 09:13:50PM +0100, Andreas Gruenbacher wrote: > This is still screwed up. We really need to unlock the page before > calling into __mark_inode_dirty() and ending the transaction. The > current page_done() hook would force us to then re-lock the page just > so that the caller can unlock it again. This just doesn't make sense, > particularly since the page_prepare and page_done hooks only exist to > allow gfs2 to do data journaling via iomap. I'll follow up with a more > useful approach ... Yes. And it would make sense to include the gfs2 patches.