From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from dpdk.org (dpdk.org [92.243.14.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 9DBC4A0526;
	Thu,  9 Jul 2020 06:44:20 +0200 (CEST)
Received: from [92.243.14.124] (localhost [127.0.0.1])
	by dpdk.org (Postfix) with ESMTP id D885D1DA46;
	Thu,  9 Jul 2020 06:44:19 +0200 (CEST)
Received: from mail-il1-f196.google.com (mail-il1-f196.google.com
 [209.85.166.196]) by dpdk.org (Postfix) with ESMTP id 3801F1DA45
 for <dev@dpdk.org>; Thu,  9 Jul 2020 06:44:18 +0200 (CEST)
Received: by mail-il1-f196.google.com with SMTP id x9so945726ila.3
 for <dev@dpdk.org>; Wed, 08 Jul 2020 21:44:18 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;
 h=mime-version:references:in-reply-to:from:date:message-id:subject:to
 :cc; bh=R94dWFWcLfGzGGvynjHH+mvMxgA2bZF7cNqxdQSaa9U=;
 b=aDnkReC7LY2bkxIx3D+1ZPHmwkIwvy0tuoBu8+nAba2fePJNaooMnIBJQFx5N7zkmN
 6pqAQgI+XnbQJZkDyM8bqQEiV6C7p1HD22GvXFy6FLohxnFJxzg8aVLs5DvECAI8WtYi
 oadnWF2eGHhTd6g15/qEht9+few2kfd2L9PGX+0FPWfxmFlU40be3yme0N6Ni8OW+ACw
 dEIo1unLjT79Mw/HAT2Z5gREABxpped5PzGCTiU8mKooPyXAtwO6hWSImQx17OCNR3up
 U3clLxCHXlQzXrIY/ct9WRQ2nsbwqhutQQTSwG7dBtciMijGXK91prbFfcgjA+KPV1ZV
 JGPg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20161025;
 h=x-gm-message-state:mime-version:references:in-reply-to:from:date
 :message-id:subject:to:cc;
 bh=R94dWFWcLfGzGGvynjHH+mvMxgA2bZF7cNqxdQSaa9U=;
 b=W6AHwFQj0C4aV48K5wmHCpzm+MpJ3xcILetClnmZ4+ZOSWZpBm2lpkRfHujZJhWgdc
 i/oTbzt5co9/kc0HvY42FnZn6evNk24jhPOexUxj8e2AQqZbC+pidGc/iJhWPMf8VdS0
 GGCrRsPFQsQAqgjJ15POgLCl88F4n4YWP6NsTaeXXdlxBYw3cuvXVyjeDBfWAXjLonfc
 ZNwAf79/CF0t+gXRcr4ta92GJhWsiMgVe4/SzmZ6XYiSedbr4p0rVQn5U0J54Q3yu0Us
 t+3ANzj4Sv6bZmCUvRjWu5ba4HrfpeIazHgR8/tBMUOU8jaazqtk2hkIcx9giwPNSwU7
 BKwg==
X-Gm-Message-State: AOAM532ebqCcPiGhvbKi8T/IMKXrNP2lr/rSn7crbKU7fE5xeHM1GS0h
 hXd6ksSTr6PwSYW7wHMXdUGCEG2jE92+rWn/XEA=
X-Google-Smtp-Source: ABdhPJw3BrZivZdYm+QEalriLCo5hSUfiSAqlngkr9PUOG2c2/vvI5fBweilwkOD013RWTU2LRtiXbkucG36PSdiscs=
X-Received: by 2002:a92:d206:: with SMTP id y6mr24196786ily.162.1594269857422; 
 Wed, 08 Jul 2020 21:44:17 -0700 (PDT)
MIME-Version: 1.0
References: <20200702120511.16315-1-andreyv@mellanox.com>
 <20200708213946.30108-1-andreyv@mellanox.com>
 <20200708213946.30108-7-andreyv@mellanox.com>
In-Reply-To: <20200708213946.30108-7-andreyv@mellanox.com>
From: Jerin Jacob <jerinjacobk@gmail.com>
Date: Thu, 9 Jul 2020 10:14:01 +0530
Message-ID: <CALBAE1NQHPheit9ZN9-bC69nmkwTxAwjRWHL9ZnXi=dTKH8Z3w@mail.gmail.com>
To: Andrey Vesnovaty <andreyv@mellanox.com>
Cc: dpdk-dev <dev@dpdk.org>, jer@marvell.com,
 Thomas Monjalon <thomas@monjalon.net>, 
 Ferruh Yigit <ferruh.yigit@intel.com>,
 Stephen Hemminger <stephen@networkplumber.org>, 
 "Richardson, Bruce" <bruce.richardson@intel.com>, Ori Kam <orika@mellanox.com>,
 Slava Ovsiienko <viacheslavo@mellanox.com>,
 Andrey Vesnovaty <andrey.vesnovaty@gmail.com>, 
 Marko Kovacevic <marko.kovacevic@intel.com>,
 Radu Nicolau <radu.nicolau@intel.com>, 
 Akhil Goyal <akhil.goyal@nxp.com>, Tomasz Kantecki <tomasz.kantecki@intel.com>,
 Sunil Kumar Kori <skori@marvell.com>,
 Pavan Nikhilesh <pbhagavatula@marvell.com>, 
 John McNamara <john.mcnamara@intel.com>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [dpdk-dev] [PATCH v2 6/6] examples/flow_filtering: utilize
	shared RSS action
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On Thu, Jul 9, 2020 at 3:09 AM Andrey Vesnovaty <andreyv@mellanox.com> wrote:
>
> This commit gives very first shared RSS action usage example and
> demonstrates shared action capability for in-place update.
>
> First application creates shared action during initialization phase.

What if PMD does not support a shared context, Then this application
fails to run?
if so, have a mode or probe the capability(if capability present then
switches to new mode else fall back to old mode) before changing the
application behavior.


> Later on the flow object created by application uses previously created
> shared RSS action with 1 queue configured instead of queue action in
> original application.
>
> On each RX queue burst shared RSS action reconfigured via
> rte_flow_shared_action_update() API to switch queue 0 to 1 & 1 to 0.
> User supposed to observe consistent queue switches on each packet burst.
>
> Signed-off-by: Andrey Vesnovaty <andreyv@mellanox.com>
> ---
>  doc/guides/sample_app_ug/flow_filtering.rst | 62 +++++++++++++++++----
>  examples/flow_filtering/flow_blocks.c       | 30 +++++-----
>  examples/flow_filtering/main.c              | 41 +++++++++++++-
>  3 files changed, 105 insertions(+), 28 deletions(-)
>
> diff --git a/doc/guides/sample_app_ug/flow_filtering.rst b/doc/guides/sample_app_ug/flow_filtering.rst
> index 5e5a6cd8a0..cfe9334717 100644
> --- a/doc/guides/sample_app_ug/flow_filtering.rst
> +++ b/doc/guides/sample_app_ug/flow_filtering.rst
> @@ -106,7 +106,7 @@ following code:
>  .. code-block:: c
>
>     /* create flow for send packet with */
> -   flow = generate_ipv4_flow(port_id, selected_queue,
> +   flow = generate_ipv4_flow(port_id, shared_action,
>                                  SRC_IP, EMPTY_MASK,
>                                  DEST_IP, FULL_MASK, &error);
>     if (!flow) {
> @@ -242,7 +242,7 @@ The Ethernet port is configured with default settings using the
>     rxq_conf = dev_info.default_rxconf;
>     rxq_conf.offloads = port_conf.rxmode.offloads;
>
> -For this example we are configuring number of rx and tx queues that are connected
> +For this example we are configuring 2 rx and 2 tx queues that are connected
>  to a single port.
>
>  .. code-block:: c
> @@ -270,13 +270,22 @@ to a single port.
>            }
>     }
>
> +Before we create the flow we create shared action in order to send it as
> +actions argument when creating a flow. The action is single queue RSS action
> +similar to action queue with the only difference that shared RSS action
> +provides update capability after action creation.
> +
> +.. code-block:: c
> +
> +   shared_action = rte_flow_shared_action_create(port_id, &action, &error);
> +
>  In the next step we create and apply the flow rule. which is to send packets
>  with destination ip equals to 192.168.1.1 to queue number 1. The detail
>  explanation of the ``generate_ipv4_flow()`` appears later in this document:
>
>  .. code-block:: c
>
> -   flow = generate_ipv4_flow(port_id, selected_queue,
> +   flow = generate_ipv4_flow(port_id, shared_action,
>                               SRC_IP, EMPTY_MASK,
>                               DEST_IP, FULL_MASK, &error);
>
> @@ -339,6 +348,21 @@ looks like the following:
>                                             printf("\n");
>                                             rte_pktmbuf_free(m);
>                                     }
> +                                   if (rss_queue[0] == 0) {
> +                                           printf(">>> switching queue 0 -> 1\n");
> +                                           rss_queue[0] = 1;
> +                                   } else {
> +                                           printf(">>> switching queue 1 -> 0\n");
> +                                           rss_queue[0] = 0;
> +                                   }
> +                                   ret = rte_flow_shared_action_update
> +                                           (port_id, shared_action, &action,
> +                                            &error);
> +                                   if (ret)
> +                                           rte_exit(EXIT_FAILURE,
> +                                                    ":: error: RSS action update "
> +                                                    "failed: %s\n",
> +                                                    rte_strerror(-ret));
>                             }
>                     }
>             }
> @@ -348,6 +372,8 @@ looks like the following:
>             rte_eth_dev_close(port_id);
>     }
>
> +On each loop eteration Rx queue switched using
> +``rte_flow_shared_action_update()`` API.
>  The main work of the application is reading the packets from all
>  queues and printing for each packet the destination queue:
>
> @@ -365,6 +391,21 @@ queues and printing for each packet the destination queue:
>                               printf(" - queue=0x%x", (unsigned int)i);
>                               printf("\n");
>                               rte_pktmbuf_free(m);
> +                             if (rss_queue[0] == 0) {
> +                                     printf(">>> switching queue 0 -> 1\n");
> +                                     rss_queue[0] = 1;
> +                             } else {
> +                                     printf(">>> switching queue 1 -> 0\n");
> +                                     rss_queue[0] = 0;
> +                             }
> +                             ret = rte_flow_shared_action_update
> +                                     (port_id, shared_action, &action,
> +                                      &error);
> +                             if (ret)
> +                                     rte_exit(EXIT_FAILURE,
> +                                              ":: error: RSS action update "
> +                                              "failed: %s\n",
> +                                              rte_strerror(-ret));
>                          }
>                  }
>             }
> @@ -378,13 +419,15 @@ The forwarding loop can be interrupted and the application closed using
>  The generate_ipv4_flow function
>  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> +
>  The generate_ipv4_flow function is responsible for creating the flow rule.
>  This function is located in the ``flow_blocks.c`` file.
>
>  .. code-block:: c
>
>     static struct rte_flow *
> -   generate_ipv4_flow(uint8_t port_id, uint16_t rx_q,
> +   generate_ipv4_flow(uint8_t port_id,
> +                   cstructrte_flow_shared_action *shared_action,
>                     uint32_t src_ip, uint32_t src_mask,
>                     uint32_t dest_ip, uint32_t dest_mask,
>                     struct rte_flow_error *error)
> @@ -393,7 +436,6 @@ This function is located in the ``flow_blocks.c`` file.
>             struct rte_flow_item pattern[MAX_PATTERN_NUM];
>             struct rte_flow_action action[MAX_ACTION_NUM];
>             struct rte_flow *flow = NULL;
> -           struct rte_flow_action_queue queue = { .index = rx_q };
>             struct rte_flow_item_ipv4 ip_spec;
>             struct rte_flow_item_ipv4 ip_mask;
>
> @@ -411,8 +453,8 @@ This function is located in the ``flow_blocks.c`` file.
>              * create the action sequence.
>              * one action only,  move packet to queue
>              */
> -           action[0].type = RTE_FLOW_ACTION_TYPE_QUEUE;
> -           action[0].conf = &queue;
> +           action[0].type = RTE_FLOW_ACTION_TYPE_SHARED;
> +           action[0].conf = shared_action;
>             action[1].type = RTE_FLOW_ACTION_TYPE_END;
>
>             /*
> @@ -468,12 +510,12 @@ The following part create the flow attributes, in our case ingress.
>     attr.ingress = 1;
>
>  The third part defines the action to be taken when a packet matches
> -the rule. In this case send the packet to queue.
> +the rule. In this case send the packet to single RSS queue.
>
>  .. code-block:: c
>
> -   action[0].type = RTE_FLOW_ACTION_TYPE_QUEUE;
> -   action[0].conf = &queue;
> +   action[0].type = RTE_FLOW_ACTION_TYPE_SHARED;
> +   action[0].conf = shared_action;
>     action[1].type = RTE_FLOW_ACTION_TYPE_END;
>
>  The fourth part is responsible for creating the pattern and is built from
> diff --git a/examples/flow_filtering/flow_blocks.c b/examples/flow_filtering/flow_blocks.c
> index 575d792810..99bfed3172 100644
> --- a/examples/flow_filtering/flow_blocks.c
> +++ b/examples/flow_filtering/flow_blocks.c
> @@ -6,11 +6,11 @@
>  #define MAX_ACTION_NUM         2
>
>  struct rte_flow *
> -generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
> -               uint32_t src_ip, uint32_t src_mask,
> -               uint32_t dest_ip, uint32_t dest_mask,
> -               struct rte_flow_error *error);
> -
> +generate_ipv4_flow(uint16_t port_id,
> +                  struct rte_flow_shared_action *shared_action,
> +                  uint32_t src_ip, uint32_t src_mask,
> +                  uint32_t dest_ip, uint32_t dest_mask,
> +                  struct rte_flow_error *error);
>
>  /**
>   * create a flow rule that sends packets with matching src and dest ip
> @@ -18,8 +18,8 @@ generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
>   *
>   * @param port_id
>   *   The selected port.
> - * @param rx_q
> - *   The selected target queue.
> + * @param shared_action
> + *   The shared RSS action with single queue
>   * @param src_ip
>   *   The src ip value to match the input packet.
>   * @param src_mask
> @@ -35,16 +35,16 @@ generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
>   *   A flow if the rule could be created else return NULL.
>   */
>  struct rte_flow *
> -generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
> -               uint32_t src_ip, uint32_t src_mask,
> -               uint32_t dest_ip, uint32_t dest_mask,
> -               struct rte_flow_error *error)
> +generate_ipv4_flow(uint16_t port_id,
> +                  struct rte_flow_shared_action *shared_action,
> +                  uint32_t src_ip, uint32_t src_mask,
> +                  uint32_t dest_ip, uint32_t dest_mask,
> +                  struct rte_flow_error *error)
>  {
>         struct rte_flow_attr attr;
>         struct rte_flow_item pattern[MAX_PATTERN_NUM];
>         struct rte_flow_action action[MAX_ACTION_NUM];
>         struct rte_flow *flow = NULL;
> -       struct rte_flow_action_queue queue = { .index = rx_q };
>         struct rte_flow_item_ipv4 ip_spec;
>         struct rte_flow_item_ipv4 ip_mask;
>         int res;
> @@ -61,10 +61,10 @@ generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
>
>         /*
>          * create the action sequence.
> -        * one action only,  move packet to queue
> +        * one action only,  move packet to shared RSS queue
>          */
> -       action[0].type = RTE_FLOW_ACTION_TYPE_QUEUE;
> -       action[0].conf = &queue;
> +       action[0].type = RTE_FLOW_ACTION_TYPE_SHARED;
> +       action[0].conf = shared_action;
>         action[1].type = RTE_FLOW_ACTION_TYPE_END;
>
>         /*
> diff --git a/examples/flow_filtering/main.c b/examples/flow_filtering/main.c
> index cc9e7e7808..d6b18d95fc 100644
> --- a/examples/flow_filtering/main.c
> +++ b/examples/flow_filtering/main.c
> @@ -32,8 +32,7 @@
>  static volatile bool force_quit;
>
>  static uint16_t port_id;
> -static uint16_t nr_queues = 5;
> -static uint8_t selected_queue = 1;
> +static uint16_t nr_queues = 2;
>  struct rte_mempool *mbuf_pool;
>  struct rte_flow *flow;
>
> @@ -42,6 +41,24 @@ struct rte_flow *flow;
>  #define FULL_MASK 0xffffffff /* full mask */
>  #define EMPTY_MASK 0x0 /* empty mask */
>
> +struct rte_flow_shared_action *shared_action;
> +uint16_t rss_queue[1] = {0};
> +
> +struct rte_flow_action_rss action_rss = {
> +               .func = RTE_ETH_HASH_FUNCTION_DEFAULT,
> +               .level = 0,
> +               .types = 0,
> +               .key_len = 0,
> +               .key = NULL,
> +               .queue = rss_queue,
> +               .queue_num = 1,
> +};
> +
> +struct rte_flow_action action = {
> +       .type = RTE_FLOW_ACTION_TYPE_RSS,
> +       .conf = &action_rss,
> +};
> +
>  #include "flow_blocks.c"
>
>  static inline void
> @@ -61,6 +78,7 @@ main_loop(void)
>         uint16_t nb_rx;
>         uint16_t i;
>         uint16_t j;
> +       int ret;
>
>         while (!force_quit) {
>                 for (i = 0; i < nr_queues; i++) {
> @@ -82,6 +100,21 @@ main_loop(void)
>
>                                         rte_pktmbuf_free(m);
>                                 }
> +                               if (rss_queue[0] == 0) {
> +                                       printf(">>> switching queue 0 -> 1\n");
> +                                       rss_queue[0] = 1;
> +                               } else {
> +                                       printf(">>> switching queue 1 -> 0\n");
> +                                       rss_queue[0] = 0;
> +                               }
> +                               ret = rte_flow_shared_action_update
> +                                       (port_id, shared_action, &action,
> +                                        &error);
> +                               if (ret)
> +                                       rte_exit(EXIT_FAILURE,
> +                                                ":: error: RSS action update "
> +                                                "failed: %s\n",
> +                                                rte_strerror(-ret));
>                         }
>                 }
>         }
> @@ -243,8 +276,10 @@ main(int argc, char **argv)
>
>         init_port();
>
> +       shared_action = rte_flow_shared_action_create(port_id, &action, &error);
> +
>         /* create flow for send packet with */
> -       flow = generate_ipv4_flow(port_id, selected_queue,
> +       flow = generate_ipv4_flow(port_id, shared_action,
>                                 SRC_IP, EMPTY_MASK,
>                                 DEST_IP, FULL_MASK, &error);
>         if (!flow) {
> --
> 2.26.2
>