[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] target-i386: kvm: Print warning when cleari
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] target-i386: kvm: Print warning when clearing mcg_cap bits |
Date: |
Wed, 25 Nov 2015 15:29:25 -0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Wed, Nov 25, 2015 at 05:45:20PM +0100, Paolo Bonzini wrote:
> On 25/11/2015 16:49, Eduardo Habkost wrote:
> > Instead of silently clearing mcg_cap bits when the host doesn't
> > support them, print a warning when doing that.
> >
> > Signed-off-by: Eduardo Habkost <address@hidden>
> > ---
> > target-i386/kvm.c | 8 +++++++-
> > 1 file changed, 7 insertions(+), 1 deletion(-)
> >
> > diff --git a/target-i386/kvm.c b/target-i386/kvm.c
> > index d63a85b..446bdfc 100644
> > --- a/target-i386/kvm.c
> > +++ b/target-i386/kvm.c
> > @@ -774,7 +774,7 @@ int kvm_arch_init_vcpu(CPUState *cs)
> > && (env->features[FEAT_1_EDX] & (CPUID_MCE | CPUID_MCA)) ==
> > (CPUID_MCE | CPUID_MCA)
> > && kvm_check_extension(cs->kvm_state, KVM_CAP_MCE) > 0) {
> > - uint64_t mcg_cap;
> > + uint64_t mcg_cap, unsupported_caps;
> > int banks;
> > int ret;
> >
> > @@ -790,6 +790,12 @@ int kvm_arch_init_vcpu(CPUState *cs)
> > return -ENOTSUP;
> > }
> >
> > + unsupported_caps = env->mcg_cap & ~(mcg_cap | MCG_CAP_BANKS_MASK);
> > + if (unsupported_caps) {
> > + error_report("warning: Unsupported MCG_CAP bits: 0x%" PRIx64
> > "\n",
>
> \n should not be at end of error_report.
>
> Fixed and applied.
MCG_CAP_BANKS_MASK is defined by patch 2/3. Have you applied the
whole series?
--
Eduardo