[dpdk-dev] examples/power: fix build

Message ID 1507897902-49657-1-git-send-email-david.hunt@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Hunt, David Oct. 13, 2017, 12:31 p.m. UTC
  Remove variable declaration from within for loop.

Fixes: f14791a8126e ("examples/vm_power_mgr: add policy to channels")

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

Comments

Thomas Monjalon Oct. 13, 2017, 1:30 p.m. UTC | #1
13/10/2017 14:31, David Hunt:
> Remove variable declaration from within for loop.
> 
> Fixes: f14791a8126e ("examples/vm_power_mgr: add policy to channels")
> 
> Signed-off-by: David Hunt <david.hunt@intel.com>

Applied, thanks
  

Patch

diff --git a/examples/vm_power_manager/channel_monitor.c b/examples/vm_power_manager/channel_monitor.c
index f16358d..f8e3491 100644
--- a/examples/vm_power_manager/channel_monitor.c
+++ b/examples/vm_power_manager/channel_monitor.c
@@ -169,8 +169,9 @@  update_policy(struct channel_packet *pkt)
 {
 
 	unsigned int updated = 0;
+	int i;
 
-	for (int i = 0; i < MAX_VMS; i++) {
+	for (i = 0; i < MAX_VMS; i++) {
 		if (strcmp(policies[i].pkt.vm_name, pkt->vm_name) == 0) {
 			policies[i].pkt = *pkt;
 			get_pcpu_to_control(&policies[i]);
@@ -184,7 +185,7 @@  update_policy(struct channel_packet *pkt)
 		}
 	}
 	if (!updated) {
-		for (int i = 0; i < MAX_VMS; i++) {
+		for (i = 0; i < MAX_VMS; i++) {
 			if (policies[i].enabled == 0) {
 				policies[i].pkt = *pkt;
 				get_pcpu_to_control(&policies[i]);
@@ -543,7 +544,9 @@  run_channel_monitor(void)
 		}
 		rte_delay_us(time_period_s*1000000);
 		if (policy_is_set) {
-			for (int j = 0; j < MAX_VMS; j++) {
+			int j;
+
+			for (j = 0; j < MAX_VMS; j++) {
 				if (policies[j].enabled == 1)
 					apply_policy(&policies[j]);
 			}