On Wed, Jun 19, 2024 at 01:39:32PM -0400, cel@xxxxxxxxxx wrote: > if (IS_ERR(bdev_file)) { > - pr_warn("pNFS: failed to open device %s (%ld)\n", > + dprintk("failed to open device %s (%ld)\n", > devname, PTR_ERR(bdev_file)); I'd just drop this one entirely. Otherwise this looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>