[PATCH 3/3] eal/x86: defer power intrinsics variable allocation

[David Marchand]

The lcore variable in this code unit is only used through
rte_power_monitor*() public symbols.

Defer the unconditional lcore variable allocation in those symbols.

Fixes: 18b5049ab4fe ("eal/x86: keep power intrinsics state in lcore variable")
Cc: sta...@dpdk.org

Signed-off-by: David Marchand <david.march...@redhat.com>
---
 lib/eal/x86/rte_power_intrinsics.c | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/lib/eal/x86/rte_power_intrinsics.c 
b/lib/eal/x86/rte_power_intrinsics.c
index e4cb913590..6fc3b2b1cb 100644
--- a/lib/eal/x86/rte_power_intrinsics.c
+++ b/lib/eal/x86/rte_power_intrinsics.c
@@ -22,8 +22,6 @@ struct power_wait_status {
 
 RTE_LCORE_VAR_HANDLE(struct power_wait_status, wait_status);
 
-RTE_LCORE_VAR_INIT(wait_status);
-
 /*
  * This function uses UMONITOR/UMWAIT instructions and will enter C0.2 state.
  * For more information about usage of these instructions, please refer to
@@ -177,6 +175,9 @@ rte_power_monitor(const struct rte_power_monitor_cond *pmc,
        if (pmc->fn == NULL)
                return -EINVAL;
 
+       if (wait_status == NULL)
+               RTE_LCORE_VAR_ALLOC(wait_status);
+
        s = RTE_LCORE_VAR_LCORE(lcore_id, wait_status);
 
        /* update sleep address */
@@ -269,6 +270,9 @@ rte_power_monitor_wakeup(const unsigned int lcore_id)
        if (lcore_id >= RTE_MAX_LCORE)
                return -EINVAL;
 
+       if (wait_status == NULL)
+               RTE_LCORE_VAR_ALLOC(wait_status);
+
        s = RTE_LCORE_VAR_LCORE(lcore_id, wait_status);
 
        /*
@@ -308,7 +312,7 @@ int
 rte_power_monitor_multi(const struct rte_power_monitor_cond pmc[],
                const uint32_t num, const uint64_t tsc_timestamp)
 {
-       struct power_wait_status *s = RTE_LCORE_VAR(wait_status);
+       struct power_wait_status *s;
        uint32_t i, rc;
 
        /* check if supported */
@@ -318,6 +322,11 @@ rte_power_monitor_multi(const struct 
rte_power_monitor_cond pmc[],
        if (pmc == NULL || num == 0)
                return -EINVAL;
 
+       if (wait_status == NULL)
+               RTE_LCORE_VAR_ALLOC(wait_status);
+
+       s = RTE_LCORE_VAR(wait_status);
+
        /* we are already inside transaction region, return */
        if (rte_xtest() != 0)
                return 0;
-- 
2.47.0