f51e2f1911
Currently instruction_pointer() returns pt_regs->ret and so return value is of type "long", which implicitly stands for "signed long". While that's perfectly fine when dealing with 32-bit values if return value of instruction_pointer() gets assigned to 64-bit variable sign extension may happen. And at least in one real use-case it happens already. In perf_prepare_sample() return value of perf_instruction_pointer() (which is an alias to instruction_pointer() in case of ARC) is assigned to (struct perf_sample_data)->ip (which type is "u64"). And what we see if instuction pointer points to user-space application that in case of ARC lays below 0x8000_0000 "ip" gets set properly with leading 32 zeros. But if instruction pointer points to kernel address space that starts from 0x8000_0000 then "ip" is set with 32 leadig "f"-s. I.e. id instruction_pointer() returns 0x8100_0000, "ip" will be assigned with 0xffff_ffff__8100_0000. Which is obviously wrong. In particular that issuse broke output of perf, because perf was unable to associate addresses like 0xffff_ffff__8100_0000 with anything from /proc/kallsyms. That's what we used to see: ----------->8---------- 6.27% ls [unknown] [k] 0xffffffff8046c5cc 2.96% ls libuClibc-0.9.34-git.so [.] memcpy 2.25% ls libuClibc-0.9.34-git.so [.] memset 1.66% ls [unknown] [k] 0xffffffff80666536 1.54% ls libuClibc-0.9.34-git.so [.] 0x000224d6 1.18% ls libuClibc-0.9.34-git.so [.] 0x00022472 ----------->8---------- With that change perf output looks much better now: ----------->8---------- 8.21% ls [kernel.kallsyms] [k] memset 3.52% ls libuClibc-0.9.34-git.so [.] memcpy 2.11% ls libuClibc-0.9.34-git.so [.] malloc 1.88% ls libuClibc-0.9.34-git.so [.] memset 1.64% ls [kernel.kallsyms] [k] _raw_spin_unlock_irqrestore 1.41% ls [kernel.kallsyms] [k] __d_lookup_rcu ----------->8---------- Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> Cc: arc-linux-dev@synopsys.com Cc: stable@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
151 lines
3.4 KiB
C
151 lines
3.4 KiB
C
/*
|
|
* Copyright (C) 2004, 2007-2010, 2011-2012 Synopsys, Inc. (www.synopsys.com)
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License version 2 as
|
|
* published by the Free Software Foundation.
|
|
*
|
|
* Amit Bhor, Sameer Dhavale: Codito Technologies 2004
|
|
*/
|
|
#ifndef __ASM_ARC_PTRACE_H
|
|
#define __ASM_ARC_PTRACE_H
|
|
|
|
#include <uapi/asm/ptrace.h>
|
|
|
|
#ifndef __ASSEMBLY__
|
|
|
|
/* THE pt_regs: Defines how regs are saved during entry into kernel */
|
|
|
|
#ifdef CONFIG_ISA_ARCOMPACT
|
|
struct pt_regs {
|
|
|
|
/* Real registers */
|
|
long bta; /* bta_l1, bta_l2, erbta */
|
|
|
|
long lp_start, lp_end, lp_count;
|
|
|
|
long status32; /* status32_l1, status32_l2, erstatus */
|
|
long ret; /* ilink1, ilink2 or eret */
|
|
long blink;
|
|
long fp;
|
|
long r26; /* gp */
|
|
|
|
long r12, r11, r10, r9, r8, r7, r6, r5, r4, r3, r2, r1, r0;
|
|
|
|
long sp; /* user/kernel sp depending on where we came from */
|
|
long orig_r0;
|
|
|
|
/*
|
|
* To distinguish bet excp, syscall, irq
|
|
* For traps and exceptions, Exception Cause Register.
|
|
* ECR: <00> <VV> <CC> <PP>
|
|
* Last word used by Linux for extra state mgmt (syscall-restart)
|
|
* For interrupts, use artificial ECR values to note current prio-level
|
|
*/
|
|
union {
|
|
struct {
|
|
#ifdef CONFIG_CPU_BIG_ENDIAN
|
|
unsigned long state:8, ecr_vec:8,
|
|
ecr_cause:8, ecr_param:8;
|
|
#else
|
|
unsigned long ecr_param:8, ecr_cause:8,
|
|
ecr_vec:8, state:8;
|
|
#endif
|
|
};
|
|
unsigned long event;
|
|
};
|
|
|
|
long user_r25;
|
|
};
|
|
#else
|
|
|
|
struct pt_regs {
|
|
|
|
long orig_r0;
|
|
|
|
union {
|
|
struct {
|
|
#ifdef CONFIG_CPU_BIG_ENDIAN
|
|
unsigned long state:8, ecr_vec:8,
|
|
ecr_cause:8, ecr_param:8;
|
|
#else
|
|
unsigned long ecr_param:8, ecr_cause:8,
|
|
ecr_vec:8, state:8;
|
|
#endif
|
|
};
|
|
unsigned long event;
|
|
};
|
|
|
|
long bta; /* bta_l1, bta_l2, erbta */
|
|
|
|
long user_r25;
|
|
|
|
long r26; /* gp */
|
|
long fp;
|
|
long sp; /* user/kernel sp depending on where we came from */
|
|
|
|
long r12;
|
|
|
|
/*------- Below list auto saved by h/w -----------*/
|
|
long r0, r1, r2, r3, r4, r5, r6, r7, r8, r9, r10, r11;
|
|
|
|
long blink;
|
|
long lp_end, lp_start, lp_count;
|
|
|
|
long ei, ldi, jli;
|
|
|
|
long ret;
|
|
long status32;
|
|
};
|
|
|
|
#endif
|
|
|
|
/* Callee saved registers - need to be saved only when you are scheduled out */
|
|
|
|
struct callee_regs {
|
|
long r25, r24, r23, r22, r21, r20, r19, r18, r17, r16, r15, r14, r13;
|
|
};
|
|
|
|
#define instruction_pointer(regs) (unsigned long)((regs)->ret)
|
|
#define profile_pc(regs) instruction_pointer(regs)
|
|
|
|
/* return 1 if user mode or 0 if kernel mode */
|
|
#define user_mode(regs) (regs->status32 & STATUS_U_MASK)
|
|
|
|
#define user_stack_pointer(regs)\
|
|
({ unsigned int sp; \
|
|
if (user_mode(regs)) \
|
|
sp = (regs)->sp;\
|
|
else \
|
|
sp = -1; \
|
|
sp; \
|
|
})
|
|
|
|
/* return 1 if PC in delay slot */
|
|
#define delay_mode(regs) ((regs->status32 & STATUS_DE_MASK) == STATUS_DE_MASK)
|
|
|
|
#define in_syscall(regs) ((regs->ecr_vec == ECR_V_TRAP) && !regs->ecr_param)
|
|
#define in_brkpt_trap(regs) ((regs->ecr_vec == ECR_V_TRAP) && regs->ecr_param)
|
|
|
|
#define STATE_SCALL_RESTARTED 0x01
|
|
|
|
#define syscall_wont_restart(reg) (reg->state |= STATE_SCALL_RESTARTED)
|
|
#define syscall_restartable(reg) !(reg->state & STATE_SCALL_RESTARTED)
|
|
|
|
#define current_pt_regs() \
|
|
({ \
|
|
/* open-coded current_thread_info() */ \
|
|
register unsigned long sp asm ("sp"); \
|
|
unsigned long pg_start = (sp & ~(THREAD_SIZE - 1)); \
|
|
(struct pt_regs *)(pg_start + THREAD_SIZE) - 1; \
|
|
})
|
|
|
|
static inline long regs_return_value(struct pt_regs *regs)
|
|
{
|
|
return regs->r0;
|
|
}
|
|
|
|
#endif /* !__ASSEMBLY__ */
|
|
|
|
#endif /* __ASM_PTRACE_H */
|