Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1198747 > unrolled thread

Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added Will Deacon <will.deacon@arm.com> - 2015-08-03 13:10 +0200
    Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added David Long <dave.long@linaro.org> - 2015-08-03 15:50 +0200
      Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added David Long <dave.long@linaro.org> - 2015-08-03 15:50 +0200
        Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added William Cohen <wcohen@redhat.com> - 2015-08-04 17:10 +0200

#1198747 — Re: [RFC PATCH V2 00/10] ARM64: Uprobe support added

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
--
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/

[toc] | [next] | [standalone]

#1198858

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?

So, also taking into consideration Steve's suggestions about 
reorganizing the asm code, I am now working on a new revision.

-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/

[toc] | [prev] | [next] | [standalone]

#1198860

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.

> So, also taking into consideration Steve's suggestions about
> reorganizing the asm code, I am now working on a new revision.
>
> -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/

[toc] | [prev] | [next] | [standalone]

#1199997

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.

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
>>
> 

--
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/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web