diff options
author | Will Deacon <will@kernel.org> | 2019-11-21 11:58:54 +0000 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2019-11-25 09:14:57 +0100 |
commit | 97a1420adf0cdf0cf6f41bab0b2acf658c96b94b (patch) | |
tree | e83eb898f3d88d3ef35fb872fc79c92fa63a614b /drivers/base | |
parent | 23e6b169c9917fbd77534f8c5f378cb073f548bd (diff) | |
download | linux-97a1420adf0cdf0cf6f41bab0b2acf658c96b94b.tar.bz2 |
locking/refcount: Ensure integer operands are treated as signed
In preparation for changing the saturation point of REFCOUNT_FULL to
INT_MIN/2, change the type of integer operands passed into the API
from 'unsigned int' to 'int' so that we can avoid casting during
comparisons when we don't want to fall foul of C integral conversion
rules for signed and unsigned types.
Since the kernel is compiled with '-fno-strict-overflow', we don't need
to worry about the UB introduced by signed overflow here. Furthermore,
we're already making heavy use of the atomic_t API, which operates
exclusively on signed types.
Signed-off-by: Will Deacon <will@kernel.org>
Reviewed-by: Ard Biesheuvel <ardb@kernel.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
Tested-by: Hanjun Guo <guohanjun@huawei.com>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Elena Reshetova <elena.reshetova@intel.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20191121115902.2551-3-will@kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'drivers/base')
0 files changed, 0 insertions, 0 deletions