diff options
| author | Ben Myers <bpm@sgi.com> | 2010-02-12 14:02:21 -0500 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2010-02-12 14:21:04 -0500 |
| commit | 7b0e4faaf3169c69828d3f7769ed8ce9142e100b (patch) | |
| tree | f174af3471072112481fd30a758fb858468a66b8 /support/nfs/rmtab.c | |
| parent | 2ba89d65421cba255a0e75a7f646e4bbb2487193 (diff) | |
| download | nfs-utils-7b0e4faaf3169c69828d3f7769ed8ce9142e100b.tar.gz nfs-utils-7b0e4faaf3169c69828d3f7769ed8ce9142e100b.tar.xz nfs-utils-7b0e4faaf3169c69828d3f7769ed8ce9142e100b.zip | |
nfs-utils: don't fdatasync the rmtab
If we're using the new caching interface the rmtab will be ignored by
exportfs so there is no need to fdatasync. This improves mountd performance.
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'support/nfs/rmtab.c')
| -rw-r--r-- | support/nfs/rmtab.c | 23 |
1 files changed, 18 insertions, 5 deletions
diff --git a/support/nfs/rmtab.c b/support/nfs/rmtab.c index 4cbd285..a28abf3 100644 --- a/support/nfs/rmtab.c +++ b/support/nfs/rmtab.c @@ -117,11 +117,24 @@ void fendrmtabent(FILE *fp) { if (fp) { - /* If it was written to, we really want - * to flush to disk before returning - */ - fflush(fp); - fdatasync(fileno(fp)); + static int have_new_cache = -1; + if (have_new_cache == -1) /* check only once */ + have_new_cache = check_new_cache(); + + if (!have_new_cache) { + /* + * If we are using the old caching interface: exportfs + * uses the rmtab to determine what should be exported, + * so it is important that it be up-to-date. + * + * If we are using the new caching interface: the rmtab + * is ignored by exportfs and the fdatasync only serves + * to slow us down. + */ + fflush(fp); + fdatasync(fileno(fp)); + } + fclose(fp); } } |