[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7c86c4470811181400r1fa56ef9o1931467ee10e4f52@mail.gmail.com>
Date: Tue, 18 Nov 2008 23:00:56 +0100
From: "stephane eranian" <eranian@...glemail.com>
To: "Markus Metzger" <markus.t.metzger@...glemail.com>
Cc: "Metzger, Markus T" <markus.t.metzger@...el.com>,
"Ingo Molnar" <mingo@...e.hu>, "Andi Kleen" <andi@...stfloor.org>,
"Andrew Morton" <akpm@...ux-foundation.org>,
linux-kernel@...r.kernel.org
Subject: Re: debugctl msr
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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists