[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/4] target-i386: move asm-x86/hyperv.h to st
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/4] target-i386: move asm-x86/hyperv.h to standard-headers |
Date: |
Thu, 10 Sep 2015 12:43:41 +0300 |
On Thu, Sep 10, 2015 at 11:39:38AM +0200, Paolo Bonzini wrote:
> The Hyper-V definitions are an industry standard and can be used
> from code that is not KVM-specific.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
> ---
> .../standard-headers}/asm-x86/hyperv.h | 10 +-
> linux-headers/asm-x86/hyperv.h | 253
> +--------------------
> scripts/update-linux-headers.sh | 9 +-
> target-i386/kvm.c | 2 +-
> 4 files changed, 13 insertions(+), 261 deletions(-)
> copy {linux-headers => include/standard-headers}/asm-x86/hyperv.h (98%)
>
> diff --git a/linux-headers/asm-x86/hyperv.h
> b/include/standard-headers/asm-x86/hyperv.h
> similarity index 98%
> copy from linux-headers/asm-x86/hyperv.h
> copy to include/standard-headers/asm-x86/hyperv.h
> index f36d56b..a2123ca 100644
> --- a/linux-headers/asm-x86/hyperv.h
> +++ b/include/standard-headers/asm-x86/hyperv.h
> @@ -1,7 +1,7 @@
> #ifndef _ASM_X86_HYPERV_H
> #define _ASM_X86_HYPERV_H
>
> -#include <linux/types.h>
> +#include "standard-headers/linux/types.h"
>
> /*
> * The below CPUID leaves are present if VersionAndFeatures.HypervisorPresent
> @@ -243,10 +243,10 @@
> #define HV_STATUS_INSUFFICIENT_BUFFERS 19
>
> typedef struct _HV_REFERENCE_TSC_PAGE {
> - __u32 tsc_sequence;
> - __u32 res1;
> - __u64 tsc_scale;
> - __s64 tsc_offset;
> + uint32_t tsc_sequence;
> + uint32_t res1;
> + uint64_t tsc_scale;
> + int64_t tsc_offset;
> } HV_REFERENCE_TSC_PAGE, *PHV_REFERENCE_TSC_PAGE;
>
> #endif
> diff --git a/linux-headers/asm-x86/hyperv.h b/linux-headers/asm-x86/hyperv.h
> index f36d56b..01af4d8 100644
> --- a/linux-headers/asm-x86/hyperv.h
> +++ b/linux-headers/asm-x86/hyperv.h
> @@ -1,252 +1 @@
> -#ifndef _ASM_X86_HYPERV_H
> -#define _ASM_X86_HYPERV_H
> -
> -#include <linux/types.h>
> -
> -/*
> - * The below CPUID leaves are present if VersionAndFeatures.HypervisorPresent
> - * is set by CPUID(HvCpuIdFunctionVersionAndFeatures).
> - */
> -#define HYPERV_CPUID_VENDOR_AND_MAX_FUNCTIONS 0x40000000
> -#define HYPERV_CPUID_INTERFACE 0x40000001
> -#define HYPERV_CPUID_VERSION 0x40000002
> -#define HYPERV_CPUID_FEATURES 0x40000003
> -#define HYPERV_CPUID_ENLIGHTMENT_INFO 0x40000004
> -#define HYPERV_CPUID_IMPLEMENT_LIMITS 0x40000005
> -
> -#define HYPERV_HYPERVISOR_PRESENT_BIT 0x80000000
> -#define HYPERV_CPUID_MIN 0x40000005
> -#define HYPERV_CPUID_MAX 0x4000ffff
> -
> -/*
> - * Feature identification. EAX indicates which features are available
> - * to the partition based upon the current partition privileges.
> - */
> -
> -/* VP Runtime (HV_X64_MSR_VP_RUNTIME) available */
> -#define HV_X64_MSR_VP_RUNTIME_AVAILABLE (1 << 0)
> -/* Partition Reference Counter (HV_X64_MSR_TIME_REF_COUNT) available*/
> -#define HV_X64_MSR_TIME_REF_COUNT_AVAILABLE (1 << 1)
> -
> -/* A partition's reference time stamp counter (TSC) page */
> -#define HV_X64_MSR_REFERENCE_TSC 0x40000021
> -
> -/*
> - * There is a single feature flag that signifies the presence of the MSR
> - * that can be used to retrieve both the local APIC Timer frequency as
> - * well as the TSC frequency.
> - */
> -
> -/* Local APIC timer frequency MSR (HV_X64_MSR_APIC_FREQUENCY) is available */
> -#define HV_X64_MSR_APIC_FREQUENCY_AVAILABLE (1 << 11)
> -
> -/* TSC frequency MSR (HV_X64_MSR_TSC_FREQUENCY) is available */
> -#define HV_X64_MSR_TSC_FREQUENCY_AVAILABLE (1 << 11)
> -
> -/*
> - * Basic SynIC MSRs (HV_X64_MSR_SCONTROL through HV_X64_MSR_EOM
> - * and HV_X64_MSR_SINT0 through HV_X64_MSR_SINT15) available
> - */
> -#define HV_X64_MSR_SYNIC_AVAILABLE (1 << 2)
> -/*
> - * Synthetic Timer MSRs (HV_X64_MSR_STIMER0_CONFIG through
> - * HV_X64_MSR_STIMER3_COUNT) available
> - */
> -#define HV_X64_MSR_SYNTIMER_AVAILABLE (1 << 3)
> -/*
> - * APIC access MSRs (HV_X64_MSR_EOI, HV_X64_MSR_ICR and HV_X64_MSR_TPR)
> - * are available
> - */
> -#define HV_X64_MSR_APIC_ACCESS_AVAILABLE (1 << 4)
> -/* Hypercall MSRs (HV_X64_MSR_GUEST_OS_ID and HV_X64_MSR_HYPERCALL)
> available*/
> -#define HV_X64_MSR_HYPERCALL_AVAILABLE (1 << 5)
> -/* Access virtual processor index MSR (HV_X64_MSR_VP_INDEX) available*/
> -#define HV_X64_MSR_VP_INDEX_AVAILABLE (1 << 6)
> -/* Virtual system reset MSR (HV_X64_MSR_RESET) is available*/
> -#define HV_X64_MSR_RESET_AVAILABLE (1 << 7)
> - /*
> - * Access statistics pages MSRs (HV_X64_MSR_STATS_PARTITION_RETAIL_PAGE,
> - * HV_X64_MSR_STATS_PARTITION_INTERNAL_PAGE,
> HV_X64_MSR_STATS_VP_RETAIL_PAGE,
> - * HV_X64_MSR_STATS_VP_INTERNAL_PAGE) available
> - */
> -#define HV_X64_MSR_STAT_PAGES_AVAILABLE (1 << 8)
> -
> -/*
> - * Feature identification: EBX indicates which flags were specified at
> - * partition creation. The format is the same as the partition creation
> - * flag structure defined in section Partition Creation Flags.
> - */
> -#define HV_X64_CREATE_PARTITIONS (1 << 0)
> -#define HV_X64_ACCESS_PARTITION_ID (1 << 1)
> -#define HV_X64_ACCESS_MEMORY_POOL (1 << 2)
> -#define HV_X64_ADJUST_MESSAGE_BUFFERS (1 << 3)
> -#define HV_X64_POST_MESSAGES (1 << 4)
> -#define HV_X64_SIGNAL_EVENTS (1 << 5)
> -#define HV_X64_CREATE_PORT (1 << 6)
> -#define HV_X64_CONNECT_PORT (1 << 7)
> -#define HV_X64_ACCESS_STATS (1 << 8)
> -#define HV_X64_DEBUGGING (1 << 11)
> -#define HV_X64_CPU_POWER_MANAGEMENT (1 << 12)
> -#define HV_X64_CONFIGURE_PROFILER (1 << 13)
> -
> -/*
> - * Feature identification. EDX indicates which miscellaneous features
> - * are available to the partition.
> - */
> -/* The MWAIT instruction is available (per section MONITOR / MWAIT) */
> -#define HV_X64_MWAIT_AVAILABLE (1 << 0)
> -/* Guest debugging support is available */
> -#define HV_X64_GUEST_DEBUGGING_AVAILABLE (1 << 1)
> -/* Performance Monitor support is available*/
> -#define HV_X64_PERF_MONITOR_AVAILABLE (1 << 2)
> -/* Support for physical CPU dynamic partitioning events is available*/
> -#define HV_X64_CPU_DYNAMIC_PARTITIONING_AVAILABLE (1 << 3)
> -/*
> - * Support for passing hypercall input parameter block via XMM
> - * registers is available
> - */
> -#define HV_X64_HYPERCALL_PARAMS_XMM_AVAILABLE (1 << 4)
> -/* Support for a virtual guest idle state is available */
> -#define HV_X64_GUEST_IDLE_STATE_AVAILABLE (1 << 5)
> -/* Guest crash data handler available */
> -#define HV_X64_GUEST_CRASH_MSR_AVAILABLE (1 << 10)
> -
> -/*
> - * Implementation recommendations. Indicates which behaviors the hypervisor
> - * recommends the OS implement for optimal performance.
> - */
> - /*
> - * Recommend using hypercall for address space switches rather
> - * than MOV to CR3 instruction
> - */
> -#define HV_X64_MWAIT_RECOMMENDED (1 << 0)
> -/* Recommend using hypercall for local TLB flushes rather
> - * than INVLPG or MOV to CR3 instructions */
> -#define HV_X64_LOCAL_TLB_FLUSH_RECOMMENDED (1 << 1)
> -/*
> - * Recommend using hypercall for remote TLB flushes rather
> - * than inter-processor interrupts
> - */
> -#define HV_X64_REMOTE_TLB_FLUSH_RECOMMENDED (1 << 2)
> -/*
> - * Recommend using MSRs for accessing APIC registers
> - * EOI, ICR and TPR rather than their memory-mapped counterparts
> - */
> -#define HV_X64_APIC_ACCESS_RECOMMENDED (1 << 3)
> -/* Recommend using the hypervisor-provided MSR to initiate a system RESET */
> -#define HV_X64_SYSTEM_RESET_RECOMMENDED (1 << 4)
> -/*
> - * Recommend using relaxed timing for this partition. If used,
> - * the VM should disable any watchdog timeouts that rely on the
> - * timely delivery of external interrupts
> - */
> -#define HV_X64_RELAXED_TIMING_RECOMMENDED (1 << 5)
> -
> -/* MSR used to identify the guest OS. */
> -#define HV_X64_MSR_GUEST_OS_ID 0x40000000
> -
> -/* MSR used to setup pages used to communicate with the hypervisor. */
> -#define HV_X64_MSR_HYPERCALL 0x40000001
> -
> -/* MSR used to provide vcpu index */
> -#define HV_X64_MSR_VP_INDEX 0x40000002
> -
> -/* MSR used to read the per-partition time reference counter */
> -#define HV_X64_MSR_TIME_REF_COUNT 0x40000020
> -
> -/* MSR used to retrieve the TSC frequency */
> -#define HV_X64_MSR_TSC_FREQUENCY 0x40000022
> -
> -/* MSR used to retrieve the local APIC timer frequency */
> -#define HV_X64_MSR_APIC_FREQUENCY 0x40000023
> -
> -/* Define the virtual APIC registers */
> -#define HV_X64_MSR_EOI 0x40000070
> -#define HV_X64_MSR_ICR 0x40000071
> -#define HV_X64_MSR_TPR 0x40000072
> -#define HV_X64_MSR_APIC_ASSIST_PAGE 0x40000073
> -
> -/* Define synthetic interrupt controller model specific registers. */
> -#define HV_X64_MSR_SCONTROL 0x40000080
> -#define HV_X64_MSR_SVERSION 0x40000081
> -#define HV_X64_MSR_SIEFP 0x40000082
> -#define HV_X64_MSR_SIMP 0x40000083
> -#define HV_X64_MSR_EOM 0x40000084
> -#define HV_X64_MSR_SINT0 0x40000090
> -#define HV_X64_MSR_SINT1 0x40000091
> -#define HV_X64_MSR_SINT2 0x40000092
> -#define HV_X64_MSR_SINT3 0x40000093
> -#define HV_X64_MSR_SINT4 0x40000094
> -#define HV_X64_MSR_SINT5 0x40000095
> -#define HV_X64_MSR_SINT6 0x40000096
> -#define HV_X64_MSR_SINT7 0x40000097
> -#define HV_X64_MSR_SINT8 0x40000098
> -#define HV_X64_MSR_SINT9 0x40000099
> -#define HV_X64_MSR_SINT10 0x4000009A
> -#define HV_X64_MSR_SINT11 0x4000009B
> -#define HV_X64_MSR_SINT12 0x4000009C
> -#define HV_X64_MSR_SINT13 0x4000009D
> -#define HV_X64_MSR_SINT14 0x4000009E
> -#define HV_X64_MSR_SINT15 0x4000009F
> -
> -/*
> - * Synthetic Timer MSRs. Four timers per vcpu.
> - */
> -#define HV_X64_MSR_STIMER0_CONFIG 0x400000B0
> -#define HV_X64_MSR_STIMER0_COUNT 0x400000B1
> -#define HV_X64_MSR_STIMER1_CONFIG 0x400000B2
> -#define HV_X64_MSR_STIMER1_COUNT 0x400000B3
> -#define HV_X64_MSR_STIMER2_CONFIG 0x400000B4
> -#define HV_X64_MSR_STIMER2_COUNT 0x400000B5
> -#define HV_X64_MSR_STIMER3_CONFIG 0x400000B6
> -#define HV_X64_MSR_STIMER3_COUNT 0x400000B7
> -
> -/* Hyper-V guest crash notification MSR's */
> -#define HV_X64_MSR_CRASH_P0 0x40000100
> -#define HV_X64_MSR_CRASH_P1 0x40000101
> -#define HV_X64_MSR_CRASH_P2 0x40000102
> -#define HV_X64_MSR_CRASH_P3 0x40000103
> -#define HV_X64_MSR_CRASH_P4 0x40000104
> -#define HV_X64_MSR_CRASH_CTL 0x40000105
> -#define HV_X64_MSR_CRASH_CTL_NOTIFY (1ULL << 63)
> -#define HV_X64_MSR_CRASH_PARAMS \
> - (1 + (HV_X64_MSR_CRASH_P4 - HV_X64_MSR_CRASH_P0))
> -
> -#define HV_X64_MSR_HYPERCALL_ENABLE 0x00000001
> -#define HV_X64_MSR_HYPERCALL_PAGE_ADDRESS_SHIFT 12
> -#define HV_X64_MSR_HYPERCALL_PAGE_ADDRESS_MASK \
> - (~((1ull << HV_X64_MSR_HYPERCALL_PAGE_ADDRESS_SHIFT) - 1))
> -
> -/* Declare the various hypercall operations. */
> -#define HV_X64_HV_NOTIFY_LONG_SPIN_WAIT 0x0008
> -
> -#define HV_X64_MSR_APIC_ASSIST_PAGE_ENABLE 0x00000001
> -#define HV_X64_MSR_APIC_ASSIST_PAGE_ADDRESS_SHIFT 12
> -#define HV_X64_MSR_APIC_ASSIST_PAGE_ADDRESS_MASK \
> - (~((1ull << HV_X64_MSR_APIC_ASSIST_PAGE_ADDRESS_SHIFT) - 1))
> -
> -#define HV_X64_MSR_TSC_REFERENCE_ENABLE 0x00000001
> -#define HV_X64_MSR_TSC_REFERENCE_ADDRESS_SHIFT 12
> -
> -#define HV_PROCESSOR_POWER_STATE_C0 0
> -#define HV_PROCESSOR_POWER_STATE_C1 1
> -#define HV_PROCESSOR_POWER_STATE_C2 2
> -#define HV_PROCESSOR_POWER_STATE_C3 3
> -
> -/* hypercall status code */
> -#define HV_STATUS_SUCCESS 0
> -#define HV_STATUS_INVALID_HYPERCALL_CODE 2
> -#define HV_STATUS_INVALID_HYPERCALL_INPUT 3
> -#define HV_STATUS_INVALID_ALIGNMENT 4
> -#define HV_STATUS_INSUFFICIENT_MEMORY 11
> -#define HV_STATUS_INVALID_CONNECTION_ID 18
> -#define HV_STATUS_INSUFFICIENT_BUFFERS 19
> -
> -typedef struct _HV_REFERENCE_TSC_PAGE {
> - __u32 tsc_sequence;
> - __u32 res1;
> - __u64 tsc_scale;
> - __s64 tsc_offset;
> -} HV_REFERENCE_TSC_PAGE, *PHV_REFERENCE_TSC_PAGE;
> -
> -#endif
> +#include "standard-headers/asm-x86/hyperv.h"
> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
> index 6a75678..11076191 100755
> --- a/scripts/update-linux-headers.sh
> +++ b/scripts/update-linux-headers.sh
> @@ -79,9 +79,6 @@ for arch in $ARCHLIST; do
> for header in kvm.h kvm_para.h; do
> cp "$tmpdir/include/asm/$header" "$output/linux-headers/asm-$arch"
> done
> - if [ $arch = x86 ]; then
> - cp "$tmpdir/include/asm/hyperv.h" "$output/linux-headers/asm-x86"
> - fi
> if [ $arch = powerpc ]; then
> cp "$tmpdir/include/asm/epapr_hcalls.h"
> "$output/linux-headers/asm-powerpc/"
> fi
> @@ -92,6 +89,9 @@ for arch in $ARCHLIST; do
> cp_portable "$tmpdir/include/asm/kvm_virtio.h"
> "$output/include/standard-headers/asm-s390/"
> cp_portable "$tmpdir/include/asm/virtio-ccw.h"
> "$output/include/standard-headers/asm-s390/"
> fi
> + if [ $arch = x86 ]; then
> + cp_portable "$tmpdir/include/asm/hyperv.h"
> "$output/include/standard-headers/asm-x86/"
> + fi
> done
>
> rm -rf "$output/linux-headers/linux"
> @@ -111,6 +111,9 @@ else
> cp "$linux/COPYING" "$output/linux-headers"
> fi
>
> +cat <<EOF >$output/linux-headers/asm-x86/hyperv.h
> +#include "standard-headers/asm-x86/hyperv.h"
> +EOF
> cat <<EOF >$output/linux-headers/linux/virtio_config.h
> #include "standard-headers/linux/virtio_config.h"
> EOF
> diff --git a/target-i386/kvm.c b/target-i386/kvm.c
> index 066d03d..d8a11be 100644
> --- a/target-i386/kvm.c
> +++ b/target-i386/kvm.c
> @@ -33,7 +33,7 @@
> #include "hw/i386/apic_internal.h"
> #include "hw/i386/apic-msidef.h"
> #include "exec/ioport.h"
> -#include <asm/hyperv.h>
> +#include "standard-headers/asm-x86/hyperv.h"
> #include "hw/pci/pci.h"
> #include "migration/migration.h"
> #include "exec/memattrs.h"
> --
> 2.4.3
- [Qemu-devel] [PATCH v3 0/2] update-linux-headers changes, Paolo Bonzini, 2015/09/10
- [Qemu-devel] [PATCH v3 1/4] pci: remove Link Training error from AER error list, Paolo Bonzini, 2015/09/10
- [Qemu-devel] [PATCH v3 3/4] update-linux-headers: copy standard-headers files one by one, Paolo Bonzini, 2015/09/10
- [Qemu-devel] [PATCH v3 2/4] update Linux headers to 4.2, Paolo Bonzini, 2015/09/10
- [Qemu-devel] [PATCH v3 4/4] target-i386: move asm-x86/hyperv.h to standard-headers, Paolo Bonzini, 2015/09/10
- Re: [Qemu-devel] [PATCH v3 0/2] update-linux-headers changes, Denis V. Lunev, 2015/09/11