YunQiang Su
2015-Sep-05 13:39 UTC
[klibc] [PATCH] mips/setjmp.S don't save and restore float point registers
Klibc FTBFS with '-mno-odd-spreg' on mips32(el) platforms, As it try to save/restore odd-number FPR. Indeed no other architectures save/restore FPR at all. It shouldn't be needed. --- usr/klibc/arch/mips/setjmp.S | 24 ------------------------ 1 file changed, 24 deletions(-) diff --git a/usr/klibc/arch/mips/setjmp.S b/usr/klibc/arch/mips/setjmp.S index 68eed19..21e4115 100644 --- a/usr/klibc/arch/mips/setjmp.S +++ b/usr/klibc/arch/mips/setjmp.S @@ -29,18 +29,6 @@ LEAF(setjmp) sw s8, 40(a0) sw ra, 44(a0) cfc1 t0,$31 - swc1 $f20,48(a0) - swc1 $f21,52(a0) - swc1 $f22,56(a0) - swc1 $f23,60(a0) - swc1 $f24,64(a0) - swc1 $f25,68(a0) - swc1 $f26,72(a0) - swc1 $f27,76(a0) - swc1 $f28,80(a0) - swc1 $f29,84(a0) - swc1 $f30,88(a0) - swc1 $f31,92(a0) sw t0,96(a0) move v0,zero jr ra @@ -61,18 +49,6 @@ LEAF(longjmp) lw s8, 40(a0) lw ra, 44(a0) lw t0, 96(a0) - lwc1 $f20,48(a0) - lwc1 $f21,52(a0) - lwc1 $f22,56(a0) - lwc1 $f23,60(a0) - lwc1 $f24,64(a0) - lwc1 $f25,68(a0) - lwc1 $f26,72(a0) - lwc1 $f27,76(a0) - lwc1 $f28,80(a0) - lwc1 $f29,84(a0) - lwc1 $f30,88(a0) - lwc1 $f31,92(a0) ctc1 t0,$31 move v0,a1 jr ra -- 2.5.0
YunQiang Su
2015-Sep-09 06:24 UTC
[klibc] [PATCH] mips/setjmp.S don't save and restore float point registers
Any idea about it? It may be a serious problem, as some distributions are using FPXX by default. On Sat, Sep 5, 2015 at 9:39 PM, YunQiang Su <wzssyqa at gmail.com> wrote:> Klibc FTBFS with '-mno-odd-spreg' on mips32(el) platforms, > As it try to save/restore odd-number FPR. > > Indeed no other architectures save/restore FPR at all. > It shouldn't be needed. > --- > usr/klibc/arch/mips/setjmp.S | 24 ------------------------ > 1 file changed, 24 deletions(-) > > diff --git a/usr/klibc/arch/mips/setjmp.S b/usr/klibc/arch/mips/setjmp.S > index 68eed19..21e4115 100644 > --- a/usr/klibc/arch/mips/setjmp.S > +++ b/usr/klibc/arch/mips/setjmp.S > @@ -29,18 +29,6 @@ LEAF(setjmp) > sw s8, 40(a0) > sw ra, 44(a0) > cfc1 t0,$31 > - swc1 $f20,48(a0) > - swc1 $f21,52(a0) > - swc1 $f22,56(a0) > - swc1 $f23,60(a0) > - swc1 $f24,64(a0) > - swc1 $f25,68(a0) > - swc1 $f26,72(a0) > - swc1 $f27,76(a0) > - swc1 $f28,80(a0) > - swc1 $f29,84(a0) > - swc1 $f30,88(a0) > - swc1 $f31,92(a0) > sw t0,96(a0) > move v0,zero > jr ra > @@ -61,18 +49,6 @@ LEAF(longjmp) > lw s8, 40(a0) > lw ra, 44(a0) > lw t0, 96(a0) > - lwc1 $f20,48(a0) > - lwc1 $f21,52(a0) > - lwc1 $f22,56(a0) > - lwc1 $f23,60(a0) > - lwc1 $f24,64(a0) > - lwc1 $f25,68(a0) > - lwc1 $f26,72(a0) > - lwc1 $f27,76(a0) > - lwc1 $f28,80(a0) > - lwc1 $f29,84(a0) > - lwc1 $f30,88(a0) > - lwc1 $f31,92(a0) > ctc1 t0,$31 > move v0,a1 > jr ra > -- > 2.5.0 >
H. Peter Anvin
2015-Sep-29 22:37 UTC
[klibc] [PATCH] mips/setjmp.S don't save and restore float point registers
On 09/08/2015 11:24 PM, YunQiang Su wrote:> Any idea about it? > It may be a serious problem, as some distributions are using FPXX by default.Sounds reasonable to me. -hpa
H. Peter Anvin
2016-Jan-07 21:04 UTC
[klibc] [PATCH] mips/setjmp.S don't save and restore float point registers
On 09/05/15 06:39, YunQiang Su wrote:> Klibc FTBFS with '-mno-odd-spreg' on mips32(el) platforms, > As it try to save/restore odd-number FPR. > > Indeed no other architectures save/restore FPR at all. > It shouldn't be needed. > --- > usr/klibc/arch/mips/setjmp.S | 24 ------------------------ > 1 file changed, 24 deletions(-)Hi, Could you please re-send this patch with a Signed-off-by:? -hpa
Apparently Analagous Threads
- [PATCH] mips/setjmp.S don't save and restore float point registers
- [klibc:master] mips: don't save floating point registers in setjmp / longjmp
- [PATCH] mips: setjmp: allow working with fpxx/fp64 abi
- [klibc 28/43] mips support for klibc
- [PATCH] add mips64 support