Skip to contentSkip to conversation

pnfs-obj: Remove unused variable from objlayout_get_deviceinfo()

PATCH In review Boaz Harrosh 2 participants 3 messages Mar 30, 2012 – Apr 3, 2012

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 pnfs-obj: Remove unused variable from objlayout_get_deviceinfo()

From: Sachin Bhamare <[email protected]>

Local variable 'sb' was not being used in objlayout_get_deviceinfo().

---
 fs/nfs/objlayout/objlayout.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)
fs/nfs/objlayout/objlayout.c +0 -2
index 1ae53eb..b158d1b 100644
@@ -604,7 +604,6 @@ int objlayout_get_deviceinfo(struct pnfs_layout_hdr *pnfslay,
 {
 	struct objlayout_deviceinfo *odi;
 	struct pnfs_device pd;
-	struct super_block *sb;
 	struct page *page, **pages;
 	u32 *p;
 	int err;
@@ -623,7 +622,6 @@ int objlayout_get_deviceinfo(struct pnfs_layout_hdr *pnfslay,
 	pd.pglen = PAGE_SIZE;
 	pd.mincount = 0;
 
-	sb = pnfslay->plh_inode->i_sb;
 	err = nfs4_proc_getdeviceinfo(NFS_SERVER(pnfslay->plh_inode), &pd);
 	dprintk("%s nfs_getdeviceinfo returned %d\n", __func__, err);
 	if (err)
Signature · 1 line
1.7.6.5
#1 lore raw
Quoted 6 lines · Boaz Harrosh

From: Sachin Bhamare <[email protected]>

Local variable 'sb' was not being used in objlayout_get_deviceinfo().

I was transitioning all my tree's to the 3.4-rc0 Kernel and noticed we forgot this patch, Please add it to what ever next-kernel branch you have.

Benny could you please make a pnfs-all-3.3 branch? It should be identical to the 3.3-rc6, probably, but just for completeness.

Tell me if you also want to add the autologin patches to it I can help out with that. Else the client side went into 3.4-rc0 so I'll send you the exofs side for your future 3.4-rc1 tree.

Thanks
Boaz

Quoted 24 lines
---
 fs/nfs/objlayout/objlayout.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)
fs/nfs/objlayout/objlayout.c +0 -2
index 1ae53eb..b158d1b 100644
@@ -604,7 +604,6 @@ int objlayout_get_deviceinfo(struct pnfs_layout_hdr *pnfslay,
 {
 	struct objlayout_deviceinfo *odi;
 	struct pnfs_device pd;
-	struct super_block *sb;
 	struct page *page, **pages;
 	u32 *p;
 	int err;
@@ -623,7 +622,6 @@ int objlayout_get_deviceinfo(struct pnfs_layout_hdr *pnfslay,
 	pd.pglen = PAGE_SIZE;
 	pd.mincount = 0;
 
-	sb = pnfslay->plh_inode->i_sb;
 	err = nfs4_proc_getdeviceinfo(NFS_SERVER(pnfslay->plh_inode), &pd);
 	dprintk("%s nfs_getdeviceinfo returned %d\n", __func__, err);
 	if (err)
#2 lore raw
Quoted 10 lines · Boaz Harrosh
Quoted 6 lines · Boaz Harrosh

From: Sachin Bhamare <[email protected]>

Local variable 'sb' was not being used in objlayout_get_deviceinfo().

Quoted 6 lines

I was transitioning all my tree's to the 3.4-rc0 Kernel and noticed we forgot this patch, Please add it to what ever next-kernel branch you have.

Benny could you please make a pnfs-all-3.3 branch? It should be identical to the 3.3-rc6, probably, but just for completeness.

I'll do that after rebasing to v3.3 (which I've been chasing my tail to do)

Quoted 4 lines

Tell me if you also want to add the autologin patches to it I can help out with that. Else the client side went into 3.4-rc0 so I'll send you the exofs side for your future 3.4-rc1 tree.

I prefer to apply it for 3.4-rc.

Quoted 2 lines

Thanks

Sure :)

Benny

Quoted 28 lines

Boaz

Quoted 24 lines

---
 fs/nfs/objlayout/objlayout.c |    2 --

 1 files changed, 0 insertions(+), 2 deletions(-)
fs/nfs/objlayout/objlayout.c +0 -2
index 1ae53eb..b158d1b 100644
@@ -604,7 +604,6 @@ int objlayout_get_deviceinfo(struct pnfs_layout_hdr *pnfslay,
  {
       struct objlayout_deviceinfo *odi;
       struct pnfs_device pd;
-     struct super_block *sb;
       struct page *page, **pages;
       u32 *p;
       int err;
@@ -623,7 +622,6 @@ int objlayout_get_deviceinfo(struct pnfs_layout_hdr *pnfslay,
       pd.pglen = PAGE_SIZE;
       pd.mincount = 0;
 
-     sb = pnfslay->plh_inode->i_sb;
       err = nfs4_proc_getdeviceinfo(NFS_SERVER(pnfslay->plh_inode), &pd);
       dprintk("%s nfs_getdeviceinfo returned %d\n", __func__, err);
       if (err)