From patchwork Sun Feb 11 13:29:04 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Sheehan,Georgina" X-Patchwork-Id: 50272 X-Patchwork-Delegate: cristian.dumitrescu@intel.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 610591B39B; Tue, 12 Feb 2019 15:03:16 +0100 (CET) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 05AC21B3A2 for ; Tue, 12 Feb 2019 15:03:13 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Feb 2019 06:03:12 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.58,362,1544515200"; d="scan'208";a="274432477" Received: from silpixa00399837.ir.intel.com (HELO silpixa00399837.ger.corp.intel.com) ([10.237.223.162]) by orsmga004.jf.intel.com with ESMTP; 12 Feb 2019 06:03:12 -0800 From: "Sheehan,Georgina" To: dev@dpdk.org Cc: cristian.dumitrescu@intel.com, Georgina Sheehan Date: Sun, 11 Feb 2018 13:29:04 +0000 Message-Id: <20180211132905.7502-2-georgina.sheehan@intel.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20180211132905.7502-1-georgina.sheehan@intel.com> References: <20180211085722.59717-1-georgina.sheehan@intel.com> <20180211132905.7502-1-georgina.sheehan@intel.com> Subject: [dpdk-dev] [PATCH v2 2/3] pipeline: add implementation for DSCP action X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Georgina Sheehan This allows the application to change the DSCP value of incoming packets v2: Added in call of function parse_table_action_dscp in softnic cli file Signed-off-by: Georgina Sheehan --- examples/ip_pipeline/action.c | 11 +++++++ examples/ip_pipeline/action.h | 1 + examples/ip_pipeline/cli.c | 52 ++++++++++++++++++++++++++++++++- examples/ip_pipeline/pipeline.h | 1 + examples/ip_pipeline/thread.c | 10 +++++++ 5 files changed, 74 insertions(+), 1 deletion(-) diff --git a/examples/ip_pipeline/action.c b/examples/ip_pipeline/action.c index d2104aad6..625c870a2 100644 --- a/examples/ip_pipeline/action.c +++ b/examples/ip_pipeline/action.c @@ -366,6 +366,17 @@ table_action_profile_create(const char *name, } } + if (params->action_mask & (1LLU << RTE_TABLE_ACTION_DSCP)) { + status = rte_table_action_profile_action_register(ap, + RTE_TABLE_ACTION_DSCP, + ¶ms->dscp); + + if (status) { + rte_table_action_profile_free(ap); + return NULL; + } + } + status = rte_table_action_profile_freeze(ap); if (status) { rte_table_action_profile_free(ap); diff --git a/examples/ip_pipeline/action.h b/examples/ip_pipeline/action.h index cde17e69a..ea8f81c56 100644 --- a/examples/ip_pipeline/action.h +++ b/examples/ip_pipeline/action.h @@ -54,6 +54,7 @@ struct table_action_profile_params { struct rte_table_action_ttl_config ttl; struct rte_table_action_stats_config stats; struct rte_table_action_sym_crypto_config sym_crypto; + struct rte_table_action_dscp_config dscp; }; struct table_action_profile { diff --git a/examples/ip_pipeline/cli.c b/examples/ip_pipeline/cli.c index a92467e63..ca3083a48 100644 --- a/examples/ip_pipeline/cli.c +++ b/examples/ip_pipeline/cli.c @@ -1045,7 +1045,8 @@ static const char cmd_table_action_profile_help[] = " [time]\n" " [sym_crypto dev offset ]\n" " [tag]\n" -" [decap]\n"; +" [decap]\n" +" [dscp]\n"; static void cmd_table_action_profile(char **tokens, @@ -1458,6 +1459,17 @@ cmd_table_action_profile(char **tokens, t0 += 1; } /* decap */ + if ((t0 < n_tokens) && (strcmp(tokens[t0], "dscp") == 0)) { + if (n_tokens < t0 + 1) { + snprintf(out, out_size, MSG_ARG_MISMATCH, + "table action profile dscp"); + return; + } + + p.action_mask |= 1LLU << RTE_TABLE_ACTION_DSCP; + t0 += 1; + } /** DSCP **/ + if (t0 < n_tokens) { snprintf(out, out_size, MSG_ARG_MISMATCH, tokens[0]); return; @@ -3116,6 +3128,7 @@ parse_match(char **tokens, * data_offset ] * [tag ] * [decap ] + * [dscp ] * * where: * ::= g | y | r | drop @@ -4108,6 +4121,26 @@ parse_table_action_decap(char **tokens, return 2; } +static uint32_t +parse_table_action_dscp(char **tokens, + uint32_t n_tokens, + struct table_rule_action *a) +{ + if ((n_tokens < 2) || + strcmp(tokens[0], "dscp")) + return 0; + + uint8_t dscp_val; + + if (parser_read_uint8(&dscp_val, tokens[1])) + return 0; + + a->dscp.dscp_val = dscp_val; + + a->action_mask |= 1 << RTE_TABLE_ACTION_DSCP; + return 2; +} + static uint32_t parse_table_action(char **tokens, uint32_t n_tokens, @@ -4293,6 +4326,20 @@ parse_table_action(char **tokens, n_tokens -= n; } + if (n_tokens && (strcmp(tokens[0], "dscp") == 0)) { + uint32_t n; + + n = parse_table_action_dscp(tokens, n_tokens, a); + if (n == 0) { + snprintf(out, out_size, MSG_ARG_INVALID, + "action dscp"); + return 0; + } + + tokens += n; + n_tokens -= n; + } + if (n_tokens0 - n_tokens == 1) { snprintf(out, out_size, MSG_ARG_INVALID, "action"); return 0; @@ -5056,6 +5103,9 @@ table_rule_show(const char *pipeline_name, if (a->action_mask & (1LLU << RTE_TABLE_ACTION_DECAP)) fprintf(f, "decap %u ", a->decap.n); + if (a->action_mask & (1LLU << RTE_TABLE_ACTION_DSCP)) + fprintf(f, "dscp "); + /* end */ fprintf(f, "\n"); } diff --git a/examples/ip_pipeline/pipeline.h b/examples/ip_pipeline/pipeline.h index 278775c2d..670174ed4 100644 --- a/examples/ip_pipeline/pipeline.h +++ b/examples/ip_pipeline/pipeline.h @@ -290,6 +290,7 @@ struct table_rule_action { struct rte_table_action_sym_crypto_params sym_crypto; struct rte_table_action_tag_params tag; struct rte_table_action_decap_params decap; + struct rte_table_action_dscp_params dscp; }; struct table_rule { diff --git a/examples/ip_pipeline/thread.c b/examples/ip_pipeline/thread.c index 272fbbeed..c9facb53e 100644 --- a/examples/ip_pipeline/thread.c +++ b/examples/ip_pipeline/thread.c @@ -2751,6 +2751,16 @@ action_convert(struct rte_table_action *a, return status; } + if (action->action_mask & (1LLU << RTE_TABLE_ACTION_DSCP)) { + status = rte_table_action_apply(a, + data, + RTE_TABLE_ACTION_DSCP, + &action->dscp); + + if (status) + return status; + } + return 0; }