On 08/26/2011 01:19 PM, J. Bruce Fields wrote:
> On Fri, Aug 12, 2011 at 05:30:12PM -0700, Boaz Harrosh wrote:
<snip>
> @@ -156,12 +155,14 @@ out_put:
> dput(dentry);
> out_unlock:
> mutex_unlock(&dir->d_inode->i_mutex);
> - if (status == 0) {
> - clp->cl_firststate = 1;
> + if (status == 0)
> vfs_fsync(rec_file, 0);
> - }
> + else
> + printk(KERN_ERR "NFSD: failed to write recovery record"
> + " (err %d); please check that recovery"
> + " directory exists and is writeable",
> + status);
Bruce Hi
Which of the variables has the directory path?
Would we like to add that information in the print. From my experience
alot of times those type of problems is the mis-communication of
the path names, and it helps to see what name was actually attempted.
Other wise looks good
Thanks
> nfs4_reset_creds(original_cred);
> - dprintk("NFSD: nfsd4_create_clid_dir returns %d\n", status);
> return status;
> }
>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
