On Mon, 23 May 2011 08:42:39 -0500 Steve French <[email protected]> wrote:
> On Mon, May 23, 2011 at 6:12 AM, Jeff Layton <[email protected]> wrote: > > On Sun, 22 May 2011 20:12:04 -0500 > > Steve French <[email protected]> wrote: > > > >> On Sun, May 22, 2011 at 7:17 AM, Jeff Layton <[email protected]> wrote: > >> > On Sun, 22 May 2011 07:59:38 -0400 > >> > Christoph Hellwig <[email protected]> wrote: > >> > > >> >> On Sun, May 22, 2011 at 07:55:24AM -0400, Jeff Layton wrote: > >> >> > This will never work properly with CIFS, as the protocol has no > >> >> > ability > >> >> > whatsoever for looking up files by filehandle. It *might* be possible > >> >> > to > >> >> > eventually do this with SMB2, but that remains to be seen. > > Shirish had done some experiments (and AFAIK has a small patch > that fixed NFS export over CIFS which works, with the usual restrictions > about having to return ESTALE if the NFS client tries to access > an inode which has been flushed from the cache on the server side). > That restriction is not usual for servers, and makes the whole scheme unreliable. > >> For nfs v3 clients that can't handle ESTALE, we are probably > >> stuck with having to wait for Samba to implement the flag > > Flag? > > NTCreateX: FILE_OPEN_BY_FILE_ID > > IIRC Shirish verified that the Windows client will emit this flag, but their > server had not gotten around to implementing it (although Samba could > implement it now that their is an open-by-handle syscall). > Again, that does absolutely nothing for directories which also need to be retrievable by filehandle. I think it would be best to remove this code, but if that's unacceptable for some reason it should at least be marked BROKEN. -- Jeff Layton <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
