revert mips64/n32 syscall asm clean-up due to regressions
authorRich Felker <dalias@aerifal.cx>
Wed, 11 Mar 2020 22:43:11 +0000 (18:43 -0400)
committerRich Felker <dalias@aerifal.cx>
Sun, 15 Mar 2020 01:21:45 +0000 (21:21 -0400)
commit5053fd2644c188ac6531e5fe6a52a1a6d0df2fc2
tree16c93914d9254fc214e0df87f37684b26ca970d1
parent2f2348c9588d61680123bbe438db38acf5dfea4c
revert mips64/n32 syscall asm clean-up due to regressions

effectivly revert commit ddc7c4f936c7a90781072f10dbaa122007e939d0
which was wrong; it caused a major regression on Linux versions prior
to 2.6.36. old kernels did not properly preserve r2 across syscall
restart, and instead restarted with the instruction right before
syscall, imposing a contract that the previous instruction must load
r2 from an immediate or a register (or memory) not clobbered by the
syscall.

since other changes were made since, including removal of the struct
stat conversion that was replaced by separate struct kstat, this is
not a direct revert, only a functional one.

the "0"(r2) input constraint added back seems useless/erroneous, but
without it most gcc versions (seems to be all prior to 9.x) fail to
honor the output register binding for r2. this seems to be a variant
of gcc bug #87733. further changes should be made later if a better
workaround is found, but this one has been working since 2012. it
seems this issue was encountered but misidentified then, when it
inspired commit 4221f154ff29ab0d6be1e7beaa5ea2d1731bc58e.
arch/mips64/syscall_arch.h
arch/mipsn32/syscall_arch.h