>>> On 15.06.15 at 17:48, <david.vra...@citrix.com> wrote: > --- a/xen/common/event_channel.c > +++ b/xen/common/event_channel.c > @@ -977,9 +977,7 @@ int evtchn_unmask(unsigned int port) > struct domain *d = current->domain; > struct evtchn *evtchn; > > - ASSERT(spin_is_locked(&d->event_lock)); > - > - if ( unlikely(!port_is_valid(d, port)) ) > + if ( port >= read_atomic(&d->valid_evtchns) ) > return -EINVAL;
Again this partial open coding looks wrong to me. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
