Re: [PATCH] nfs-utils: make auth_reload respect sub-second timestamps on etab

[Neil Brown <neilb@suse.de>]

On Wednesday April 25, chuck.lever@oracle.com wrote:
> 
> Hi Jeff-
> 
> I don't know much about the export logic, but why not use inode change 
> notification instead of time stamps?

inotify is 2.6 only I think.  I'm not sure I want to exclude 2.4
people just yet (or do I ??).

Given that exportfs replaces the file with a new one, would we need to
get notification on the directory, or just on the file?  Does it make
it more complicated, or easier?

Actually, mountd could open etab and hold it open. auth_reload could
stat "/var/lib/nfs/etab" and if it is a different file, then the
content is presumably different, while if it is the same file, we can
be certain the content is unchanged.

Further, we would be certain only to read each version once - zero
risk of re-parsing data when there is no change.

That might be an even better option .... anyone want to code it up??

Or is it wrong to depend on current exportfs behaviour?

NeilBrown

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs