On Tue, Mar 08, 2016 at 01:32:58PM -0600, Chong Li wrote:
> On Tue, Mar 8, 2016 at 1:09 PM, Wei Liu <wei.l...@citrix.com> wrote:
> > On Sun, Mar 06, 2016 at 11:55:56AM -0600, Chong Li wrote:
> >> Add xc_sched_rtds_vcpu_get/set functions to interact with
> >> Xen to get/set a domain's per-VCPU parameters.
> >>
> >> Signed-off-by: Chong Li <chong...@wustl.edu>
> >> Signed-off-by: Meng Xu <men...@cis.upenn.edu>
> >> Signed-off-by: Sisu Xi <xis...@gmail.com>
> >>
> >> ---
> >> Changes on PATCH v5:
> >> 1) In xc_sched_rtds_vcpu_get/set, re-issueing the hypercall
> >> if it is preempted.
> >>
> >> Changes on PATCH v4:
> >> 1) Minor modifications on the function parameters.
> >>
> >> Changes on PATCH v2:
> >> 1) Minor modifications due to the change of struct xen_domctl_scheduler_op.
> >>
> >> CC: <dario.faggi...@citrix.com>
> >> CC: <george.dun...@eu.citrix.com>
> >> CC: <dgol...@seas.upenn.edu>
> >> CC: <men...@cis.upenn.edu>
> >> CC: <wei.l...@citrix.com>
> >> CC: <lichong...@gmail.com>
> >> ---
> >> tools/libxc/include/xenctrl.h | 16 +++++++---
> >> tools/libxc/xc_rt.c | 68
> >> +++++++++++++++++++++++++++++++++++++++++++
> >> 2 files changed, 80 insertions(+), 4 deletions(-)
> >>
> >> diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h
> >> index 01a6dda..9462271 100644
> >> --- a/tools/libxc/include/xenctrl.h
> >> +++ b/tools/libxc/include/xenctrl.h
> >> @@ -888,11 +888,19 @@ int xc_sched_credit2_domain_get(xc_interface *xch,
> >> struct xen_domctl_sched_credit2 *sdom);
> >>
> >> int xc_sched_rtds_domain_set(xc_interface *xch,
> >> - uint32_t domid,
> >> - struct xen_domctl_sched_rtds *sdom);
> >> + uint32_t domid,
> >> + struct xen_domctl_sched_rtds *sdom);
> >> int xc_sched_rtds_domain_get(xc_interface *xch,
> >> - uint32_t domid,
> >> - struct xen_domctl_sched_rtds *sdom);
> >> + uint32_t domid,
> >> + struct xen_domctl_sched_rtds *sdom);
> >
> > Please don't mix fixing indentation with new functionality. Besides, the
> > new indentation looks wrong.
> >
> >> +int xc_sched_rtds_vcpu_set(xc_interface *xch,
> >> + uint32_t domid,
> >> + struct xen_domctl_schedparam_vcpu *vcpus,
> >> + uint32_t num_vcpus);
> >> +int xc_sched_rtds_vcpu_get(xc_interface *xch,
> >> + uint32_t domid,
> >> + struct xen_domctl_schedparam_vcpu *vcpus,
> >> + uint32_t num_vcpus);
> >>
> >
> > Indentation also looks wrong. Maybe you used tab? Please use spaces
> > instead.
> >
> I'm not using tab. This is just what I'm confused with. The
> indentation for these
> four *rtds* functions is the same as the two *credit2* functions
> above. I can not
> find the indentation rules for function calls with many / long parameters.
>
Use white space and stay aligned with previous line wherever you can.
So:
int xc_sched_rtds_vcpu_get(xc_interface *xch,
uint32_t domid,
struct xen_domctl_schedparam_vcpu *vcpus,
uint32_t num_vcpus);
I know there is inconsistency in the code base. I'm sorry it makes you
feel confused.
Wei.
> Chong
>
> --
> Chong Li
> Department of Computer Science and Engineering
> Washington University in St.louis
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
