Commit 0aac63ce authored by Jim Harris's avatar Jim Harris Committed by Tomasz Zawadzki
Browse files

bdevperf: do not append cpumask to thread name 



spdk_thread_create() does not require unique thread
names, and we already print out the CPU mask for each
job. So there is no need to append the cpumask to the
thread name.

Removing it has the added advantage of not modifying
the job name specified by the user if they are using
a job config file.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I6d75ea2bdc50061d7338ad41749e458efa62b48e
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/6331


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarPaul Luse <paul.e.luse@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatar <dongx.yi@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Community-CI: Mellanox Build Bot
parent 783293de

test/bdev/bdevperf/bdevperf.c

+1 −6
Original line number Diff line number Diff line
@@ -1118,7 +1118,6 @@ typedef struct spdk_thread *spdk_thread_t; 
static spdk_thread_t

construct_job_thread(struct spdk_cpuset *cpumask, const char *tag)

{

	char thread_name[32];

	struct spdk_cpuset tmp;



	/* This function runs on the main thread. */
@@ -1136,11 +1135,7 @@ construct_job_thread(struct spdk_cpuset *cpumask, const char *tag) 
		fprintf(stderr, "cpumask for '%s' is too big\n", tag);

	}



	snprintf(thread_name, sizeof(thread_name), "%s_%s",

		 tag,

		 spdk_cpuset_fmt(cpumask));



	return spdk_thread_create(thread_name, cpumask);

	return spdk_thread_create(tag, cpumask);

}



static uint32_t