From 3969779e1ef31ad49a26e551ca49dab038cb70d4 Mon Sep 17 00:00:00 2001 From: Brooks Davis Date: Mon, 22 Apr 2024 10:48:58 -0700 Subject: [PATCH] ztest: use ASSERT3P to compare pointers With a sufficiently modern gcc (I saw this with gcc13), gcc complains when casting pointers to an integer of a different type (even a larger one). On 32-bt ASSERT3U does this on 32-bit systems by casting a 32-bit pointer to uint64_t so use ASSERT3P which uses uintptr_t. Fixes: 5caeef02fa53 RAID-Z expansion feature Reviewed-by: Brian Behlendorf Signed-off-by: Brooks Davis Closes #16115 --- cmd/ztest.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/ztest.c b/cmd/ztest.c index 684ab586bb93..b0fea8b3cfb4 100644 --- a/cmd/ztest.c +++ b/cmd/ztest.c @@ -8045,7 +8045,7 @@ ztest_raidz_expand_run(ztest_shared_t *zs, spa_t *spa) ztest_expand_io_t *thread_args; ASSERT3U(ztest_opts.zo_raidz_expand_test, !=, RAIDZ_EXPAND_NONE); - ASSERT3U(rzvd->vdev_ops, ==, &vdev_raidz_ops); + ASSERT3P(rzvd->vdev_ops, ==, &vdev_raidz_ops); ztest_opts.zo_raidz_expand_test = RAIDZ_EXPAND_STARTED; /* Setup a 1 MiB buffer of random data */