On Wed, 2016-03-16 at 11:47 -0500, Chong Li wrote:
> Add xc_sched_rtds_vcpu_get/set functions to interact with
> Xen to get/set a domain's per-VCPU parameters.
> 
Patch is ok, IMO, with only two minor comments.

First, the subject is a bit long. I'd go for:
"libxc: enable per-VCPU parameters for RTDS"

Second, here...

> Signed-off-by: Chong Li <chong.li@wustl.edu>
> Signed-off-by: Meng Xu <mengxu@cis.upenn.edu>
> Signed-off-by: Sisu Xi <xisisu@gmail.com>

> --- a/tools/libxc/xc_rt.c
> +++ b/tools/libxc/xc_rt.c
> @@ -62,3 +62,71 @@ int xc_sched_rtds_domain_get(xc_interface *xch,
>  
>      return rc;
>  }
> +
> +int xc_sched_rtds_vcpu_set(xc_interface *xch,
> +                           uint32_t domid,
> +                           struct xen_domctl_schedparam_vcpu *vcpus,
> +                           uint32_t num_vcpus)
> +{
> +    int rc = 0;
>
rc does not need to be initialized to 0 (and hence it shouldn't be).

Even with only this fixed (i.e., even if the subject stays as it is):

Reviewed-by: Dario Faggioli <dario.faggioli@citrix.com>

Thanks and 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)