[CRIU] [PATCH v2 5/5] introduce --enable-fs cli option

Pavel Emelyanov xemul at parallels.com
Thu Apr 9 12:17:34 PDT 2015


I like the set, but one thing worries me

> +
> +bool add_fsname_auto(const char *names)
> +{
> +	xfree(fsauto_names);
> +	fsauto_names = xstrdup(names);
> +	return fsauto_names != NULL;
>  }

If I do "criu blah-blah --enable-fs xfs --enable-fs yfs --enable-fs zfs"
then only zfs will be _silently_ enabled. Can we somehow make this fact
be explicit to users?

-- Pavel


More information about the CRIU mailing list