diff options
| author | Brooks Davis <brooks@one-eyed-alien.net> | 2024-04-22 17:48:58 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-04-22 17:48:58 +0000 |
| commit | 7e52795aad561ec39e76a3ef6fea9e5c254b2e16 (patch) | |
| tree | f227624c4e76da7dc7731e2d077c1f20dd32d85f | |
| parent | cdae59e1530061cf4caa549a062994161c4383c6 (diff) | |
| download | src-7e52795aad561ec39e76a3ef6fea9e5c254b2e16.tar.gz src-7e52795aad561ec39e76a3ef6fea9e5c254b2e16.zip | |
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 <behlendorf1@llnl.gov>
Signed-off-by: Brooks Davis <brooks.davis@sri.com>
Closes #16115
| -rw-r--r-- | cmd/ztest.c | 2 |
1 files changed, 1 insertions, 1 deletions
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 */ |