serialqueue: Rename clock estimation variable names

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
This commit is contained in:
Kevin O'Connor 2017-06-27 20:08:18 -04:00
parent c7d0358c41
commit 067fac05a8
3 changed files with 17 additions and 16 deletions

View File

@ -59,8 +59,8 @@ defs_serialqueue = """
, uint64_t min_clock, uint64_t req_clock); , uint64_t min_clock, uint64_t req_clock);
void serialqueue_pull(struct serialqueue *sq, struct pull_queue_message *pqm); void serialqueue_pull(struct serialqueue *sq, struct pull_queue_message *pqm);
void serialqueue_set_baud_adjust(struct serialqueue *sq, double baud_adjust); void serialqueue_set_baud_adjust(struct serialqueue *sq, double baud_adjust);
void serialqueue_set_clock_est(struct serialqueue *sq, double est_clock void serialqueue_set_clock_est(struct serialqueue *sq, double est_freq
, double last_ack_time, uint64_t last_ack_clock); , double last_clock_time, uint64_t last_clock);
void serialqueue_get_stats(struct serialqueue *sq, char *buf, int len); void serialqueue_get_stats(struct serialqueue *sq, char *buf, int len);
int serialqueue_extract_old(struct serialqueue *sq, int sentq int serialqueue_extract_old(struct serialqueue *sq, int sentq
, struct pull_queue_message *q, int max); , struct pull_queue_message *q, int max);

View File

@ -344,8 +344,8 @@ struct serialqueue {
int receive_waiting; int receive_waiting;
// Baud / clock tracking // Baud / clock tracking
double baud_adjust, idle_time; double baud_adjust, idle_time;
double est_clock, last_ack_time; double est_freq, last_clock_time;
uint64_t last_ack_clock; uint64_t last_clock;
double last_receive_sent_time; double last_receive_sent_time;
// Retransmit support // Retransmit support
uint64_t send_seq, receive_seq; uint64_t send_seq, receive_seq;
@ -671,8 +671,9 @@ check_send_command(struct serialqueue *sq, double eventtime)
// Check for stalled messages now ready // Check for stalled messages now ready
double idletime = eventtime > sq->idle_time ? eventtime : sq->idle_time; double idletime = eventtime > sq->idle_time ? eventtime : sq->idle_time;
idletime += MESSAGE_MIN * sq->baud_adjust; idletime += MESSAGE_MIN * sq->baud_adjust;
double timedelta = idletime - sq->last_ack_time; double timedelta = idletime - sq->last_clock_time;
uint64_t ack_clock = (uint64_t)(timedelta * sq->est_clock) + sq->last_ack_clock; uint64_t ack_clock = ((uint64_t)(timedelta * sq->est_freq)
+ sq->last_clock);
uint64_t min_stalled_clock = MAX_CLOCK, min_ready_clock = MAX_CLOCK; uint64_t min_stalled_clock = MAX_CLOCK, min_ready_clock = MAX_CLOCK;
struct command_queue *cq; struct command_queue *cq;
list_for_each_entry(cq, &sq->pending_queues, node) { list_for_each_entry(cq, &sq->pending_queues, node) {
@ -702,20 +703,20 @@ check_send_command(struct serialqueue *sq, double eventtime)
// Check for messages to send // Check for messages to send
if (sq->ready_bytes >= MESSAGE_PAYLOAD_MAX) if (sq->ready_bytes >= MESSAGE_PAYLOAD_MAX)
return PR_NOW; return PR_NOW;
if (! sq->est_clock) { if (! sq->est_freq) {
if (sq->ready_bytes) if (sq->ready_bytes)
return PR_NOW; return PR_NOW;
sq->need_kick_clock = MAX_CLOCK; sq->need_kick_clock = MAX_CLOCK;
return PR_NEVER; return PR_NEVER;
} }
uint64_t reqclock_delta = MIN_REQTIME_DELTA * sq->est_clock; uint64_t reqclock_delta = MIN_REQTIME_DELTA * sq->est_freq;
if (min_ready_clock <= ack_clock + reqclock_delta) if (min_ready_clock <= ack_clock + reqclock_delta)
return PR_NOW; return PR_NOW;
uint64_t wantclock = min_ready_clock - reqclock_delta; uint64_t wantclock = min_ready_clock - reqclock_delta;
if (min_stalled_clock < wantclock) if (min_stalled_clock < wantclock)
wantclock = min_stalled_clock; wantclock = min_stalled_clock;
sq->need_kick_clock = wantclock; sq->need_kick_clock = wantclock;
return idletime + (wantclock - ack_clock) / sq->est_clock; return idletime + (wantclock - ack_clock) / sq->est_freq;
} }
// Callback timer to send data to the serial port // Callback timer to send data to the serial port
@ -981,13 +982,13 @@ serialqueue_set_baud_adjust(struct serialqueue *sq, double baud_adjust)
// Set the estimated clock rate of the mcu on the other end of the // Set the estimated clock rate of the mcu on the other end of the
// serial port // serial port
void void
serialqueue_set_clock_est(struct serialqueue *sq, double est_clock serialqueue_set_clock_est(struct serialqueue *sq, double est_freq
, double last_ack_time, uint64_t last_ack_clock) , double last_clock_time, uint64_t last_clock)
{ {
pthread_mutex_lock(&sq->lock); pthread_mutex_lock(&sq->lock);
sq->est_clock = est_clock; sq->est_freq = est_freq;
sq->last_ack_time = last_ack_time; sq->last_clock_time = last_clock_time;
sq->last_ack_clock = last_ack_clock; sq->last_clock = last_clock;
pthread_mutex_unlock(&sq->lock); pthread_mutex_unlock(&sq->lock);
} }

View File

@ -59,8 +59,8 @@ void serialqueue_encode_and_send(struct serialqueue *sq, struct command_queue *c
, uint64_t min_clock, uint64_t req_clock); , uint64_t min_clock, uint64_t req_clock);
void serialqueue_pull(struct serialqueue *sq, struct pull_queue_message *pqm); void serialqueue_pull(struct serialqueue *sq, struct pull_queue_message *pqm);
void serialqueue_set_baud_adjust(struct serialqueue *sq, double baud_adjust); void serialqueue_set_baud_adjust(struct serialqueue *sq, double baud_adjust);
void serialqueue_set_clock_est(struct serialqueue *sq, double est_clock void serialqueue_set_clock_est(struct serialqueue *sq, double est_freq
, double last_ack_time, uint64_t last_ack_clock); , double last_clock_time, uint64_t last_clock);
void serialqueue_get_stats(struct serialqueue *sq, char *buf, int len); void serialqueue_get_stats(struct serialqueue *sq, char *buf, int len);
int serialqueue_extract_old(struct serialqueue *sq, int sentq int serialqueue_extract_old(struct serialqueue *sq, int sentq
, struct pull_queue_message *q, int max); , struct pull_queue_message *q, int max);