On 18.04.2022 14:22, Penny Zheng wrote: > --- a/xen/include/xen/sched.h > +++ b/xen/include/xen/sched.h > @@ -591,6 +591,9 @@ struct domain > struct ioreq_server *server[MAX_NR_IOREQ_SERVERS]; > } ioreq_server; > #endif > + > + /* Holding CDF_* constant. Internal flags for domain creation. */ > + uint32_t flags;
There's no need to use a fixed width type here; unsigned int will do. See ./CODING_STYLE. I'd also like to ask for the field to be given a more descriptive name. Just "flags" can mean about anything. Maybe simply "cdf"? Jan
