diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/dd.c | 7 | ||||
-rw-r--r-- | src/sort.c | 30 |
2 files changed, 10 insertions, 27 deletions
@@ -27,10 +27,10 @@ #include "system.h" #include "close-stream.h" #include "error.h" +#include "fadvise.h" #include "fd-reopen.h" #include "gethrxtime.h" #include "human.h" -#include "ignore-value.h" #include "long-options.h" #include "quote.h" #include "quotearg.h" @@ -849,12 +849,9 @@ iwrite (int fd, char const *buf, size_t size) posix_fadvise to tell the system not to pollute the buffer cache with this data. Don't bother to diagnose lseek or posix_fadvise failure. */ -#ifdef POSIX_FADV_DONTNEED off_t off = lseek (STDOUT_FILENO, 0, SEEK_CUR); if (0 <= off) - ignore_value (posix_fadvise (STDOUT_FILENO, - off, 0, POSIX_FADV_DONTNEED)); -#endif + fdadvise (STDOUT_FILENO, off, 0, FADVISE_DONTNEED); /* Attempt to ensure that that final block is committed to disk as quickly as possible. */ diff --git a/src/sort.c b/src/sort.c index afa45a8b3..fa59e6ac3 100644 --- a/src/sort.c +++ b/src/sort.c @@ -30,11 +30,11 @@ #include "system.h" #include "argmatch.h" #include "error.h" +#include "fadvise.h" #include "filevercmp.h" #include "hard-locale.h" #include "hash.h" #include "heap.h" -#include "ignore-value.h" #include "md5.h" #include "mbswidth.h" #include "nproc.h" @@ -856,9 +856,13 @@ create_temp_file (int *pfd, bool survive_fd_exhaustion) return node; } -/* Predeclare an access pattern for input files. - Ignore any errors -- this is only advisory. +/* Return a stream for FILE, opened with mode HOW. A null FILE means + standard output; HOW should be "w". When opening for input, "-" + means standard input. To avoid confusion, do not return file + descriptors STDIN_FILENO, STDOUT_FILENO, or STDERR_FILENO when + opening an ordinary FILE. Return NULL if unsuccessful. + fadvise() is used to specify an access pattern for input files. There are a few hints we could possibly provide, and after careful testing it was decided that specifying POSIX_FADV_SEQUENTIAL was not detrimental @@ -899,24 +903,6 @@ create_temp_file (int *pfd, bool survive_fd_exhaustion) cache immediately after processing. This is done implicitly however when the files are unlinked. */ -static void -fadvise_input (FILE *fp) -{ -#if HAVE_POSIX_FADVISE - if (fp) - { - int fd = fileno (fp); - ignore_value (posix_fadvise (fd, 0, 0, POSIX_FADV_SEQUENTIAL)); - } -#endif -} - -/* Return a stream for FILE, opened with mode HOW. A null FILE means - standard output; HOW should be "w". When opening for input, "-" - means standard input. To avoid confusion, do not return file - descriptors STDIN_FILENO, STDOUT_FILENO, or STDERR_FILENO when - opening an ordinary FILE. Return NULL if unsuccessful. */ - static FILE * stream_open (const char *file, const char *how) { @@ -932,7 +918,7 @@ stream_open (const char *file, const char *how) } else fp = fopen (file, how); - fadvise_input (fp); + fadvise (fp, FADVISE_SEQUENTIAL); return fp; } return fopen (file, how); |