Re: [PATCH v2] mountlist: additional dummy FS names for Linux

[Eric Wong]

Jim Meyering <address@hidden> wrote:
> Thanks.  I've tweaked the log and added a matching ChangeLog entry.
> Testing via df on my F18 laptop, diff -u <(src/df-prev) <(src/df)
> shows no difference.  What do people think of this entry from
> that F18 system?
> 
>     devtmpfs         1538304        0   1538304   0% /dev
> 
> If we're omitting "devfs", then should "devtmpfs" also be omitted?

Yes, I believe devtmpfs should be omitted.  I never noticed the
kernel option for it until now :x

Since I've been playing with SystemTap, I've also noticed "debugfs"
on some systems.

Here is an updated patch with debugfs and devtmpfs:

>From b0904cbd02932009e0b9de568ce4742d27a18e78 Mon Sep 17 00:00:00 2001
From: Eric Wong <address@hidden>
Date: Fri, 7 Dec 2012 23:14:34 +0000
Subject: [PATCH] mountlist: recognize more "dummy" file systems

* lib/mountlist.c (ME_DUMMY_0):
  Add these dummy FS names to the list:
  - "debugfs" virtual filesystem for kernel debugging
  - "devpts" PTY slave filesystem
  - "devtmpfs" device filesystem on top of tmpfs/ramfs
  - "fusectl" control filesystem for FUSE
  - "mqueue" enumerates POSIX message queues
  - "rpc_pipefs" kernel <-> userspace bridge for NFS
  - "sysfs" is for exporting kernel objects
  - "devfs" device filesystem for Linux 2.4 and FreeBSD
---
 ChangeLog       | 14 ++++++++++++++
 lib/mountlist.c | 10 ++++++++++
 2 files changed, 24 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index a49fc07..0fdcb11 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2012-12-07  Eric Wong  <address@hidden>
+
+       mountlist: recognize more "dummy" file systems
+       * lib/mountlist.c (ME_DUMMY_0):
+       Add these dummy FS names to the list:
+       - "debugfs" virtual filesystem for kernel debugging
+       - "devpts" PTY slave filesystem
+       - "devtmpfs" device filesystem on top of tmpfs/ramfs
+       - "fusectl" control filesystem for FUSE
+       - "mqueue" enumerates POSIX message queues
+       - "rpc_pipefs" kernel <-> userspace bridge for NFS
+       - "sysfs" is for exporting kernel objects
+       - "devfs" device filesystem for Linux 2.4 and FreeBSD
+
 2012-12-10  Eli Zaretskii  <address@hidden>
 
        canonicalize, canonicalize-lgpl: Microsoft Windows prefix fixes
diff --git a/lib/mountlist.c b/lib/mountlist.c
index d0fe1b2..e61d37a 100644
--- a/lib/mountlist.c
+++ b/lib/mountlist.c
@@ -153,6 +153,16 @@
   (strcmp (Fs_type, "autofs") == 0              \
    || strcmp (Fs_type, "proc") == 0             \
    || strcmp (Fs_type, "subfs") == 0            \
+   /* for Linux 2.6/3.x */                      \
+   || strcmp (Fs_type, "debugfs") == 0          \
+   || strcmp (Fs_type, "devpts") == 0           \
+   || strcmp (Fs_type, "devtmpfs") == 0         \
+   || strcmp (Fs_type, "fusectl") == 0          \
+   || strcmp (Fs_type, "mqueue") == 0           \
+   || strcmp (Fs_type, "rpc_pipefs") == 0       \
+   || strcmp (Fs_type, "sysfs") == 0            \
+   /* FreeBSD, Linux 2.4 */                     \
+   || strcmp (Fs_type, "devfs") == 0            \
    /* for NetBSD 3.0 */                         \
    || strcmp (Fs_type, "kernfs") == 0           \
    /* for Irix 6.5 */                           \
-- 
Eric Wong