From: Laszlo Ersek <lersek@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>,
qemu-devel@nongnu.org, Richard Henderson <rth@twiddle.net>,
Eduardo Habkost <ehabkost@redhat.com>,
"Michael S . Tsirkin" <mst@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
kvm@vger.kernel.org, Boris Petkov <bp@suse.de>,
Tony Luck <tony.luck@intel.com>,
Andi Kleen <andi.kleen@intel.com>,
rkrcmar@redhat.com, Ashok Raj <ashok.raj@intel.com>
Subject: Re: [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg
Date: Wed, 22 Jun 2016 17:51:53 +0200 [thread overview]
Message-ID: <5a3b61d2-c981-2424-a94b-e8d1131b2751@redhat.com> (raw)
In-Reply-To: <20160622101833.kb37aukooifeer32@hz-desktop>
On 06/22/16 12:18, Haozhong Zhang wrote:
> On 06/17/16 19:31, Laszlo Ersek wrote:
>> Hi Haozhong,
>>
>> On 06/16/16 13:19, Haozhong Zhang wrote:
>>> On 06/16/16 11:52, Paolo Bonzini wrote:
> [..]
>>>> Have you prepared a patch for SeaBIOS already?
>>>
>>> Yes, I'll send it after I fix the type error (uint32_t => uint64_t) in
>>> next version.
>>
>> This should be supported by OVMF as well (thanks Paolo for the heads-up).
>>
>> I'm glad to code that up, but I'd like to ask you to file an RFE
>> (Request For Enhancement) in the upstream edk2 tracker for OVMF:
>>
>> https://github.com/tianocore/edk2/issues/new
>>
>> In the RFE,
>> - the subject line should include "OvmfPkg",
>> - please describe what the feature is good for (going into the details
>> of a specific use case is welcome),
>> - please specify the pathname and the internal format of the fw_cfg
>> file,
>> - please clarify if this MSR is considered part of the "chipset state"
>> that the firmware is responsible for at *every* boot. In particular
>> whether you expect that the firmware program this MSR at S3 resume as
>> well.
>>
>
> Done. Pls check https://github.com/tianocore/edk2/issues/97.
Thanks. Looks good. I asked a few more questions in there. Especially if
the APs are involved in this, then the feature is going to be
significantly more complex than I expected.
Thanks
Laszlo
WARNING: multiple messages have this Message-ID (diff)
From: Laszlo Ersek <lersek@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>,
qemu-devel@nongnu.org, Richard Henderson <rth@twiddle.net>,
Eduardo Habkost <ehabkost@redhat.com>,
"Michael S . Tsirkin" <mst@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
kvm@vger.kernel.org, Boris Petkov <bp@suse.de>,
Tony Luck <tony.luck@intel.com>,
Andi Kleen <andi.kleen@intel.com>,
rkrcmar@redhat.com, Ashok Raj <ashok.raj@intel.com>
Subject: Re: [Qemu-devel] [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg
Date: Wed, 22 Jun 2016 17:51:53 +0200 [thread overview]
Message-ID: <5a3b61d2-c981-2424-a94b-e8d1131b2751@redhat.com> (raw)
In-Reply-To: <20160622101833.kb37aukooifeer32@hz-desktop>
On 06/22/16 12:18, Haozhong Zhang wrote:
> On 06/17/16 19:31, Laszlo Ersek wrote:
>> Hi Haozhong,
>>
>> On 06/16/16 13:19, Haozhong Zhang wrote:
>>> On 06/16/16 11:52, Paolo Bonzini wrote:
> [..]
>>>> Have you prepared a patch for SeaBIOS already?
>>>
>>> Yes, I'll send it after I fix the type error (uint32_t => uint64_t) in
>>> next version.
>>
>> This should be supported by OVMF as well (thanks Paolo for the heads-up).
>>
>> I'm glad to code that up, but I'd like to ask you to file an RFE
>> (Request For Enhancement) in the upstream edk2 tracker for OVMF:
>>
>> https://github.com/tianocore/edk2/issues/new
>>
>> In the RFE,
>> - the subject line should include "OvmfPkg",
>> - please describe what the feature is good for (going into the details
>> of a specific use case is welcome),
>> - please specify the pathname and the internal format of the fw_cfg
>> file,
>> - please clarify if this MSR is considered part of the "chipset state"
>> that the firmware is responsible for at *every* boot. In particular
>> whether you expect that the firmware program this MSR at S3 resume as
>> well.
>>
>
> Done. Pls check https://github.com/tianocore/edk2/issues/97.
Thanks. Looks good. I asked a few more questions in there. Especially if
the APs are involved in this, then the feature is going to be
significantly more complex than I expected.
Thanks
Laszlo
next prev parent reply other threads:[~2016-06-22 15:52 UTC|newest]
Thread overview: 81+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-16 6:06 [PATCH v4 0/3] Add QEMU support for Intel local MCE Haozhong Zhang
2016-06-16 6:06 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 6:06 ` [PATCH v4 1/3] target-i386: KVM: add basic Intel LMCE support Haozhong Zhang
2016-06-16 6:06 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 9:50 ` Paolo Bonzini
2016-06-16 9:50 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 10:16 ` Haozhong Zhang
2016-06-16 10:16 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 10:23 ` Paolo Bonzini
2016-06-16 10:23 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 10:34 ` Haozhong Zhang
2016-06-16 10:34 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 10:42 ` Paolo Bonzini
2016-06-16 10:42 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 18:05 ` Eduardo Habkost
2016-06-16 18:05 ` [Qemu-devel] " Eduardo Habkost
2016-06-16 18:17 ` Paolo Bonzini
2016-06-16 18:17 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 19:37 ` Eduardo Habkost
2016-06-16 19:37 ` [Qemu-devel] " Eduardo Habkost
2016-06-17 1:26 ` Haozhong Zhang
2016-06-17 1:26 ` [Qemu-devel] " Haozhong Zhang
2016-06-17 16:20 ` Eduardo Habkost
2016-06-17 16:20 ` [Qemu-devel] " Eduardo Habkost
2016-06-20 2:04 ` Haozhong Zhang
2016-06-20 2:04 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 6:06 ` [PATCH v4 2/3] target-i386: add migration support for Intel LMCE Haozhong Zhang
2016-06-16 6:06 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 9:51 ` Paolo Bonzini
2016-06-16 9:51 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 10:29 ` Haozhong Zhang
2016-06-16 10:29 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 10:41 ` Paolo Bonzini
2016-06-16 10:41 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 10:55 ` Haozhong Zhang
2016-06-16 10:55 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 17:36 ` Eduardo Habkost
2016-06-16 17:36 ` [Qemu-devel] " Eduardo Habkost
2016-06-16 17:40 ` Paolo Bonzini
2016-06-16 17:40 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 17:58 ` Eduardo Habkost
2016-06-16 17:58 ` [Qemu-devel] " Eduardo Habkost
2016-06-17 2:01 ` Haozhong Zhang
2016-06-17 2:01 ` [Qemu-devel] " Haozhong Zhang
2016-06-17 17:20 ` Eduardo Habkost
2016-06-17 17:20 ` [Qemu-devel] " Eduardo Habkost
2016-06-17 17:26 ` Paolo Bonzini
2016-06-17 17:26 ` [Qemu-devel] " Paolo Bonzini
2016-06-20 2:11 ` Haozhong Zhang
2016-06-20 2:11 ` [Qemu-devel] " Haozhong Zhang
2016-06-20 6:58 ` Paolo Bonzini
2016-06-20 6:58 ` [Qemu-devel] " Paolo Bonzini
2016-06-20 7:26 ` Haozhong Zhang
2016-06-16 19:53 ` Eduardo Habkost
2016-06-16 19:53 ` [Qemu-devel] " Eduardo Habkost
2016-06-16 6:06 ` [PATCH v4 3/3] i386: publish advised value of MSR_IA32_FEATURE_CONTROL via fw_cfg Haozhong Zhang
2016-06-16 6:06 ` [Qemu-devel] " Haozhong Zhang
2016-06-16 9:52 ` Paolo Bonzini
2016-06-16 9:52 ` [Qemu-devel] " Paolo Bonzini
2016-06-16 11:19 ` Haozhong Zhang
2016-06-16 11:19 ` [Qemu-devel] " Haozhong Zhang
2016-06-17 17:31 ` Laszlo Ersek
2016-06-17 17:31 ` [Qemu-devel] " Laszlo Ersek
2016-06-17 20:21 ` Raj, Ashok
2016-06-17 20:21 ` [Qemu-devel] " Raj, Ashok
2016-06-17 20:48 ` Laszlo Ersek
2016-06-17 20:48 ` [Qemu-devel] " Laszlo Ersek
2016-06-17 20:55 ` Raj, Ashok
2016-06-17 20:55 ` [Qemu-devel] " Raj, Ashok
2016-06-17 21:30 ` Laszlo Ersek
2016-06-17 21:30 ` [Qemu-devel] " Laszlo Ersek
2016-06-20 3:09 ` Haozhong Zhang
2016-06-20 3:09 ` [Qemu-devel] " Haozhong Zhang
2016-06-20 6:56 ` Paolo Bonzini
2016-06-20 6:56 ` [Qemu-devel] " Paolo Bonzini
2016-06-20 7:20 ` Haozhong Zhang
2016-06-20 7:20 ` [Qemu-devel] " Haozhong Zhang
2016-06-22 10:18 ` Haozhong Zhang
2016-06-22 10:18 ` [Qemu-devel] " Haozhong Zhang
2016-06-22 15:51 ` Laszlo Ersek [this message]
2016-06-22 15:51 ` Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5a3b61d2-c981-2424-a94b-e8d1131b2751@redhat.com \
--to=lersek@redhat.com \
--cc=andi.kleen@intel.com \
--cc=ashok.raj@intel.com \
--cc=bp@suse.de \
--cc=ehabkost@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mst@redhat.com \
--cc=mtosatti@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=rkrcmar@redhat.com \
--cc=rth@twiddle.net \
--cc=tony.luck@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.