Re: [PATCH v3 05/46] perf/x86/intel/cmt: add per-package locks

From: Thomas Gleixner
Date: Wed Nov 16 2016 - 14:03:34 EST


On Mon, 14 Nov 2016, David Carrillo-Cisneros wrote:

> > Also, "monr" is a horribly 'word'.
>
> What makes it so bad? (honest question) . Some alternatives:
>
> - res_mon, resm, rmon (Resource Monitor)
> - rmnode, rnode, rmon_node (Resource Monitoring node, similar to
> Resource Monitor ID, but to reflect that it's a node in a
> tree/hierarchy)
> - rdt_mon, rdtm (something with RDT + Monitoring)
> - ment, rdt_ment (Monitoring Entity)
>
> Other suggestions?

The naming is the least of my worries right now. Before you start to rework
the series can we please get the information about:

- what you want to achieve and why

- the design of your approach

so we can avoid staring at another series of 40+ patches just to figure out
that something is wrong at the conceptual level?

We sort out the naming convention once we are done with the above.

Thanks,

tglx