Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753221AbbHDPnE (ORCPT ); Tue, 4 Aug 2015 11:43:04 -0400 Received: from mail-qg0-f53.google.com ([209.85.192.53]:33482 "EHLO mail-qg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751073AbbHDPnC (ORCPT ); Tue, 4 Aug 2015 11:43:02 -0400 Message-ID: <55C0DD84.5010002@linaro.org> Date: Tue, 04 Aug 2015 11:43:00 -0400 From: David Long User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: William Cohen , Will Deacon , Pratyush Anand , "steve.capper@linaro.org" CC: "linux-arm-kernel@lists.infradead.org" , "linux@arm.linux.org.uk" , Catalin Marinas , "srikar@linux.vnet.ibm.com" , "vijaya.kumar@caviumnetworks.com" , "linux-kernel@vger.kernel.org" , "oleg@redhat.com" Subject: Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added References: <20150803110908.GC10501@arm.com> <55BF7019.9080007@linaro.org> <55BF707F.7020208@linaro.org> <55C0D53F.6080403@redhat.com> In-Reply-To: <55C0D53F.6080403@redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2374 Lines: 54 On 08/04/15 11:07, William Cohen wrote: > On 08/03/2015 09:45 AM, David Long wrote: >> On 08/03/15 09:43, David Long wrote: >>> On 08/03/15 07:09, Will Deacon wrote: >>>> On Thu, Jun 18, 2015 at 04:58:47AM +0100, Pratyush Anand wrote: >>>>> These patches have been prepared on top of ARM64 kprobe v7 patches [1]. >>>>> Keeping as RFC, because kprobe-v7 still need to be ACKed. >>>> >>>> Unfortunately, I've not seen any movement on the kprobes patches >>>> recently, >>>> so this is out of the picture for 4.3. >>>> >>>> Dave: did you plan to respin your series after Steve's comments on v7? >>>> >>>> Will >>>> >>> >>> Catalin's comments last week helped me clarify in my own mind that we >>> should be able to simplfy the register pushing somewhat. Will, does that >>> also make sense to you? >>> >> >> ^^ That question was aimed at Will Cohen, just to clarify. > > Hi Dave, > > The suggestion to only save the caller saved registers would make the save and restore code shorter and faster. There would be cases reduced saved and restore could be visible to systemtap scripts using print_reg() and register() functions such when a call to a return probed function is followed by a call to a function with a kprobe on entry. I worry about cases where code is broken (not following the calling conventions maybe assembly code functions) and using a return probe causes things to fail in a different way making it more difficult to diagnose the problem. There are less than 50 "*.S" files for arm64, but I prefer to minimize the chances that the return probe changes something. > Perhaps the best argument for leaving it as-is is so that user-written kprobe modules can alter the values of these registers, although specifically altering callee-saved registers on function returnd does not sound as useful as say modifying x0. > There is room for improvements for the patch. The magic offset numbers in the save restore code could be eliminated. > > -Will > >> >>> So, also taking into consideration Steve's suggestions about >>> reorganizing the asm code, I am now working on a new revision. >>> >>> -dl >>> >> > -dl -- 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/