diff mbox series

[v1] examples/power: fix policy handling for fifo

Message ID 20190716111112.32427-1-david.hunt@intel.com (mailing list archive)
State Accepted, archived
Headers
Series [v1] examples/power: fix policy handling for fifo |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail Compilation issues
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS

Commit Message

Hunt, David July 16, 2019, 11:11 a.m. UTC 
  While the core frequency scale-up and scale down all works fine with the
fifo-per-core functionaliry, there was a gap for policy handling. When
creating or destroying a policy, the core associated with the fifo
needs to be automatically added to the core list, so the policy is
associated with the correct core.

Fixes: 221e7026d521 ("examples/power: add FIFO per core for JSON interface")

Signed-off-by: David Hunt <david.hunt@intel.com>
---
 examples/vm_power_manager/channel_monitor.c | 24 ++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

Comments

Burakov, Anatoly July 16, 2019, 1:44 p.m. UTC | #1 
On 16-Jul-19 12:11 PM, David Hunt wrote:
> While the core frequency scale-up and scale down all works fine with the
> fifo-per-core functionaliry, there was a gap for policy handling. When
> creating or destroying a policy, the core associated with the fifo
> needs to be automatically added to the core list, so the policy is
> associated with the correct core.
> 
> Fixes: 221e7026d521 ("examples/power: add FIFO per core for JSON interface")
> 
> Signed-off-by: David Hunt <david.hunt@intel.com>
> ---

LGTM

Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
Thomas Monjalon July 17, 2019, 8:52 p.m. UTC | #2 
16/07/2019 15:44, Burakov, Anatoly:
> On 16-Jul-19 12:11 PM, David Hunt wrote:
> > While the core frequency scale-up and scale down all works fine with the
> > fifo-per-core functionaliry, there was a gap for policy handling. When
> > creating or destroying a policy, the core associated with the fifo
> > needs to be automatically added to the core list, so the policy is
> > associated with the correct core.
> > 
> > Fixes: 221e7026d521 ("examples/power: add FIFO per core for JSON interface")
> > 
> > Signed-off-by: David Hunt <david.hunt@intel.com>
> > ---
> 
> LGTM
> 
> Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>

Applied, thanks
diff mbox series

Patch

diff --git a/examples/vm_power_manager/channel_monitor.c b/examples/vm_power_manager/channel_monitor.c
index 496772f8a..fe6088a18 100644
--- a/examples/vm_power_manager/channel_monitor.c
+++ b/examples/vm_power_manager/channel_monitor.c
@@ -835,6 +835,8 @@  read_json_packet(struct channel_info *chan_info)
 	json_t *root;
 	json_error_t error;
 	const char *resource_name;
+	char *start, *end;
+	uint32_t n;
 
 
 	/* read opening brace to closing brace */
@@ -882,7 +884,27 @@  read_json_packet(struct channel_info *chan_info)
 					"Error validating JSON profile data\n");
 				break;
 			}
-			process_request(&pkt, chan_info);
+			start = strstr(pkt.vm_name,
+					CHANNEL_MGR_FIFO_PATTERN_NAME);
+			if (start != NULL) {
+				/* move past pattern to start of fifo id */
+				start += strlen(CHANNEL_MGR_FIFO_PATTERN_NAME);
+
+				end = start;
+				n = (uint32_t)strtoul(start, &end, 10);
+
+				if (end[0] == '\0') {
+					/* Add core id to core list */
+					pkt.num_vcpu = 1;
+					pkt.vcpu_to_control[0] = n;
+					process_request(&pkt, chan_info);
+				} else {
+					RTE_LOG(ERR, CHANNEL_MONITOR,
+						"Cannot extract core id from fifo name\n");
+				}
+			} else {
+				process_request(&pkt, chan_info);
+			}
 		} else {
 			RTE_LOG(ERR, CHANNEL_MONITOR,
 					"JSON error on line %d: %s\n",