Re: [PATCH 3/3] perf tool: Add non arch events for SandyBridge microarchitecture

[Andi Kleen <andi@firstfloor.org>]

Jiri Olsa <jolsa@redhat.com> writes:

> Adding non architectural event aliases for Sandy Bridge
> microarchitecture.

"SandyBridge" is not necessarily unique. There are flavours
of sandy bridge with slightly different events (e.g. in offcore)
Also there's uncores too, which are very different between
different SandyBridges.

I would merge all these little files into a single file.
I don't think anyone wants hundreds of new files for this, 
if you do it for multiple CPUs.

Besides it would waste insane amounts of space on most file systems
without tail packing -- each of your little files takes 4K plus an inode
and directory entry.

Intel normally uses a spreadsheet CSV format.

Best would be probably to just teach perf about reading one of those
standard formats, then they could be just downloaded somewhere.
I have some python scripts that do it as a wrapper.

-Andi

-- 
ak@linux.intel.com -- Speaking for myself only