diff options
author | Christian Borntraeger <borntraeger@de.ibm.com> | 2015-01-12 06:13:39 -0500 |
---|---|---|
committer | Christian Borntraeger <borntraeger@de.ibm.com> | 2015-01-19 08:15:00 -0500 |
commit | c5b19946eb76c67566aae6a84bf2b10ad59295ea (patch) | |
tree | b52119176597d541599f4c230e1513042ce14b6e /include/linux/compiler.h | |
parent | 378af02b1aecabb3756e19c0cbb8cdd9c3b9637f (diff) |
kernel: Fix sparse warning for ACCESS_ONCE
Commit 927609d622a3 ("kernel: tighten rules for ACCESS ONCE") results in
sparse warnings like "Using plain integer as NULL pointer" - Let's add a
type cast to the dummy assignment.
To avoid warnings lik "sparse: warning: cast to restricted __hc32" we also
use __force on that cast.
Fixes: 927609d622a3 ("kernel: tighten rules for ACCESS ONCE")
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Diffstat (limited to 'include/linux/compiler.h')
-rw-r--r-- | include/linux/compiler.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/compiler.h b/include/linux/compiler.h index 5e186bf6f6c1..7bebf0563d18 100644 --- a/include/linux/compiler.h +++ b/include/linux/compiler.h | |||
@@ -461,7 +461,7 @@ static __always_inline void __assign_once_size(volatile void *p, void *res, int | |||
461 | * If possible use READ_ONCE/ASSIGN_ONCE instead. | 461 | * If possible use READ_ONCE/ASSIGN_ONCE instead. |
462 | */ | 462 | */ |
463 | #define __ACCESS_ONCE(x) ({ \ | 463 | #define __ACCESS_ONCE(x) ({ \ |
464 | __maybe_unused typeof(x) __var = 0; \ | 464 | __maybe_unused typeof(x) __var = (__force typeof(x)) 0; \ |
465 | (volatile typeof(x) *)&(x); }) | 465 | (volatile typeof(x) *)&(x); }) |
466 | #define ACCESS_ONCE(x) (*__ACCESS_ONCE(x)) | 466 | #define ACCESS_ONCE(x) (*__ACCESS_ONCE(x)) |
467 | 467 | ||