d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

examples/l3fwd-power: add Rx interrupt timeout

Browse Source 
Currently, thread waiting on an interrupt does not have a timeout, so
it will not ever wake up until traffic arrives. This means that, when
time comes to exit the application, it will not quit unless there
happens to be traffic coming in and waking up the thread from sleep.

Fix it so that the interrupt thread sleeps for 10ms before waking up
and attempting to poll again. Additionally, remove the log message
to avoid spamming about entering interrupt mode.

Fixes: 613ce6691c ("examples/l3fwd-power: implement proper shutdown")
Cc: stable@dpdk.org

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: David Hunt <david.hunt@intel.com>
Tested-by: Lihong Ma <lihongx.ma@intel.com>
This commit is contained in:
Anatoly Burakov 2020-05-07 11:46:28 +01:00 committed by Thomas Monjalon
parent ea153cc853
commit f4d1e19c29
1 changed files with 14 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
examples/l3fwd-power/main.c
View File

@ -824,17 +824,24 @@ power_freq_scaleup_heuristic(unsigned lcore_id,
static int
sleep_until_rx_interrupt(int num)
{
/*
* we want to track when we are woken up by traffic so that we can go
* back to sleep again without log spamming.
*/
static bool timeout;
struct rte_epoll_event event[num];
int n, i;
uint16_t port_id;
uint8_t queue_id;
void *data;
RTE_LOG(INFO, L3FWD_POWER,
"lcore %u sleeps until interrupt triggers\n",
rte_lcore_id());
if (!timeout) {
RTE_LOG(INFO, L3FWD_POWER,
"lcore %u sleeps until interrupt triggers\n",
rte_lcore_id());
}
n = rte_epoll_wait(RTE_EPOLL_PER_THREAD, event, num, -1);
n = rte_epoll_wait(RTE_EPOLL_PER_THREAD, event, num, 10);
for (i = 0; i < n; i++) {
data = event[i].epdata.data;
port_id = ((uintptr_t)data) >> CHAR_BIT;
@ -845,6 +852,7 @@ sleep_until_rx_interrupt(int num)
" port %d queue %d\n",
rte_lcore_id(), port_id, queue_id);
}
timeout = n == 0;
return 0;
}
@ -1307,7 +1315,8 @@ main_loop(__rte_unused void *dummy)
/**
* start receiving packets immediately
*/
goto start_rx;
if (likely(!is_done()))
goto start_rx;
}
}
stats[lcore_id].sleep_time += lcore_idle_hint;