char *name;
int fd;
int width;
- int last_read;
- int ttl;
+ struct timespec last_read;
+ struct timespec ttl;
int pos_init; /* Initial position on the output buffer. */
int pos_curr; /* Current position on the output buffer. */
int pos_final; /* Final position on the output buffer. */
};
enum read_status {
- FAIL_FINAL = -2,
- FAIL_TMP = -1,
- END_OF_FILE = 0,
- END_OF_MESSAGE = 1
+ END_OF_FILE,
+ END_OF_MESSAGE,
+ RETRY,
+ FAILURE
};
void
" name = %s,"
" fd = %d,"
" width = %d,"
- " last_read = %d,"
- " ttl = %d,"
+ " last_read = {tv_sec = %ld, tv_nsec = %ld}"
+ " ttl = {tv_sec = %ld, tv_nsec = %ld},"
" pos_init = %d,"
" pos_curr = %d,"
" pos_final = %d,"
f->name,
f->fd,
f->width,
- f->last_read,
- f->ttl,
+ f->last_read.tv_sec,
+ f->last_read.tv_nsec,
+ f->ttl.tv_sec,
+ f->ttl.tv_nsec,
f->pos_init,
f->pos_curr,
f->pos_final,
" SPEC = FILE_PATH DATA_WIDTH DATA_TTL\n"
" FILE_PATH = string\n"
" DATA_WIDTH = int (* (positive) number of characters *)\n"
- " DATA_TTL = int (* (positive) number of seconds *)\n"
+ " DATA_TTL = float (* (positive) number of seconds *)\n"
" OPTION = -i INTERVAL\n"
" | -s SEPARATOR\n"
" | -x (* Output to X root window *)\n"
" | -l LOG_LEVEL\n"
" SEPARATOR = string\n"
- " INTERVAL = int (* (positive) number of seconds *)\n"
+ " INTERVAL = float (* (positive) number of seconds *)\n"
" LOG_LEVEL = int (* %d through %d *)\n"
"\n",
argv0,
char *w = argv[i++];
char *t = argv[i++];
+ struct timespec last_read;
+
if (!is_pos_num(w))
usage("[spec] Invalid width: \"%s\", for fifo \"%s\"\n", w, n);
- if (!is_pos_num(t))
+ if (!is_decimal(t))
usage("[spec] Invalid TTL: \"%s\", for fifo \"%s\"\n", t, n);
+ last_read.tv_sec = 0;
+ last_read.tv_nsec = 0;
Fifo *f = calloc(1, sizeof(struct Fifo));
if (f) {
f->name = n;
f->fd = -1;
f->width = atoi(w);
- f->ttl = atoi(t);
- f->last_read = 0;
+ f->ttl = timespec_of_float(atof(t));
+ f->last_read = last_read;
f->pos_init = cfg->total_width;
f->pos_curr = f->pos_init;
f->pos_final = f->pos_init + f->width - 1;
}
}
+void
+fifo_expire_one(Fifo *f, struct timespec t, char *buf)
+{
+ struct timespec td;
+
+ timespecsub(&t, &(f->last_read), &td);
+ if (timespeccmp(&td, &(f->ttl), >=)) {
+ /* TODO: Maybe configurable expiry character. */
+ memset(buf + f->pos_init, '_', f->pos_final - f->pos_init);
+ warn("Data source expired: \"%s\"\n", f->name);
+ }
+}
+
+void
+fifo_expire_all(Config *cfg, struct timespec t, char *buf)
+{
+ Fifo *f;
+
+ for (f = cfg->fifos; f; f = f->next)
+ fifo_expire_one(f, t, buf);
+}
+
void
fifo_read_error(Fifo *f, char *buf)
{
}
enum read_status
-fifo_read_one(Fifo *f, char *buf)
+fifo_read_one(Fifo *f, struct timespec t, char *buf)
{
- /* Initialize all to an impossible value: */
- ssize_t n = -5; /* Number of bytes read. */
- char c = -1; /* Character read. */
- int r = -1; /* Remaining unused slots in buffer range. */
+ char c; /* Character read. */
+ int r; /* Remaining unused slots in buffer range. */
for (;;) {
- n = read(f->fd, &c, 1);
- assert(n >= -1 && n <= 1);
- switch (n) {
+ switch (read(f->fd, &c, 1)) {
case -1:
error("Failed to read: \"%s\". errno: %d, msg: %s\n",
f->name, errno, strerror(errno));
- if (errno == 11)
- return FAIL_TMP;
- else
- return FAIL_FINAL;
+ switch (errno) {
+ case EINTR:
+ case EAGAIN:
+ return RETRY;
+ default:
+ return FAILURE;
+ }
case 0:
debug("%s: End of FILE\n", f->name);
f->pos_curr = f->pos_init;
if (r > 0)
memset(buf + f->pos_curr, ' ', r);
f->pos_curr = f->pos_init;
+ f->last_read = t;
return END_OF_MESSAGE;
} else {
if (f->pos_curr <= f->pos_final)
assert(0);
}
}
- /* TODO Record timestamp read */
}
void
-fifo_read_all(Config *cfg, char *buf)
+fifo_read_all(Config *cfg, struct timespec t, char *buf)
{
fd_set fds;
int maxfd = -1;
- int ready;
+ int ready = 0;
struct stat st;
FD_ZERO(&fds);
debug("ready: %d\n", ready);
assert(ready != 0);
if (ready < 0)
+ /* TODO: Do we really want to fail here? */
fatal("%s", strerror(errno));
- for (Fifo *f = cfg->fifos; f; f = f->next) {
- if (FD_ISSET(f->fd, &fds)) {
- debug("reading: %s\n", f->name);
- switch (fifo_read_one(f, buf)) {
- case END_OF_FILE:
- case FAIL_FINAL:
- close(f->fd);
- f->fd = -1;
- break;
- case END_OF_MESSAGE:
- case FAIL_TMP:
- break;
- default:
- assert(0);
+ while (ready) {
+ for (Fifo *f = cfg->fifos; f; f = f->next) {
+ if (FD_ISSET(f->fd, &fds)) {
+ debug("reading: %s\n", f->name);
+ switch (fifo_read_one(f, t, buf)) {
+ /*
+ * ### MESSAGE LOSS ###
+ * is introduced by closing at EOM in addition
+ * to EOF, since there may be unread messages
+ * remaining in the pipe. However,
+ *
+ * ### INTER-MESSAGE PUSHBACK ###
+ * is also gained, since pipes block at the
+ * "open" call.
+ *
+ * This is an acceptable trade-off because we
+ * are a stateless reporter of a _most-recent_
+ * status, not a stateful accumulator.
+ */
+ case END_OF_MESSAGE:
+ case END_OF_FILE:
+ case FAILURE:
+ close(f->fd);
+ f->fd = -1;
+ ready--;
+ break;
+ case RETRY:
+ break;
+ default:
+ assert(0);
+ }
}
}
}
+ assert(ready == 0);
}
int
/* TODO: Handle signals */
for (;;) {
clock_gettime(CLOCK_MONOTONIC, &t0); // FIXME: check errors
- /* TODO: Cache expiration. i.e. use the TTL */
- /* TODO: How to trigger TTL check? On select? Alarm signal? */
/* TODO: Set timeout on fifo_read_all based on diff of last time of
- * fifo_read_all and desired time of next TTL check.
+ * fifo_read_all and desired time of next TTL check?
*/
/* TODO: How long to wait on IO? Max TTL? */
- fifo_read_all(cfg, buf);
+ fifo_read_all(cfg, t0, buf);
if (cfg->output_to_x_root_window) {
if (XStoreName(display, DefaultRootWindow(display), buf) < 0)
fatal("XStoreName failed.\n");
puts(buf);
fflush(stdout);
}
+
+ /*
+ * This is a good place for expiry check, since we're about to
+ * sleep anyway and the time taken by the check will be
+ * subtracted from the sleep period.
+ */
+ fifo_expire_all(cfg, t0, buf);
+
clock_gettime(CLOCK_MONOTONIC, &t1); // FIXME: check errors
timespecsub(&t1, &t0, &td);
debug("td {tv_sec = %ld, tv_nsec = %ld}\n", td.tv_sec, td.tv_nsec);