[v3,2/8] lib/power: add changes for host commands/policies

Message ID 20180914135406.52190-3-david.hunt@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series add json power policy interface for containers |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Hunt, David Sept. 14, 2018, 1:54 p.m. UTC
  Signed-off-by: David Hunt <david.hunt@intel.com>
---
 lib/librte_power/channel_commands.h | 5 +++++
 1 file changed, 5 insertions(+)
  

Comments

Burakov, Anatoly Sept. 25, 2018, 9:21 a.m. UTC | #1
On 14-Sep-18 2:54 PM, David Hunt wrote:
> Signed-off-by: David Hunt <david.hunt@intel.com>

The commit message is not very descriptive as to what these chages are 
and why they are added.
  
Hunt, David Sept. 25, 2018, 1:47 p.m. UTC | #2
On 25/9/2018 10:21 AM, Burakov, Anatoly wrote:
> On 14-Sep-18 2:54 PM, David Hunt wrote:
>> Signed-off-by: David Hunt <david.hunt@intel.com>
>
> The commit message is not very descriptive as to what these chages are 
> and why they are added.
>

Sure, I'll add a more descriptive comment next version.

Thanks,
Dave.
  

Patch

diff --git a/lib/librte_power/channel_commands.h b/lib/librte_power/channel_commands.h
index ee638eefa..e7b93a797 100644
--- a/lib/librte_power/channel_commands.h
+++ b/lib/librte_power/channel_commands.h
@@ -19,6 +19,7 @@  extern "C" {
 #define CPU_POWER               1
 #define CPU_POWER_CONNECT       2
 #define PKT_POLICY              3
+#define PKT_POLICY_REMOVE       4
 
 /* CPU Power Command Scaling */
 #define CPU_POWER_SCALE_UP      1
@@ -58,6 +59,9 @@  struct traffic {
 	uint32_t max_max_packet_thresh;
 };
 
+#define CORE_TYPE_VIRTUAL 0
+#define CORE_TYPE_PHYSICAL 1
+
 struct channel_packet {
 	uint64_t resource_id; /**< core_num, device */
 	uint32_t unit;        /**< scale down/up/min/max */
@@ -70,6 +74,7 @@  struct channel_packet {
 	uint8_t vcpu_to_control[MAX_VCPU_PER_VM];
 	uint8_t num_vcpu;
 	struct timer_profile timer_policy;
+	bool core_type;
 	enum workload workload;
 	enum policy_to_use policy_to_use;
 	struct t_boost_status t_boost_status;