Re: [PATCH] Update the exportfs documentation and comments and fix some exportfs issues

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 03, 2008 at 08:27:48AM +0000, Phillip Lougher wrote:
> You define these in the filesystem c files themselves as #defines.  UDF  
> and BTRFS place theirs in the enum fid_type definition itself (in  
> linux/exportfs.h).  Is there any reason why you didn't do this?

The reason why the IDs for new filesystems go into exportfs.h to make
sure they don't clash for different filesystems.  While re-using the
same ID to mean a different format is technically fine, it's not very
nice for people trying to e.g. decipher NFS traffic in wireshark.

Somewhere on my TODO list is an item to walk through all the filesystems
and make them send out different IDs for new requests that don't overlap
while still accepting their legacy IDs for traffic from the time before
a server reboot.

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

[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux