bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] mountlist: recognize more file system types as remote


From: Pádraig Brady
Subject: Re: [PATCH] mountlist: recognize more file system types as remote
Date: Tue, 3 Nov 2020 14:08:01 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:83.0) Gecko/20100101 Thunderbird/83.0

On 03/11/2020 08:31, Kamil Dudka wrote:
On Tuesday, October 27, 2020 10:23:15 PM CET Pádraig Brady wrote:
Sync "remote" file systems from stat.c in coreutils.
Note we only consider file systems that do not use host:resource
mount source.  I.e. those that don't generally use a colon when
mounting, as that case is already considered.  Searching for
"<fstype> /etc/fstab" was informative for identifying these.
The full list of "remote" file systems in coreutils is currently:
   acfs afs ceph cifs coda fhgfs fuseblk fusectl
   gfs gfs2 gpfs ibrix k-afs lustre novell nfs nfsd
   ocfs2 panfs prl_fs smb smb2 snfs vboxsf vmhgfs vxfs
Note also we include virtual machine file systems,
as though they may not be distributed to separate hosts,
they're remote to the current kernel.

* lib/mountlist.c (ME_REMOTE): Sync previously unconsidered
"remote" file systems from stat.c in coreutils.

Looks good to me.

Kamil

---
  ChangeLog       |  7 +++++++
  lib/mountlist.c | 11 ++++++++++-
  2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 4eb5bcb1d..41e284558 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2020-10-27  Pádraig Brady  <P@draigBrady.com>
+
+       mountlist: recognize more file system types as remote
+
+       * lib/mountlist.c (ME_REMOTE): Sync previously unconsidered
+       "remote" file systems from stat.c in coreutils.
+
  2020-10-26  Paul Eggert  <eggert@cs.ucla.edu>

        sys_stat: update comments for S_IRWXUGO, S_IXUGO
diff --git a/lib/mountlist.c b/lib/mountlist.c
index ca1be63f2..8205c29bd 100644
--- a/lib/mountlist.c
+++ b/lib/mountlist.c
@@ -224,7 +224,7 @@ me_remote (char const *fs_name, char const *fs_type
_GL_UNUSED) #ifndef ME_REMOTE
  /* A file system is "remote" if its Fs_name contains a ':'
     or if (it is of type (smbfs or cifs) and its Fs_name starts with '//')
-   or if it is of type (afs or auristorfs)
+   or if it is of any other of the listed types
     or Fs_name is equal to "-hosts" (used by autofs to mount remote fs).  */
# define ME_REMOTE(Fs_name, Fs_type)            \
      (strchr (Fs_name, ':') != NULL              \
@@ -233,8 +233,17 @@ me_remote (char const *fs_name, char const *fs_type
_GL_UNUSED) && (strcmp (Fs_type, "smbfs") == 0     \

               || strcmp (Fs_type, "smb3") == 0   \
               || strcmp (Fs_type, "cifs") == 0)) \

+     || strcmp (Fs_type, "acfs") == 0           \

       || strcmp (Fs_type, "afs") == 0            \

+     || strcmp (Fs_type, "coda") == 0           \

       || strcmp (Fs_type, "auristorfs") == 0     \

+     || strcmp (Fs_type, "fhgfs") == 0          \
+     || strcmp (Fs_type, "gpfs") == 0           \
+     || strcmp (Fs_type, "ibrix") == 0          \
+     || strcmp (Fs_type, "ocfs2") == 0          \
+     || strcmp (Fs_type, "prl_fs") == 0         \
+     || strcmp (Fs_type, "vboxsf") == 0         \
+     || strcmp (Fs_type, "vxfs") == 0           \

       || strcmp ("-hosts", Fs_name) == 0)

  #endif

Thanks for the review.

Upon further consideration I'm reluctant to include
the "VM" file systems here. I.e. vboxsf and prl_fs,
as they do consume space on the local machine.
I'll add a comment about those instead, and will push.

cheers,
Pádraig



reply via email to

[Prev in Thread] Current Thread [Next in Thread]