Re: [Xen-devel] [PATCH 2/3] xen: sched: add wakeup flags to the scheduler interface

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 11.02.16 at 12:38, <dario.faggioli@xxxxxxxxxx> wrote:
> --- a/xen/include/xen/sched-if.h
> +++ b/xen/include/xen/sched-if.h
> @@ -144,7 +144,8 @@ struct scheduler {
>      void         (*remove_vcpu)    (const struct scheduler *, struct vcpu *);
>  
>      void         (*sleep)          (const struct scheduler *, struct vcpu *);
> -    void         (*wake)           (const struct scheduler *, struct vcpu *);
> +    void         (*wake)           (const struct scheduler *, struct vcpu *,
> +                                    unsigned int);

Just one cosmetic comment: You properly use "unsigned int" here,
but just "unsigned" everywhere else in this patch. May I ask that
you use the canonical full form everywhere?

Jan


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