On Mon, Mar 23, 2020 at 10:14:56PM -0300, Jason Gunthorpe wrote:
> From: Jason Gunthorpe <j...@mellanox.com>
> Most places that return an error code, like -EFAULT, do not set
> HMM_PFN_ERROR, only two places do this.
> Resolve this inconsistency by never setting the pfns on an error
> exit. This doesn't seem like a worthwhile thing to do anyhow.
> If for some reason it becomes important, it makes more sense to directly
> return the address of the failing page rather than have the caller scan
> for the HMM_PFN_ERROR.
> No caller inspects the pnfs output array if hmm_range_fault() fails.
> Signed-off-by: Jason Gunthorpe <j...@mellanox.com>

Looks good,

Reviewed-by: Christoph Hellwig <h...@lst.de>
amd-gfx mailing list

Reply via email to