aboutsummaryrefslogtreecommitdiffstats
path: root/fs/dlm
diff options
context:
space:
mode:
authorFabian Frederick <fabf@skynet.be>2014-08-08 17:23:37 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2014-08-08 18:57:27 -0400
commite0d9bf4cc0888befd00b1a7db383681be68aada9 (patch)
tree913dc84ad78bb8b120729c0008c111953a7295be /fs/dlm
parentfda9f9903be6c3b590472c175c514b0834bb3c83 (diff)
fs/dlm/debug_fs.c: remove unnecessary null test before debugfs_remove
This fixes checkpatch warning: WARNING: debugfs_remove(NULL) is safe this check is probably not required Signed-off-by: Fabian Frederick <fabf@skynet.be> Cc: Christine Caulfield <ccaulfie@redhat.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/dlm')
-rw-r--r--fs/dlm/debug_fs.c15
1 files changed, 5 insertions, 10 deletions
diff --git a/fs/dlm/debug_fs.c b/fs/dlm/debug_fs.c
index 8d77ba7b1756..1323c568e362 100644
--- a/fs/dlm/debug_fs.c
+++ b/fs/dlm/debug_fs.c
@@ -718,16 +718,11 @@ static const struct file_operations waiters_fops = {
718 718
719void dlm_delete_debug_file(struct dlm_ls *ls) 719void dlm_delete_debug_file(struct dlm_ls *ls)
720{ 720{
721 if (ls->ls_debug_rsb_dentry) 721 debugfs_remove(ls->ls_debug_rsb_dentry);
722 debugfs_remove(ls->ls_debug_rsb_dentry); 722 debugfs_remove(ls->ls_debug_waiters_dentry);
723 if (ls->ls_debug_waiters_dentry) 723 debugfs_remove(ls->ls_debug_locks_dentry);
724 debugfs_remove(ls->ls_debug_waiters_dentry); 724 debugfs_remove(ls->ls_debug_all_dentry);
725 if (ls->ls_debug_locks_dentry) 725 debugfs_remove(ls->ls_debug_toss_dentry);
726 debugfs_remove(ls->ls_debug_locks_dentry);
727 if (ls->ls_debug_all_dentry)
728 debugfs_remove(ls->ls_debug_all_dentry);
729 if (ls->ls_debug_toss_dentry)
730 debugfs_remove(ls->ls_debug_toss_dentry);
731} 726}
732 727
733int dlm_create_debug_file(struct dlm_ls *ls) 728int dlm_create_debug_file(struct dlm_ls *ls)