Re: [PATCH v8 2/3] perf x86: Topology max dies for whole system

[Alexander Shishkin]

roman.sudarikov@xxxxxxxxxxxxxxx writes:

> From: Roman Sudarikov <roman.sudarikov@xxxxxxxxxxxxxxx>
>
> Helper function to return number of dies on the platform.

It's not a function though, it's a macro. And maybe more of an
"accessor" than a "helper".

It's also good to mention in the commit message, why this change is
needed and sometimes (though maybe not here) why this way is better than
the alternatives.

Another thing about the subject line is that it has to describe the
action that the patch is taking, in imperative mood. In this case it
could be "Turn the max die calculation into a function".

The prefix of the subject line should be: perf/x86/intel/uncore. Check
out git log arch/x86/events/intel/uncore.c for examples.

I also think that it won't be unreasonable to fold this patch into the
next one. That's up to you, of course.

Regards,
--
Alex