> +static int > +xfs_vn_change_ok( > + struct dentry *dentry, > + struct iattr *iattr) Please don't use the _vn prefix for something that's not an actual inode operation (and we should do a mess rename to _iop one day as well.) > +int > +xfs_vn_setattr_nonsize( > + struct dentry *dentry, > + struct iattr *iattr) Same here. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html