>>> On 08.09.16 at 07:30, <dongli.zh...@oracle.com> wrote: > --- a/xen/include/xen/sched.h > +++ b/xen/include/xen/sched.h > @@ -474,6 +474,9 @@ struct domain > unsigned int guest_request_enabled : 1; > unsigned int guest_request_sync : 1; > } monitor; > + > + /* set to 1 the first time this domain gets scheduled. */ > + bool_t already_scheduled;
Did you go through and check that there is nothing this information can already get derived from? I can't immediately point you at anything, but it feels like there should. And if indeed there isn't, then - to extend on someone else's comments (I think it was Dario) - please use plain bool in new additions. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel