Re: [PATCH] Add MCE support to KVM

From: Anthony Liguori
Date: Sat Apr 18 2009 - 23:34:01 EST


Avi Kivity wrote:

Then we would need to tell which read-only MSRs are setup writeable and which aren't...

I'm okay with an ioctl to setup MCE, but just make sure userspace has all the information to know what the kernel can do rather than the try-and-see-if-it-works approach. We can publish this information via KVM_CAP things, or via another ioctl (see KVM_GET_SUPPORTED_CPUID2 for an example).

Why not introduce a new exit type for MSR reads/writes that aren't handled by the kernel? You just need a bit on the return that indicates whether to GPF because of an invalid MSR access.

KVM_SET_MSRs should be reserved for MSRs that are performance sensitive. Not all of them will be.

Regards,

Anthony Liguori

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/