Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753330AbaKQPIx (ORCPT ); Mon, 17 Nov 2014 10:08:53 -0500 Received: from mga11.intel.com ([192.55.52.93]:11303 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753271AbaKQPIv (ORCPT ); Mon, 17 Nov 2014 10:08:51 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.97,862,1389772800"; d="scan'208";a="417616264" From: kan.liang@intel.com To: acme@kernel.org, jolsa@redhat.com, namhyung@kernel.org Cc: linux-kernel@vger.kernel.org, ak@linux.intel.com, Kan Liang Subject: [PATCH V3 2/3] perf tool:perf diff support for different binaries Date: Mon, 17 Nov 2014 09:52:18 -0500 Message-Id: <1416235939-29565-3-git-send-email-kan.liang@intel.com> X-Mailer: git-send-email 1.8.3.2 In-Reply-To: <1416235939-29565-1-git-send-email-kan.liang@intel.com> References: <1416235939-29565-1-git-send-email-kan.liang@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Kan Liang Currently, the perf diff only works with same binaries. That's because it compares the symbol start address. It doesn't work if the perf.data comes from different binaries. This patch matches the function names. Signed-off-by: Kan Liang Acked-by: Namhyung Kim --- tools/perf/util/sort.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c index 9402885..5b6c57d 100644 --- a/tools/perf/util/sort.c +++ b/tools/perf/util/sort.c @@ -1430,6 +1430,15 @@ int sort_dimension__add(const char *tok) sort__has_parent = 1; } else if (sd->entry == &sort_sym) { sort__has_sym = 1; + /* + * perf diff displays the performance difference amongst + * two or more perf.data files. Those files could come + * from different binaries. So we should not compare + * their ips, but the name of symble. + */ + if (sort__mode == SORT_MODE__DIFF) + sd->entry->se_collapse = sort__sym_sort; + } else if (sd->entry == &sort_dso) { sort__has_dso = 1; } -- 1.8.3.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/