void bch2_io_timer_add(struct io_clock *clock, struct io_timer *timer)
  {
 -      size_t i;
+       const struct min_heap_callbacks callbacks = {
+               .less = io_timer_cmp,
+               .swp = io_timer_swp,
+       };
+ 
        spin_lock(&clock->timer_lock);
  
 -      if (time_after_eq((unsigned long) atomic64_read(&clock->now),
 -                        timer->expire)) {
 +      if (time_after_eq64((u64) atomic64_read(&clock->now), timer->expire)) {
                spin_unlock(&clock->timer_lock);
                timer->fn(timer);
                return;
        }
  
-       for (size_t i = 0; i < clock->timers.used; i++)
 -      for (i = 0; i < clock->timers.nr; i++)
++      for (size_t i = 0; i < clock->timers.nr; i++)
                if (clock->timers.data[i] == timer)
                        goto out;
  
  
  void bch2_io_timer_del(struct io_clock *clock, struct io_timer *timer)
  {
 -      size_t i;
+       const struct min_heap_callbacks callbacks = {
+               .less = io_timer_cmp,
+               .swp = io_timer_swp,
+       };
+ 
        spin_lock(&clock->timer_lock);
  
-       for (size_t i = 0; i < clock->timers.used; i++)
 -      for (i = 0; i < clock->timers.nr; i++)
++      for (size_t i = 0; i < clock->timers.nr; i++)
                if (clock->timers.data[i] == timer) {
-                       heap_del(&clock->timers, i, io_timer_cmp, NULL);
+                       min_heap_del(&clock->timers, i, &callbacks, NULL);
                        break;
                }
  
        bch2_io_timer_del(clock, &wait.io_timer);
  }
  
 -static struct io_timer *get_expired_timer(struct io_clock *clock,
 -                                        unsigned long now)
 +static struct io_timer *get_expired_timer(struct io_clock *clock, u64 now)
  {
        struct io_timer *ret = NULL;
 -      spin_lock(&clock->timer_lock);
 -
+       const struct min_heap_callbacks callbacks = {
+               .less = io_timer_cmp,
+               .swp = io_timer_swp,
+       };
+ 
 -          time_after_eq(now, clock->timers.data[0]->expire)) {
+       if (clock->timers.nr &&
++          time_after_eq64(now, clock->timers.data[0]->expire)) {
+               ret = *min_heap_peek(&clock->timers);
+               min_heap_pop(&clock->timers, &callbacks, NULL);
+       }
  
-       if (clock->timers.used &&
-           time_after_eq64(now, clock->timers.data[0]->expire))
-               heap_pop(&clock->timers, ret, io_timer_cmp, NULL);
 -      spin_unlock(&clock->timer_lock);
 -
        return ret;
  }
  
  
  void bch2_io_timers_to_text(struct printbuf *out, struct io_clock *clock)
  {
 -      unsigned long now;
 -      unsigned i;
 -
        out->atomic++;
        spin_lock(&clock->timer_lock);
 -      now = atomic64_read(&clock->now);
 +      u64 now = atomic64_read(&clock->now);
 +
 +      printbuf_tabstop_push(out, 40);
 +      prt_printf(out, "current time:\t%llu\n", now);
  
-       for (unsigned i = 0; i < clock->timers.used; i++)
 -      for (i = 0; i < clock->timers.nr; i++)
 -              prt_printf(out, "%ps:\t%li\n",
++      for (unsigned i = 0; i < clock->timers.nr; i++)
 +              prt_printf(out, "%ps %ps:\t%llu\n",
                       clock->timers.data[i]->fn,
 -                     clock->timers.data[i]->expire - now);
 +                     clock->timers.data[i]->fn2,
 +                     clock->timers.data[i]->expire);
        spin_unlock(&clock->timer_lock);
        --out->atomic;
  }