NUM_FPU_REGS just makes it clearer what's going on, rather than the magic hard coded 32. Signed-off-by: Paul Burton <paul.burton@xxxxxxxxxx> --- arch/mips/kernel/signal.c | 4 ++-- arch/mips/kernel/signal32.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/mips/kernel/signal.c b/arch/mips/kernel/signal.c index e0178e1..0f97c7d 100644 --- a/arch/mips/kernel/signal.c +++ b/arch/mips/kernel/signal.c @@ -69,7 +69,7 @@ static int copy_fp_to_sigcontext(struct sigcontext __user *sc) int i; int err = 0; - for (i = 0; i < 32; i++) { + for (i = 0; i < NUM_FPU_REGS; i++) { err |= __put_user(get_fpr64(¤t->thread.fpu.fpr[i], 0), &sc->sc_fpregs[i]); @@ -85,7 +85,7 @@ static int copy_fp_from_sigcontext(struct sigcontext __user *sc) int err = 0; u64 fpr_val; - for (i = 0; i < 32; i++) { + for (i = 0; i < NUM_FPU_REGS; i++) { err |= __get_user(fpr_val, &sc->sc_fpregs[i]); set_fpr64(¤t->thread.fpu.fpr[i], 0, fpr_val); } diff --git a/arch/mips/kernel/signal32.c b/arch/mips/kernel/signal32.c index aec5821..bae2e6e 100644 --- a/arch/mips/kernel/signal32.c +++ b/arch/mips/kernel/signal32.c @@ -84,7 +84,7 @@ static int copy_fp_to_sigcontext32(struct sigcontext32 __user *sc) int err = 0; int inc = test_thread_flag(TIF_32BIT_FPREGS) ? 2 : 1; - for (i = 0; i < 32; i += inc) { + for (i = 0; i < NUM_FPU_REGS; i += inc) { err |= __put_user(get_fpr64(¤t->thread.fpu.fpr[i], 0), &sc->sc_fpregs[i]); @@ -101,7 +101,7 @@ static int copy_fp_from_sigcontext32(struct sigcontext32 __user *sc) int inc = test_thread_flag(TIF_32BIT_FPREGS) ? 2 : 1; u64 fpr_val; - for (i = 0; i < 32; i += inc) { + for (i = 0; i < NUM_FPU_REGS; i += inc) { err |= __get_user(fpr_val, &sc->sc_fpregs[i]); set_fpr64(¤t->thread.fpu.fpr[i], 0, fpr_val); } -- 1.8.5.3