On Sat, 2007-06-23 at 08:46 +0100, Christoph Hellwig wrote:
> On Fri, Jun 22, 2007 at 01:03:21PM -0700, Dave Hansen wrote:
> > Signed-off-by: Dave Hansen <[EMAIL PROTECTED]>
> 
> We really want a guaranteed non-NULL file here, but I don't want to put
> this on your plate also.  Please add a comment about bloody NFS exports
> for now.

How does this look?

-       if (IS_RDONLY(inode))
+       /*
+        * Ideally, we want to guarantee that 'f_vfsmnt'
+        * is non-NULL here.  But, NFS exports need to
+        * be fixed up before we can do that.  So, check
+        * it for now. - Dave Hansen
+        */
+       if (file->f_vfsmnt)
+               err = mnt_want_write(file->f_vfsmnt);
+       if (err)
                return;


-- Dave

-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to