Quoting Oren Laadan ([email protected]): > From: Dan Smith <[email protected]> > > Changes: > - Update to match UTS changes > > Signed-off-by: Dan Smith <[email protected]> > Signed-off-by: Oren Laadan <[email protected]>
Acked-by: Serge Hallyn <[email protected]> However... > + if (!!ipc_ns ^ !(flags & CLONE_NEWIPC)) > + return -EINVAL; Every time I see this I have to think about whether it is right or not. I'm not sure whether it's worth commenting (at each such meme) that CLONE_NEWIPC only needed to be set the first time we ran across that ipcns, or whether it's indicative that there is a simpler way the code could be done. But if it just took me a twice-over to see that it's right, when I'd already confirmed that with the CLONE_NEWUTS version last week, then a fresh reviewer will be cursing your name... -serge _______________________________________________ Containers mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/containers _______________________________________________ Devel mailing list [email protected] https://openvz.org/mailman/listinfo/devel
