Re: [Xen-devel] [PATCH] common/vm_event: Fix comment

[Razvan Cojocaru <rcojocaru@xxxxxxxxxxxxxxx>]

On 08/11/2016 01:38 PM, Jan Beulich wrote:
>>>> On 09.08.16 at 10:32, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
>> --- a/xen/common/vm_event.c
>> +++ b/xen/common/vm_event.c
>> @@ -255,7 +255,7 @@ static inline void vm_event_release_slot(struct domain 
>> *d,
>>  
>>  /*
>>   * vm_event_mark_and_pause() tags vcpu and put it to sleep.
>> - * The vcpu will resume execution in vm_event_wake_waiters().
>> + * The vcpu will resume execution in vm_event_wake().
>>   */
>>  void vm_event_mark_and_pause(struct vcpu *v, struct vm_event_domain *ved)
>>  {
> 
> I was about to commit this without further waiting for an ack, as
> being supposedly trivial, but then I checked and also found
> vm_event_wake{blocked,queued}(), and now I'm not sure
> whether the reference wouldn't better be to
> vm_event_wake_blocked(). Could you clarify that for me please?

Indeed, that's more precise. I'm happy to send V2 if you'd like.


Thanks,
Razvan

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