From: Timothy Day Date: Mon, 24 Feb 2025 01:25:30 +0000 (-0500) Subject: LU-16518 pcc: fix unused-but-set-variable warnings X-Git-Tag: 2.16.53~63 X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=refs%2Fchanges%2F77%2F58177%2F2;p=fs%2Flustre-release.git LU-16518 pcc: fix unused-but-set-variable warnings 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 Change-Id: I7fde9d49bd6309335e6f9083ba588fc86d495b1c Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/58177 Reviewed-by: Andreas Dilger Reviewed-by: Qian Yingjin Reviewed-by: Max Wang Reviewed-by: James Simmons Reviewed-by: Oleg Drokin Tested-by: jenkins Tested-by: Maloo --- diff --git a/lustre/llite/pcc.c b/lustre/llite/pcc.c index a07315d..5213da6 100644 --- a/lustre/llite/pcc.c +++ b/lustre/llite/pcc.c @@ -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);