Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-08-01 Thread Paolo Bonzini
On 01/08/2018 17:21, Brian Gerst wrote: > On Tue, Jul 31, 2018 at 9:00 AM Paolo Bonzini wrote: >> >> On 31/07/2018 14:57, tedheadster wrote: This shouldn't be necessary; for systems that don't have virtualization extensions, the comment explains why setting X86_FEATURE_VMMCALL is

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-08-01 Thread Paolo Bonzini
On 01/08/2018 17:21, Brian Gerst wrote: > On Tue, Jul 31, 2018 at 9:00 AM Paolo Bonzini wrote: >> >> On 31/07/2018 14:57, tedheadster wrote: This shouldn't be necessary; for systems that don't have virtualization extensions, the comment explains why setting X86_FEATURE_VMMCALL is

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-08-01 Thread Brian Gerst
On Tue, Jul 31, 2018 at 9:00 AM Paolo Bonzini wrote: > > On 31/07/2018 14:57, tedheadster wrote: > >> > >> This shouldn't be necessary; for systems that don't have virtualization > >> extensions, the comment explains why setting X86_FEATURE_VMMCALL is safe. > >> > >> But it is also wrong, because

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-08-01 Thread Brian Gerst
On Tue, Jul 31, 2018 at 9:00 AM Paolo Bonzini wrote: > > On 31/07/2018 14:57, tedheadster wrote: > >> > >> This shouldn't be necessary; for systems that don't have virtualization > >> extensions, the comment explains why setting X86_FEATURE_VMMCALL is safe. > >> > >> But it is also wrong, because

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-31 Thread Paolo Bonzini
On 31/07/2018 14:57, tedheadster wrote: >> >> This shouldn't be necessary; for systems that don't have virtualization >> extensions, the comment explains why setting X86_FEATURE_VMMCALL is safe. >> >> But it is also wrong, because you can run a 32-bit kernel as a guest on >> a 64-bit processor,

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-31 Thread Paolo Bonzini
On 31/07/2018 14:57, tedheadster wrote: >> >> This shouldn't be necessary; for systems that don't have virtualization >> extensions, the comment explains why setting X86_FEATURE_VMMCALL is safe. >> >> But it is also wrong, because you can run a 32-bit kernel as a guest on >> a 64-bit processor,

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-31 Thread tedheadster
> > This shouldn't be necessary; for systems that don't have virtualization > extensions, the comment explains why setting X86_FEATURE_VMMCALL is safe. > > But it is also wrong, because you can run a 32-bit kernel as a guest on > a 64-bit processor, and then it should set X86_FEATURE_VMMCALL

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-31 Thread tedheadster
> > This shouldn't be necessary; for systems that don't have virtualization > extensions, the comment explains why setting X86_FEATURE_VMMCALL is safe. > > But it is also wrong, because you can run a 32-bit kernel as a guest on > a 64-bit processor, and then it should set X86_FEATURE_VMMCALL

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-31 Thread Paolo Bonzini
On 30/07/2018 21:40, Matthew Whitehead wrote: > early_init_amd() is called on all AMD processors, both 64 and 32 bit. > Presently 32 bit processors get the X86_FEATURE_VMMCALL feature set, which > they do not support. Add conditionals to restrict it to 64 bit processors. This shouldn't be

Re: [PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-31 Thread Paolo Bonzini
On 30/07/2018 21:40, Matthew Whitehead wrote: > early_init_amd() is called on all AMD processors, both 64 and 32 bit. > Presently 32 bit processors get the X86_FEATURE_VMMCALL feature set, which > they do not support. Add conditionals to restrict it to 64 bit processors. This shouldn't be

[PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-30 Thread Matthew Whitehead
early_init_amd() is called on all AMD processors, both 64 and 32 bit. Presently 32 bit processors get the X86_FEATURE_VMMCALL feature set, which they do not support. Add conditionals to restrict it to 64 bit processors. Cc: Paolo Bonzini Signed-off-by: Matthew Whitehead ---

[PATCH] x86: kvm: Restrict X86_FEATURE_VMMCALL to x86_64 platform

2018-07-30 Thread Matthew Whitehead
early_init_amd() is called on all AMD processors, both 64 and 32 bit. Presently 32 bit processors get the X86_FEATURE_VMMCALL feature set, which they do not support. Add conditionals to restrict it to 64 bit processors. Cc: Paolo Bonzini Signed-off-by: Matthew Whitehead ---