diff options
| author | Colin Ian King <colin.i.king@gmail.com> | 2023-06-21 14:02:56 +0100 |
|---|---|---|
| committer | Vishal Verma <vishal.l.verma@intel.com> | 2023-06-23 01:31:31 -0600 |
| commit | dd0c64258a9d9e74b4896f05c7e77fa3365b5f12 (patch) | |
| tree | f87b78766f1c63f341c3bc848fbadd4aa7ddaff4 | |
| parent | 2532f41607c4308733239dd43278f8a5540f3ec7 (diff) | |
fsdax: remove redundant variable 'error'
The variable 'error' is being assigned a value that is never read,
the assignment and the variable and redundant and can be removed.
Cleans up clang scan build warning:
fs/dax.c:1880:10: warning: Although the value stored to 'error' is
used in the enclosing expression, the value is never actually read
from 'error' [deadcode.DeadStores]
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20230621130256.2676126-1-colin.i.king@gmail.com
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
| -rw-r--r-- | fs/dax.c | 3 |
1 files changed, 1 insertions, 2 deletions
@@ -1830,7 +1830,6 @@ static vm_fault_t dax_iomap_pmd_fault(struct vm_fault *vmf, pfn_t *pfnp, vm_fault_t ret = VM_FAULT_FALLBACK; pgoff_t max_pgoff; void *entry; - int error; if (vmf->flags & FAULT_FLAG_WRITE) iter.flags |= IOMAP_WRITE; @@ -1877,7 +1876,7 @@ static vm_fault_t dax_iomap_pmd_fault(struct vm_fault *vmf, pfn_t *pfnp, } iter.pos = (loff_t)xas.xa_index << PAGE_SHIFT; - while ((error = iomap_iter(&iter, ops)) > 0) { + while (iomap_iter(&iter, ops) > 0) { if (iomap_length(&iter) < PMD_SIZE) continue; /* actually breaks out of the loop */ |