Commit 636a415b authored by Helge Deller's avatar Helge Deller

parisc: Reduce irq overhead when run in qemu

When run under QEMU, calling mfctl(16) creates some overhead because the
qemu timer has to be scaled and moved into the register. This patch
reduces the number of calls to mfctl(16) by moving the calls out of the
loops.

Additionally, increase the minimal time interval to 8000 cycles instead
of 500 to compensate possible QEMU delays when delivering interrupts.
Signed-off-by: default avatarHelge Deller <deller@gmx.de>
Cc: stable@vger.kernel.org # 4.14+
parent 5ffa8518
...@@ -76,10 +76,10 @@ irqreturn_t __irq_entry timer_interrupt(int irq, void *dev_id) ...@@ -76,10 +76,10 @@ irqreturn_t __irq_entry timer_interrupt(int irq, void *dev_id)
next_tick = cpuinfo->it_value; next_tick = cpuinfo->it_value;
/* Calculate how many ticks have elapsed. */ /* Calculate how many ticks have elapsed. */
now = mfctl(16);
do { do {
++ticks_elapsed; ++ticks_elapsed;
next_tick += cpt; next_tick += cpt;
now = mfctl(16);
} while (next_tick - now > cpt); } while (next_tick - now > cpt);
/* Store (in CR16 cycles) up to when we are accounting right now. */ /* Store (in CR16 cycles) up to when we are accounting right now. */
...@@ -103,16 +103,17 @@ irqreturn_t __irq_entry timer_interrupt(int irq, void *dev_id) ...@@ -103,16 +103,17 @@ irqreturn_t __irq_entry timer_interrupt(int irq, void *dev_id)
* if one or the other wrapped. If "now" is "bigger" we'll end up * if one or the other wrapped. If "now" is "bigger" we'll end up
* with a very large unsigned number. * with a very large unsigned number.
*/ */
while (next_tick - mfctl(16) > cpt) now = mfctl(16);
while (next_tick - now > cpt)
next_tick += cpt; next_tick += cpt;
/* Program the IT when to deliver the next interrupt. /* Program the IT when to deliver the next interrupt.
* Only bottom 32-bits of next_tick are writable in CR16! * Only bottom 32-bits of next_tick are writable in CR16!
* Timer interrupt will be delivered at least a few hundred cycles * Timer interrupt will be delivered at least a few hundred cycles
* after the IT fires, so if we are too close (<= 500 cycles) to the * after the IT fires, so if we are too close (<= 8000 cycles) to the
* next cycle, simply skip it. * next cycle, simply skip it.
*/ */
if (next_tick - mfctl(16) <= 500) if (next_tick - now <= 8000)
next_tick += cpt; next_tick += cpt;
mtctl(next_tick, 16); mtctl(next_tick, 16);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment