From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Hunt Subject: [PATCH] examples/power: fix wrong core_id with JSON cmds Date: Mon, 7 Jan 2019 11:39:20 +0000 Message-ID: <20190107113920.37748-1-david.hunt@intel.com> Cc: david.hunt@intel.com, lei.a.yao@intel.com To: dev@dpdk.org Return-path: Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id 6F7161B445 for ; Mon, 7 Jan 2019 12:39:40 +0100 (CET) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" This patch fixes a bug introduced in the 64-core limtation enhancement where the core_id is inadvertently converted from virtual to physical even though it may already be a physical core_id. We should be using the core_type field, and only converting via hypervisor when core_type is set to CORE_TYPE_VIRTUAL Fixes: 5776b7a371d1 ("examples/power: allow VM to use lcores over 63") Signed-off-by: David Hunt --- examples/vm_power_manager/channel_monitor.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/examples/vm_power_manager/channel_monitor.c b/examples/vm_power_manager/channel_monitor.c index 85622e7cb..1a3a0fa76 100644 --- a/examples/vm_power_manager/channel_monitor.c +++ b/examples/vm_power_manager/channel_monitor.c @@ -640,7 +640,10 @@ process_request(struct channel_packet *pkt, struct channel_info *chan_info) if (pkt->command == CPU_POWER) { unsigned int core_num; - core_num = get_pcpu(chan_info, pkt->resource_id); + if (pkt->core_type == CORE_TYPE_VIRTUAL) + core_num = get_pcpu(chan_info, pkt->resource_id); + else + core_num = pkt->resource_id; switch (pkt->unit) { case(CPU_POWER_SCALE_MIN): -- 2.17.1