summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Dagnelie <pcd@delphix.com>2019-12-18 09:35:57 -0700
committerJerry Jelinek <jerry.jelinek@joyent.com>2019-12-19 13:45:25 -0700
commit028b5df8ad1713f1c0c8ba060ec660fe0a20261d (patch)
tree3359ca37a182becc0b493b98f7b1ea17a428f8b2
parent2c4824dbe94f4768dd1a5058fc108fc83513cf59 (diff)
downloadillumos-joyent-028b5df8ad1713f1c0c8ba060ec660fe0a20261d.tar.gz
12095 Don't call sizeof on void
Portions contributed by: Jerry Jelinek <jerry.jelinek@joyent.com> Reviewed by: Ryan Moeller <ryan@ixsystems.com> Reviewed by: Igor Kozhukhov <igor@dilos.org> Reviewed by: Matt Macy <mmacy@FreeBSD.org> Reviewed by: Toomas Soome <tsoome@me.com> Reviewed by: C Fraire <cfraire@me.com> Reviewed by: Garrett D'Amore <garrett@damore.org> Reviewed by: Joice McIntosh <joicemcintosh@yahoo.com> Approved by: Dan McDonald <danmcd@joyent.com>
-rw-r--r--usr/src/uts/common/fs/zfs/dsl_scan.c2
-rw-r--r--usr/src/uts/common/fs/zfs/sys/range_tree.h12
2 files changed, 7 insertions, 7 deletions
diff --git a/usr/src/uts/common/fs/zfs/dsl_scan.c b/usr/src/uts/common/fs/zfs/dsl_scan.c
index 9d2b7a509c..81c7d599a4 100644
--- a/usr/src/uts/common/fs/zfs/dsl_scan.c
+++ b/usr/src/uts/common/fs/zfs/dsl_scan.c
@@ -1241,7 +1241,7 @@ dsl_scan_should_clear(dsl_scan_t *scn)
if (queue != NULL) {
/* # extents in exts_by_size = # in exts_by_addr */
mused += zfs_btree_numnodes(&queue->q_exts_by_size) *
- sizeof (range_seg_t) + queue->q_sio_memused;
+ sizeof (range_seg_gap_t) + queue->q_sio_memused;
}
mutex_exit(&tvd->vdev_scan_io_queue_lock);
}
diff --git a/usr/src/uts/common/fs/zfs/sys/range_tree.h b/usr/src/uts/common/fs/zfs/sys/range_tree.h
index d80faa933e..6e0025a2f1 100644
--- a/usr/src/uts/common/fs/zfs/sys/range_tree.h
+++ b/usr/src/uts/common/fs/zfs/sys/range_tree.h
@@ -126,11 +126,11 @@ rs_get_start_raw(const range_seg_t *rs, const range_tree_t *rt)
ASSERT3U(rt->rt_type, <=, RANGE_SEG_NUM_TYPES);
switch (rt->rt_type) {
case RANGE_SEG32:
- return (((range_seg32_t *)rs)->rs_start);
+ return (((const range_seg32_t *)rs)->rs_start);
case RANGE_SEG64:
- return (((range_seg64_t *)rs)->rs_start);
+ return (((const range_seg64_t *)rs)->rs_start);
case RANGE_SEG_GAP:
- return (((range_seg_gap_t *)rs)->rs_start);
+ return (((const range_seg_gap_t *)rs)->rs_start);
default:
VERIFY(0);
return (0);
@@ -143,11 +143,11 @@ rs_get_end_raw(const range_seg_t *rs, const range_tree_t *rt)
ASSERT3U(rt->rt_type, <=, RANGE_SEG_NUM_TYPES);
switch (rt->rt_type) {
case RANGE_SEG32:
- return (((range_seg32_t *)rs)->rs_end);
+ return (((const range_seg32_t *)rs)->rs_end);
case RANGE_SEG64:
- return (((range_seg64_t *)rs)->rs_end);
+ return (((const range_seg64_t *)rs)->rs_end);
case RANGE_SEG_GAP:
- return (((range_seg_gap_t *)rs)->rs_end);
+ return (((const range_seg_gap_t *)rs)->rs_end);
default:
VERIFY(0);
return (0);