Can you explain why you think it is redundant? Goldwyn Rodrigues wrote: > Remove redundant BUG_ON() > > Signed-off-by: Goldwyn Rodrigues <rgold...@suse.de> > --- > > diff --git a/fs/ocfs2/dlm/dlmast.c b/fs/ocfs2/dlm/dlmast.c > index d07ddbe..81eff8e 100644 > --- a/fs/ocfs2/dlm/dlmast.c > +++ b/fs/ocfs2/dlm/dlmast.c > @@ -103,7 +103,6 @@ static void __dlm_queue_ast(struct dlm_ctxt *dlm, struct > dlm_lock *lock) > lock->ast_pending, lock->ml.type); > BUG(); > } > - BUG_ON(!list_empty(&lock->ast_list)); > if (lock->ast_pending) > mlog(0, "lock has an ast getting flushed right now\n"); > > >
_______________________________________________ Ocfs2-devel mailing list Ocfs2-devel@oss.oracle.com http://oss.oracle.com/mailman/listinfo/ocfs2-devel