Re: [PATCH] Btrfs: change BUG_ON()'s to ASSERT()'s in backref_cache_cleanup()
From: David Sterba <hidden>
Date: 2016-07-12 16:17:11
On Mon, Jul 11, 2016 at 06:52:57PM -0700, Liu Bo wrote:
Since it is just an in-memory building of the backrefs of several btree blocks, nothing is fatal other than memory leaks, so this changes BUG_ON()'s to ASSERT()'s.
Agreed, this a good justification. Sometimes the BUG_ON serves as a last-resort sanity check so it's not like changing all BUG_ON to ASSERT.
Signed-off-by: Liu Bo <redacted>
Reviewed-by: David Sterba <dsterba@suse.com>