On Mon, Nov 07, 2011 at 04:45:57PM +0100, Peter Zijlstra wrote: > On Mon, 2011-11-07 at 17:41 +0200, Gleb Natapov wrote: > > > > + entry->eax = min(cap.version, 2) > > > > + | (cap.num_counters_gp << 8) > > > > + | (cap.bit_width_gp << 16) > > > > + | (cap.events_mask_len << 24); > > > > Do you mean by doing cpuid here directly instead of checking perf > > capability? > > No I meant something like: > > union cpuid10_eax eax; > > eax.version = min(cap.version, 2); > eax.bit_width = cap.bit_width; > eax.num_counters = cap.num_counters; > eax.mask_length = cap.mask_length; > > entry->eax = eax.full; > Ah, yes. Will do that. -- Gleb. -- 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