Re: [PATCH 2/3] perfmon x86 infrastructure definitions

From: stephane eranian
Date: Fri Mar 07 2008 - 16:01:31 EST


Robert,

On Fri, Mar 7, 2008 at 3:42 AM, Robert Richter <robert.richter@xxxxxxx> wrote:
>
> On 06.03.08 22:25:26, stephane eranian wrote:
> > adds AMD Northbridge config MSR definition
> >
> > Signed-off-by: Stephane Eranian <eranian@xxxxxxxxx>
> > Signed-off-by: Robert Richter <robert.richter@xxxxxxx>
>
> > --- a/include/asm-x86/msr-index.h
> > +++ b/include/asm-x86/msr-index.h
> > @@ -83,6 +83,7 @@
> > /* AMD64 MSRs. Not complete. See the architecture manual for a more
> > complete list. */
> >
> > +#define MSR_AMD64_NB_CFG 0xc001001f
>
> This will probably not be needed after mm config works fine (patches
> from Yinghai Lu). Since the usage of CF8 extended configuration cycles
> is not recommended, the current code to enable IBS interrupts can be
> seen as temporary. I will rework the IBS code so that it uses mm
> config.
>
Ok, but I want to push what we have now. We can change this later.
--
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/