[CRIU] [PATCH 0/5] revert 246367e4e483 "add walk_all flag to walk_namespaces"
Oleg Nesterov
oleg at redhat.com
Tue Apr 14 09:53:47 PDT 2015
Hello.
Perhaps I am totally confused, this is quite possible. But it seems
that we need at least to try to cleanup this code. Before we try to
fix it. Lets start with the "we need the criu's mi if auto" logic.
To me, this "walk_all" logic buys nothing and just complicates the code.
Just look at this "ns->pid != getpid() || !(root_ns_mask & CLONE_NEWNS)"
check in collect_mntns(). It is horrible! At least it is certainly not
easy to understand it.
Tycho, could you please review? I am _not_ sure I understood this code
correctly. Help.
Oleg.
More information about the CRIU
mailing list