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

Reply via email to