]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit
parisc: Fix validity check of pointer size argument in new CAS implementation
authorJohn David Anglin <dave.anglin@bell.net>
Sat, 11 Nov 2017 22:11:16 +0000 (17:11 -0500)
committerHelge Deller <deller@gmx.de>
Fri, 17 Nov 2017 14:27:13 +0000 (15:27 +0100)
commit05f016d2ca7a4fab99d5d5472168506ddf95e74f
treed020d10f9e2c17fee084cd5471c36f957ded604c
parentbebc6082da0a9f5d47a1ea2edc099bf671058bd4
parisc: Fix validity check of pointer size argument in new CAS implementation

As noted by Christoph Biedl, passing a pointer size of 4 in the new CAS
implementation causes a kernel crash.  The attached patch corrects the
off by one error in the argument validity check.

In reviewing the code, I noticed that we only perform word operations
with the pointer size argument.  The subi instruction intentionally uses
a word condition on 64-bit kernels.  Nullification was used instead of a
cmpib instruction as the branch should never be taken.  The shlw
pseudo-operation generates a depw,z instruction and it clears the target
before doing a shift left word deposit.  Thus, we don't need to clip the
upper 32 bits of this argument on 64-bit kernels.

Tested with a gcc testsuite run with a 64-bit kernel.  The gcc atomic
code in libgcc is the only direct user of the new CAS implementation
that I am aware of.

Signed-off-by: John David Anglin <dave.anglin@bell.net>
Cc: stable@vger.kernel.org # 3.13+
Signed-off-by: Helge Deller <deller@gmx.de>
arch/parisc/kernel/syscall.S