On Thu, Nov 29, 2012 at 08:37:50PM +0900, Namhyung Kim wrote: > On Wed, 28 Nov 2012 14:52:43 +0100, Jiri Olsa wrote: > > Adding diff command the flexibility to specify multiple data > > files on input. If not input file is given the standard behaviour > > stands and diff inspects 'perf.data' and 'perf.data.old' files. > > > > Also changing the processing and output display for data files. > > > > For 'perf diff A B' command: > > > > - the current way is to iterate over B data and display > > A (baseline) data (symbol samples) only if found in B > > > > - this patch iterates over A (baseline) data and display > > B data (symbol samples) only if found in A > > > > For 'perf diff A B C' command, the diff now iterates the > > A (baseline) data and display B and C data (symbol samples) > > only if they found in A (baseline) > > > > All other standard diff command computation features > > (delta,ratio,wdiff) stays. > > This is quite a large change and IMHO can be separated into 3 patches at > least.
you should have seen this patch before ;-) > > 1) change he->pairs from list to array ok > 2) introduce diff_data__fmt > 3) convert to diff_data__fmt will try ;) thanks, jirka -- 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/