From d191bfe8a688d40aac0e3626b434b54ddbafd072 Mon Sep 17 00:00:00 2001 From: Martial Bornet Date: Thu, 20 Oct 2011 19:18:09 +0200 Subject: [PATCH] tail: with -f, use nanosleep (not inotify) on a GPFS file system 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 | 4 ++++ src/tail.c | 1 + 2 files changed, 5 insertions(+) diff --git a/NEWS b/NEWS index 3ed44b2..4d210b5 100644 --- 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] diff --git a/src/tail.c b/src/tail.c index f315776..1641a12 100644 --- a/src/tail.c +++ b/src/tail.c @@ -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: -- 2.7.4