[Devel] Re: [PATCH] checkpoint/powerpc: fix up checkpoint syscall, tidy restart

Oren Laadan orenl at cs.columbia.edu
Fri Feb 5 13:30:13 PST 2010


Thanks !

This one and the other two (user-cr) applied and pushed.

Oren.


Nathan Lynch wrote:
> Fix up fallout from changing checkpoint to ptregscall; use named
> parameters in sys_restart() instead of redundantly loading the syscall
> arguments from ptregs.
> 
> Signed-off-by: Nathan Lynch <ntl at pobox.com>
> ---
>  arch/powerpc/kernel/process.c |   23 +++++++++++------------
>  1 files changed, 11 insertions(+), 12 deletions(-)
> 
> diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c
> index f9da9eb..4b9019b 100644
> --- a/arch/powerpc/kernel/process.c
> +++ b/arch/powerpc/kernel/process.c
> @@ -991,21 +991,20 @@ out:
>  	return error;
>  }
>  
> -int sys_restart(unsigned long a0, unsigned long a1, unsigned long a2,
> -	       unsigned long a3, unsigned long a4, unsigned long a5,
> -	       struct pt_regs *regs)
> +int sys_checkpoint(unsigned long pid, unsigned long fd, unsigned long flags,
> +		   unsigned long logfd, unsigned long p5, unsigned long p6,
> +		   struct pt_regs *regs)
>  {
> -	unsigned long flags;
> -	pid_t pid;
> -	int logfd;
> -	int fd;
> -
>  	CHECK_FULL_REGS(regs);
>  
> -	pid = a0;
> -	fd = a1;
> -	flags = a2;
> -	logfd = a3;
> +	return do_sys_checkpoint(pid, fd, flags, logfd);
> +}
> +
> +int sys_restart(unsigned long pid, unsigned long fd, unsigned long flags,
> +		unsigned long logfd, unsigned long p5, unsigned long p6,
> +		struct pt_regs *regs)
> +{
> +	CHECK_FULL_REGS(regs);
>  
>  	return do_sys_restart(pid, fd, flags, logfd);
>  }
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list