On Fri, Mar 09, 2018 at 01:18:40PM +0000, Andrew Cooper wrote: > The only relevent initialisation for the idle domain is the context switch and > poisoned pointers. Collect these bits together early in the function and exit > when complete (although as a consequence, the e820 and vtsc lock > initialisation are moved forwards). This allows us to remove subsequent > is_idle_domain() checks and unindent most of the logic. > > Furthermore, we no longer call these functions for the idle domain: > * mapcache_domain_init() and tsc_set_info() were previously guarded against > the idle domain, and have had their guards turned into ASSERT()s. > * pit_init() is implicitly guarded by has_vpit(). > * psr_domain_init() no longer allocates a socket array. > > Finally, two changes are introduced for the benefit of the following patch: > * For PV hardware domains, or XEN_X86_EMU_PIT into emflags rather than into > config->emulation_flags, to facilitating config becoming const. > * References to domcr_flags are moved until after the idle early exist, to > facilitiate them being unavailable for system domains. > > No practical change in behaviour. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Reviewed-by: Wei Liu <wei.l...@citrix.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel