[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <527D5630.3000906@amd.com>
Date: Sat, 9 Nov 2013 06:22:56 +0900
From: Suravee Suthikulpanit <suravee.suthikulpanit@....com>
To: Borislav Petkov <bp@...en8.de>,
Frederic Weisbecker <fweisbec@...il.com>
CC: Borislav Petkov <bp@...e.de>, <mingo@...nel.org>,
<mingo@...hat.com>, <jacob.w.shin@...il.com>, <oleg@...hat.com>,
<a.p.zijlstra@...llo.nl>, <acme@...stprotocols.net>,
<hpa@...or.com>, <linux-kernel@...r.kernel.org>,
<sherry.hurwitz@....com>, Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH 1/3] perf/x86/amd: AMD support for bp_len > HW_BREAKPOINT_LEN_8
On 11/02/2013 01:34 PM, Borislav Petkov wrote:
> On Thu, Oct 31, 2013 at 12:23:30PM +0100, Frederic Weisbecker wrote:
>> Ok we can keep that naming then, at least on the feature symbol. But
>> add a comment on it.
> Great, in the latest F16h BKDG the CPUID bit is called
> "DataBreakpointExtension". So BPEXT could mean anything :)
>
> So the comment is with the definition of the bit:
>
> +#define X86_FEATURE_BPEXT (6*32+26) /* data breakpoint extension */
>
> Oh well...
>
Sorry for late reply. And yes, the BDKG called it "Breakpoint Extension". Keeping the name consistent with the documentation might be best for now to avoid confusion.
So, would you like me to send in a new patch to add this comment?
Suravee
--
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