[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h |
Date: |
Fri, 25 May 2018 15:19:07 +0300 |
On Fri, May 25, 2018 at 12:06:17PM +0100, Peter Maydell wrote:
> On 23 May 2018 at 15:43, Michael S. Tsirkin <address@hidden> wrote:
> > Switch to the header we imported from Linux,
> > this allows us to drop a hack in kvm_i386.h.
> > More code will be dropped in the next patch.
> >
> > Signed-off-by: Michael S. Tsirkin <address@hidden>
>
> > --- a/target/i386/cpu.h
> > +++ b/target/i386/cpu.h
> > @@ -688,8 +688,6 @@ typedef uint32_t FeatureWordArray[FEATURE_WORDS];
> > #define CPUID_7_0_EDX_SPEC_CTRL (1U << 26) /* Speculation Control */
> > #define CPUID_7_0_EDX_SPEC_CTRL_SSBD (1U << 31) /* Speculative Store
> > Bypass Disable */
> >
> > -#define KVM_HINTS_DEDICATED (1U << 0)
> > -
> > #define CPUID_8000_0008_EBX_IBPB (1U << 12) /* Indirect Branch
> > Prediction Barrier */
> >
> > #define CPUID_XSAVE_XSAVEOPT (1U << 0)
>
> Hi -- this seems like it will break compilation when we next
> update our copy of the Linux kernel headers,
That just means we'll need to update kvm.c when we do it.
> because (as of
> 4.17-rc6, at least), asm-x86/kvm_para.h doesn't define
> KVM_HINTS_DEDICATED. Here's the diff I get as part of
> my attempt to run update-linux-headers:
>
> --- a/include/standard-headers/asm-x86/kvm_para.h
> +++ b/include/standard-headers/asm-x86/kvm_para.h
> @@ -29,7 +29,7 @@
> #define KVM_FEATURE_PV_TLB_FLUSH 9
> #define KVM_FEATURE_ASYNC_PF_VMEXIT 10
>
> -#define KVM_HINTS_DEDICATED 0
> +#define KVM_HINTS_REALTIME 0
>
> /* The last 8 bits are used to indicate how to interpret the flags field
> * in pvclock structure. If no bits are set, all flags are ignored.
>
> I'm not sure what's going on here -- commit 633711e8287 in
> the kernel just renames the constant, but doesn't that
> break userspace API ?
>
> thanks
> -- PMM
- [Qemu-devel] [PULL 09/28] update-linux-headers.sh: unistd.h, kvm consistency, (continued)
- [Qemu-devel] [PULL 09/28] update-linux-headers.sh: unistd.h, kvm consistency, Michael S. Tsirkin, 2018/05/23
- [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Michael S. Tsirkin, 2018/05/23
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Peter Maydell, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Peter Maydell, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Michael S. Tsirkin, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Peter Maydell, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Michael S. Tsirkin, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Peter Maydell, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Michael S. Tsirkin, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Peter Maydell, 2018/05/25
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PULL 07/28] x86/cpu: use standard-headers/asm-x86.kvm_para.h, Paolo Bonzini, 2018/05/25
[Qemu-devel] [PULL 12/28] vhost-user: add Net prefix to internal state structure, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 14/28] virtio: support setting memory region based host notifier, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 13/28] vhost-user: support receiving file descriptors in slave_read, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 11/28] linux-headers: add kvm header for mips, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 16/28] libvhost-user: Send messages with no data, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 15/28] vhost-user+postcopy: Use qemu_set_nonblock, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 10/28] linux-headers: add unistd.h on all arches, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 17/28] hw/virtio: Fix brace Werror with clang 6.0.0, Michael S. Tsirkin, 2018/05/23
[Qemu-devel] [PULL 20/28] intel-iommu: send PSI always even if across PDEs, Michael S. Tsirkin, 2018/05/23