[dpdk-stable] [dpdk-dev] [PATCH v3 3/6] power: rename public structs
Burakov, Anatoly
anatoly.burakov at intel.com
Wed Jan 20 15:27:51 CET 2021
On 20-Jan-21 1:55 PM, David Hunt wrote:
> From: Bruce Richardson <bruce.richardson at intel.com>
>
> 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 at dpdk.org
>
> Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
> Signed-off-by: David Hunt <david.hunt at intel.com>
> ---
<snip>
> 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?
--
Thanks,
Anatoly
More information about the stable
mailing list