[V3,2/2] tests/*: Remove old DPDK flags from power tests

Message ID 20220329102408.1137392-3-tadhg.kearney@intel.com (mailing list archive)
State Accepted
Headers
Series Remove old DPDK flags from power tests |

Checks

Context Check Description
ci/Intel-dts-suite-test warning SKIPPED
ci/Intel-dts-doc-test success Testing OK

Commit Message

Tadhg Kearney March 29, 2022, 10:24 a.m. UTC
  * CONFIG_RTE_LIBRTE_I40E_PMD, CONFIG_RTE_LIBRTE_POWER + _CONFIG option do not exist anymore
* Telemetry is built by default + _LIBRTE_ option also does not exist anymore

Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>
---
 tests/TestSuite_power_branch_ratio.py | 39 +--------------------------
 tests/TestSuite_power_telemetry.py    | 18 +------------
 2 files changed, 2 insertions(+), 55 deletions(-)
  

Comments

Jun Dong March 30, 2022, 7:23 a.m. UTC | #1
Acked-by: Jun Dong <junx.dong@intel.com>

> -----Original Message-----
> From: Tadhg Kearney <tadhg.kearney@intel.com>
> Sent: Tuesday, March 29, 2022 6:24 PM
> To: dts@dpdk.org
> Cc: Pattan, Reshma <reshma.pattan@intel.com>; Kearney, Tadhg
> <tadhg.kearney@intel.com>
> Subject: [dts][V3 2/2] tests/*: Remove old DPDK flags from power tests
> 
> * CONFIG_RTE_LIBRTE_I40E_PMD, CONFIG_RTE_LIBRTE_POWER + _CONFIG
> option do not exist anymore
> * Telemetry is built by default + _LIBRTE_ option also does not exist anymore
> 
> Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>
  
Tu, Lijuan March 31, 2022, 6:31 a.m. UTC | #2
On Tue, 29 Mar 2022 10:24:08 +0000, Tadhg Kearney <tadhg.kearney@intel.com> wrote:
> * CONFIG_RTE_LIBRTE_I40E_PMD, CONFIG_RTE_LIBRTE_POWER + _CONFIG option do not exist anymore
> * Telemetry is built by default + _LIBRTE_ option also does not exist anymore
> 
> Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>


Series applied, thanks
  

Patch

diff --git a/tests/TestSuite_power_branch_ratio.py b/tests/TestSuite_power_branch_ratio.py
index deb271e0..a2c28478 100644
--- a/tests/TestSuite_power_branch_ratio.py
+++ b/tests/TestSuite_power_branch_ratio.py
@@ -44,8 +44,7 @@  from pprint import pformat
 from framework.exception import VerifyFailure
 from framework.packet import Packet
 from framework.pktgen import TRANSMIT_CONT
-from framework.qemu_libvirt import LibvirtKvm
-from framework.settings import HEADER_SIZE, HOST_BUILD_TYPE_SETTING, load_global_setting
+from framework.settings import HEADER_SIZE
 from framework.test_case import TestCase
 from framework.utils import create_mask as dts_create_mask
 
@@ -227,27 +226,6 @@  class TestPowerBranchRatio(TestCase):
 
         return result
 
-    @property
-    def compile_switch(self):
-        sw_table = [
-            "CONFIG_RTE_LIBRTE_POWER",
-            "CONFIG_RTE_LIBRTE_POWER_DEBUG",
-        ]
-        return sw_table
-
-    def preset_compilation(self):
-        if "meson" == load_global_setting(HOST_BUILD_TYPE_SETTING):
-            compile_SWs = self.compile_switch + ["CONFIG_RTE_LIBRTE_I40E_PMD"]
-            self.dut.set_build_options(dict([(sw[7:], "y") for sw in compile_SWs]))
-        else:
-            for sw in self.compile_switch:
-                cmd = (
-                    "sed -i -e " "'s/{0}=n$/{0}=y/' " "{1}/config/common_base"
-                ).format(sw, self.target_dir)
-                self.d_a_con(cmd)
-        # re-compile dpdk source code
-        self.dut.build_install_dpdk(self.target)
-
     @contextmanager
     def restore_environment(self):
         try:
@@ -258,19 +236,6 @@  class TestPowerBranchRatio(TestCase):
                 self.restore_port_drv()
             except Exception as e:
                 self.logger.error(traceback.format_exc())
-            # restore compilation
-            if "meson" == load_global_setting(HOST_BUILD_TYPE_SETTING):
-                self.dut.set_build_options(
-                    dict([(sw[7:], "n") for sw in self.compile_switch])
-                )
-            else:
-                for sw in self.compile_switch:
-                    cmd = (
-                        "sed -i -e " "'s/{0}=y$/{0}=n/' " "{1}/config/common_base"
-                    ).format(sw, self.target_dir)
-                    self.d_a_con(cmd)
-            # re-compile dpdk source code
-            self.dut.build_install_dpdk(self.target)
 
     def restore_port_drv(self):
         driver = self.drivername
@@ -532,8 +497,6 @@  class TestPowerBranchRatio(TestCase):
         # modprobe msr module to let the application can get the CPU HW info
         self.d_a_con("modprobe msr")
         self.d_a_con("cpupower frequency-set -g userspace > /dev/null 2>&1")
-        # compile
-        self.preset_compilation()
         # init binary
         self.init_vm_power_mgr()
         self.init_testpmd()
diff --git a/tests/TestSuite_power_telemetry.py b/tests/TestSuite_power_telemetry.py
index afb55262..39acdef2 100644
--- a/tests/TestSuite_power_telemetry.py
+++ b/tests/TestSuite_power_telemetry.py
@@ -45,7 +45,7 @@  from pprint import pformat
 from framework.exception import VerifyFailure
 from framework.packet import Packet
 from framework.pktgen import TRANSMIT_CONT
-from framework.settings import HEADER_SIZE, HOST_BUILD_TYPE_SETTING, load_global_setting
+from framework.settings import HEADER_SIZE
 from framework.test_case import TestCase
 from framework.utils import create_mask as dts_create_mask
 
@@ -125,20 +125,6 @@  class TestPowerTelemetry(TestCase):
 
         return result
 
-    def preset_compilation(self):
-        if self.dut.skip_setup:
-            return
-        SW = "CONFIG_RTE_LIBRTE_TELEMETRY"
-        if "meson" == load_global_setting(HOST_BUILD_TYPE_SETTING):
-            self.dut.set_build_options({SW[7:]: "y"})
-        else:
-            cmd = "sed -i -e 's/{0}=n$/{0}=y/' {1}/config/common_base".format(
-                SW, self.target_dir
-            )
-            self.d_a_con(cmd)
-        # re-compile dpdk source code
-        self.dut.build_install_dpdk(self.target)
-
     def prepare_binary(self, name):
         example_dir = "examples/" + name
         out = self.dut.build_dpdk_apps("./" + example_dir)
@@ -445,8 +431,6 @@  class TestPowerTelemetry(TestCase):
 
     def preset_test_environment(self):
         self.is_l3fwd_on = None
-        # open compile switch and re-compile target source code
-        self.preset_compilation()
         # init binary
         self.init_l3fwd_power()
         self.init_telemetry()