From 1369c6a44b622df3577223ce68ff16a1ea7cc8aa Mon Sep 17 00:00:00 2001 From: imarom Date: Mon, 30 Jan 2017 13:56:01 +0200 Subject: move service mode check to RX queue enable Signed-off-by: imarom --- src/rpc-server/commands/trex_rpc_cmd_general.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/rpc-server/commands/trex_rpc_cmd_general.cpp b/src/rpc-server/commands/trex_rpc_cmd_general.cpp index 54798abb..60180659 100644 --- a/src/rpc-server/commands/trex_rpc_cmd_general.cpp +++ b/src/rpc-server/commands/trex_rpc_cmd_general.cpp @@ -682,10 +682,6 @@ TrexRpcCmdSetRxFeature::_run(const Json::Value ¶ms, Json::Value &result) { uint8_t port_id = parse_port(params, result); TrexStatelessPort *port = get_stateless_obj()->get_port_by_id(port_id); - if (!port->is_service_mode_on()) { - generate_execute_err(result, "rx_feature - available only under service mode"); - } - /* decide which feature is being set */ const std::string type = parse_choice(params, "type", {"queue", "server"}, result); @@ -707,6 +703,10 @@ TrexRpcCmdSetRxFeature::parse_queue_msg(const Json::Value &msg, TrexStatelessPor bool enabled = parse_bool(msg, "enabled", result); if (enabled) { + + if (!port->is_service_mode_on()) { + generate_execute_err(result, "setting RX queue is only available under service mode"); + } uint64_t size = parse_uint32(msg, "size", result); @@ -745,7 +745,7 @@ TrexRpcCmdGetRxQueuePkts::_run(const Json::Value ¶ms, Json::Value &result) { TrexStatelessPort *port = get_stateless_obj()->get_port_by_id(port_id); if (!port->is_service_mode_on()) { - generate_execute_err(result, "get_rx_queue_pkts - available only under service mode"); + generate_execute_err(result, "fetching RX queue packets is only available under service mode"); } -- cgit 1.2.3-korg