bin/115361: [zfs] mount(8) gets into a state where it won't set/unset ZFS properties (atime, exec, setuid)

[Scot Hetzel]

The following reply was made to PR bin/115361; it has been noted by GNATS.

From: Scot Hetzel <swhetzel at gmail.com>
To: Martin Matuska <mm at freebsd.org>
Cc: bug-followup at freebsd.org
Subject: Re: bin/115361: [zfs] mount(8) gets into a state where it won't
 set/unset ZFS properties (atime, exec, setuid)
Date: Sat, 9 Nov 2013 11:45:46 -0600

 On Tue, Oct 11, 2011 at 2:23 AM, Martin Matuska <mm at freebsd.org> wrote:
 > If there are no objections, I would like to close this PR.
 >
 > --
 > Martin Matuska
 > FreeBSD committer
 > http://blog.vx.sk
 >
 
 I just ran the zfstest script on:
 
 FreeBSD fbsd10 11.0-CURRENT FreeBSD 11.0-CURRENT #0 r257220: Sun Oct
 27 22:42:08 CDT 2013     root at fbsd10:/usr/obj/usr/src/sys/GENERIC
 i386
 
 The results below shows that mount still can't change devices, setuid
 or xattr attributes on the zfs filesystem (zfs get
 devices,setuid,xattr ${ZFS_FILESYSTEM}).  they should show as off, and
  SOURCE should change to temporary, but they stay at default.
 
 NOTE: change ZFS_FILESYSTEM and ZFS_MOUNTPOINT in the zfstest script
 to the appropriate zfs filesystem and mount point.
 
 ================================================================================
 
 Test setting/unsetting of devices
 
 ===> Current settings for Scratch/ports/distfiles
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, local, nfsv4acls)
 
 ===> mount -u -o nodevices /usr/ports/distfiles
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default <- should be off
 and temporary
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls) <- missing nodevices
 
 ===> mount -u -o devices /usr/ports/distfiles
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls)
 
 ================================================================================
 
 Test setting/unsetting of setuid
 ===> Current settings for Scratch/ports/distfiles
 NAME                     PROPERTY  VALUE   SOURCE
 Scratch/ports/distfiles  setuid    on      default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, local, nfsv4acls)
 
 ===> mount -u -o nosetuid /usr/ports/distfiles
 NAME                     PROPERTY  VALUE   SOURCE
 Scratch/ports/distfiles  setuid    on      default <- should be off
 and temporary
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nosuid, nfsv4acls)
 
 ===> mount -u -o setuid /usr/ports/distfiles
 NAME                     PROPERTY  VALUE   SOURCE
 Scratch/ports/distfiles  setuid    on      default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls)
 
 ================================================================================
 
 Test setting/unsetting of suid
 ===> Current settings for Scratch/ports/distfiles
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default
 Scratch/ports/distfiles  setuid    on       default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, local, nfsv4acls)
 
 ===> mount -u -o nosuid /usr/ports/distfiles
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default <- should be off
 and temporary
 Scratch/ports/distfiles  setuid    on       default  <- should be off
 and temporary
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nosuid, nfsv4acls) <- missing nodevices
 
 ===> mount -u -o nonosuid /usr/ports/distfiles
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default
 Scratch/ports/distfiles  setuid    on       default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls)
 
 ===> mount -u -o suid /usr/ports/distfiles
 =====> nosuid already set
 NAME                     PROPERTY  VALUE    SOURCE
 Scratch/ports/distfiles  devices   on       default
 Scratch/ports/distfiles  setuid    on       default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls)
 
 ================================================================================
 
 Test setting/unsetting of xattr
 ===> Current settings for Scratch/ports/distfiles
 NAME                     PROPERTY  VALUE  SOURCE
 Scratch/ports/distfiles  xattr     off    temporary
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, local, nfsv4acls)
 
 ===> mount -u -o noxattr /usr/ports/distfiles
 NAME                     PROPERTY  VALUE  SOURCE
 Scratch/ports/distfiles  xattr     off    temporary
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls) <- missing noxattr
 
 ===> mount -u -o xattr /usr/ports/distfiles
 NAME                     PROPERTY  VALUE  SOURCE
 Scratch/ports/distfiles  xattr     off    temporary <- should be on and default
 Scratch/ports/distfiles on /usr/ports/distfiles (zfs, NFS exported,
 local, nfsv4acls)
 
 ================================================================================
 
 -- 
 DISCLAIMER:
 
 No electrons were maimed while sending this message. Only slightly bruised.