On Tue, Sep 14, 2021 at 05:31:30PM -0600, Jane Chu wrote: > + if ((map_len == -EIO) && (iov_iter_rw(iter) == WRITE)) { No need for the inner braces. > + if (dax_clear_poison(dax_dev, pgoff, PHYS_PFN(size)) == 0) Overly long line. Otherwise looks good, but it might need a rebase to the iomap_iter changes.