auracaster/openocd
3
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Call poll at a fixed interval.

Browse Source 
The existing implementation blocks in select() for a fixed amount of
time. This change tracks when the next event (likely poll()) wants to be
run, and uses a shorter timeout in select() if necessary.

Also track all these timeouts using milliseconds as returned by
timeval_ms() instead of `struct timeval` to simplify the code.

This feature is helpful if poll() wants to do something like sample PCs
or memory values for basically the entire time that otherwise OpenOCD
would be hung in select(). See
https://github.com/riscv/riscv-openocd/pull/541 for an example of that.
The RISC-V code using this change will be upstreamed some day, too.

Signed-off-by: Tim Newsome <tim@sifive.com>
Change-Id: I67104a7cf69ed07c8399c14aa55963fc5116a67d
Reviewed-on: http://openocd.zylin.com/6363
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
This commit is contained in:
Tim Newsome
2021-07-09 12:58:07 -07:00
committed by Antonio Borneo
parent beff3de2ce
commit db16b3dc5b
3 changed files with 42 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
src/target/target.c
View File

@@ -154,9 +154,10 @@ static struct target_type *target_types[] = {
struct target *all_targets;
static struct target_event_callback *target_event_callbacks;
static struct target_timer_callback *target_timer_callbacks;
static int64_t target_timer_next_event_value;
static LIST_HEAD(target_reset_callback_list);
static LIST_HEAD(target_trace_callback_list);
static const int polling_interval = 100;
static const int polling_interval = TARGET_DEFAULT_POLLING_INTERVAL;
static const struct jim_nvp nvp_assert[] = {
{ .name = "assert", NVP_ASSERT },
@@ -1733,8 +1734,8 @@ int target_register_timer_callback(int (*callback)(void *priv),
(*callbacks_p)->time_ms = time_ms;
(*callbacks_p)->removed = false;
gettimeofday(&(*callbacks_p)->when, NULL);
timeval_add_time(&(*callbacks_p)->when, 0, time_ms * 1000);
(*callbacks_p)->when = timeval_ms() + time_ms;
target_timer_next_event_value = MIN(target_timer_next_event_value, (*callbacks_p)->when);
(*callbacks_p)->priv = priv;
(*callbacks_p)->next = NULL;
@@ -1868,15 +1869,14 @@ int target_call_trace_callbacks(struct target *target, size_t len, uint8_t *data
}
static int target_timer_callback_periodic_restart(
struct target_timer_callback *cb, struct timeval *now)
struct target_timer_callback *cb, int64_t *now)
{
cb->when = *now;
timeval_add_time(&cb->when, 0, cb->time_ms * 1000L);
cb->when = *now + cb->time_ms;
return ERROR_OK;
}
static int target_call_timer_callback(struct target_timer_callback *cb,
struct timeval *now)
int64_t *now)
{
cb->callback(cb->priv);
@@ -1898,8 +1898,12 @@ static int target_call_timer_callbacks_check_time(int checktime)
keep_alive();
struct timeval now;
gettimeofday(&now, NULL);
int64_t now = timeval_ms();
/* Initialize to a default value that's a ways into the future.
* The loop below will make it closer to now if there are
* callbacks that want to be called sooner. */
target_timer_next_event_value = now + 1000;
/* Store an address of the place containing a pointer to the
* next item; initially, that's a standalone "root of the
@@ -1915,11 +1919,14 @@ static int target_call_timer_callbacks_check_time(int checktime)
bool call_it = (*callback)->callback &&
((!checktime && (*callback)->type == TARGET_TIMER_TYPE_PERIODIC) ||
timeval_compare(&now, &(*callback)->when) >= 0);
now >= (*callback)->when);
if (call_it)
target_call_timer_callback(*callback, &now);
if (!(*callback)->removed && (*callback)->when < target_timer_next_event_value)
target_timer_next_event_value = (*callback)->when;
callback = &(*callback)->next;
}
@@ -1927,17 +1934,22 @@ static int target_call_timer_callbacks_check_time(int checktime)
return ERROR_OK;
}
int target_call_timer_callbacks(void)
int target_call_timer_callbacks()
{
return target_call_timer_callbacks_check_time(1);
}
/* invoke periodic callbacks immediately */
int target_call_timer_callbacks_now(void)
int target_call_timer_callbacks_now()
{
return target_call_timer_callbacks_check_time(0);
}
int64_t target_timer_next_event(void)
{
return target_timer_next_event_value;
}
/* Prints the working area layout for debug purposes */
static void print_wa_layout(struct target *target)
{