On 16.06.2020 18:30, Roger Pau Monné wrote:
> On Tue, Jun 16, 2020 at 05:20:39PM +0200, Michał Leszczyński wrote:
>> Check if Intel Processor Trace feature is supported by current
>> processor. Define hvm_ipt_supported function.
>>
>> Signed-off-by: Michal Leszczynski <michal.leszczyn...@cert.pl>
>> ---
>> xen/arch/x86/hvm/vmx/vmx.c | 24 +++++++++++++++++++++
>> xen/include/asm-x86/cpufeature.h | 1 +
>> xen/include/asm-x86/hvm/hvm.h | 9 ++++++++
>> xen/include/asm-x86/hvm/vmx/vmcs.h | 1 +
>> xen/include/public/arch-x86/cpufeatureset.h | 1 +
>> 5 files changed, 36 insertions(+)
>>
>> diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c
>> index ab19d9424e..a91bbdb798 100644
>> --- a/xen/arch/x86/hvm/vmx/vmx.c
>> +++ b/xen/arch/x86/hvm/vmx/vmx.c
>> @@ -2484,6 +2484,7 @@ static bool __init has_if_pschange_mc(void)
>>
>> const struct hvm_function_table * __init start_vmx(void)
>> {
>> + u64 _vmx_misc_cap;
>
> Please use uint64_t, and you can drop the leading _vmx prefix, this is
> already vmx specific.
Actually, all of _vmx_ should be dropped (i.e. in particular local
variables shouldn't start with an underscore).
Jan
