* Andi Kleen <a...@firstfloor.org> wrote:
> This is based on v7 of the full Haswell PMU support,
> rebased, and stripped down to the bare bones
Ok, I found some time to still squeeze this into the v3.10 x86 PMU bits
merge window but ran into problems.
You say it's barebones, yet it does not work :-( How well was this
patch-set tested on non-Haswell hardware, which makes up 99.99% of our
installed base?
In particular, after applying your patches, 'perf top' stopped working on
an Intel testbox of mine:
processor : 15
vendor_id : GenuineIntel
cpu family : 6
model : 26
model name : Intel(R) Xeon(R) CPU X55600 @ 2.80GHz
stepping : 5
'perf top' just does not produce any profiling output - it says 0 events.
> Most interesting new features are not in this patchkit
Sigh, we don't want esoteric Haswell-only features that only a very small
subset of people will use.
As I mentioned before, we want _EXISTING_ tools to work and we want the
patchset to be well-tested. Your Haswell-only extensions might be
'interesting' to you but are completely uninteresting to most users who
just want bog standard profiling to work!
Until you don't understand that you will run into problems like this,
putting priorities not into making existing stuff work, but working on
esoteric extensions and featurities, missing the forest from all the
trees...
Your stubborn incompetence is the reason why we are at version 11 of the
patch-set (!) which is _still_ trivially unacceptable and there's _still_
no Haswell support in the upstream kernel...
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
