Re: [Xen-devel] [PATCH] xen: put cpupool's member 'n_dom' after 'cpupool_id'

[chenbaodong <chenbaodong@xxxxxxxxxx>]

On 5/31/19 18:52, Julien Grall wrote:
> Hi,
>
> On 5/31/19 4:18 AM, Baodong Chen wrote:
> > Thus, sizeof(struct cpupool) will save 8 bytes for 64-bit system.
> I am happy with the change, although AFAIK cpupool is not instantiated 
> that often. Are you planning to have more instantiation of it?
> Cheers,
No, I'm not planning to create lots of cpupool instance.

I'm studying xen for a few weeks and my plan is:

run xen for embeded automotive use case with dom0less, null scheduler, 
small code base for safety certified maybe a plus(not sure whether can 
do this).

> > Signed-off-by: Baodong Chen <chenbaodong@xxxxxxxxxx>
> > ---
> >   xen/include/xen/sched-if.h | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/xen/include/xen/sched-if.h b/xen/include/xen/sched-if.h
> > index 92bc7a0..f0cf210 100644
> > --- a/xen/include/xen/sched-if.h
> > +++ b/xen/include/xen/sched-if.h
> > @@ -213,9 +213,9 @@ static inline void sched_free_domdata(const 
> > struct scheduler *s,
> >   struct cpupool
> >   {
> >       int              cpupool_id;
> > +    unsigned int     n_dom;
> >       cpumask_var_t    cpu_valid;      /* all cpus assigned to pool */
> >       struct cpupool   *next;
> > -    unsigned int     n_dom;
> >       struct scheduler *sched;
> >       atomic_t         refcnt;
> >   };
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel