[dpdk-dev] [PATCH v3 3/6] power: rename public structs
David Hunt
david.hunt at intel.com
Wed Jan 20 16:03:29 CET 2021
On 20/1/2021 2:27 PM, Burakov, Anatoly wrote:
> 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?
>
Good point. Will do.
Thanks.
More information about the dev
mailing list