tail: with -f, use nanosleep (not inotify) on a GPFS file system
authorMartial Bornet <mbornet.pro@gmail.com>
Thu, 20 Oct 2011 17:18:09 +0000 (19:18 +0200)
committerJim Meyering <meyering@redhat.com>
Fri, 21 Oct 2011 07:03:30 +0000 (09:03 +0200)
Because tail's fremote function did not designate GPFS as
a remote file system type, tail -f would mistakenly attempt
to use inotify, which cannot work with a remote file system.
* src/tail.c (fremote): List GPFS as a remote file system type.
* NEWS (Bug fixes): Mention it.
Reported in http://bugs.debian.org/646022.

NEWS
src/tail.c

diff --git a/NEWS b/NEWS
index 3ed44b2..4d210b5 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -7,6 +7,10 @@ GNU coreutils NEWS                                    -*- outline -*-
   tac no longer fails to handle two or more non-seekable inputs
   [bug introduced in coreutils-5.3.0]
 
+  tail -f no longer tries to use inotify on GPFS file systems
+  [you might say this was introduced in coreutils-7.5, along with inotify
+   support, but the GPFS magic number wasn't in the usual places then.]
+
 
 * Noteworthy changes in release 8.14 (2011-10-12) [stable]
 
index f315776..1641a12 100644 (file)
@@ -904,6 +904,7 @@ fremote (int fd, const char *name)
         case S_MAGIC_FUSEBLK:
         case S_MAGIC_FUSECTL:
         case S_MAGIC_GFS:
+        case S_MAGIC_GPFS:
         case S_MAGIC_KAFS:
         case S_MAGIC_LUSTRE:
         case S_MAGIC_NCP: