Re: debugctl msr

[stephane eranian]

Markus,

I think your definition for ds_cfg_64 is wrong. On Core, the PEBS
record ALWAYS includes r8-r15 even on
32 bits (zero filled). Also the DS_AREA has 9 fields, not 8.
Consequently, I think the structure should be defined
as follows:

static const struct ds_configuration ds_cfg_64 = {
        .sizeof_ds    = 8 * 9,
        .sizeof_field = 8,
        .sizeof_rec[ds_bts]   = 8 * 3,
        .sizeof_rec[ds_pebs]  = 8 * 18
};

Do you agree?

Thanks.
--
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/