VMCI: Use 32bit atomics for queue headers on X86_32
authorJorgen Hansen <jhansen@vmware.com>
Thu, 12 Nov 2015 09:29:32 +0000 (01:29 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Feb 2016 05:36:02 +0000 (21:36 -0800)
commitf42a0fd13bd281811e7457b28d939c8e8b808868
treec738c01147cfc4f5d68f10a55309adeb9139f74d
parentfe760e4d64fe5c17c39e86c410d41f6587ee88bc
VMCI: Use 32bit atomics for queue headers on X86_32

This change restricts the reading and setting of the head and tail
pointers on 32bit X86 to 32bit for both correctness and
performance reasons. On uniprocessor X86_32, the atomic64_read
may be implemented as a non-locked cmpxchg8b. This may result in
updates to the pointers done by the VMCI device being overwritten.
On MP systems, there is no such correctness issue, but using 32bit
atomics avoids the overhead of the locked 64bit operation. All this
is safe because the queue size on 32bit systems will never exceed
a 32bit value.

Reviewed-by: Thomas Hellstrom <thellstrom@vmware.com>
Signed-off-by: Jorgen Hansen <jhansen@vmware.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/vmw_vmci/vmci_driver.c
include/linux/vmw_vmci_defs.h