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

[chenbaodong <chenbaodong@mxnavi.com>]

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@mxnavi.com>
>> ---
>>   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@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel