[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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




On 6/3/19 17:30, Julien Grall wrote:
Hi,

On 03/06/2019 02:48, chenbaodong wrote:

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).

FWIW, there are discussion to get Xen safety certified. They are captured on [1].

Cheers,

[1] https://wiki.xenproject.org/wiki/Category:Safety_Certification/FuSa_SIG

Hello Julien, Thanks for your info.

I will follow the wiki link and see what i can do about safety certification.





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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.