Whamcloud - gitweb
LU-16518 pcc: fix unused-but-set-variable warnings 77/58177/2
authorTimothy Day <timday@amazon.com>
Mon, 24 Feb 2025 01:25:30 +0000 (20:25 -0500)
committerOleg Drokin <green@whamcloud.com>
Thu, 6 Mar 2025 08:07:57 +0000 (08:07 +0000)
In several places, pcc_file is set but never used.
Clang doesn't like this, so discard this variable.

Test-Parameters: trivial
Signed-off-by: Timothy Day <timday@amazon.com>
Change-Id: I7fde9d49bd6309335e6f9083ba588fc86d495b1c
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/58177
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Qian Yingjin <qian@ddn.com>
Reviewed-by: Max Wang <wamax@amazon.com>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
lustre/llite/pcc.c

index a07315d..5213da6 100644 (file)
@@ -3190,7 +3190,6 @@ void pcc_vm_open(struct vm_area_struct *vma)
 {
        struct pcc_vma *pccv = (struct pcc_vma *)vma->vm_private_data;
        struct vvp_object *vob;
-       struct pcc_file *pccf;
        struct inode *inode;
 
        ENTRY;
@@ -3202,7 +3201,6 @@ void pcc_vm_open(struct vm_area_struct *vma)
        LASSERT(atomic_read(&vob->vob_mmap_cnt) >= 0);
        atomic_inc(&vob->vob_mmap_cnt);
 
-       pccf = ll_file2pccf(pccv->pccv_file);
        atomic_inc(&pccv->pccv_refcnt);
        if (pccv->pccv_vm_ops->open)
                pccv->pccv_vm_ops->open(vma);
@@ -3216,7 +3214,6 @@ void pcc_vm_close(struct vm_area_struct *vma)
 {
        struct pcc_vma *pccv = (struct pcc_vma *)vma->vm_private_data;
        struct vvp_object *vob;
-       struct pcc_file *pccf;
        struct inode *inode;
 
        ENTRY;
@@ -3232,7 +3229,6 @@ void pcc_vm_close(struct vm_area_struct *vma)
        if (pccv->pccv_vm_ops && pccv->pccv_vm_ops->close)
                pccv->pccv_vm_ops->close(vma);
 
-       pccf = ll_file2pccf(pccv->pccv_file);
        pcc_inode_mmap_put(inode);
        if (atomic_dec_and_test(&pccv->pccv_refcnt)) {
                fput(pccv->pccv_file);