| Message ID | 20191025091618.10153-6-liuwe@microsoft.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Implement Hyper-V reference TSC based clock source | expand |
On 25.10.2019 11:16, Wei Liu wrote: > --- a/xen/arch/x86/setup.c > +++ b/xen/arch/x86/setup.c > @@ -1577,7 +1577,7 @@ void __init noreturn __start_xen(unsigned long mbi_p) > max_cpus = nr_cpu_ids; > } > > - if ( xen_guest ) > + if ( running_on_hypervisor ) > hypervisor_setup(); This code is using hypervisor_name already, so I guess the patch has become unnecessary? Jan
On Tue, Dec 10, 2019 at 05:17:28PM +0100, Jan Beulich wrote: > On 25.10.2019 11:16, Wei Liu wrote: > > --- a/xen/arch/x86/setup.c > > +++ b/xen/arch/x86/setup.c > > @@ -1577,7 +1577,7 @@ void __init noreturn __start_xen(unsigned long mbi_p) > > max_cpus = nr_cpu_ids; > > } > > > > - if ( xen_guest ) > > + if ( running_on_hypervisor ) > > hypervisor_setup(); > > This code is using hypervisor_name already, so I guess the patch > has become unnecessary? Yes. I basically squashed this patch into my previous series while reworking that. This patch here is not needed anymore. Wei. > > Jan
diff --git a/xen/arch/x86/setup.c b/xen/arch/x86/setup.c index 4aa0af5a12..044c45be36 100644 --- a/xen/arch/x86/setup.c +++ b/xen/arch/x86/setup.c @@ -1577,7 +1577,7 @@ void __init noreturn __start_xen(unsigned long mbi_p) max_cpus = nr_cpu_ids; } - if ( xen_guest ) + if ( running_on_hypervisor ) hypervisor_setup(); /* Low mappings were only needed for some BIOS table parsing. */
The hypervisor_setup method is not unique to Xen guest. Signed-off-by: Wei Liu <liuwe@microsoft.com> --- xen/arch/x86/setup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)