All of lore.kernel.org
 help / color / mirror / Atom feed
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 17:43:08 +0900	[thread overview]
Message-ID: <4CEE219C.6070901@np.css.fujitsu.com> (raw)
In-Reply-To: <1290663884.2903.309.camel@yhuang-dev>

Hi, Huang-san

(2010/11/25 14:44), Huang Ying wrote:
> On Thu, 2010-11-25 at 13:30 +0800, Jin Dongming wrote:
>> 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.
> 
> This can be done in QEMU via KVM_SET_MSRS too.
Yes, it could be done in Guest OS with APL. So I will cancel this patch.

Best Regards,
Jin Dongming
>  
>>> 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,
> 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
> 
> 



      reply	other threads:[~2010-11-25  8:41 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
2010-11-25  5:44     ` Huang Ying
2010-11-25  8:43       ` Jin Dongming [this message]

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=4CEE219C.6070901@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.