[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_
From: |
Haozhong Zhang |
Subject: |
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg |
Date: |
Mon, 20 Jun 2016 11:09:41 +0800 |
User-agent: |
Mutt/1.6.1-neo (2016-05-02) |
On 06/17/16 13:21, Raj, Ashok wrote:
> On Fri, Jun 17, 2016 at 07:31:08PM +0200, Laszlo Ersek wrote:
> > >>
> > >> On 16/06/2016 08:06, Haozhong Zhang wrote:
> > >>> It's a prerequisite that certain bits of MSR_IA32_FEATURE_CONTROL should
> > >>> be set before some features (e.g. VMX and LMCE) can be used, which is
> > >>> usually done by the firmware. This patch adds a fw_cfg file
> > >>> "etc/msr_feature_control" which contains the advised value of
> > >>> MSR_IA32_FEATURE_CONTROL and can be used by guest firmware (e.g.
> > >>> SeaBIOS).
> > >>>
>
> I'm sorry i'm joining this discussion a bit late returning from vacation.
> In a real platform supporting LMCE, BIOS is responsible for setting the bits
> for IA32_FEATURE_CONTROL correctly. There are good reasons why we want the
> BIOS to play this role.
>
> in a virtualized environment, do we really have to push the same requirement
> or would it suffice to just emulate it as we did in the early patches.
>
> Not sure what exact problem is created by just simply supporting it within
> kvm/qemu and not needing the bios for the guest to also adapt these changes.
>
In the current nested VMX implementation in QEMU, setup
MSR_IA32_FEATURE_CONTROL is left to guest. So I think, for LMCE which
is another feature involving MSR_IA32_FEATURE_CONTROL, we may follow
the existing code.
Paolo and Radim, is there any case that objects to setting
MSR_IA32_FEATURE_CONTROL in QEMU?
Thanks,
Haozhong
- Re: [Qemu-devel] [PATCH v4 1/3] target-i386: KVM: add basic Intel LMCE support, (continued)
[Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Haozhong Zhang, 2016/06/16
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Paolo Bonzini, 2016/06/16
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Haozhong Zhang, 2016/06/16
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Laszlo Ersek, 2016/06/17
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Raj, Ashok, 2016/06/17
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Laszlo Ersek, 2016/06/17
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Raj, Ashok, 2016/06/17
- Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Laszlo Ersek, 2016/06/17
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg,
Haozhong Zhang <=
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Paolo Bonzini, 2016/06/20
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Haozhong Zhang, 2016/06/20
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Haozhong Zhang, 2016/06/22
Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg, Laszlo Ersek, 2016/06/22
[Qemu-devel] [PATCH v4 2/3] target-i386: add migration support for Intel LMCE, Haozhong Zhang, 2016/06/16