On Thu, Nov 08, 2018 at 08:34:35AM -0700, Jan Beulich wrote: > >>> On 08.11.18 at 15:58, <[email protected]> wrote: > > --- a/xen/arch/x86/cpu/amd.c > > +++ b/xen/arch/x86/cpu/amd.c > > @@ -631,7 +631,9 @@ static void init_amd(struct cpuinfo_x86 *c) > > case 0xf ... 0x17: > > disable_c1e(NULL); > > if (acpi_smi_cmd && (acpi_enable_value | acpi_disable_value)) { > > +#ifdef CONFIG_PV > > pv_post_outb_hook = amd_check_disable_c1e; > > +#endif > > amd_acpi_c1e_quirk = true; > > } > > break; > > Hmm, I would have hoped for this to get properly cleaned up, > as outlined in that other thread, if you already touch it. Anyway, > I see you've got the necessary acks, so the subsequent > cleanup patch will just have to delete two more lines.
Do you mean to switch PV to use the same mechanism as HVM? I don't have a machine to test this code path at the moment, so it will need to wait a bit. In the mean time I would like to tick one of the big work items off of my list. Wei. > > Jan > > > > _______________________________________________ > Xen-devel mailing list > [email protected] > https://lists.xenproject.org/mailman/listinfo/xen-devel _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
