Eric W. Biederman [ebiederm@xxxxxxxxxxxx] wrote: | Oleg Nesterov <oleg@xxxxxxxxxx> writes: | | > Yes! I meant we should change INIT_SIGNALS(), currently it does | > | > #define INIT_SIGNALS(sig) { | > ... | > .pgrp = 1, | > { .__session = 1}, | > | > and this confuses (I think) set_special_pids(1,1) above. Because | > __set_special_pids() still deals with pid_t, not "struct pid". | > | > Unless I missed something, we should kill these 2 initializations | > above. | | Got it. I agree we should initialize those fields to 0. | | Sukadev you want to get that? Sure. Will do that. Thanks Oleg for your detailed review/comments. Suka _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers