Commit 6726f459 authored by Krzysztof Karas's avatar Krzysztof Karas Committed by Tomasz Zawadzki
Browse files

spdk_top: delete free_data() function 



Delete free_data() function, because it is called only once
and contains only three lines of code. This is to make
spdk_top code cleaner.

Signed-off-by: default avatarKrzysztof Karas <krzysztof.karas@intel.com>
Change-Id: I78e0edd941e8fb81ac58ea13bdc91148db0a2417
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7972


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 3154d4c9

app/spdk_top/spdk_top.c

+5 −9
Original line number Diff line number Diff line
@@ -735,14 +735,6 @@ end: 
	return rc;

}



static void

free_data(void)

{

	free_rpc_threads_stats(&g_threads_stats);

	free_rpc_pollers_stats(&g_pollers_stats);

	free_rpc_cores_stats(&g_cores_stats);

}



enum str_alignment {

	ALIGN_LEFT,

	ALIGN_RIGHT,
@@ -2490,7 +2482,11 @@ show_stats(pthread_t *data_thread) 
	pthread_join(*data_thread, NULL);



	free_resources();

	free_data();



	/* Free memory holding current data states before quitting application */

	free_rpc_threads_stats(&g_threads_stats);

	free_rpc_pollers_stats(&g_pollers_stats);

	free_rpc_cores_stats(&g_cores_stats);

}



static void