Commit c8238508 authored by Jan Kara's avatar Jan Kara
Browse files

quota: Replace BUG_ON in dqput() 



The BUG_ON in dqput() will likely take the whole machine down when it
hits. Replace it with WARN_ON_ONCE() instead and stop hiding that behind
CONFIG_QUOTA_DEBUG.

Signed-off-by: default avatarJan Kara <jack@suse.cz>
parent 6613476e

fs/quota/dquot.c

+1 −4
Original line number Diff line number Diff line
@@ -875,10 +875,7 @@ void dqput(struct dquot *dquot) 
	}



	/* Need to release dquot? */

#ifdef CONFIG_QUOTA_DEBUG

	/* sanity check */

	BUG_ON(!list_empty(&dquot->dq_free));

#endif

	WARN_ON_ONCE(!list_empty(&dquot->dq_free));

	put_releasing_dquots(dquot);

	atomic_dec(&dquot->dq_count);

	spin_unlock(&dq_list_lock);