]> git.proxmox.com Git - mirror_ubuntu-artful-kernel.git/commit
pstore/core: drop cmpxchg based updates
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>
Thu, 8 Sep 2016 11:48:06 +0000 (13:48 +0200)
committerLuis Henriques <luis.henriques@canonical.com>
Tue, 8 Nov 2016 16:45:04 +0000 (16:45 +0000)
commitb8092e43061dd2a300336c92c4a3b0e17b15aceb
tree834090c1abe7b1a5e0c656fda6171cf59735ecc9
parent2d426e9a9839553898293a2932729aad3d0e76bf
pstore/core: drop cmpxchg based updates

BugLink: http://bugs.launchpad.net/bugs/1637510
commit d5a9bf0b38d2ac85c9a693c7fb851f74fd2a2494 upstream.

I have here a FPGA behind PCIe which exports SRAM which I use for
pstore. Now it seems that the FPGA no longer supports cmpxchg based
updates and writes back 0xff…ff and returns the same.  This leads to
crash during crash rendering pstore useless.
Since I doubt that there is much benefit from using cmpxchg() here, I am
dropping this atomic access and use the spinlock based version.

Cc: Anton Vorontsov <anton@enomsg.org>
Cc: Colin Cross <ccross@android.com>
Cc: Kees Cook <keescook@chromium.org>
Cc: Tony Luck <tony.luck@intel.com>
Cc: Rabin Vincent <rabinv@axis.com>
Tested-by: Rabin Vincent <rabinv@axis.com>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
[kees: remove "_locked" suffix since it's the only option now]
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
fs/pstore/ram_core.c