On Thu, Aug 12, 2021 at 12:57:58PM +0100, David Howells wrote: > Make swap_readpage(), when accessing a swap file (SWP_FS_OPS) use > the ->direct_IO() method on the filesystem rather then ->readpage(). ->direct_IO is just a helper for ->read_iter and ->write_iter, so please don't call it directly. It actually is slowly on its way out, with at at least all of the iomap implementations not using it, as well as various other file systems. > + ki = kzalloc(sizeof(*ki), GFP_KERNEL); > + if (!ki) > + return -ENOMEM; for the synchronous case we could avoid this allocation and just use arguments on stack.