On 09/20/2011 01:05 PM, Chuck Lever wrote: > > On Sep 20, 2011, at 12:57 PM, Steve Dickson wrote: > >> To allow greater flexibility to where statd's state is kept, >> statd's state path can now be decoupled from the normal >> NFS state directory. >> >> In configure.ac, the NSM_STATD_PATH definition will now define > > In the description: > s/NSM_STATD_PATH/NSM_DEFAULT_STATEDIR/ > > Otherwise, > > Reviewed-by: Chuck Lever <chuck.lever@xxxxxxxxxx> Committed with the above change... steved. > >> the path to where the state information is kept. The default >> value, /var/lib/nfs, can be redefined with the --with-statdpath >> flag. >> >> Signed-off-by: Steve Dickson <steved@xxxxxxxxxx> >> --- >> configure.ac | 9 +++++++++ >> support/nsm/file.c | 8 -------- >> 2 files changed, 9 insertions(+), 8 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index b5934c4..500172b 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -23,6 +23,14 @@ AC_ARG_WITH(statedir, >> statedir=$withval, >> statedir=/var/lib/nfs) >> AC_SUBST(statedir) >> +AC_ARG_WITH(statdpath, >> + [AC_HELP_STRING([--with-statdpath=/foo @<:@default=/var/lib/nfs@:>@], >> + [define statd's state dir as /foo instead of the NFS statedir] >> + )], >> + statdpath=$withval, >> + statdpath=$statedir >> + ) >> + AC_SUBST(statdpath) >> AC_ARG_WITH(statduser, >> [AC_HELP_STRING([--with-statduser=rpcuser], >> [statd to run under @<:@rpcuser or nobody@:>@] >> @@ -387,6 +395,7 @@ dnl ************************************************************* >> dnl Export some path names to config.h >> dnl ************************************************************* >> AC_DEFINE_UNQUOTED(NFS_STATEDIR, "$statedir", [This defines the location of the NFS state files. Warning: this must match definitions in config.mk!]) >> +AC_DEFINE_UNQUOTED(NSM_DEFAULT_STATEDIR, "$statdpath", [Define this to the pathname where statd keeps its state file]) >> >> if test "x$cross_compiling" = "xno"; then >> CFLAGS_FOR_BUILD=${CFLAGS_FOR_BUILD-"$CFLAGS"} >> diff --git a/support/nsm/file.c b/support/nsm/file.c >> index a12c753..5dd52c1 100644 >> --- a/support/nsm/file.c >> +++ b/support/nsm/file.c >> @@ -94,14 +94,6 @@ >> >> #define NSM_KERNEL_STATE_FILE "/proc/sys/fs/nfs/nsm_local_state" >> >> -/* >> - * Some distributions place statd's files in a subdirectory >> - */ >> -#define NSM_PATH_EXTENSION >> -/* #define NSM_PATH_EXTENSION "/statd" */ >> - >> -#define NSM_DEFAULT_STATEDIR NFS_STATEDIR NSM_PATH_EXTENSION >> - >> static char nsm_base_dirname[PATH_MAX] = NSM_DEFAULT_STATEDIR; >> >> #define NSM_MONITOR_DIR "sm" >> -- >> 1.7.6.2 >> >> -- >> 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 > -- 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