Re: [dpdk-dev] [PATCH v4 3/4] power: process incoming confirmation cmds

[Pattan, Reshma]

> -----Original Message-----
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Hajkowski
> +             RTE_LOG(DEBUG, GUEST_CLI, "Not expected command has
> been received.\n");

Instead of : "Not expected command has been received ".  How about, Received 
invalid response from host, expecting NACK/ACK?
But let's hear from others.

> +     int result;
> +     ret = check_response_cmd(res->lcore_id, &result);
> +     if (ret < 0) {
> +             RTE_LOG(DEBUG, GUEST_CLI, "No confirmation for sent msg
> received\n");
> +     } else {
> +             cmdline_printf(cl, "Ack for sent msg received with result:
> %s.\n",
> +                             result == 1 ? "SUCCESS" : "ERROR");
> +     }
>  }

In else statement how about just checking for result ACK/NACK and have log 
saying
received ACK /received NACK for msg

Thanks,
Reshma