From: Mikulas Patocka Date: Fri, 21 Jul 2017 17:16:06 +0000 (-0400) Subject: dm integrity: WARN_ON if variables representing journal usage get out of sync X-Git-Tag: v4.13-rc3~15^2~10 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aa03a91ffaefcffb397cddf88b97215b3eff726d;p=platform%2Fkernel%2Flinux-exynos.git dm integrity: WARN_ON if variables representing journal usage get out of sync If this WARN_ON triggers it speaks to programmer error, and likely implies corruption, but no released kernel should trigger it. This WARN_ON serves to assist DM integrity developers as changes are made/tested in the future. BUG_ON is excessive for catching programmer error, if a user or developer would like warnings to trigger a panic, they can enable that via /proc/sys/kernel/panic_on_warn Signed-off-by: Mikulas Patocka Signed-off-by: Mike Snitzer --- diff --git a/drivers/md/dm-integrity.c b/drivers/md/dm-integrity.c index be3b6f4..a7a3708 100644 --- a/drivers/md/dm-integrity.c +++ b/drivers/md/dm-integrity.c @@ -1729,6 +1729,8 @@ static void pad_uncommitted(struct dm_integrity_c *ic) wraparound_section(ic, &ic->free_section); ic->n_uncommitted_sections++; } + WARN_ON(ic->journal_sections * ic->journal_section_entries != + (ic->n_uncommitted_sections + ic->n_committed_sections) * ic->journal_section_entries + ic->free_sectors); } static void integrity_commit(struct work_struct *w)