On Mon, Mar 23, 2020 at 10:14:56PM -0300, Jason Gunthorpe wrote: > From: Jason Gunthorpe <[email protected]> > > 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 <[email protected]>
Looks good, Reviewed-by: Christoph Hellwig <[email protected]> _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
