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

Re: [Xen-devel] [PATCH v1] x86: Restore IA32_MISC_ENABLE on wakeup

On 8/19/19 11:04 AM, Andrew Cooper wrote:
> On 19/08/2019 03:23, Michał Kowalczyk wrote:
>> diff --git a/xen/arch/x86/boot/trampoline.S b/xen/arch/x86/boot/trampoline.S
>> index 7c6a2328d2..fcaa3eeaf1 100644
>> --- a/xen/arch/x86/boot/trampoline.S
>> +++ b/xen/arch/x86/boot/trampoline.S
>> @@ -85,7 +85,7 @@ trampoline_gdt:
>>          .long   trampoline_gdt + BOOT_PSEUDORM_DS + 2 - .
>>          .popsection
>> -GLOBAL(trampoline_misc_enable_off)
>> +GLOBAL(misc_enable_off)
> The overall change is fine, but why have you renamed this variable?
The old name had "trampoline_" prefix because the only place where it
was used was trampoline_protmode_entry in arch/x86/boot/trampoline.S.
Now it's also used in the wakeup code, so I removed the prefix which
could be (IMO) misleading.
> Without the rename, the patch would be just the single hunk in wakeup.S
> and therefore easier to backport.

True. Anyway, the decision is on your side, I can leave the old name if
you prefer.

Xen-devel mailing list



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