]> git.itanic.dy.fi Git - linux-stable/commit
metag/usercopy: Set flags before ADDZ
authorJames Hogan <james.hogan@imgtec.com>
Tue, 4 Apr 2017 10:43:26 +0000 (11:43 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 7 Jun 2017 22:47:07 +0000 (00:47 +0200)
commitcabb4ce1458c43c74f250c6f95dd8fce0a3c21f9
treebd0687499553d0d49cdace05f6694e707f26c9ec
parent9a83add8a964a6ccb9da9094856b702049c46173
metag/usercopy: Set flags before ADDZ

commit fd40eee1290ad7add7aa665e3ce6b0f9fe9734b4 upstream.

The fixup code for the copy_to_user rapf loops reads TXStatus.LSM_STEP
to decide how far to rewind the source pointer. There is a special case
for the last execution of an MGETL/MGETD, since it leaves LSM_STEP=0
even though the number of MGETLs/MGETDs attempted was 4. This uses ADDZ
which is conditional upon the Z condition flag, but the AND instruction
which masked the TXStatus.LSM_STEP field didn't set the condition flags
based on the result.

Fix that now by using ANDS which does set the flags, and also marking
the condition codes as clobbered by the inline assembly.

Fixes: 373cd784d0fc ("metag: Memory handling")
Signed-off-by: James Hogan <james.hogan@imgtec.com>
Cc: linux-metag@vger.kernel.org
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Willy Tarreau <w@1wt.eu>
arch/metag/lib/usercopy.c