Thanks, Hrvoje. I think the code looks good, I just have a few cosmetic nits.
Patch set 2:Code-Review -1
4 comments:
Patch Set #2, Line 371: provide
s/provide a/cause
on a large
number of hosts
Although you're correct, I think it's best to leave this out since "a large number of hosts" is an uncommon scenario. The "repeated execution" aspect is convincing on its own.
Patch Set #2, Line 236: Ignore using the system clock for loop timing
This comment seems more confusing than helpful. IIUC we're just ignoring the externally provided value.
Patch Set #2, Line 242: /* Ignore using the system clock for loop timing */
Similar as above, the comment seems confusing (if I understand the intention correctly)
To view, visit change 44517. To unsubscribe, or for help writing mail filters, visit settings.