@@ -17,6 +17,8 @@
#include "qom/object_interfaces.h"
#include "system/kvm-pmu.h"
+#define UINT12_MAX (4095)
+
static void kvm_pmu_filter_set_action(Object *obj, int value,
Error **errp G_GNUC_UNUSED)
{
@@ -54,6 +56,12 @@ static void kvm_pmu_filter_get_event(Object *obj, Visitor *v, const char *name,
str_event->u.raw.code = g_strdup_printf("0x%lx",
event->u.raw.code);
break;
+ case KVM_PMU_EVENT_FMT_X86_DEFAULT:
+ str_event->u.x86_default.select =
+ g_strdup_printf("0x%x", event->u.x86_default.select);
+ str_event->u.x86_default.umask =
+ g_strdup_printf("0x%x", event->u.x86_default.umask);
+ break;
default:
g_assert_not_reached();
}
@@ -98,6 +106,60 @@ static void kvm_pmu_filter_set_event(Object *obj, Visitor *v, const char *name,
goto fail;
}
break;
+ case KVM_PMU_EVENT_FMT_X86_DEFAULT: {
+ uint64_t select, umask;
+
+ ret = qemu_strtou64(str_event->u.x86_default.select, NULL,
+ 0, &select);
+ if (ret < 0) {
+ error_setg(errp,
+ "Invalid %s PMU event (select: %s): %s. "
+ "The select must be a "
+ "12-bit unsigned number string.",
+ KVMPMUEventEncodeFmt_str(str_event->format),
+ str_event->u.x86_default.select,
+ strerror(-ret));
+ g_free(event);
+ goto fail;
+ }
+ if (select > UINT12_MAX) {
+ error_setg(errp,
+ "Invalid %s PMU event (select: %s): "
+ "Numerical result out of range. "
+ "The select must be a "
+ "12-bit unsigned number string.",
+ KVMPMUEventEncodeFmt_str(str_event->format),
+ str_event->u.x86_default.select);
+ g_free(event);
+ goto fail;
+ }
+ event->u.x86_default.select = select;
+
+ ret = qemu_strtou64(str_event->u.x86_default.umask, NULL,
+ 0, &umask);
+ if (ret < 0) {
+ error_setg(errp,
+ "Invalid %s PMU event (umask: %s): %s. "
+ "The umask must be a uint8 string.",
+ KVMPMUEventEncodeFmt_str(str_event->format),
+ str_event->u.x86_default.umask,
+ strerror(-ret));
+ g_free(event);
+ goto fail;
+ }
+ if (umask > UINT8_MAX) {
+ error_setg(errp,
+ "Invalid %s PMU event (umask: %s): "
+ "Numerical result out of range. "
+ "The umask must be a uint8 string.",
+ KVMPMUEventEncodeFmt_str(str_event->format),
+ str_event->u.x86_default.umask);
+ g_free(event);
+ goto fail;
+ }
+ event->u.x86_default.umask = umask;
+ break;
+ }
default:
g_assert_not_reached();
}
@@ -27,4 +27,17 @@ struct KVMPMUFilter {
KVMPMUFilterEventList *events;
};
+/*
+ * Stolen from Linux kernel (RAW_EVENT at tools/testing/selftests/kvm/include/
+ * x86_64/pmu.h).
+ *
+ * Encode an eventsel+umask pair into event-select MSR format. Note, this is
+ * technically AMD's format, as Intel's format only supports 8 bits for the
+ * event selector, i.e. doesn't use bits 24:16 for the selector. But, OR-ing
+ * in '0' is a nop and won't clobber the CMASK.
+ */
+#define X86_PMU_RAW_EVENT(eventsel, umask) (((eventsel & 0xf00UL) << 24) | \
+ ((eventsel) & 0xff) | \
+ ((umask) & 0xff) << 8)
+
#endif /* KVM_PMU_H */
@@ -27,11 +27,13 @@
#
# @raw: the encoded event code that KVM can directly consume.
#
+# @x86-default: standard x86 encoding format with select and umask.
+#
# Since 10.0
##
{ 'enum': 'KVMPMUEventEncodeFmt',
'prefix': 'KVM_PMU_EVENT_FMT',
- 'data': ['raw'] }
+ 'data': ['raw', 'x86-default'] }
##
# @KVMPMURawEvent:
@@ -46,6 +48,25 @@
{ 'struct': 'KVMPMURawEvent',
'data': { 'code': 'uint64' } }
+##
+# @KVMPMUX86DefalutEvent:
+#
+# x86 PMU event encoding with select and umask.
+# raw_event = ((select & 0xf00UL) << 24) | \
+# (select) & 0xff) | \
+# ((umask) & 0xff) << 8)
+#
+# @select: x86 PMU event select field, which is a 12-bit unsigned
+# number.
+#
+# @umask: x86 PMU event umask field.
+#
+# Since 10.0
+##
+{ 'struct': 'KVMPMUX86DefalutEvent',
+ 'data': { 'select': 'uint16',
+ 'umask': 'uint8' } }
+
##
# @KVMPMUFilterEvent:
#
@@ -58,7 +79,8 @@
{ 'union': 'KVMPMUFilterEvent',
'base': { 'format': 'KVMPMUEventEncodeFmt' },
'discriminator': 'format',
- 'data': { 'raw': 'KVMPMURawEvent' } }
+ 'data': { 'raw': 'KVMPMURawEvent',
+ 'x86-default': 'KVMPMUX86DefalutEvent' } }
##
# @KVMPMUFilterProperty:
@@ -86,6 +108,23 @@
{ 'struct': 'KVMPMURawEventVariant',
'data': { 'code': 'str' } }
+##
+# @KVMPMUX86DefalutEventVariant:
+#
+# The variant of KVMPMUX86DefalutEvent with the string, rather than
+# the numeric value.
+#
+# @select: x86 PMU event select field. This field is a 12-bit
+# unsigned number string.
+#
+# @umask: x86 PMU event umask field. This field is a uint8 string.
+#
+# Since 10.0
+##
+{ 'struct': 'KVMPMUX86DefalutEventVariant',
+ 'data': { 'select': 'str',
+ 'umask': 'str' } }
+
##
# @KVMPMUFilterEventVariant:
#
@@ -98,7 +137,8 @@
{ 'union': 'KVMPMUFilterEventVariant',
'base': { 'format': 'KVMPMUEventEncodeFmt' },
'discriminator': 'format',
- 'data': { 'raw': 'KVMPMURawEventVariant' } }
+ 'data': { 'raw': 'KVMPMURawEventVariant',
+ 'x86-default': 'KVMPMUX86DefalutEventVariant' } }
##
# @KVMPMUFilterPropertyVariant:
@@ -5970,6 +5970,10 @@ static bool kvm_config_pmu_event(KVMPMUFilter *filter,
case KVM_PMU_EVENT_FMT_RAW:
code = event->u.raw.code;
break;
+ case KVM_PMU_EVENT_FMT_X86_DEFAULT:
+ code = X86_PMU_RAW_EVENT(event->u.x86_default.select,
+ event->u.x86_default.umask);
+ break;
default:
g_assert_not_reached();
}
@@ -6640,6 +6644,7 @@ static void kvm_arch_check_pmu_filter(const Object *obj, const char *name,
switch (event->format) {
case KVM_PMU_EVENT_FMT_RAW:
+ case KVM_PMU_EVENT_FMT_X86_DEFAULT:
break;
default:
error_setg(errp,
The select&umask is the common way for x86 to identify the PMU event, so support this way as the "x86-default" format in kvm-pmu-filter object. Signed-off-by: Zhao Liu <zhao1.liu@intel.com> --- Changes since RFC v1: * Bump up the supported QAPI version to 10.0. --- accel/kvm/kvm-pmu.c | 62 ++++++++++++++++++++++++++++++++++++++++ include/system/kvm-pmu.h | 13 +++++++++ qapi/kvm.json | 46 +++++++++++++++++++++++++++-- target/i386/kvm/kvm.c | 5 ++++ 4 files changed, 123 insertions(+), 3 deletions(-)