diff options
| author | Will Deacon <will@kernel.org> | 2019-12-19 20:40:21 +0300 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2020-04-15 23:36:41 +0300 |
| commit | 9b4fb5cec031f81ef436bf2cfd9fc265e25f6e45 (patch) | |
| tree | b33080e6d87ae7af66b5ad77987d6a4fa321b0fe /lib/fault-inject.c | |
| parent | 9a8939490d401fefddf53cd5e4cb3e20a52b98a7 (diff) | |
| download | linux-9b4fb5cec031f81ef436bf2cfd9fc265e25f6e45.tar.xz | |
fault_inject: Don't rely on "return value" from WRITE_ONCE()
It's a bit weird that WRITE_ONCE() evaluates to the value it stores and
it's different to smp_store_release(), which can't be used this way.
In preparation for preventing this in WRITE_ONCE(), change the fault
injection code to use a local variable instead.
Cc: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'lib/fault-inject.c')
| -rw-r--r-- | lib/fault-inject.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/fault-inject.c b/lib/fault-inject.c index 8186ca84910b..ce12621b4275 100644 --- a/lib/fault-inject.c +++ b/lib/fault-inject.c @@ -106,7 +106,9 @@ bool should_fail(struct fault_attr *attr, ssize_t size) unsigned int fail_nth = READ_ONCE(current->fail_nth); if (fail_nth) { - if (!WRITE_ONCE(current->fail_nth, fail_nth - 1)) + fail_nth--; + WRITE_ONCE(current->fail_nth, fail_nth); + if (!fail_nth) goto fail; return false; |