Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751884AbbHKRgI (ORCPT ); Tue, 11 Aug 2015 13:36:08 -0400 Received: from foss.arm.com ([217.140.101.70]:54576 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751191AbbHKRgF (ORCPT ); Tue, 11 Aug 2015 13:36:05 -0400 Date: Tue, 11 Aug 2015 18:36:01 +0100 From: Will Deacon To: David Long Cc: Catalin Marinas , "linux-arm-kernel@lists.infradead.org" , Russell King , "sandeepa.s.prabhu@gmail.com" , William Cohen , Steve Capper , "Jon Medhurst (Tixy)" , Masami Hiramatsu , Ananth N Mavinakayanahalli , Anil S Keshavamurthy , "davem@davemloft.net" , Mark Brown , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v8 0/7] arm64: Add kernel probes (kprobes) support Message-ID: <20150811173601.GD29880@arm.com> References: <1439254364-15362-1-git-send-email-dave.long@linaro.org> <20150811165645.GA29880@arm.com> <55CA2AF2.6040202@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <55CA2AF2.6040202@linaro.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1919 Lines: 44 On Tue, Aug 11, 2015 at 06:03:46PM +0100, David Long wrote: > On 08/11/15 12:56, Will Deacon wrote: > > On Tue, Aug 11, 2015 at 01:52:37AM +0100, David Long wrote: > >> From: "David A. Long" > >> > >> This patchset is heavily based on Sandeepa Prabhu's ARM v8 kprobes patches, > >> first seen in October 2013. This version attempts to address concerns raised by > >> reviewers and also fixes problems discovered during testing. > >> > >> This patchset adds support for kernel probes(kprobes), jump probes(jprobes) > >> and return probes(kretprobes) support for ARM64. > >> > >> The kprobes mechanism makes use of software breakpoint and single stepping > >> support available in the ARM v8 kernel. > >> > >> This patch depends on: > >> [PATCH 1/2] Move the pt_regs_offset struct definition from arch to common include file > >> [PATCH 2/2] Consolidate redundant register/stack access code > > > > Are these two queued somewhere? This series doesn't even build without > > them. > > I posted the last revision of that on July 27. I don't know if they're > "queued" somewhere. They do affect multiple architectures so I'm not > certain where they would get queued. They also live in a branch in my > own Linaro repo. Well they're not in linux-next, so I guess they're not currently destined for mainline :( > Was there a better way for me to deal with this dependency? I was > reluctant to make this into one patch set as the other patch really does > stand alone as a useful fix. I agree (well, it's a cleanup not a fix). Perhaps it's worth reposting them with akpm also on Cc, as he sometimes picks up tree-wide patches like that. Will -- 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/