Re: [PATCH 1/2] evtchn/fifo: initialize priority when events are bound

[David Vrabel <david.vrabel@citrix.com>]

On 10/12/13 14:50, Jan Beulich wrote:
>>>> On 10.12.13 at 14:56, David Vrabel <david.vrabel@citrix.com> wrote:
>> From: David Vrabel <david.vrabel@citrix.com>
>>
>> Event channel ports that are reused or that were not in the initial
>> bucket would have a non-default priority.
>>
>> Add an init evtchn_port_op hook and use this to set the priority when
>> an event channel is bound.
>>
>> Within this new evtchn_fifo_init() call, also check if the event is
>> already on a queue and print a warning, as this event may have its
>> first event delivered on a queue with the wrong VCPU or priority.
>> This guest is expected to prevent this (if it cares) by not unbinding
>> events that are still linked.
>>
> 
> Reported-by: Jan Beulich <jbeulich@suse.com>

Yes, sorry. Not the first time I've not properly attributed someone.  I
shall have to be more careful in future.

>> Signed-off-by: David Vrabel <david.vrabel@citrix.com>
> 
> Reviewed-by: Jan Beulich <jbeulich@suse.com>

Thanks!

David