summaryrefslogtreecommitdiffstats
path: root/virt
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2021-08-10 06:21:30 -0400
committerPaolo Bonzini <pbonzini@redhat.com>2021-08-13 03:35:15 -0400
commitee3b6e41bc26c628b0c06aacd2758a316fe583be (patch)
tree33d9c2bbcd87edf98f066225360a5ac5d95378dd /virt
parent1ccb6f983a063e794daeb03f90b3517f87dfae8f (diff)
downloadlinux-ee3b6e41bc26c628b0c06aacd2758a316fe583be.tar.gz
linux-ee3b6e41bc26c628b0c06aacd2758a316fe583be.tar.xz
KVM: stats: remove dead stores
These stores are copied and pasted from the "if" statements above. They are dead and while they are not really a bug, they can be confusing to anyone reading the code as well. Remove them. Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'virt')
-rw-r--r--virt/kvm/binary_stats.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/virt/kvm/binary_stats.c b/virt/kvm/binary_stats.c
index e609d428811a..eefca6c69f51 100644
--- a/virt/kvm/binary_stats.c
+++ b/virt/kvm/binary_stats.c
@@ -136,9 +136,7 @@ ssize_t kvm_stats_read(char *id, const struct kvm_stats_header *header,
src = stats + pos - header->data_offset;
if (copy_to_user(dest, src, copylen))
return -EFAULT;
- remain -= copylen;
pos += copylen;
- dest += copylen;
}
*offset = pos;