test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts][PATCH v2 0/2] remove power_empty_poll test_plan and conf
@ 2023-05-08  9:30 Tadhg Kearney
  2023-05-08  9:30 ` [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove power_empty_poll test_plan Tadhg Kearney
  2023-05-08  9:30 ` [dts][PATCH v2 2/2] conf/power_empty_poll: remove power_empty_poll configuration Tadhg Kearney
  0 siblings, 2 replies; 5+ messages in thread
From: Tadhg Kearney @ 2023-05-08  9:30 UTC (permalink / raw)
  To: dts; +Cc: reshma.pattan, karen.kelly, Tadhg Kearney

The experimental empty poll API has been removed from the power
management library in DPDK 22.03. Removing test_plan and configuration,
power_empty_poll test has already been removed.

Tadhg Kearney (2):
  test_plan/power_empty_poll: remove power_empty_poll test_plan
  conf/power_empty_poll: remove power_empty_poll configuration

 conf/power_empty_poll.cfg                 |   3 -
 test_plans/index.rst                      |   1 -
 test_plans/power_empty_poll_test_plan.rst | 115 ----------------------
 3 files changed, 119 deletions(-)
 delete mode 100644 conf/power_empty_poll.cfg
 delete mode 100644 test_plans/power_empty_poll_test_plan.rst

-- 
2.34.1


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

