Remove duplicated code in tmpfs_chflags()

Antonio Huete Jimenez ahuete.devel at gmail.com
Sat Jun 29 04:28:35 PDT 2013


Hi,

Please see attached patch:

http://leaf.dragonflybsd.org/~tuxillo/archive/patches/tmpfs_chflags01.diff

The code that was removed seems duplicated to me as
vop_helper_setattr_flags function already does the trick. Also
SF_NOCACHE is set on VOP_MOUNT, see
http://bxr.su/DragonFly/sys/vfs/tmpfs/tmpfs_vfsops.c#267. There is no
need for a check to VROOT in every single chflags call either IMO.

The patch has been tested with schg/noschg:

-rw-r--r--  1 root  wheel  schg 0 Jun 29 13:05 e
vkernel64 # rm e
override rw-r--r--  root/wheel schg for e? y
rm: e: Operation not permitted
vkernel64 # chflags noschg e
vkernel64 # rm e

Cheers,
Antonio Huete



More information about the Kernel mailing list