leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

powerpc: PTRACE_PEEKUSR/PTRACE_POKEUSER of FPR registers in little endian builds

Browse Source 
FPRs overlap the high 64bits of the first 32 VSX registers. The
ptrace FP read/write code assumes big endian ordering and grabs
the lowest 64 bits.

Fix this by using the TS_FPR macro which does the right thing.

Signed-off-by: Anton Blanchard <anton@samba.org>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
Anton Blanchard 2013-09-23 12:04:38 +10:00 committed by Benjamin Herrenschmidt
parent e156bd8ad7
commit 87fec0514f
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
arch/powerpc/kernel/ptrace.c
View File

@ -1554,8 +1554,8 @@ long arch_ptrace(struct task_struct *child, long request,
flush_fp_to_thread(child); flush_fp_to_thread(child);
if (fpidx < (PT_FPSCR - PT_FPR0)) if (fpidx < (PT_FPSCR - PT_FPR0))
tmp = ((unsigned long *)child->thread.fpr) memcpy(&tmp, &child->thread.TS_FPR(fpidx),
[fpidx * TS_FPRWIDTH]; sizeof(long));
else else
tmp = child->thread.fpscr.val; tmp = child->thread.fpscr.val;
} }
@ -1587,8 +1587,8 @@ long arch_ptrace(struct task_struct *child, long request,
flush_fp_to_thread(child); flush_fp_to_thread(child);
if (fpidx < (PT_FPSCR - PT_FPR0)) if (fpidx < (PT_FPSCR - PT_FPR0))
((unsigned long *)child->thread.fpr) memcpy(&child->thread.TS_FPR(fpidx), &data,
[fpidx * TS_FPRWIDTH] = data; sizeof(long));
else else
child->thread.fpscr.val = data; child->thread.fpscr.val = data;
ret = 0; ret = 0;