Re: [dpdk-dev] [PATCH v3 3/6] power: rename public structs

[David Hunt <david.hunt@intel.com>]

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@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@dpdk.org
>>
>> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
>> Signed-off-by: David Hunt <david.hunt@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.