test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] crypto: Rebuild DPDK at the end of suite
@ 2019-01-09  6:27 Phanendra Vukkisala
  2019-01-11  2:00 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: Phanendra Vukkisala @ 2019-01-09  6:27 UTC (permalink / raw)
  To: dts; +Cc: Vijaya Bhaskar Annayyolla, Faisal Masood, Phanendra Vukkisala

From: pvukkisala <pvukkisala@marvell.com>

Crypto tests building DPDK with changes before starting tests and undoing
changes at end, but missed to rebuild. This patch will take care of rebuilding
DPDK, so that further tests will not effect while running multiple tests at once

Signed-off-by: phanendra,vukkisala <pvukkisala@marvell.com>
---
 tests/TestSuite_crypto_perf.py          |    1 +
 tests/TestSuite_unit_tests_cryptodev.py |    1 +
 2 files changed, 2 insertions(+)

diff --git a/tests/TestSuite_crypto_perf.py b/tests/TestSuite_crypto_perf.py
index 1715e5c..cf5cce4 100755
--- a/tests/TestSuite_crypto_perf.py
+++ b/tests/TestSuite_crypto_perf.py
@@ -331,6 +331,7 @@ class TestCryptoPerf(TestCase):
         self.dut.send_expect("sed -i 's/CONFIG_RTE_LIBRTE_PMD_OPENSSL=y$/CONFIG_RTE_LIBRTE_PMD_OPENSSL=n/' config/common_base", "# ")
         self.dut.send_expect("sed -i 's/CONFIG_RTE_LIBRTE_PMD_AESNI_MB=y$/CONFIG_RTE_LIBRTE_PMD_AESNI_MB=n/' config/common_base", "# ")
         self.dut.send_expect("sed -i 's/CONFIG_RTE_LIBRTE_PMD_ZUC=y$/CONFIG_RTE_LIBRTE_PMD_ZUC=n/' config/common_base", "# ")
+        self.dut.build_install_dpdk(self.dut.target)
         f.close()
 
     def __check_buffer_size_number(self):
diff --git a/tests/TestSuite_unit_tests_cryptodev.py b/tests/TestSuite_unit_tests_cryptodev.py
index df83fd9..db6552e 100644
--- a/tests/TestSuite_unit_tests_cryptodev.py
+++ b/tests/TestSuite_unit_tests_cryptodev.py
@@ -105,6 +105,7 @@ class UnitTestsCryptodev(TestCase):
         self.dut.send_expect("sed -i 's/CONFIG_RTE_LIBRTE_PMD_OPENSSL=y$/CONFIG_RTE_LIBRTE_PMD_OPENSSL=n/' config/common_base", "# ")
         self.dut.send_expect("sed -i 's/CONFIG_RTE_LIBRTE_PMD_AESNI_MB=y$/CONFIG_RTE_LIBRTE_PMD_AESNI_MB=n/' config/common_base", "# ")
         self.dut.send_expect("sed -i 's/CONFIG_RTE_LIBRTE_PMD_ZUC=y$/CONFIG_RTE_LIBRTE_PMD_ZUC=n/' config/common_base", "# ")
+        self.dut.build_install_dpdk(self.dut.target)
 
     def test_cryptodev_qat_autotest(self):
         self.__run_unit_test("cryptodev_qat_autotest")
-- 
1.7.9.5

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dts] [PATCH] crypto: Rebuild DPDK at the end of suite
  2019-01-09  6:27 [dts] [PATCH] crypto: Rebuild DPDK at the end of suite Phanendra Vukkisala
@ 2019-01-11  2:00 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2019-01-11  2:00 UTC (permalink / raw)
  To: Phanendra Vukkisala, dts; +Cc: Vijaya Bhaskar Annayyolla, Faisal Masood

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Phanendra Vukkisala
> Sent: Wednesday, January 9, 2019 2:28 PM
> To: dts@dpdk.org
> Cc: Vijaya Bhaskar Annayyolla <avijay@marvell.com>; Faisal Masood
> <fmasood@marvell.com>; Phanendra Vukkisala <pvukkisala@marvell.com>
> Subject: [dts] [PATCH] crypto: Rebuild DPDK at the end of suite
> 
> From: pvukkisala <pvukkisala@marvell.com>
> 
> Crypto tests building DPDK with changes before starting tests and undoing
> changes at end, but missed to rebuild. This patch will take care of rebuilding
> DPDK, so that further tests will not effect while running multiple tests at once
> 
> Signed-off-by: phanendra,vukkisala <pvukkisala@marvell.com>
> ---
>  tests/TestSuite_crypto_perf.py          |    1 +
>  tests/TestSuite_unit_tests_cryptodev.py |    1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/tests/TestSuite_crypto_perf.py b/tests/TestSuite_crypto_perf.py
> index 1715e5c..cf5cce4 100755
> --- a/tests/TestSuite_crypto_perf.py
> +++ b/tests/TestSuite_crypto_perf.py
> @@ -331,6 +331,7 @@ class TestCryptoPerf(TestCase):
>          self.dut.send_expect("sed -i
> 's/CONFIG_RTE_LIBRTE_PMD_OPENSSL=y$/CONFIG_RTE_LIBRTE_PMD_OPENS
> SL=n/' config/common_base", "# ")
>          self.dut.send_expect("sed -i
> 's/CONFIG_RTE_LIBRTE_PMD_AESNI_MB=y$/CONFIG_RTE_LIBRTE_PMD_AESN
> I_MB=n/' config/common_base", "# ")
>          self.dut.send_expect("sed -i
> 's/CONFIG_RTE_LIBRTE_PMD_ZUC=y$/CONFIG_RTE_LIBRTE_PMD_ZUC=n/'
> config/common_base", "# ")
> +        self.dut.build_install_dpdk(self.dut.target)
>          f.close()
> 
>      def __check_buffer_size_number(self):
> diff --git a/tests/TestSuite_unit_tests_cryptodev.py
> b/tests/TestSuite_unit_tests_cryptodev.py
> index df83fd9..db6552e 100644
> --- a/tests/TestSuite_unit_tests_cryptodev.py
> +++ b/tests/TestSuite_unit_tests_cryptodev.py
> @@ -105,6 +105,7 @@ class UnitTestsCryptodev(TestCase):
>          self.dut.send_expect("sed -i
> 's/CONFIG_RTE_LIBRTE_PMD_OPENSSL=y$/CONFIG_RTE_LIBRTE_PMD_OPENS
> SL=n/' config/common_base", "# ")
>          self.dut.send_expect("sed -i
> 's/CONFIG_RTE_LIBRTE_PMD_AESNI_MB=y$/CONFIG_RTE_LIBRTE_PMD_AESN
> I_MB=n/' config/common_base", "# ")
>          self.dut.send_expect("sed -i
> 's/CONFIG_RTE_LIBRTE_PMD_ZUC=y$/CONFIG_RTE_LIBRTE_PMD_ZUC=n/'
> config/common_base", "# ")
> +        self.dut.build_install_dpdk(self.dut.target)
> 
>      def test_cryptodev_qat_autotest(self):
>          self.__run_unit_test("cryptodev_qat_autotest")
> --
> 1.7.9.5

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-01-11  2:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-09  6:27 [dts] [PATCH] crypto: Rebuild DPDK at the end of suite Phanendra Vukkisala
2019-01-11  2:00 ` Tu, Lijuan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).