There is no functional change. Calculated steal could be used in the
subsequent changes. Hence reordering the function.

Signed-off-by: Srikar Dronamraju <[email protected]>
---
 arch/powerpc/platforms/pseries/lpar.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/arch/powerpc/platforms/pseries/lpar.c 
b/arch/powerpc/platforms/pseries/lpar.c
index 6a415febc53b..dde12b27ba60 100644
--- a/arch/powerpc/platforms/pseries/lpar.c
+++ b/arch/powerpc/platforms/pseries/lpar.c
@@ -662,15 +662,20 @@ machine_device_initcall(pseries, 
vcpudispatch_stats_procfs_init);
 u64 pseries_paravirt_steal_clock(int cpu)
 {
        struct lppaca *lppaca = &lppaca_of(cpu);
+       unsigned long steal;
+
+       steal = be64_to_cpu(READ_ONCE(lppaca->ready_enqueue_tb));
+       steal += be64_to_cpu(READ_ONCE(lppaca->enqueue_dispatch_tb));
 
        /*
         * VPA steal time counters are reported at TB frequency. Hence do a
-        * conversion to ns before returning
+        * conversion to ns before using.
         */
-       return tb_to_ns(be64_to_cpu(READ_ONCE(lppaca->enqueue_dispatch_tb)) +
-                       be64_to_cpu(READ_ONCE(lppaca->ready_enqueue_tb)));
+       steal = tb_to_ns(steal);
+
+       return steal;
 }
-#endif
+#endif /* CONFIG_PARAVIRT_TIME_ACCOUNTING */
 
 #endif /* CONFIG_PPC_SPLPAR */
 
-- 
2.43.7


Reply via email to