On Thu, Sep 20, 2018 at 01:31:27PM -0700, Darrick J. Wong wrote: > > { > > struct xfs_writepage_ctx wpc = { > > - .io_type = XFS_IO_INVALID, > > + .io_type = XFS_IO_HOLE, > > Hm. So I guess if we ever saw XFS_IO_INVALID that meant "we never did > find any extents and so never set io_type", right? Yes.