[CRIU] [PATCH] dump/x86: sanitize the ERESTART_RESTARTBLOCK -> EINTR transition
Pavel Emelyanov
xemul at parallels.com
Thu Mar 26 04:10:41 PDT 2015
On 03/24/2015 10:22 PM, Oleg Nesterov wrote:
> 1. The -ERESTART_RESTARTBLOCK case in get_task_regs() depends on kernel
> internals too much, and for no reason. We shouldn't rely on fact that
> a) we are going to do sigreturn() and b) restore_sigcontext() always
> sets restart_block->fn = do_no_restart_syscall which returns -EINTR.
>
> Just change this code to enforce -EINTR after restore, this is what
> we actually want until we teach criu to handle ERESTART_RESTARTBLOCK.
>
> 2. Add pr_warn() to make the potential bug-reports more understandable,
> a sane application should handle -EINTR correctly but this is not
> always the case.
>
> Signed-off-by: Oleg Nesterov <oleg at redhat.com>
Applied, thanks!
More information about the CRIU
mailing list