From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755438Ab3LDCuV (ORCPT ); Tue, 3 Dec 2013 21:50:21 -0500 Received: from ozlabs.org ([203.10.76.45]:57283 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753776Ab3LDCuS (ORCPT ); Tue, 3 Dec 2013 21:50:18 -0500 Message-ID: <1386125416.2602.1.camel@concordia> Subject: Re: [V3 00/10] perf: New conditional branch filter From: Michael Ellerman To: Anshuman Khandual Cc: linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org, mikey@neuling.org, sukadev@linux.vnet.ibm.com, eranian@google.com, Arnaldo Carvalho de Melo Date: Wed, 04 Dec 2013 13:50:16 +1100 In-Reply-To: <1381906617-11392-1-git-send-email-khandual@linux.vnet.ibm.com> References: <1381906617-11392-1-git-send-email-khandual@linux.vnet.ibm.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.8.4-0ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2013-10-16 at 12:26 +0530, Anshuman Khandual wrote: > This patchset is the re-spin of the original branch stack sampling > patchset which introduced new PERF_SAMPLE_BRANCH_COND branch filter. This patchset > also enables SW based branch filtering support for book3s powerpc platforms which > have PMU HW backed branch stack sampling support. > > Summary of code changes in this patchset: > > (1) Introduces a new PERF_SAMPLE_BRANCH_COND branch filter > (2) Add the "cond" branch filter options in the "perf record" tool > (3) Enable PERF_SAMPLE_BRANCH_COND in X86 platforms > (4) Enable PERF_SAMPLE_BRANCH_COND in POWER8 platform > (5) Update the documentation regarding "perf record" tool Can you please address my comments and then resend patches 1-5. And make sure you send them to the perf maintainers. Those three touch the generic code, powerpc and x86, so we'll get those merged first, and then focus on the remaining patches, which are powerpc specific. cheers