[spp] [PATCH] spp_vf: change exit command to return response

Yasufumi Ogawa ogawa.yasufumi at lab.ntt.co.jp
Tue Dec 25 08:46:55 CET 2018


> From: Hideyuki Yamashita <yamashita.hideyuki at po.ntt-tx.co.jp>
> 
> For secondary processes use common process_request(), it causes an
> error because spp-ctl expects a response but these processes do not
> return anything. This patch is to change it to return message.
Applied, thanks.

> 
> Signed-off-by: Hideyuki Yamashita <yamashita.hideyuki at po.ntt-tx.co.jp>
> Signed-off-by: Naoki Takada <takada.naoki at lab.ntt.co.jp>
> ---
>   src/vf/common/command_proc.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/vf/common/command_proc.c b/src/vf/common/command_proc.c
> index cf35ce0..a612061 100644
> --- a/src/vf/common/command_proc.c
> +++ b/src/vf/common/command_proc.c
> @@ -1600,8 +1600,10 @@ process_request(int *sock, const char *request_str, size_t request_str_len)
>   	if (request.is_requested_exit) {
>   		/* Terminated by process exit command.                       */
>   		/* Other route is normal end because it responds to command. */
> +		set_command_results(&command_results[0], CRES_SUCCESS, "");
> +		send_command_result_response(sock, &request, command_results);
>   		RTE_LOG(INFO, SPP_COMMAND_PROC,
> -				"No response with process exit command.\n");
> +				"Terminate process for exit.\n");
>   		return SPP_RET_NG;
>   	}
>   
> 



More information about the spp mailing list