[dpdk-dev] [PATCH 4/4] power: send confirmation cmd to vm guest

Hajkowski, MarcinX marcinx.hajkowski at intel.com
Mon Mar 18 13:40:52 CET 2019

Logging  error in case of invalid command is enough in my opinion. In this case we distinguish between invalid command (log error) and unsuccessful power operation (NAK cmd). 
I agree though that error log might be more precise (separate for error during sending ACK/NAK and separate for  invalid command).

Best Regards,

-----Original Message-----
From: Pattan, Reshma 
Sent: Friday, March 15, 2019 6:14 PM
To: Hajkowski, MarcinX <marcinx.hajkowski at intel.com>; Hunt, David <david.hunt at intel.com>
Cc: dev at dpdk.org; Hajkowski, MarcinX <marcinx.hajkowski at intel.com>
Subject: RE: [dpdk-dev] [PATCH 4/4] power: send confirmation cmd to vm guest

> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Hajkowski

> +		int ret = -1;
> +		if (valid_unit)
> +			ret = send_ack_for_received_cmd(pkt,
> +						chan_info,
> +						scale_res ?

> +		if (ret < 0)
> sending ack command "
> +					"or unexpected unit type.\n");
> +
>  	}

This if check should go inside for if(valid_unit)?,  otherwise this would applicable for valid_unit == false also.
Also do we need to handle valid_unit ==false case? Do we need to send back  message to guest saying invalid command.


More information about the dev mailing list