diff options
author | Vasily Gorbik <gor@linux.ibm.com> | 2018-06-18 17:51:29 +0200 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2018-07-02 11:24:54 +0200 |
commit | ad82a928eb58471adb2dec2001f5fbe57e5ee4b5 (patch) | |
tree | 87704376889741ce43bac62e0e2bd3e8109a6d5f /mm | |
parent | s390: fix gcc 8 stringop-truncation warnings in proc handlers (diff) | |
download | linux-ad82a928eb58471adb2dec2001f5fbe57e5ee4b5.tar.xz linux-ad82a928eb58471adb2dec2001f5fbe57e5ee4b5.zip |
s390/perf: fix gcc 8 array-bounds warning
arch/s390/kernel/perf_regs.c:36:19: warning: array subscript 16 is above
array bounds of 'long unsigned int[16]' [-Warray-bounds]
return regs->gprs[idx];
gcc tries to be smart here and since there is a condition:
if (idx >= PERF_REG_S390_R0 && idx <= PERF_REG_S390_R15)
return regs->gprs[idx];
which covers all possible array subscripts, it gives the warning
for the last function return statement:
return regs->gprs[idx];
which in presence of that condition does not really make sense and
should be replaced with "return 0;"
Also move WARN_ON_ONCE((u32)idx >= PERF_REG_S390_MAX) to the end of the
function.
Reviewed-by: Hendrik Brueckner <brueckner@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions