From patchwork Sun Nov 11 04:59:39 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Leo Yan X-Patchwork-Id: 150755 Delivered-To: patch@linaro.org Received: by 2002:a2e:299d:0:0:0:0:0 with SMTP id p29-v6csp1705980ljp; Sat, 10 Nov 2018 21:00:20 -0800 (PST) X-Google-Smtp-Source: AJdET5e2RqC9V5xFwGuIA4s4Ky/HnPhc6YZOp7ABaMoR40tNBrKVQRVLrciwfmLx166TcueOi4qz X-Received: by 2002:a63:554b:: with SMTP id f11mr3449308pgm.37.1541912420762; Sat, 10 Nov 2018 21:00:20 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1541912420; cv=none; d=google.com; s=arc-20160816; b=vLrJ97fa67bkbZTQUA2PT/MV/7xWR5maDxXXJZINm9OQTuB0/Z+NYdRC6uOqA3lOre BAfrpqElI3HXvDtz/5Zl+4k4TSoZIbhJBRbYUTL0K5i4uyaw0dBzU6S72GGCvaeIem+a Gx9roair/4rcodVyT7fC6KS1nKYX5efLc1Jal7ugzAV0gDExzXdGCR3eC80M/wN4xUSK DhP5ieuue26lu/tZolwYFApy7ym+mEegWpNeyYMayWoSfuYYgVYP4xp+ZhUtLpRowRpc EzxVHeeIsGIBRlnD36wZ37suGG8PpxvCF52fjJXX6QkJLF8iZ4vVoRNo3I03T8elDzY2 lDTw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:dkim-signature; bh=ZbLjNsAHRTQqCj6a1vHW0MpbOVGBpYLMhVIToqZ+NHM=; b=wP2UCS1epKl+A6mzwL8nThNRRk+8HFkvta+oyRzY884xy7Y+exl7iEl5oLG0HVEI2F +idVg7T/tDpnQBh3XyyuDcnNSR6R+AQ+P142LuVx0tv2lTnCZ+1roNkINIkMYAfiQPM2 1i3WbfBtOMPqRea4wdlpOcE0glsMFXwJdpLKRHGL6XWFlbEwXHjPFugpS6gPk/1obwMp BFoTMk5d98q1oI4evUSon8VZghuWNt+UE1X0cT6rFszoXf22SZp5TFynPzmq04LxqZlP XX38pbsi3ErWa7KBqoLNVILhKCpKti9CK7qFDZIomwIaNcpHyNm+huVg6SE3yhrsnmd/ M74Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kX3Ru026; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id d9si10798807pgb.105.2018.11.10.21.00.20; Sat, 10 Nov 2018 21:00:20 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kX3Ru026; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727503AbeKKOrn (ORCPT + 32 others); Sun, 11 Nov 2018 09:47:43 -0500 Received: from mail-wm1-f67.google.com ([209.85.128.67]:36476 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727184AbeKKOrn (ORCPT ); Sun, 11 Nov 2018 09:47:43 -0500 Received: by mail-wm1-f67.google.com with SMTP id v70-v6so4777763wmd.1 for ; Sat, 10 Nov 2018 21:00:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=ZbLjNsAHRTQqCj6a1vHW0MpbOVGBpYLMhVIToqZ+NHM=; b=kX3Ru026f/ESYt6B1mTZGk6Ey5ia7fEi5PJ8IB30wxK77FSFx+/TfmzhW8Rt1b9Mws oOALFaXam6s1cb/uM7MDCMsKmVuZD/KEnnc/V2aTzpS+VR8Sng/OTGhv8T2Hcedq4IC5 /gqUm0MrB1tnHKUHCkfhRdA9rVW2rc1/3D1h0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=ZbLjNsAHRTQqCj6a1vHW0MpbOVGBpYLMhVIToqZ+NHM=; b=nKVLLSOz1CWdNmJp5vXSngP3otcpnop2L8gxUNAGtKOsthsQBFUmhFLKrmpftqRUY2 FjHOAd4h6luqUeV9oUa20hC1tqaaBAs9uvk/eqC3YK4MjfAcU/hN1jaLc2v2uBZz5dE3 wPPBEctpNa7qPmOujPxXAAyuit0BfGCudOLr3a1n4kABbA5lHIECHnNmWZXDuH/gggu8 vnit9x1G/M4el/w2MCXlrXMrk2RGxfD9WbKy7iZd9WD8uU5Z8GCvSTKB01QgDEt+3gLS djyAinaGTomr3VAGeNoOgepI5F4SAstx9Ap/4Bvh8Gh7ruQYTWjdCJyJVt/mVybqRG+D xsaA== X-Gm-Message-State: AGRZ1gLlXH++AxEu3E3rvnUDfTIuOWsZ0ddLdcRgnTj4inmDVrGcNhpE q4QiwjUI4dQLXcrwDRjR+zuMbw== X-Received: by 2002:a1c:1a53:: with SMTP id a80-v6mr3928909wma.120.1541912413637; Sat, 10 Nov 2018 21:00:13 -0800 (PST) Received: from localhost.localdomain ([209.250.228.18]) by smtp.gmail.com with ESMTPSA id t187-v6sm4463802wmt.45.2018.11.10.21.00.09 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sat, 10 Nov 2018 21:00:12 -0800 (PST) From: Leo Yan To: Arnaldo Carvalho de Melo , Mathieu Poirier , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Mike Leach , Robert Walker , Al Grant , Coresight ML Cc: Leo Yan Subject: [PATCH v1 1/5] perf cs-etm: Correct packets swapping in cs_etm__flush() Date: Sun, 11 Nov 2018 12:59:39 +0800 Message-Id: <1541912383-19915-2-git-send-email-leo.yan@linaro.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1541912383-19915-1-git-send-email-leo.yan@linaro.org> References: <1541912383-19915-1-git-send-email-leo.yan@linaro.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The structure cs_etm_queue uses 'prev_packet' to point to previous packet, this can be used to combine with new coming packet to generate samples. In function cs_etm__flush() it swaps packets only when the flag 'etm->synth_opts.last_branch' is true, this means that it will not swap packets if without option '--itrace=il' to generate last branch entries; thus for this case the 'prev_packet' doesn't point to the correct previous packet and the stale packet still will be used to generate sequential sample. Thus if dump trace with 'perf script' command we can see the incorrect flow with the stale packet's address info. This patch corrects packets swapping in cs_etm__flush(); except using the flag 'etm->synth_opts.last_branch' it also checks the another flag 'etm->sample_branches', if any flag is true then it swaps packets so can save correct content to 'prev_packet'. Finally this can fix the wrong program flow dumping issue. Signed-off-by: Leo Yan --- tools/perf/util/cs-etm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.7.4 Reviewed-by: Mathieu Poirier diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c index 48ad217..fe18d7b 100644 --- a/tools/perf/util/cs-etm.c +++ b/tools/perf/util/cs-etm.c @@ -997,7 +997,7 @@ static int cs_etm__flush(struct cs_etm_queue *etmq) } swap_packet: - if (etmq->etm->synth_opts.last_branch) { + if (etm->sample_branches || etmq->etm->synth_opts.last_branch) { /* * Swap PACKET with PREV_PACKET: PACKET becomes PREV_PACKET for * the next incoming packet.