From: Jin Dongming <jin.dongming@np.css.fujitsu.com>
To: Huang Ying <ying.huang@intel.com>
Cc: Avi Kivity <avi@redhat.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
Andi Kleen <andi@firstfloor.org>,
Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>,
KVM list <kvm@vger.kernel.org>
Subject: Re: [PATCH] Add function for clearing the requested VCPUs' mce registers.
Date: Thu, 25 Nov 2010 14:30:02 +0900 [thread overview]
Message-ID: <4CEDF45A.1000901@np.css.fujitsu.com> (raw)
In-Reply-To: <1290648466.2903.179.camel@yhuang-dev>
Hi, Huang-san
(2010/11/25 10:27), Huang Ying wrote:
> Hi, Dongming,
>
> On Thu, 2010-11-25 at 09:14 +0800, Jin Dongming wrote:
>> In some case of mce test, the injected data can be remained
>> in the registers and ca affect to the result of following test cases.
>> So add codes for clearing mce registers of given VCPU.
>>
>> mce registers of give VCPU could be cleared from kernel by calling
>> the function in this patch. What need to be paid attention is that
>> the status and mcg_status of mce must be set with 0. If not, mce
>> registers will not be cleared.
>
> Why do you need this? To use "mce=3" in guest Linux MCE testing?
When I set fake_panic==1 on Guest OS and tested mce via qemu-monitor,
I got the wrong result sometimes. The reason why I got the wrong result
was that mce registers were not cleared. So I made this patch.
>
> If it is, why not use full reboot? MCE registers are not cleared in real
> machine too. And it is not so pain to reboot the guest.
Though we know that mce registers could be cleared by rebooting the Guest
Machine and the reboot of Guest Machine could save much more time than
the reboot of Host Machine, it is still slower than setting
fake_panic==1 to mce test.
Best Regards,
Jin Dongming
>
> Best Regards,
> Huang Ying
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
next prev parent reply other threads:[~2010-11-25 5:27 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-25 1:14 [PATCH] Add function for clearing the requested VCPUs' mce registers Jin Dongming
2010-11-25 1:27 ` Huang Ying
2010-11-25 5:30 ` Jin Dongming [this message]
2010-11-25 5:44 ` Huang Ying
2010-11-25 8:43 ` Jin Dongming
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=4CEDF45A.1000901@np.css.fujitsu.com \
--to=jin.dongming@np.css.fujitsu.com \
--cc=andi@firstfloor.org \
--cc=avi@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mtosatti@redhat.com \
--cc=seto.hidetoshi@jp.fujitsu.com \
--cc=ying.huang@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.