diff options
author | Andre Noll <maan@systemlinux.org> | 2008-07-11 08:02:20 -0400 |
---|---|---|
committer | Neil Brown <neilb@suse.de> | 2008-07-11 08:02:20 -0400 |
commit | ce0c8e05f8ef93d991d665aade8c4bf35806ea1a (patch) | |
tree | e3fb5bb279452161aca0351993f76b98ffbbb662 /drivers/md/md.c | |
parent | 05710466c9ef2e3ee55166934c801a2393c32f80 (diff) |
md: Simplify sb_equal().
The only caller of sb_equal() tests the return value against
zero, so it's OK to return the negated return value of memcmp().
Signed-off-by: Andre Noll <maan@systemlinux.org>
Signed-off-by: Neil Brown <neilb@suse.de>
Diffstat (limited to 'drivers/md/md.c')
-rw-r--r-- | drivers/md/md.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 8a03c953b887..20e03db8fdfc 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c | |||
@@ -572,11 +572,7 @@ static int sb_equal(mdp_super_t *sb1, mdp_super_t *sb2) | |||
572 | tmp1->nr_disks = 0; | 572 | tmp1->nr_disks = 0; |
573 | tmp2->nr_disks = 0; | 573 | tmp2->nr_disks = 0; |
574 | 574 | ||
575 | if (memcmp(tmp1, tmp2, MD_SB_GENERIC_CONSTANT_WORDS * 4)) | 575 | ret = (memcmp(tmp1, tmp2, MD_SB_GENERIC_CONSTANT_WORDS * 4) == 0); |
576 | ret = 0; | ||
577 | else | ||
578 | ret = 1; | ||
579 | |||
580 | abort: | 576 | abort: |
581 | kfree(tmp1); | 577 | kfree(tmp1); |
582 | kfree(tmp2); | 578 | kfree(tmp2); |