Skip to contentSkip to conversation

ocfs2: remove obscure BUG_ON in dlmglue

PATCH Applied Joseph Qi 2 participants 2 messages Jul 1, 2016 – Jul 4, 2016 OCFS2

lore
Share & tools
lore

Joseph Qi

author
#0 lore raw

PATCH 1/1 ocfs2: remove obscure BUG_ON in dlmglue

These BUG_ON(!inode) are obscure because we have already used inode to get osb. And actually we can guarantee here inode is valid in the context. So we can safely remove them.

---
 fs/ocfs2/dlmglue.c | 9 ---------
 1 file changed, 9 deletions(-)
fs/ocfs2/dlmglue.c +0 -9
index 1cfa1b6bf..2108960 100644
@@ -1634,7 +1634,6 @@ int ocfs2_create_new_inode_locks(struct inode *inode)
 	int ret;
 	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
 	BUG_ON(!ocfs2_inode_is_new(inode));
 
 	mlog(0, "Inode %llu\n", (unsigned long long)OCFS2_I(inode)->ip_blkno);
@@ -1677,8 +1676,6 @@ int ocfs2_rw_lock(struct inode *inode, int write)
 	struct ocfs2_lock_res *lockres;
 	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
-
 	mlog(0, "inode %llu take %s RW lock\n",
 	     (unsigned long long)OCFS2_I(inode)->ip_blkno,
 	     write ? "EXMODE" : "PRMODE");
@@ -1721,8 +1718,6 @@ int ocfs2_open_lock(struct inode *inode)
 	struct ocfs2_lock_res *lockres;
 	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
-
 	mlog(0, "inode %llu take PRMODE open lock\n",
 	     (unsigned long long)OCFS2_I(inode)->ip_blkno);
 
@@ -1746,8 +1741,6 @@ int ocfs2_try_open_lock(struct inode *inode, int write)
 	struct ocfs2_lock_res *lockres;
 	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
-
 	mlog(0, "inode %llu try to take %s open lock\n",
 	     (unsigned long long)OCFS2_I(inode)->ip_blkno,
 	     write ? "EXMODE" : "PRMODE");
@@ -2325,8 +2318,6 @@ int ocfs2_inode_lock_full_nested(struct inode *inode,
 	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 	struct buffer_head *local_bh = NULL;
 
-	BUG_ON(!inode);
-
 	mlog(0, "inode %llu, take %s META lock\n",
 	     (unsigned long long)OCFS2_I(inode)->ip_blkno,
 	     ex ? "EXMODE" : "PRMODE");
Signature · 1 line
1.8.4.3
#1 lore raw

PATCH 1/1 ocfs2: remove obscure BUG_ON in dlmglue

Good catch, thanks!

Quoted 58 lines · Joseph Qi

These BUG_ON(!inode) are obscure because we have already used inode to get osb. And actually we can guarantee here inode is valid in the context. So we can safely remove them.

---
  fs/ocfs2/dlmglue.c | 9 ---------
  1 file changed, 9 deletions(-)
fs/ocfs2/dlmglue.c +0 -9
index 1cfa1b6bf..2108960 100644
@@ -1634,7 +1634,6 @@ int ocfs2_create_new_inode_locks(struct inode *inode)
  	int ret;
  	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
  	BUG_ON(!ocfs2_inode_is_new(inode));
 
  	mlog(0, "Inode %llu\n", (unsigned long long)OCFS2_I(inode)->ip_blkno);
@@ -1677,8 +1676,6 @@ int ocfs2_rw_lock(struct inode *inode, int write)
  	struct ocfs2_lock_res *lockres;
  	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
-
  	mlog(0, "inode %llu take %s RW lock\n",
  	     (unsigned long long)OCFS2_I(inode)->ip_blkno,
  	     write ? "EXMODE" : "PRMODE");
@@ -1721,8 +1718,6 @@ int ocfs2_open_lock(struct inode *inode)
  	struct ocfs2_lock_res *lockres;
  	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
-
  	mlog(0, "inode %llu take PRMODE open lock\n",
  	     (unsigned long long)OCFS2_I(inode)->ip_blkno);
 
@@ -1746,8 +1741,6 @@ int ocfs2_try_open_lock(struct inode *inode, int write)
  	struct ocfs2_lock_res *lockres;
  	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
 
-	BUG_ON(!inode);
-
  	mlog(0, "inode %llu try to take %s open lock\n",
  	     (unsigned long long)OCFS2_I(inode)->ip_blkno,
  	     write ? "EXMODE" : "PRMODE");
@@ -2325,8 +2318,6 @@ int ocfs2_inode_lock_full_nested(struct inode *inode,
  	struct ocfs2_super *osb = OCFS2_SB(inode->i_sb);
  	struct buffer_head *local_bh = NULL;
 
-	BUG_ON(!inode);
-
  	mlog(0, "inode %llu, take %s META lock\n",
  	     (unsigned long long)OCFS2_I(inode)->ip_blkno,
  	     ex ? "EXMODE" : "PRMODE");