[Skiboot] [RFC PATCH 1/4] ibm-fsp/firenze: uncore header file definitions

Michael Ellerman mpe at ellerman.id.au
Wed Apr 22 11:57:06 AEST 2015


On Wed, 2015-04-22 at 11:44 +1000, Stewart Smith wrote:
> Madhavan Srinivasan <maddy at linux.vnet.ibm.com> writes:
> > +/*
> > + * Power8 has Nest Instrumentation support with which per-chip
> > + * utilisation metrics like memory bandwidth, IO bandwidth and
> > + * many other component metrics can be obtained. These Nest
> > + * counters can be programmed via scoms or HW PORE Engine,
> > + * called PORE_SLW_IMA.
> > + *
> > + * In the Linux kernel side, have used intel's "uncore" framework
> > + * to plug the Power8 nest instrumentation support. For the
> > + * consistency, have used "uncore" for in the variable/file names
> > + * here for services provided by opal.
> > + *
> 
> Why not just call it nest_pmu everywhere? I can't see anywhere that's
> really forcing us to use uncore terminology.

+1

I'm not entirely convinced it makes sense to use the uncore stuff in Linux, but
it definitely doesn't make sense to propagate the name down into skiboot.

cheers




More information about the Skiboot mailing list