> For PEBS-LL and possibly other special cases, it is important to remember > that perf report always end up sorting by period (hist_collapse_resort). But > for PEBS-LL we want to sort on nr_events * weight. Thus, with your patch, > the only way, I found, to achieve this is by passing: > > add_hist_entry(self, &entry, al, weight, weight);
Seems like a hack. IMHO it should always sort by all the keys i specified with --sort in exactly that order I specified I had a similar thing in a really old version of my patches, but I gave it up because it was too unintuitive. -Andi -- 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/
