Subject: Re: Netchannles: first stage has been completed. Further ideas.

From: Christian Borntraeger <[email-masked]>
Date: Tue, 18 Jul 2006 13:16:18 +0200

Hello Evgeniy,

> +asmlinkage long sys_netchannel_control(void __user *arg)
[...]
> +	if (copy_from_user(&ctl, arg, sizeof(struct unetchannel_control)))
> +		return -ERESTARTSYS;
                        ^^^^^^^^^^^
[...]
> +	if (copy_to_user(arg, &ctl, sizeof(struct unetchannel_control)))
> +		return -ERESTARTSYS;
                        ^^^^^^^^^^^

I think this should be -EFAULT instead of -ERESTARTSYS, right?

-- 
Mit freundlichen Grüßen / Best Regards

Christian Borntraeger
Linux Software Engineer zSeries Linux & Virtualization