<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body>
    <p><br>
    </p>
    <div class="moz-cite-prefix">On 20/07/2024 17:50, Sivaprasad Tummala
      wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:20240720165030.246294-4-sivaprasad.tummala@amd.com">
      <pre class="moz-quote-pre" wrap="">After refactoring the power library, power management operations are now
consistently supported regardless of the operating environment, making
function pointer checks unnecessary and thus removed from applications.

Signed-off-by: Sivaprasad Tummala <a class="moz-txt-link-rfc2396E" href="mailto:sivaprasad.tummala@amd.com"><sivaprasad.tummala@amd.com></a>
---
 app/test/test_power.c         | 95 -----------------------------------
 app/test/test_power_cpufreq.c | 52 -------------------
 app/test/test_power_kvm_vm.c  | 36 -------------
 3 files changed, 183 deletions(-)

</pre>
    </blockquote>
    <p><br>
    </p>
    <p>Hi Sivaprasad, <br>
    </p>
    <p>    Nice work on the patch-set. <br>
    </p>
    <p>There's just four function pointer checks remaining that my
      compiler is complaining about. They are in
      examples/l3fwd-power/main.c (lines 443, 452, 1350, 1353). It would
      be nice to have these removed as well, seeing as the functions are
      now inlines and don't need these checks.<br>
    </p>
    <p>I'm running the patch set through some tests here, will keep you
      posted on progress.</p>
    <p>Rgds,<br>
      Dave.</p>
    <p><br>
    </p>
    ---snip---<span style="white-space: pre-wrap">
</span>
  </body>
</html>