[dpdk-dev] examples/vm_power: fix improper physical core mask

Message ID 1439275763-12481-1-git-send-email-yong.liu@intel.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Marvin Liu Aug. 11, 2015, 6:49 a.m. UTC
  From: Marvin Liu <yong.liu@intel.com>

Host cpu mapping structure can only support 64 cpus. When run vm_power sample
on platform with more than 64 cpus, will generate improper physical core mask.
After limited supported host cpus to 64 will fix this issue.

Signed-off-by: Marvin Liu <yong.liu@intel.com>
  

Comments

Thomas Monjalon Aug. 11, 2015, 9:41 a.m. UTC | #1
2015-08-11 14:49, Yong Liu:
> Host cpu mapping structure can only support 64 cpus. When run vm_power sample
> on platform with more than 64 cpus, will generate improper physical core mask.
> After limited supported host cpus to 64 will fix this issue.
> 
> Signed-off-by: Marvin Liu <yong.liu@intel.com>

Fixes: e9f64db94637 ("examples/vm_power: show warning when more than 64 cores")

Applied, thanks
  

Patch

diff --git a/examples/vm_power_manager/channel_manager.c b/examples/vm_power_manager/channel_manager.c
index 35ff40a..db76f2e 100644
--- a/examples/vm_power_manager/channel_manager.c
+++ b/examples/vm_power_manager/channel_manager.c
@@ -764,11 +764,13 @@  channel_manager_init(const char *path)
 	}
 	global_n_host_cpus = (unsigned)n_cpus;
 
-	if (global_n_host_cpus > CHANNEL_CMDS_MAX_CPUS)
+	if (global_n_host_cpus > CHANNEL_CMDS_MAX_CPUS) {
 		RTE_LOG(WARNING, CHANNEL_MANAGER, "The number of host CPUs(%u) exceeds the "
 				"maximum of %u. No cores over %u should be used.\n",
 				global_n_host_cpus, CHANNEL_CMDS_MAX_CPUS,
 				CHANNEL_CMDS_MAX_CPUS - 1);
+		global_n_host_cpus = CHANNEL_CMDS_MAX_CPUS;
+	}
 
 	return 0;
 error: