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_iterchanges.