From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1757554AbcIVO6b (ORCPT <rfc822;w@1wt.eu>);
        Thu, 22 Sep 2016 10:58:31 -0400
Received: from foss.arm.com ([217.140.101.70]:41068 "EHLO foss.arm.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1757521AbcIVO62 (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 22 Sep 2016 10:58:28 -0400
Date: Thu, 22 Sep 2016 09:58:26 -0500
From: Kim Phillips <kim.phillips@arm.com>
To: Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com>
Cc: <linux-kernel@vger.kernel.org>, <acme@kernel.org>,
        <linuxppc-dev@lists.ozlabs.org>, <peterz@infradead.org>,
        <mingo@redhat.com>, <alexander.shishkin@linux.intel.com>,
        <treeze.taeung@gmail.com>, <naveen.n.rao@linux.vnet.ibm.com>,
        <markus@trippelsdorf.de>, <namhyung@kernel.org>, <pawel.moll@arm.com>,
        <chris.ryder@arm.com>, <jolsa@kernel.org>, <mhiramat@kernel.org>
Subject: Re: [PATCH v7 0/6] perf annotate: Cross arch support + few fixes
Message-Id: <20160922095826.e3e5986e5acf0b2f692ef55b@arm.com>
In-Reply-To: <57E36995.4090101@linux.vnet.ibm.com>
References: <1474472876-2706-1-git-send-email-ravi.bangoria@linux.vnet.ibm.com>
        <20160921143427.8db0abae30690baadc18de01@arm.com>
        <57E36995.4090101@linux.vnet.ibm.com>
Organization: ARM
X-Mailer: Sylpheed 3.5.0 (GTK+ 2.24.30; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, 22 Sep 2016 10:48:13 +0530
Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> wrote:

> On Thursday 22 September 2016 01:04 AM, Kim Phillips wrote:
> > On Wed, 21 Sep 2016 21:17:50 +0530
> > Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> wrote:
> >
> >> Kim, I don't have arm test machine. Can you please help me to test
> >> this on arm.
> > This works for me:  hitting return on return instructions yields
> > "Invalid jump offset", but I'll get that later.
> 
> Thanks Kim.
> 
> Hmm.. so, ins__find_arm does not contain logic for return instructions. Navigation
> with return instruction is working fine for x86 and powerpc.

Right, for ARM, in order to match return instructions, 'lr' must be
found to be the operand of the branch instruction, which is not
contained in the 'name' variable passed to ins__find().

I don't want this to inhibit acceptance of this series, however: I plan
on addressing it afterwards, unless, of course, it is perceived as a
problem somehow (compatibility-wise, it is not because it is an error
at the moment).

Thanks,

Kim