Re: [PATCH 2/3] dax: introduce dax_clear_poison to dax pwrite operation

From: Christoph Hellwig
Date: Thu Nov 04 2021 - 13:54:25 EST


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.