Messages in this thread | | | Date | Mon, 16 Apr 2012 23:26:06 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH] pidns: reboot_pid_ns: use SEND_SIG_FORCED instead of force_sig() |
| |
On 04/16, Serge Hallyn wrote: > > Quoting Oleg Nesterov (oleg@redhat.com): > > On 04/16, Serge Hallyn wrote: > > > > > > Quoting Oleg Nesterov (oleg@redhat.com): > > > > > > > > Strange... I even tested this change, but only in the root namespace. > > > > OK, thanks, I'll investigate. I hope you verified your kernel has > > > > 629d362b commit ;) > > > > > > I took friday's HEAD of Linus tree. All went fine. Added this patch. > > > The container can shutdown, but reboot causes shutdown. Don't undersrtand > > > why. > > > > Heh. This looks "impossible". I'd like to understand the reason. > > So, LINUX_REBOOT_CMD_RESTART results in WIFSIGNALED() == SIGINT? > > > > OK, I'll try to test this patch in the non-root namespace. > > Playing with the test case below (based on the one Daniel submitted > alongside his container reboot patches), it looks like > WIFSIGNALED(status) and WTERMSIG(status) are 0 with this kernel, not > 1 and sig.
Great, thanks Serge. I'll investigate and report tomorrow.
Oleg.
| |