[RFC PATCH 1/4] linsched: remove process_all_softirqs() in main loop for accuracy

[RFC PATCH 1/4] linsched: remove process_all_softirqs() in main loop for accuracy

[Michael Wang]

From: Michael Wang <wangyun@linux.vnet.ibm.com>

process_all_softirqs() will handle softirq for all the cpu even
it's not the right timing for them, this will cause inaccuracy.

This patch stop invoke process_all_softirqs(), so softirq will
only be handled after timer interrupt arrived.

Signed-off-by: Michael Wang <wangyun@linux.vnet.ibm.com>
---
 tools/linsched/hrtimer.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/tools/linsched/hrtimer.c b/tools/linsched/hrtimer.c
index 26be1d8..de88b25 100644
--- a/tools/linsched/hrtimer.c
+++ b/tools/linsched/hrtimer.c
@@ -206,7 +206,6 @@ void linsched_run_sim(int sim_ticks)
                        /* a handler should never leave this state changed */
                        BUG_ON(smp_processor_id() != active_cpu);

-                       process_all_softirqs();
                        linsched_rcu_invoke();

                        BUG_ON(irqs_disabled());

[RFC PATCH 1/4] linsched: remove "process_all_softirqs" in main loop for accuracy

[Michael Wang]

From: Michael Wang <wangyun@linux.vnet.ibm.com>

"process_all_softirqs" will simulate an extra interrupt which will
cause cpu do extra load balance work.

This will make some cpu more active then others, the unfair will
cause inaccuracy.

Signed-off-by: Michael Wang <wangyun@linux.vnet.ibm.com>
---
 tools/linsched/hrtimer.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/tools/linsched/hrtimer.c b/tools/linsched/hrtimer.c
index 26be1d8..de88b25 100644
--- a/tools/linsched/hrtimer.c
+++ b/tools/linsched/hrtimer.c
@@ -206,7 +206,6 @@ void linsched_run_sim(int sim_ticks)
 			/* a handler should never leave this state changed */
 			BUG_ON(smp_processor_id() != active_cpu);

-			process_all_softirqs();
 			linsched_rcu_invoke();

 			BUG_ON(irqs_disabled());
-- 
1.7.1