[Devel] [PATCH 1/2] vzctl: synchronize CRIU with vzctl (v2)

Andrew Vagin avagin at parallels.com
Sat Jun 1 05:43:42 PDT 2013


On Fri, May 31, 2013 at 01:38:06PM -0700, Kir Kolyshkin wrote:
> >+	snprintf(ctpath, STR_SIZE, "%s/%d", NETNS_RUN_DIR, arg->veid);
> >+	snprintf(pidpath, STR_SIZE, "/proc/%d/ns/net", ret);
> >+	if (symlink(pidpath, ctpath)) {
> >+		logger(-1, errno, "Can't symlink into netns file %s", ctpath);
> >+		destroy_container(arg->veid);
> >+		return -VZ_RESOURCE_ERROR;
> >+	}
> >+
> 
> Can you please explain why are you doing this linking in two places --
> here and in vps-rst-env script?
>

We need to create this symlink before sending signal to signal_p,
because configuration scripts use it.

If we are talking about CT start, we can create it immediatly after
forking init. In case of criu we can do it only in setup-namespaces
scripts.



More information about the Devel mailing list