On 19.06.2020 01:40, Michał Leszczyński wrote:
> @@ -630,6 +633,12 @@ static inline bool hvm_altp2m_supported(void)
> return hvm_funcs.altp2m_supported;
> }
>
> +/* returns true if hardware supports Intel Processor Trace */
> +static inline bool hvm_pt_supported(void)
> +{
> + return hvm_funcs.pt_supported;
> +}
This is vendor-independent code, hence I'd like to see "Intel"
dropped from the comment.
> --- a/xen/include/asm-x86/hvm/vmx/vmcs.h
> +++ b/xen/include/asm-x86/hvm/vmx/vmcs.h
> @@ -285,6 +285,7 @@ extern u64 vmx_ept_vpid_cap;
>
> #define VMX_MISC_CR3_TARGET 0x01ff0000
> #define VMX_MISC_VMWRITE_ALL 0x20000000
> +#define VMX_MISC_PT_SUPPORTED 0x00004000
Move this up by two lines, so the values continue to be numerically
sorted?
Jan