Re: [PATCH] sysctl: adjust XEN_SYSCTL_numainfo behavior

[Dario Faggioli <dario.faggioli@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1147 bytes --]

On Tue, 2015-07-14 at 10:52 +0100, Jan Beulich wrote:
> ... to match XEN_SYSCTL_cputopoinfo, allowing the caller to get what it
> needs (if e.g. it's after the data for just one specific node) with
> just one hypercall, without caring about the total number of nodes in
> the system.
> 
> Suggested-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 
Reviewed-by: Dario Faggioli <dario.faggioli@citrix.com>

One question:

> --- a/xen/common/sysctl.c
> +++ b/xen/common/sysctl.c
>  
> @@ -335,7 +329,7 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xe
>          else
>              i = num_nodes;
>  
> -        if ( (!ret || (ret == -ENOBUFS)) && (ni->num_nodes != i) )
> +        if ( !ret && (ni->num_nodes != i) )
>          {
Can't we kill the parentheses around the second argument of the && ?

Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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