diff options
| author | Nathan Chancellor <natechancellor@gmail.com> | 2019-10-24 22:28:03 +0200 |
|---|---|---|
| committer | Mike Snitzer <snitzer@redhat.com> | 2019-11-07 17:59:38 +0100 |
| commit | 35ad035b8398c634d888324a299ec50a84cbc6cc (patch) | |
| tree | 81112c3baed20ffe14bbac05a34da9b19b3f2477 /drivers/md/dm-builtin.c | |
| parent | dm zoned: reduce overhead of backing device checks (diff) | |
| download | linux-35ad035b8398c634d888324a299ec50a84cbc6cc.tar.xz linux-35ad035b8398c634d888324a299ec50a84cbc6cc.zip | |
dm raid: Remove unnecessary negation of a shift in raid10_format_to_md_layout
When building with Clang + -Wtautological-constant-compare:
drivers/md/dm-raid.c:619:8: warning: converting the result of '<<' to a
boolean always evaluates to true [-Wtautological-constant-compare]
r = !RAID10_OFFSET;
^
drivers/md/dm-raid.c:517:28: note: expanded from macro 'RAID10_OFFSET'
#define RAID10_OFFSET (1 << 16) /* stripes with data
copies area adjacent on devices */
^
1 warning generated.
Negating a non-zero number will always make it zero, which is the
default value of r in this function so this statement is unnecessary;
remove it so that clang no longer warns.
Link: https://github.com/ClangBuiltLinux/linux/issues/753
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Acked-by: Heinz Mauelshagen <heinzm@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md/dm-builtin.c')
0 files changed, 0 insertions, 0 deletions