>From b2b55d54582e6ca9895461f20adbc979d492291f Mon Sep 17 00:00:00 2001 From: Aaron Burgemeister Date: Thu, 15 Jul 2010 19:54:49 -0600 Subject: [PATCH] stat: add %m to output the mount point for a file MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit * src/find-mount-point.c: A new file refactoring find_mount_point() out from df.c * src/find-mount-point.h: Likewise. * src/df.c: Use the new find-mount-point module. * src/stat.c (print_stat): Using the new module, output the mount point for the new %m format directive. * src/stat.c (find_bind_mount): A new function to return the bind mount for a file if any. (usage): Document the %m format directive. * src/Makefile.am: Reference the refactored find-mount-point.c * po/POTFILES.in: Add find_mount_point.c to the translation list * doc/coreutils.texi (stat invocation): Document %m * test/misc/stat-mount: A new test to correlate mount points output from df and stat. * tests/Makefile.am: Reference the new test. * NEWS: Mention the new feature * THANKS: Add the author Signed-off-by: Pádraig Brady --- NEWS | 3 + THANKS | 1 + doc/coreutils.texi | 12 +++++ po/POTFILES.in | 1 + src/Makefile.am | 4 ++ src/df.c | 91 +-------------------------------------- src/find-mount-point.c | 113 ++++++++++++++++++++++++++++++++++++++++++++++++ src/find-mount-point.h | 17 +++++++ src/stat.c | 93 +++++++++++++++++++++++++++++++++++++++ tests/Makefile.am | 1 + tests/misc/stat-mount | 27 +++++++++++ 11 files changed, 273 insertions(+), 90 deletions(-) create mode 100644 src/find-mount-point.c create mode 100644 src/find-mount-point.h create mode 100755 tests/misc/stat-mount diff --git a/NEWS b/NEWS index ae7b839..bece07b 100644 --- a/NEWS +++ b/NEWS @@ -24,6 +24,9 @@ GNU coreutils NEWS -*- outline -*- sort now supports -d, -f, -i, -R, and -V in any combination. + stat now accepts the %m format directive to output + the mount point for a file. + ** Changes in behavior du now uses less than half as much memory when operating on trees diff --git a/THANKS b/THANKS index 34b95f1..c368a2c 100644 --- a/THANKS +++ b/THANKS @@ -8,6 +8,7 @@ the bug-report mailing list (as seen on last line of e.g., cp --help). ??? address@hidden A Costa address@hidden +Aaron Burgemeister address@hidden Aaron Hawley address@hidden Achim Blumensath address@hidden Adam Jimerson address@hidden diff --git a/doc/coreutils.texi b/doc/coreutils.texi index 7eedec7..9a3bd95 100644 --- a/doc/coreutils.texi +++ b/doc/coreutils.texi @@ -10685,6 +10685,7 @@ The valid @var{format} directives for files with @option{--format} and @item %G - Group name of owner @item %h - Number of hard links @item %i - Inode number address@hidden %m - Mount point (See note below) @item %n - File name @item %N - Quoted file name with dereference if symbolic link @item %o - I/O block size @@ -10701,6 +10702,17 @@ The valid @var{format} directives for files with @option{--format} and @item %Z - Time of last change as seconds since Epoch @end itemize +The mount point printed by @samp{%m} is similar to that output +by @command{df}, except that: address@hidden @bullet address@hidden stat does not dereference symlinks +by default (unless @option{-L} is specified) address@hidden stat does not search for specified device nodes in the +file system list, instead operating on them directly address@hidden stat outputs the aliased target of a bind mounted file, +rather than its backing device address@hidden itemize + When listing file system information (@option{--file-system} (@option{-f})), you must use a different set of @var{format} directives: diff --git a/po/POTFILES.in b/po/POTFILES.in index be20f4c..673a7d9 100644 --- a/po/POTFILES.in +++ b/po/POTFILES.in @@ -61,6 +61,7 @@ src/expand.c src/expr.c src/factor.c src/false.c +src/find-mount-point.c src/fmt.c src/fold.c src/getlimits.c diff --git a/src/Makefile.am b/src/Makefile.am index 1a19fa6..00c7ff7 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -145,6 +145,7 @@ noinst_HEADERS = \ copy.h \ cp-hash.h \ dircolors.h \ + find-mount-point.h \ fs.h \ group-list.h \ ls.h \ @@ -478,6 +479,9 @@ rm_SOURCES = rm.c remove.c mkdir_SOURCES = mkdir.c prog-fprintf.c rmdir_SOURCES = rmdir.c prog-fprintf.c +df_SOURCES = df.c find-mount-point.c +stat_SOURCES = stat.c find-mount-point.c + uname_SOURCES = uname.c uname-uname.c arch_SOURCES = uname.c uname-arch.c nproc_SOURCES = nproc.c diff --git a/src/df.c b/src/df.c index 76622fb..97b1637 100644 --- a/src/df.c +++ b/src/df.c @@ -29,8 +29,7 @@ #include "human.h" #include "mountlist.h" #include "quote.h" -#include "save-cwd.h" -#include "xgetcwd.h" +#include "find-mount-point.h" /* The official name of this program (e.g., no `g' prefix). */ #define PROGRAM_NAME "df" @@ -522,94 +521,6 @@ show_dev (char const *disk, char const *mount_point, putchar ('\n'); } -/* Return the root mountpoint of the file system on which FILE exists, in - malloced storage. FILE_STAT should be the result of stating FILE. - Give a diagnostic and return NULL if unable to determine the mount point. - Exit if unable to restore current working directory. */ -static char * -find_mount_point (const char *file, const struct stat *file_stat) -{ - struct saved_cwd cwd; - struct stat last_stat; - char *mp = NULL; /* The malloced mount point. */ - - if (save_cwd (&cwd) != 0) - { - error (0, errno, _("cannot get current directory")); - return NULL; - } - - if (S_ISDIR (file_stat->st_mode)) - /* FILE is a directory, so just chdir there directly. */ - { - last_stat = *file_stat; - if (chdir (file) < 0) - { - error (0, errno, _("cannot change to directory %s"), quote (file)); - return NULL; - } - } - else - /* FILE is some other kind of file; use its directory. */ - { - char *xdir = dir_name (file); - char *dir; - ASSIGN_STRDUPA (dir, xdir); - free (xdir); - - if (chdir (dir) < 0) - { - error (0, errno, _("cannot change to directory %s"), quote (dir)); - return NULL; - } - - if (stat (".", &last_stat) < 0) - { - error (0, errno, _("cannot stat current directory (now %s)"), - quote (dir)); - goto done; - } - } - - /* Now walk up FILE's parents until we find another file system or /, - chdiring as we go. LAST_STAT holds stat information for the last place - we visited. */ - while (true) - { - struct stat st; - if (stat ("..", &st) < 0) - { - error (0, errno, _("cannot stat %s"), quote ("..")); - goto done; - } - if (st.st_dev != last_stat.st_dev || st.st_ino == last_stat.st_ino) - /* cwd is the mount point. */ - break; - if (chdir ("..") < 0) - { - error (0, errno, _("cannot change to directory %s"), quote ("..")); - goto done; - } - last_stat = st; - } - - /* Finally reached a mount point, see what it's called. */ - mp = xgetcwd (); - -done: - /* Restore the original cwd. */ - { - int save_errno = errno; - if (restore_cwd (&cwd) != 0) - error (EXIT_FAILURE, errno, - _("failed to return to initial working directory")); - free_cwd (&cwd); - errno = save_errno; - } - - return mp; -} - /* If DISK corresponds to a mount point, show its usage and return true. Otherwise, return false. */ static bool diff --git a/src/find-mount-point.c b/src/find-mount-point.c new file mode 100644 index 0000000..f3a2074 --- /dev/null +++ b/src/find-mount-point.c @@ -0,0 +1,113 @@ +/* find-mount-point.c -- find the root mount point for a file. + Copyright (C) 2010 Free Software Foundation, Inc. + + This program is free software: you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation, either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +#include +#include + +#include "system.h" +#include "error.h" +#include "quote.h" +#include "save-cwd.h" +#include "xgetcwd.h" +#include "find-mount-point.h" + +/* Return the root mountpoint of the file system on which FILE exists, in + malloced storage. FILE_STAT should be the result of stating FILE. + Give a diagnostic and return NULL if unable to determine the mount point. + Exit if unable to restore current working directory. */ +extern char * +find_mount_point (const char *file, const struct stat *file_stat) +{ + struct saved_cwd cwd; + struct stat last_stat; + char *mp = NULL; /* The malloced mount point. */ + + if (save_cwd (&cwd) != 0) + { + error (0, errno, _("cannot get current directory")); + return NULL; + } + + if (S_ISDIR (file_stat->st_mode)) + /* FILE is a directory, so just chdir there directly. */ + { + last_stat = *file_stat; + if (chdir (file) < 0) + { + error (0, errno, _("cannot change to directory %s"), quote (file)); + return NULL; + } + } + else + /* FILE is some other kind of file; use its directory. */ + { + char *xdir = dir_name (file); + char *dir; + ASSIGN_STRDUPA (dir, xdir); + free (xdir); + + if (chdir (dir) < 0) + { + error (0, errno, _("cannot change to directory %s"), quote (dir)); + return NULL; + } + + if (stat (".", &last_stat) < 0) + { + error (0, errno, _("cannot stat current directory (now %s)"), + quote (dir)); + goto done; + } + } + + /* Now walk up FILE's parents until we find another file system or /, + chdiring as we go. LAST_STAT holds stat information for the last place + we visited. */ + while (true) + { + struct stat st; + if (stat ("..", &st) < 0) + { + error (0, errno, _("cannot stat %s"), quote ("..")); + goto done; + } + if (st.st_dev != last_stat.st_dev || st.st_ino == last_stat.st_ino) + /* cwd is the mount point. */ + break; + if (chdir ("..") < 0) + { + error (0, errno, _("cannot change to directory %s"), quote ("..")); + goto done; + } + last_stat = st; + } + + /* Finally reached a mount point, see what it's called. */ + mp = xgetcwd (); + +done: + /* Restore the original cwd. */ + { + int save_errno = errno; + if (restore_cwd (&cwd) != 0) + error (EXIT_FAILURE, errno, + _("failed to return to initial working directory")); + free_cwd (&cwd); + errno = save_errno; + } + + return mp; +} diff --git a/src/find-mount-point.h b/src/find-mount-point.h new file mode 100644 index 0000000..49be9ba --- /dev/null +++ b/src/find-mount-point.h @@ -0,0 +1,17 @@ +/* find-mount-point.h -- find the root mount point for a file. + Copyright (C) 2010 Free Software Foundation, Inc. + + This program is free software: you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation, either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +extern char* find_mount_point (const char *, const struct stat *); diff --git a/src/stat.c b/src/stat.c index c3730f0..0a2162c 100644 --- a/src/stat.c +++ b/src/stat.c @@ -64,10 +64,12 @@ #include "filemode.h" #include "fs.h" #include "getopt.h" +#include "mountlist.h" #include "quote.h" #include "quotearg.h" #include "stat-time.h" #include "strftime.h" +#include "find-mount-point.h" #if USE_STATVFS # define STRUCT_STATVFS struct statvfs @@ -179,6 +181,9 @@ static bool interpret_backslash_escapes; "" for --printf=FMT, "\n" for --format=FMT (-c). */ static char const *trailing_delim = ""; +/* Linked list of mounted file systems. */ +static struct mount_entry *mount_list; + /* Return the type of the specified file system. Some systems have statfvs.f_basetype[FSTYPSZ] (AIX, HP-UX, and Solaris). Others have statvfs.f_fstypename[_VFS_NAMELEN] (NetBSD 3.0). @@ -604,6 +609,90 @@ print_statfs (char *pformat, size_t prefix_len, char m, char const *filename, return fail; } +/* Return any bind mounted target for a path. + The result does _not_ have to be freed. + NULL is returned if no bind mount is found. */ +static char const * ATTRIBUTE_WARN_UNUSED_RESULT +find_bind_mount (char const * name) +{ + char const * bind_mount = NULL; + + static bool tried_mount_list = false; + if (!tried_mount_list) /* attempt/warn once per process. */ + { + if (!(mount_list = read_file_system_list (false))) + error (0, errno, "%s", _("cannot read table of mounted file systems")); + tried_mount_list = true; + } + + struct mount_entry *me; + for (me = mount_list; me; me = me->me_next) + { + if (me->me_dummy && me->me_devname[0] == '/' + && STREQ (me->me_mountdir, name)) + { + struct stat name_stats; + struct stat dev_stats; + + if (stat (name, &name_stats) == 0 + && stat (me->me_devname, &dev_stats) == 0 + && SAME_INODE (name_stats, dev_stats)) + { + bind_mount = me->me_devname; + break; + } + } + } + + return bind_mount; +} + +/* Print mount point. Return zero upon success, nonzero upon failure. */ + /* TODO: see if there is a way to correlate the test also */ +static bool ATTRIBUTE_WARN_UNUSED_RESULT +out_mount_point (char const *filename, char *pformat, size_t prefix_len, + const struct stat *statp) +{ + /* Look for bind mounts first. Note we ouput the immediate target, + rather than further resolving to a base device mount point. */ + char const *bind_mount = NULL; + + /* It's not possible to bind mount a symlink, + so there is no inconsistency here with + canonicalizing when -L is not specified. */ + char *resolved = canonicalize_file_name (filename); + if (!resolved) + { + out_string (pformat, prefix_len, "?"); + return true; + } + bind_mount = find_bind_mount (resolved); + free (resolved); + if (bind_mount) + { + out_string (pformat, prefix_len, bind_mount); + return false; + } + + /* If there is no direct bind mount, then + traverse back up the tree looking for a device change. */ + char *mp = find_mount_point (filename, statp); + if (!mp) + { + out_string (pformat, prefix_len, "?"); + return true; + } + else + { + /* This dir might be bind mounted to another device, + so we resolve the bound target in that case also. */ + bind_mount = find_bind_mount (mp); + out_string (pformat, prefix_len, bind_mount ? bind_mount : mp); + free (mp); + return false; + } +} + /* Print stat info. Return zero upon success, nonzero upon failure. */ static bool print_stat (char *pformat, size_t prefix_len, char m, @@ -679,6 +768,9 @@ print_stat (char *pformat, size_t prefix_len, char m, case 't': out_uint_x (pformat, prefix_len, major (statbuf->st_rdev)); break; + case 'm': + fail |= out_mount_point (filename, pformat, prefix_len, statbuf); + break; case 'T': out_uint_x (pformat, prefix_len, minor (statbuf->st_rdev)); break; @@ -1025,6 +1117,7 @@ The valid format sequences for files (without --file-system):\n\ fputs (_("\ %h Number of hard links\n\ %i Inode number\n\ + %m Mount point\n\ %n File name\n\ %N Quoted file name with dereference if symbolic link\n\ %o I/O block size\n\ diff --git a/tests/Makefile.am b/tests/Makefile.am index 4aa93bf..5619d0b 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -241,6 +241,7 @@ TESTS = \ misc/split-l \ misc/stat-fmt \ misc/stat-hyphen \ + misc/stat-mount \ misc/stat-printf \ misc/stat-slash \ misc/stdbuf \ diff --git a/tests/misc/stat-mount b/tests/misc/stat-mount new file mode 100755 index 0000000..99e29cb --- /dev/null +++ b/tests/misc/stat-mount @@ -0,0 +1,27 @@ +#!/bin/sh +# Test stat -c%m + +# Copyright (C) 2010 Free Software Foundation, Inc. + +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation, either version 3 of the License, or +# (at your option) any later version. + +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. + +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +. "${srcdir=.}/init.sh"; path_prepend_ ../src + + +df_mnt=$(df -P . | sed -n '2s/.* \([^ ]*$\)/\1/p') +stat_mnt=$(stat -c%m .) + +test "$df_mnt" = "$stat_mnt" || fail=1 + +Exit $fail -- 1.6.2.5