On 21 Sep 2014 21:40, "Alex Shi" <alex.shi@linaro.org> wrote:
>
> I am going to cherry pick d34a3ebd8d25cf691a94fae66a957a480cf46430 into lsk/v3.10/topic/android-fixed branch
> and merge it into lsk-anroid branch to fix the build failure issue. Any objections for this?

None from me.

>
> /arch/arm64/kernel/ptrace.c:1112:3: error: implicit declaration of function 'regs_return_value' [-Werror=implicit-function-declaration]
>
> Regards!
> Alex
>
>
> ===============
> commit d34a3ebd8d25cf691a94fae66a957a480cf46430
> Author: AKASHI Takahiro <takahiro.akashi@linaro.org>
> Date:   Wed Apr 30 10:51:31 2014 +0100
>
>     arm64: Add regs_return_value() in syscall.h
>
>     This macro, regs_return_value, is used mainly for audit to record system
>     call's results, but may also be used in test_kprobes.c.
>
>     Acked-by: Will Deacon <will.deacon@arm.com>
>     Acked-by: Richard Guy Briggs <rgb@redhat.com>
>     Signed-off-by: AKASHI Takahiro <takahiro.akashi@linaro.org>
>     Signed-off-by: Will Deacon <will.deacon@arm.com>
>     Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
>
> diff --git a/arch/arm64/include/asm/ptrace.h b/arch/arm64/include/asm/ptrace.h
> index c7ba261..a429b59 100644
> --- a/arch/arm64/include/asm/ptrace.h
> +++ b/arch/arm64/include/asm/ptrace.h
> @@ -135,6 +135,11 @@ struct pt_regs {
>  #define user_stack_pointer(regs) \
>         (!compat_user_mode(regs)) ? ((regs)->sp) : ((regs)->compat_sp)
>
> +static inline unsigned long regs_return_value(struct pt_regs *regs)
> +{
> +       return regs->regs[0];
> +}
> +
>  /*
>   * Are the current registers suitable for user mode? (used to maintain
>   * security in signal handlers)
>
> --
> Thanks
>     Alex