On Thu, 2011-11-03 at 14:33 +0200, Gleb Natapov wrote: > + case 0xa: { /* Architectural Performance Monitoring */ > + struct x86_pmu_capability cap; > + > + perf_get_x86_pmu_capability(&cap); > + > + /* > + * Only support guest architectural pmu on a host > + * with architectural pmu. > + */ > + if (!cap.version) > + memset(&cap, 0, sizeof(cap)); > + > + entry->eax = min(cap.version, 2) > + | (cap.num_counters_gp << 8) > + | (cap.bit_width_gp << 16) > + | (cap.events_mask_len << 24); > + entry->ebx = cap.events_mask; > + entry->ecx = 0; > + entry->edx = cap.num_counters_fixed > + | (cap.bit_width_fixed << 5); > + break; > + } would it make sense to use the cpuid10_e[ad]x unions to fill out that data? -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html