* [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove power_empty_poll test_plan
  2023-05-08  9:30 [dts][PATCH v2 0/2] remove power_empty_poll test_plan and conf Tadhg Kearney
@ 2023-05-08  9:30 ` Tadhg Kearney
  2023-05-08 11:52   ` Pattan, Reshma
  2023-05-08  9:30 ` [dts][PATCH v2 2/2] conf/power_empty_poll: remove power_empty_poll configuration Tadhg Kearney
  1 sibling, 1 reply; 5+ messages in thread
From: Tadhg Kearney @ 2023-05-08  9:30 UTC (permalink / raw)
  To: dts; +Cc: reshma.pattan, karen.kelly, Tadhg Kearney

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=y, Size: 6141 bytes --]

Removing test_plan index and file, power_empty_poll test has already been
removed.

Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>
---
 test_plans/index.rst                      |   1 -
 test_plans/power_empty_poll_test_plan.rst | 115 ----------------------
 2 files changed, 116 deletions(-)
 delete mode 100644 test_plans/power_empty_poll_test_plan.rst

diff --git a/test_plans/index.rst b/test_plans/index.rst
index a0c056cd..5f81951f 100644
--- a/test_plans/index.rst
+++ b/test_plans/index.rst
@@ -211,7 +211,6 @@ The following are the test plans for the DPDK DTS automated test system.
     vm_pw_mgmt_policy_test_plan
     power_bidirection_channel_test_plan
     power_branch_ratio_test_plan
-    power_empty_poll_test_plan
     power_pbf_test_plan
     power_pmd_test_plan
     power_pstate_test_plan
diff --git a/test_plans/power_empty_poll_test_plan.rst b/test_plans/power_empty_poll_test_plan.rst
deleted file mode 100644
index 70784092..00000000
--- a/test_plans/power_empty_poll_test_plan.rst
+++ /dev/null
@@ -1,115 +0,0 @@
-.. SPDX-License-Identifier: BSD-3-Clause
-   Copyright(c) 2010-2019 Intel Corporation
-
-=========================
-Power Lib Empty Poll Test
-=========================
-
-Inband Policy Control
-=====================
-
-For packet processing workloads such as DPDK polling is continuous. This means
-CPU cores always show 100% busy independent of how much work those cores are
-doing. It is critical to accurately determine how busy a core is hugely
-important for the following reasons:
-
-   * No indication of overload conditions
-
-   * User do not know how much real load is on a system meaning resulted in
-     wasted energy as no power management is utilized
-
-Tried and failed schemes include calculating the cycles required from the load
-on the core, in other words the busyness. For example, how many cycles it costs
-to handle each packet and determining the frequency cost per core. Due to the
-varying nature of traffic, types of frames and cost in cycles to process, this
-mechanism becomes complex quickly where a simple scheme is required to solve
-the problems.
-
-For all polling mechanism, the proposed solution focus on how many times empty
-poll executed instead of calculating how many cycles it cost to handle each
-packet. The less empty poll number means current core is busy with processing
-workload, therefore,  the higher frequency is needed. The high empty poll
-number indicate current core has lots spare time, therefore, we can lower the
-frequency.
-
-2.1 Power state definition:
-
-LOW:  the frequency is used for purge mode.
-
-MED:  the frequency is used to process modest traffic workload.
-
-HIGH: the frequency is used to process busy traffic workload.
-
-2.2 There are two phases to establish the power management system:
-
-a.Initialization/Training phase. There is no traffic pass-through, the system
-will test average empty poll numbers  with LOW/MED/HIGH  power state. Those
-average empty poll numbers will be the baseline for the normal phase. The
-system will collect all core's counter every 100ms. The Training phase will
-take 5 seconds.
-
-b.Normal phase. When the real traffic pass-though, the system will compare
-run-time empty poll moving average value with base line then make decision to
-move to HIGH power state of MED  power state. The system will collect all
-core's counter every 10ms.
-
-``training_flag`` : optional, enable/disable training mode. Default value is 0.
- If the training_flag is set as 1(true), then the application will start in
- training mode and print out the trained threshold values. If the training_flag
- is set as 0(false), the application will start in normal mode, and will use
- either the default thresholds or those supplied on the command line. The
- trained threshold values are specific to the user’s system, may give a better
- power profile when compared to the default threshold values.
-
-``med_threshold`` : optional, sets the empty poll threshold of a modestly busy
-system state. If this is not supplied, the application will apply the default
-value of 350000.
-
-``high_threshold`` : optional, sets the empty poll threshold of a busy system
-state. If this is not supplied, the application will apply the default value of
-580000.
-
-
-Preparation Work for Settings
-=============================
-BIOS setting::
-
-    1. Turn on Speedstep option in BIOS
-    2. Turn on Turbo in BIOS
-    3. Turn off Hyper Threading
-
-Linux setting::
-
-    1. Use intel_pstate driver for CPU frequency control
-    2. modprobe msr
-
-sys_min=/sys/devices/system/cpu/cpu{}/cpufreq/cpuinfo_min_freq
-sys_max=/sys/devices/system/cpu/cpu{}/cpufreq/cpuinfo_max_freq
-no_turbo_max=$(rdmsr -p 1 0x0CE -f 15:8 -d)00000
-
-cur_min=/sys/devices/system/cpu/cpu{}/cpufreq/scaling_min_freq
-cur_max=/sys/devices/system/cpu/cpu{}/cpufreq/scaling_max_freq
-
-
-Test Case1 : Basic Training mode test based on one NIC with l3fwd-power
-=======================================================================
-Step 1. Bind One NIC to DPDK driver, launch l3fwd-power with empty-poll enabled
-
-    ./<build_target>/examples/dpdk-l3fwd-power -l 1-2 -n 4 -- -p 0x1 -P --config="(0,0,2)" --empty-poll="1,0,0" -l 10 -m 6 -h 1
-
-Step 2. Check the log also when changing the inject packet rate as following:
-
-    Injected Rate(1024B, dst_ip=1.1.1.1): 10G -> 0.1G -> 10G -> 0.1G -> 10G ->
-    0.1G The frequency will be set to MED when we inject 0.1G and return to HGH
-    when inject 10G Rate, check the frequency of the forwarding core(core 2)
-    When traffic is 10G:  cur_min=cur_max=no_turbo_max
-    When traffic is 0.1G: cur_min=cur_max=[no_turbo_max-500000]
-
-
-Test Case2: No-Training mode test based on one NIC with l3fwd-power
-===================================================================
-Step 1. Bind One NIC to DPDK driver, launch l3fwd-power with empty-poll enabled
-
-   ./<build_target>/examples/dpdk-l3fwd-power -l 1-2 -n 4  -- -p 0x1 -P --config="(0,0,2)" --empty-poll="0,350000,500000" -l 10 -m 6 -h 1
-
-Step 2. Check no training steps are executed in sample's launch log.
-- 
2.34.1


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

* [dts][PATCH v2 2/2] conf/power_empty_poll: remove power_empty_poll configuration
  2023-05-08  9:30 [dts][PATCH v2 0/2] remove power_empty_poll test_plan and conf Tadhg Kearney
  2023-05-08  9:30 ` [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove power_empty_poll test_plan Tadhg Kearney
@ 2023-05-08  9:30 ` Tadhg Kearney
  2023-05-08 11:54   ` Pattan, Reshma
  1 sibling, 1 reply; 5+ messages in thread
From: Tadhg Kearney @ 2023-05-08  9:30 UTC (permalink / raw)
  To: dts; +Cc: reshma.pattan, karen.kelly, Tadhg Kearney

Power_empty_poll test has been removed, so removing configuration to
follow.

Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>
---
 conf/power_empty_poll.cfg | 3 ---
 1 file changed, 3 deletions(-)
 delete mode 100644 conf/power_empty_poll.cfg

diff --git a/conf/power_empty_poll.cfg b/conf/power_empty_poll.cfg
deleted file mode 100644
index 4533250a..00000000
--- a/conf/power_empty_poll.cfg
+++ /dev/null
@@ -1,3 +0,0 @@
-# frame_size: select different traffic frame size.
-[suite]
-frame_size = 256
-- 
2.34.1


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

* RE: [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove power_empty_poll test_plan
  2023-05-08  9:30 ` [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove power_empty_poll test_plan Tadhg Kearney
@ 2023-05-08 11:52   ` Pattan, Reshma
  0 siblings, 0 replies; 5+ messages in thread
From: Pattan, Reshma @ 2023-05-08 11:52 UTC (permalink / raw)
  To: Kearney, Tadhg, dts; +Cc: Kelly, Karen



> -----Original Message-----
> From: Kearney, Tadhg <tadhg.kearney@intel.com>
> Sent: Monday, May 8, 2023 10:31 AM
> To: dts@dpdk.org
> Cc: Pattan, Reshma <reshma.pattan@intel.com>; Kelly, Karen
> <karen.kelly@intel.com>; Kearney, Tadhg <tadhg.kearney@intel.com>
> Subject: [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove
> power_empty_poll test_plan
> 
> Removing test_plan index and file, power_empty_poll test has already been
> removed.
> 
> Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>
Acked-by: Reshma Pattan <reshma.pattan@intel.com>

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

* RE: [dts][PATCH v2 2/2] conf/power_empty_poll: remove power_empty_poll configuration
  2023-05-08  9:30 ` [dts][PATCH v2 2/2] conf/power_empty_poll: remove power_empty_poll configuration Tadhg Kearney
@ 2023-05-08 11:54   ` Pattan, Reshma
  0 siblings, 0 replies; 5+ messages in thread
From: Pattan, Reshma @ 2023-05-08 11:54 UTC (permalink / raw)
  To: Kearney, Tadhg, dts; +Cc: Kelly, Karen



> -----Original Message-----
> From: Kearney, Tadhg <tadhg.kearney@intel.com>
> Sent: Monday, May 8, 2023 10:31 AM
> To: dts@dpdk.org
> Cc: Pattan, Reshma <reshma.pattan@intel.com>; Kelly, Karen
> <karen.kelly@intel.com>; Kearney, Tadhg <tadhg.kearney@intel.com>
> Subject: [dts][PATCH v2 2/2] conf/power_empty_poll: remove
> power_empty_poll configuration
> 
> Power_empty_poll test has been removed, so removing configuration to follow.
> 
> Signed-off-by: Tadhg Kearney <tadhg.kearney@intel.com>
Acked-by: Reshma Pattan <reshma.pattan@intel.com>

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

end of thread, other threads:[~2023-05-08 11:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-08  9:30 [dts][PATCH v2 0/2] remove power_empty_poll test_plan and conf Tadhg Kearney
2023-05-08  9:30 ` [dts][PATCH v2 1/2] test_plan/power_empty_poll: remove power_empty_poll test_plan Tadhg Kearney
2023-05-08 11:52   ` Pattan, Reshma
2023-05-08  9:30 ` [dts][PATCH v2 2/2] conf/power_empty_poll: remove power_empty_poll configuration Tadhg Kearney
2023-05-08 11:54   ` Pattan, Reshma

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).