Call path below SIGALRM isn't safe
We do allocations, open files, printf, etc from the SIGALRM signal
handler which gets run every 250 msecs. This isn't necessarily
safe and could deadlock. Move it to thread context instead.
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
diff --git a/eta.c b/eta.c
index 81dbfa7..608e0dd 100644
--- a/eta.c
+++ b/eta.c
@@ -328,7 +328,11 @@
}
disp_time = mtime_since(&disp_prev_time, &now);
- if (disp_time < 1000)
+
+ /*
+ * Allow a little slack, the target is to print it every 1000 msecs
+ */
+ if (disp_time < 900)
return;
calc_rate(disp_time, io_bytes, disp_io_bytes, rate);