From 333fd3ee3265a7311503a08b791744cbf63e9c44 Mon Sep 17 00:00:00 2001 From: Shuhei Matsumoto Date: Wed, 25 Dec 2019 20:56:32 -0500 Subject: [PATCH] bdevperf: Delete g_coremap and use lcore of io_target_group instead The role done by the variables has been replaced by lcore of struct io_target_group. So we can delete them safely. Signed-off-by: Shuhei Matsumoto Change-Id: I2ed856c8ccfda1d72b3fc97677fecaf295df37af Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/478706 Tested-by: SPDK CI Jenkins Community-CI: SPDK CI Jenkins Reviewed-by: Jim Harris Reviewed-by: Seth Howell Reviewed-by: Ben Walker Reviewed-by: Changpeng Liu --- test/bdev/bdevperf/bdevperf.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/test/bdev/bdevperf/bdevperf.c b/test/bdev/bdevperf/bdevperf.c index 2c100a0f0..b58c1be97 100644 --- a/test/bdev/bdevperf/bdevperf.c +++ b/test/bdev/bdevperf/bdevperf.c @@ -120,7 +120,6 @@ struct io_target_group { }; struct io_target_group *g_head; -uint32_t *g_coremap; static uint32_t g_target_count = 0; /* @@ -240,18 +239,8 @@ blockdev_heads_init(void) TAILQ_INIT(&g_head[i].targets); } - g_coremap = calloc(core_count, sizeof(uint32_t)); - if (!g_coremap) { - free(g_head); - fprintf(stderr, "Cannot allocate coremap array with size=%u\n", - core_count); - return -1; - } - SPDK_ENV_FOREACH_CORE(i) { - g_coremap[idx] = i; - g_head[idx].lcore = i; - idx++; + g_head[idx++].lcore = i; } return 0; @@ -297,7 +286,6 @@ blockdev_heads_destroy(void) { bdevperf_free_targets(); free(g_head); - free(g_coremap); } static void