From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM01-SN1-obe.outbound.protection.outlook.com (mail-sn1nam01on0047.outbound.protection.outlook.com [104.47.32.47]) by dpdk.org (Postfix) with ESMTP id 63B5E1B1CC for ; Wed, 1 Aug 2018 19:26:55 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Ll8SEwP4eQG79nUKxzyz7aoCs+rWsFjO8/PMO+YsKdg=; b=Zl/I8yFAwE4ZSbOQy8Rs95FZ3kshZo2SYdsXoAzI6SLdTRukMJurcEWGBN13oBd7IHbEQ36SX1C6gCeP5+to81Q+DMXNubxC06wRdTx04fhlr/va2xHCe3Hsuw9uM+0WLHFsP51X/Uw4135jtjv3EB+5AwcwErJE6OjWohNVjXo= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.JacobKollanukkaran@cavium.com; Received: from jerin (106.201.44.195) by BYAPR07MB4998.namprd07.prod.outlook.com (2603:10b6:a03:5b::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1017.15; Wed, 1 Aug 2018 17:26:46 +0000 Date: Wed, 1 Aug 2018 22:56:30 +0530 From: Jerin Jacob To: Stephen Hemminger Cc: "Joseph, Anoob" , Thomas Monjalon , dev@dpdk.org, Bruce Richardson , Pablo de Lara , Narayana Prasad , Hemant Agrawal , "Ananyev, Konstantin" , Sunil Kumar Kori , Nikhil Rao Message-ID: <20180801172628.GA471@jerin> References: <1528976946-14396-1-git-send-email-anoob.joseph@caviumnetworks.com> <1531289248-20025-1-git-send-email-anoob.joseph@caviumnetworks.com> <3685021.sWt9K18E1B@xps> <20180801095414.7bc34b30@xeon-e3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180801095414.7bc34b30@xeon-e3> User-Agent: Mutt/1.10.1 (2018-07-13) X-Originating-IP: [106.201.44.195] X-ClientProxiedBy: PN1PR0101CA0064.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00:d::26) To BYAPR07MB4998.namprd07.prod.outlook.com (2603:10b6:a03:5b::23) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 254eabcc-8ef5-4a99-f4eb-08d5f7d3f89c X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(7020095)(4652040)(8989117)(4534165)(7168020)(4627221)(201703031133081)(201702281549075)(8990107)(5600074)(711020)(2017052603328)(7153060)(7193020); SRVR:BYAPR07MB4998; X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4998; 3:7i0m35qDLw4HAOqH5i4acVXeC+Vq/IAKzA4EJm/PXBxA5wRlySHR3RQkHBN6zyWaBPncig1OlI6N6/SiDcvUOw7E9EXUTcPM67umx1a34s5B/cqa95VBopW2ACYZoOpU7qnN1d+NEraVkyKzHf1zMTRlJhE8NamSbS+8Ez6bYnm+8Y9OO8W2Vfn4t64FNZqbGiBmcqh/aYLKPWuhNgT81oN8EzqjXtpphJzsYheV6meMwKAyzrL2YEyFkImJDbF9; 25:3qsrfLk8xPksHEMBgFT2CoCLOT2kvkuQ3anEkmzpkkvkgN2b34zpPC57IiVpjIjMSZfbO3eyoIXFuS7yFGQ1sNhdjTpom+PYS+ZeXn/6YosknHLZaB3xTFDjhx/pM5D3zD/ps5uZVt1Cl6l9MGYIClSRecaA+Q/Hmq9/RlHvf9fTeWH4M4dCRrAsVaxUi/DsXVlbikLK1rWciwaJOEj6cc+2OfZC6goM6HkB+khqJFMh64q+kblRjawWzhjAZ6aYgp8g2frPiJV0Jty6iLAUHn3tz0+/djU51bh8bBM4k/vGtLUVI0G1yVGnzsv/siKRC41x5c8SyHpYNLwf/C8GHw==; 31:+w6ws3hI4aj8lp8M3/UlYHF2jqymM2BbgFzuPvhHnUMdNh3EGjFnpZI0zHxDzEnZulLNP7U/lGqu7QrF8SnEzvZh4eSo6SABn/XISC1V6XepCQUoYpQsZiXCb+WPzvPU+g5yuHX5Uzxbmr89L9rURtg9NGXrR0wqJbu3Rh+iBjWNGxB7YPMqsOZUSQV0XyQV9L5/xQvICdLdIEaM/MCZDn8UpgDvZs+WgQyIBxTv6yE= X-MS-TrafficTypeDiagnostic: BYAPR07MB4998: X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4998; 20:QSfOfMzMYxZ5BYVr9Jbg3tdDhWY+dLZUw09QAzQcUnSaKb1zOZcI8ufo8cxvuKWHXi/bsNBnAqqLy9/6ABhQpMlovID456+DbVZ9Gl5eCwRc7YfQT9jktQF9TfSZ7cuvpLU0ZkL7rO7i1tghHFH6NNL+YcZlS81MVqQAFUuPgSPA6JzZZdJq8DEb3aGskADyw97gRdf2zGyuJx4iedt2aCtucatJTHa2pnh21dM+7Qsac/7yEsFv+gGJUMY5rJ+2pM18S50gEqlSJeaofrw+FTTpBa3YpIZWz1793LfTsLm0WHWvNTSGMZi/E3Y2qfR6cQWJEYC/lpHvqJ9HeP9lNd4iyUtrVC2QCUYOpXsjQln3IyRWH1EnGMYj7oCDFHdWB9Bkp+16JayIV0f+HcQI2M+88vU1iw1MEyqD/81b1TZlUdl8GxTdVS99hZ+swXH9q6xTkZCt3GEY+LeCP+TqkTASCVTgsKunuk1Qg0/7UlS5FDbYhQCtQfGFuf5F9Ue9V5sNxNc20QlgwoyJqWAcMzeh/OsSCRh8+6dZYqnT0vdBYWt5A3LduLAHzAOZERL8PRx5aaIQll/nP99HXqMoDvZrPoDYhqbeyaQ+n0Y7JgM= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(278428928389397)(185117386973197)(228905959029699); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(3002001)(93006095)(10201501046)(3231311)(944501410)(52105095)(149027)(150027)(6041310)(20161123560045)(20161123564045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123562045)(6072148)(201708071742011)(7699016); SRVR:BYAPR07MB4998; BCL:0; PCL:0; RULEID:; SRVR:BYAPR07MB4998; X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4998; 4:8UDjYuyr4VE5DwPfqmEH6isQNiWzKsEfBOl7wpXDMkQSJ3ju8EDNisyIIJlU+5QYJAY9rBTs2RAQIW5nBCR3o/3PE+sRzy7L+D0UrpzyZ92W5schRYMHkMsdDZsVm0m9S+FFJp/f3EgdKDdoH+2rfnOgin4iiB8reEK0mccu6enASgeesG+j6HwwHEaNv7uYEzQtDngSPUPyXpSw91QMuRtucNQL6nHT8SaPklmIew2discgWvmpCjpghYWGiwIXvcnBdc8jdmuAGSDC1BnzFPspIFJtYEhG3DiQdOaZrpV8NPAYq99S3GMTy6OyMvzQR9Te4zPAgRjIm0PIYJW/hsBxxsWKRTtgpev/rvYlV74wRJHhKBBy97QrJWSxfO8k X-Forefront-PRVS: 0751474A44 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(376002)(136003)(39850400004)(396003)(346002)(366004)(52164004)(189003)(199004)(13464003)(16526019)(966005)(26005)(446003)(14444005)(476003)(9686003)(6306002)(55016002)(72206003)(8676002)(81156014)(33896004)(386003)(53546011)(6246003)(33656002)(55236004)(44832011)(5009440100003)(956004)(11346002)(50466002)(76176011)(486006)(53936002)(81166006)(186003)(58126008)(42882007)(54906003)(47776003)(7736002)(97736004)(6496006)(68736007)(316002)(4326008)(52116002)(305945005)(2906002)(25786009)(106356001)(478600001)(229853002)(1076002)(66066001)(5660300001)(23726003)(105586002)(6666003)(33716001)(16586007)(6916009)(93886005)(6116002)(8936002)(3846002)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:BYAPR07MB4998; H:jerin; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BYAPR07MB4998; 23:KAzlmArie2B+no57DfhRlHKPtVyq4XKyQcNyisBuo?= =?us-ascii?Q?O42xnUh6Zia0K5UCx+xM0UgDFvUzvpiUCO+y47gYTHMOOzohTMtg3wlQN1FV?= =?us-ascii?Q?guzoSMfkw4SIyu/1V/ZYnbDsyqvrkewVWqZ5jq40d2BqL1OOzlfPW1V1F0dX?= =?us-ascii?Q?zW/MZhHVsekPHbC31MSGNwnQr5/c9vLU9/l5TGbcXZyzwJHjtX/lPX6eZGTR?= =?us-ascii?Q?cwhfMc0GOI8uSyIno2sPLNaCZZqfKli5duRO2YJG4vxZUFq/fjwwIHttSHwp?= =?us-ascii?Q?PqGeMdN2zwRoYmtfjEIY8YfQB5sNsKCxIeFSYFYalMllD2pqxdMtDRWELk0m?= =?us-ascii?Q?HCYIJSj7PnBsE0PCX3A8E3lHvuOIybKb4ETSn5bkHBLq4cYEd87STMapK1PP?= =?us-ascii?Q?RVjhKLMM32fEeFX67Lvh8tXdaW3lJDSkkrkze9I/y0Dow+LxknBLShy2hCWY?= =?us-ascii?Q?t72R0VaLAbvWby1iV6eEBavljcfleaABUbbf1LjnaTwSoHE7tMIVxS+5awn3?= =?us-ascii?Q?NcRea/fonL2acAAoq86VcjpIHLdsMIa5EUOPYiNpybdCUG9agTrOhjWZ6RjB?= =?us-ascii?Q?i6SrhC0hbxG32tJYxbfvbVe5HA24mC+KS1Wv3+pfC4SiQ46dowu7lxY14IhA?= =?us-ascii?Q?yQpkVHzxHmLt8R/UBK6NZtgy42dWevgpIjE/DlM2C4SYEwe2rqOlUrMKwfqu?= =?us-ascii?Q?awpCBqrd5Wuo30sNLegPv9dgbn94Ug26uj5tvEbAs5aQF1f+x0L8p332Z1Fw?= =?us-ascii?Q?elHRWRn17mDngCjUD549kkKhIYhFf4G/yj0xIcpeZZ3qKMy9n5PY8Ek9mdwk?= =?us-ascii?Q?pikv79tOLXhCb8ZOBbohjegXLPBA/DyBskZMNRXCiQOaN/286ZQxrp9cHEku?= =?us-ascii?Q?3KDYGQbi+TwWcdUiCr2BGFne+D8+Bu+76P1vixNQi1wKeRKpc6HXaxiZjCW/?= =?us-ascii?Q?qKoCr2wXm0cV7o8GclVwm8TMy1tDqXRbsGhP1T4NJPDRoUOJzfGKw19ScwcK?= =?us-ascii?Q?SdXpo2kEvUKeFwTMgLFNJUEu3++GdwNjYVpsZc2vxPOpX1aQN6/SkooF45AN?= =?us-ascii?Q?6C6/ltzVRlDCBeTlHLeQR68z2j7mWABSVvDXh4d+fcy7NlCT77WHxiduAJL3?= =?us-ascii?Q?9bllh+DzwqZFd9lMRajOcV90rxrQx/gSD7t/ZtPG3nz5p/S1hAuskPQsGt/v?= =?us-ascii?Q?D56BRQd0bhuRgi43MP3Dys8ZC8qTA17dwUu3iBkjWXFkUqwq4KhwYlL+5BHT?= =?us-ascii?Q?caU/BH+MmWsZYjHMTPSaxC0uh4Jo1II8QoAIes/9P1CHV8j91x8znK5g3SwM?= =?us-ascii?Q?aWpEUU5z9YtTDmokiERNpQdEbVzSpZwzz8GF/OgcVx1yN6IerHwQEijeMzkT?= =?us-ascii?Q?K5+Ds4a0Vk6xqg7N1+y/WAohUsdIwM+LZg8uAcFyON1SdHWXSrwIfcPn7K/b?= =?us-ascii?Q?sCQZhdK3w2/Q5fbpY8VInNkbWHS6dQhigucJKD4lOhtrAQwnv1jiVFzLn7w+?= =?us-ascii?Q?JllGgHSxOezOY+Tm5RnY/wmfyQWHKd8/2M=3D?= X-Microsoft-Antispam-Message-Info: 9bSrT6sjEwELLwN4gnFa+leYeu4AxGvWwTea07w3M2JAoLKh9GwSAJ606uD33ev5V/kV5iLLmsWJFrVtZxIqERz25n34KdPbnnSr5zVA7bDYw18uTjn1QqyvkZLrL0un8y0zvO9M7nRNdM4vm1CNwiz6yjo9QjLZFPXCXpKyLz1BYaxHEKijm6PI/ruMIyqGYpcjdRoelvPwVRlTaeMNgxzHf6Qmn1ut47JkuiAXBhop8AmfUnHCXF/XxRbZNjnW6CPkHEDj67SfAPEpvv85DDvsS9sG8DGnkntquA4sGgDvBpi0khrB5Vyk5S3DJVu5HZlYFrJJc/7+UegoRRGkjHuDVXtFmmjuP2gZHrYoVqo= X-Microsoft-Exchange-Diagnostics: 1; BYAPR07MB4998; 6:RIrfZ3yt1WilZ0BRWGd1zrP10XzEDm0Cn02NLwOl8OQFsffmi23hW9vpTeULSkjZxsa9SkyQHp8daV8UIdHZwwLWRje6I6Wr4TBfdr3oMNalty8Ygeqm3qDBPECEsls2hvn9Y6uMuZ80HTlbhY/3U5qTpSthppBuWta2I1J7xUb7NbbiWvIEsyarnUu7mUmqStoPLeXvK9LX//++LnQAIGOVkycmjhrbVPxYKK2GWncDKmixp5KFA8Wb6bcPrABho7SriZ4i/8jbDuQkWyFdjQswECyiGRy+z99Agrs28uD6b4OxzaMbIvDG4FvLXcVx5OJj5QkVoZNuEjHObKuTAynEkKyXuQlOp14DVmjj+7wW6GB1dN5MzHAKd6FPcbGwXEH5wEcHhMv+s3kashxoLCUmZYZUdfB9k6mVeLg95DLU7Z9NtRstUSia44OLsgZ3SQOR/3DRwOxSuwRLYmFZXg==; 5:qz37v7eiELf3mjRgB1QTuDaNZHZSGMMC4aqFpk4IAjtjZQn88VMY5PhYMkFmDreVkM+kyiO0wG8YQwcnSQhRQH0afD+TZG0d9hS1gfzyetAjMd6yI+AGO0tBf0yKFB0BbWP9iwH1H7cm2NNaDcjfdC4lP1TqCrAtGldA41VksII=; 7:/OsaQkC/MkFqsqAIKwr4nBmrQ4W0cJ+Uu7TGledtJ9PPees4P+EFdJAgYGLCS7ThLf85GlgFK2wBKmIiGpuwLcCbABlcgd3f/WZbb8h0K73SNru3kO6gycTX06IV6OXO8JsB89fpLxgescLrA6LUYi7R3RZX7bfm3pEgo1mVGrnP22AvwTpf8+LngYR40prO/zXGKSHzaH60ladTbVBrN4qn8XHjMIS9Hg7570dJHeXwomHRGQ2/84iZKAVoX0sE SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Aug 2018 17:26:46.5198 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 254eabcc-8ef5-4a99-f4eb-08d5f7d3f89c X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR07MB4998 Subject: Re: [dpdk-dev] [PATCH v2 00/12] preparing l2fwd for eventmode additions 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: , X-List-Received-Date: Wed, 01 Aug 2018 17:26:56 -0000 -----Original Message----- > Date: Wed, 1 Aug 2018 09:54:14 -0700 > From: Stephen Hemminger > To: "Joseph, Anoob" > Cc: Thomas Monjalon , dev@dpdk.org, Bruce Richardson > , Pablo de Lara > , Jerin Jacob > , Narayana Prasad > , Hemant Agrawal > , "Ananyev, Konstantin" > , Sunil Kumar Kori , > Nikhil Rao > Subject: Re: [dpdk-dev] [PATCH v2 00/12] preparing l2fwd for eventmode > additions > > > On Wed, 1 Aug 2018 12:29:50 +0530 > "Joseph, Anoob" wrote: > > > Hi Thomas, > > > > On 26-07-2018 22:27, Thomas Monjalon wrote: > > > External Email > > > > > >> Anoob Joseph (12): > > >> examples/l2fwd: move macro definitions to common header > > >> examples/l2fwd: move structure definitions to common header > > >> examples/l2fwd: move globally accessed vars to common header > > >> examples/l2fwd: move dataplane code to new file > > >> examples/l2fwd: remove unused header includes > > >> examples/l2fwd: move drain buffers to new function > > >> examples/l2fwd: optimize check for master core > > >> examples/l2fwd: move periodic tasks to new function > > >> examples/l2fwd: skip timer updates for non master cores > > >> examples/l2fwd: move pkt send code to a new function > > >> examples/l2fwd: use fprint instead of printf for usage print > > >> examples/l2fwd: improvements to the usage print > > > Maintainers of this app look to be against adding complexity. > > > > > > In order to get this series accepted, we need more discussions > > > with more people involved. > > > So it will miss 18.08. > > > > > > It can be discussed in a more global discussion about examples maintenance. > > > If discussion does not happen, you can request it to the technical board. > > > > > Event dev framework and various adapters enable multiple packet handling > > schemes, as opposed to the traditional polling on queues. But these > > features are not integrated into any established example application. > > There are specific example applications for event dev etc, which can be > > used to analyze an event device or a particular eventdev adapter, but > > there is no standard application which can be used to compare the real > > world performance for a system when it's using event device for packet > > handling and when it's done via polling on queues. > > > > The following patch submitted by Sunil was looking to address this issue > > with l3fwd, > > https://mails.dpdk.org/archives/dev/2018-March/093131.html > > > > Bruce & Jerin reviewed the patch and suggested the addition of helper > > functions to abstract the event mode additions in applications, > > https://mails.dpdk.org/archives/dev/2018-April/096879.html > > > > This effort of adding helper functions for eventmode was taken up > > following the above suggestion. The idea is to add eventmode without > > touching the existing code path. All the eventmode specific additions > > would go into library so that these need not be repeated for every > > application. And since there is no change in the existing code path, > > performance for any vendor should not have any impact with the additions. > > > > The scope of this effort has increased since the submission, as now we > > have Tx adapter as well. Sunil & Konstantin had clarified their > > concerns, and gave green flag to this approach. > > https://mails.dpdk.org/archives/dev/2018-June/105730.html > > https://mails.dpdk.org/archives/dev/2018-July/106453.html > > > > I guess Bruce was opening this question to the community. For compute > > intense applications like ipsec-secgw, eventmode might be the right > > approach in the first place. Such complex applications would need a > > scheduler to perform dynamic load balancing. Addition of eventmode in > > l2fwd was to float around the idea which can then be scaled for more > > complex applications. > > > > If maintainers doesn't have any objection to this, I'm fine with adding > > this in the next release. > > > > Thanks, > > Anoob > > It is important that DPDK has good examples of how to use existing > frameworks and libraries. These applications are what most customers > build their applications from and they provide basis for testing. > > The DPDK needs to continue to support multiple usage models. This > is one of its strong points. I would rather leave existing l2fwd > and l3fwd alone and instead make new examples that use the frameworks. > If nothing else haveing l2fwd and l2fwd-eventdev would allow for > performance comparisons. Unlike other applications example, there wont be any change in packet processing functions in eventdev vs poll mode case. Only worker schematics will change and that can be moved to separated files. something like worker_poll.c and worker_event.c and both of them use common packet processing functions using mbuf. The only disadvantage of having separate application would be packet processing code duplication. Which is non trivial for l3fwd, IPSec application IMO. # Are we fine with code duplication in example application like l3fwd and IPSec? # if yes, Are we fine with keeping l2fwd _as is_ to reduce the complexity and l2fwd-eventdev supports both modes wherever possible? > > As the number of examples increases, probably also need to have > a roadmap or decision chart to explain the advangage/disadvantage > of each architecture. >