Putting here for posterity...

Not of much consequence while we expect exactly one margo instance in each process, but the global timer mutex and linked list should be associated with a margo_instance and the timer functions should take a margo_instance_id - the timer is dependent on the margo loop. This also eliminates the need to have a separate init function for the timer subsystem.