>>> On 13.01.15 at 13:11, <wei.l...@citrix.com> wrote:
>          rc = do_memory_op(cmd, nat.hnd);
> +        if ( rc == -ENOBUFS && op == XENMEM_get_vnumainfo )
> +        {
> +            cmp.vnuma.nr_vnodes = nat.vnuma->nr_vnodes;
> +            cmp.vnuma.nr_vcpus = nat.vnuma->nr_vcpus;
> +            cmp.vnuma.nr_vmemranges = nat.vnuma->nr_vmemranges;
> +            if ( __copy_to_guest(compat, &cmp.vnuma, 1) )
> +                rc = -EFAULT;
> +        }
>          if ( rc < 0 )
>              break;

Please move the new if() inside the body of the existing one.
Ack with that change.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to