On 20/1/2021 2:27 PM, Burakov, Anatoly wrote:

On 20-Jan-21 1:55 PM, David Hunt wrote:

quoted

From: Bruce Richardson <redacted>

rename the public structs to have an rte_power_ prefix.

Fixes: 210c383e247b ("power: packet format for vm power management")
Fixes: cd0d5547e873 ("power: vm communication channels in guest")
Cc: stable@dpdk.org

Signed-off-by: Bruce Richardson <redacted>
Signed-off-by: David Hunt <redacted>
---

<snip>

quoted

          RTE_LOG(ERR, GUEST_CLI, "Error receiving message.\n");

@@ -293,14 +294,15 @@ cmd_query_caps_list_parsed(void *parsed_result,

  {
      struct cmd_query_caps_result *res = parsed_result;
      unsigned int lcore_id;
-    struct channel_packet_caps_list pkt_caps_list;
-    struct channel_packet pkt;
+    struct rte_power_channel_packet_caps_list pkt_caps_list;
+    struct rte_power_channel_packet pkt;
      bool query_list = false;
      int ret;
      char *ep;
  -    memset(&pkt, 0, sizeof(struct channel_packet));
-    memset(&pkt_caps_list, 0, sizeof(struct channel_packet_caps_list));
+    memset(&pkt, 0, sizeof(struct rte_power_channel_packet));
+    memset(&pkt_caps_list, 0,
+            sizeof(struct rte_power_channel_packet_caps_list));

Here and in other places where you do sizeof(), you might want to 
avoid future changes by replacing it with sizeof(*pkt) instead?

Good point. Will do.

Thanks.