git: sbin/hammer: Don't print mountpoint of PFS

Tomohiro Kusumi tkusumi at crater.dragonflybsd.org
Tue Aug 23 05:34:46 PDT 2016


commit d6c59c720c8103f8d9c75cf66dcfbd14d3ecdc90
Author: Tomohiro Kusumi <kusumi.tomohiro at gmail.com>
Date:   Tue Aug 23 19:59:26 2016 +0900

    sbin/hammer: Don't print mountpoint of PFS
    
    /sbin/hammer should not care if PFS are null mounted or where PFS are
    mounted. It has nothing to do with the filesystem itself. PFS is just
    a pointer to jump into a subset of HAMMER's B-Tree (clustered by
    localization parameter). Just because DragonFly's installer used to
    use PFS (not anymore) and null mount by default doesn't mean /sbin
    /hammer needs to provide null mount info.
    
    If someone does mount PFS to somewhere and want to know where PFS are
    mounted, one needs to just type mount. In fact this is much better
    than getting mountpoints info via hammer info command because mount(8)
    doesn't use hammer's ioctl which may cause B-Tree lookup, but instead
    retrieves mountpoints directly from vfs.
    
    Also note that printing such info gives users wrong assumption that
    PFS in HAMMER is something equivalent of ZFS (created out of storage
    zpool) which is not true. As mentioned above, PFS is just a pointer to
    a subset of a single HAMMER filesystem. These are not independent fs.
    
    Also note that hammer info still has bugs due to bugs from libhammer
    and its attached ioctl as mentioned in e0f42079, including things under
    investigation now. Whatever it is, hammer info needs to be rewritten
    (as a different command like hammer fsinfo) in short and straight
    forward code like other commands, without unnecessary abstraction.
    I've been fixing and adding missing stuff to this hammer info command,
    however hammer info is basically going nowhere because this command is
    too tightly attached to libhammer.

Summary of changes:
 sbin/hammer/cmd_info.c | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/d6c59c720c8103f8d9c75cf66dcfbd14d3ecdc90


-- 
DragonFly BSD source repository



More information about the Commits mailing list