[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] common/sched: Correct function prototypes



>>> On 11.10.13 at 11:40, Tim Deegan <tim@xxxxxxx> wrote:
> At 07:55 +0100 on 11 Oct (1381478143), Jan Beulich wrote:
>> >>> On 10.10.13 at 20:25, Keir Fraser <keir.xen@xxxxxxxxx> wrote:
>> > On 10/10/2013 18:05, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx> wrote:
>> >> --- a/xen/include/xen/sched.h
>> >> +++ b/xen/include/xen/sched.h
>> >> @@ -567,9 +567,9 @@ void sched_set_node_affinity(struct domain *, 
>> >> nodemask_t *);
>> >>  int  sched_id(void);
>> >>  void sched_tick_suspend(void);
>> >>  void sched_tick_resume(void);
>> >> -void vcpu_wake(struct vcpu *d);
>> >> -void vcpu_sleep_nosync(struct vcpu *d);
>> >> -void vcpu_sleep_sync(struct vcpu *d);
>> >> +void vcpu_wake(struct vcpu *v);
>> >> +void vcpu_sleep_nosync(struct vcpu *v);
>> >> +void vcpu_sleep_sync(struct vcpu *v);
>> 
>> ... I'd much prefer if we stopped naming parameters in declarations
>> where the parameter types are already sufficiently describing them.
> 
> I'd rather not -- I prefer the declarations to match the definitions.

Any specific reason for that?

Jan


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.