Question about hrtimer_restart apic_timer_fn in lapic.c

      /*
       * There is a race window between reading and incrementing, but we do
       * not care about potentially losing timer events in the !reinject
       * case anyway. Note: KVM_REQ_PENDING_TIMER is implicitly checked
       * in vcpu_enter_guest.
       */
       if (!atomic_read(&ktimer->pending)) {
                atomic_inc(&ktimer->pending);
                /* FIXME: this code should not know anything about vcpus */
                kvm_make_request(KVM_REQ_PENDING_TIMER, vcpu);
       }

       if (waitqueue_active(q))
                wake_up_interruptible(q);

        if (lapic_is_periodic(apic)) {
               hrtimer_add_expires_ns(&ktimer->timer, ktimer->period);
               return HRTIMER_RESTART;
         } else
                return HRTIMER_NORESTART;
}

What is point of the comment, clearly we need access to vcpu in the call as it's a pointer to the apic structure's vcpu and we need to call on that particular CPU a request of KVM_REQ_PENDING_TIMER? Please let me known if I am missing any important information.

Paolo,
If that is the case why don't I just move this function over if we don't need it in this file into x86.c. This seems to be the easiest solution from my understanding.
Regards,

Sorry Paolo,
I am confused of where you would want me to move it as I looked around and there seems to be no real place to put it.