Date: Thu, 15 Jun 2017 08:10:18 -0700
From: Andi Kleen <ak@linux.intel.com>
To: Stephane Eranian <eranian@google.com>
Cc: linux-kernel@vger.kernel.org, acme@redhat.com, peterz@infradead.org,
        mingo@elte.hu, kan.liang@intel.com, jolsa@redhat.com
Subject: Re: [PATCH 0/5] perf: add support for capturing skid IP
Message-ID: <20170615151018.GA4789@tassilo.jf.intel.com>
References: <1497534989-29231-1-git-send-email-eranian@google.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1497534989-29231-1-git-send-email-eranian@google.com>
User-Agent: Mutt/1.8.0 (2017-02-23)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 462
Lines: 13

On Thu, Jun 15, 2017 at 06:56:24AM -0700, Stephane Eranian wrote:
> This patchs adds a new sample record type called
> PERF_SAMPLE_SKID_IP. The goal is to record
> the unmodified interrupted instruction pointer (IP) as seen by
> the kernel and reflected in the machine state.

Patches look reasonable for me.

If you only cared about branches it would be more natural to model
it like a 1 entry LBR. That would make a lot more tooling work
automatically.

-Andi