Claudio Jeker <cje...@diehard.n-r-g.com> writes:

> This are obvious and easy to fix unused but set variables.
> There are more in vioscsi.c but those are actually used if compiled with
> DEBUG set.

ok dv@, but maybe just nuke that fd variable in vioqcow2 instead?

Index: vioqcow2.c
===================================================================
RCS file: /opt/cvs/src/usr.sbin/vmd/vioqcow2.c,v
retrieving revision 1.16
diff -u -p -r1.16 vioqcow2.c
--- vioqcow2.c  16 Jun 2021 16:55:02 -0000      1.16
+++ vioqcow2.c  4 Jan 2022 13:30:09 -0000
@@ -497,12 +497,10 @@ mkcluster(struct qcdisk *disk, struct qc
 {
        off_t l2sz, l1off, l2tab, l2off, cluster, clusteroff, orig;
        uint64_t buf;
-       int fd;

        pthread_rwlock_wrlock(&disk->lock);

        cluster = -1;
-       fd = disk->fd;
        /* L1 entries always exist */
        l2sz = disk->clustersz / 8;
        l1off = off / (disk->clustersz * l2sz);

Reply via email to