From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 11B1DA00C2; Tue, 27 Sep 2022 15:26:36 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B267241133; Tue, 27 Sep 2022 15:26:35 +0200 (CEST) Received: from mail-qk1-f182.google.com (mail-qk1-f182.google.com [209.85.222.182]) by mails.dpdk.org (Postfix) with ESMTP id 5025840694 for ; Tue, 27 Sep 2022 15:26:35 +0200 (CEST) Received: by mail-qk1-f182.google.com with SMTP id 3so5977937qka.5 for ; Tue, 27 Sep 2022 06:26:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date; bh=/1hzA6dQYfbdfh7omKeNJCPgKtqPmobte4lz3gTUWAo=; b=fEi+pcTnEyPqrnjo00I30kJWOBpliYsJZB43l99dibV/iPOPPuO4IF13GvHKvNo592 i3NRJ+DmSbtVybOhkP9MXDCYX4ViDws9KxJNTouOYqZPpP3TBLrQ2M1Kz20GK36mHMYV ndjNxxt3ILs5XCm44HaSV8WLFHSFXFbKkIx81+7ZqxQlDd78mOiIxzxmNQkzKzlNiOsA 8QBjmnaqTV1Sp0TpjHX5g5OlRByRFjGthg1pKyXF35KUcH3rbQgFWEqGPnaTmKYIt5+/ PZMx9LlP/oFIdzbFqWM1vQ+IFLhHPquZKnKvDhiFFceOObMoOeNxxoS8B3oIk7sz185z ID+g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date; bh=/1hzA6dQYfbdfh7omKeNJCPgKtqPmobte4lz3gTUWAo=; b=VzgeCytlO/Ek6wEG/M1pWYSI15503cywz3SLqzSQGOYNzydy3N5BReFHQoaqzkWTvj AQuLvjk1oHGvKVWQv69XbrPrPEFE3g9tyTHNlrJEsjembm3Rk7C2Uv+ty7lCR90y9m21 UCDeR+/6HJv001v2/IOsaP6EAkgK/wAHLdnALt5NSweqaBxfSHcmzxEBG2AlX/CF/zTZ oVhJxnMq1Osu6wqSfS2Jj8Zl902vJ8u4RUjmG0pYeK6chR+AeOvTIHKfptLo8e2c8xZM 0Of0y+J9skG77fuh8kty9UakqbkUdmppM1YJYjoL9LskOB9wJP0rO9uETVVUAgy48jtc VHtw== X-Gm-Message-State: ACrzQf0kctw8ZzRkoFA6JlZ1lSP2MsYC9coTBmWgEffccSalymU1K29K iwKOjb/GrtL+F7q5JquTkf2I1tfL9LLBr4ToWZI= X-Google-Smtp-Source: AMsMyM48EeR1ZaTYmic8XZXM+ll94c2FC5krhCF/dQmWWrZAoaWR9tPhbcCfY//CGugvtYZhe5oqbSgY8ridgDv8QpA= X-Received: by 2002:a05:620a:2214:b0:6ce:3426:1363 with SMTP id m20-20020a05620a221400b006ce34261363mr17765115qkh.662.1664285194671; Tue, 27 Sep 2022 06:26:34 -0700 (PDT) MIME-Version: 1.0 References: <20220622013839.405771-1-vfialko@marvell.com> <20220804095907.97895-1-vfialko@marvell.com> <20220804095907.97895-2-vfialko@marvell.com> In-Reply-To: From: Jerin Jacob Date: Tue, 27 Sep 2022 18:56:08 +0530 Message-ID: Subject: Re: [PATCH 1/3] eventdev: introduce event cryptodev vector type To: "Gujjar, Abhinandan S" Cc: Volodymyr Fialko , "dev@dpdk.org" , Jerin Jacob Kollanukkaran , Pavan Nikhilesh Bhagavatula , Shijith Thotton , Hemant Agrawal , Sachin Saxena , "Jayatheerthan, Jay" , Akhil Goyal , Anoob Joseph Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org > > > > > > > > > Right now we are targeting crypto_cn10k PMD and ipsec-secgw event > > > > mode to support vectorization. > > > Is there a way to test this? When can be dataplane changes expected? > > > > > If the spec looks okay, support in s/w crypto adapter and other h/w PMDs can > > be added by respective maintainers. Currently, we are adding library change, > > support for one PMD and an application to test the feature. Feature is exposed > > with capability flag to not break existing functionality. > Got it. How do I test this feature without data plane changes? Hi @Gujjar, Abhinandan S > If there is a way to test this, please let me know. Dataplane changes can be tested on the cn10k platform. This feature is a hardware assisted feature. > This design is right now tested for cn10k, I am not sure this works for sw adapter. SW driver support is not added in this series as in order to accept a API change, one would need, 1)API spec 2)One of the driver 3)Test application to exercise the API. It is a similar case for all ethdev, rte_flow features etc. Community can add SW driver support just like any other subsystem APIs. Also, The proposed library changes don't differentiate between SW & HW PMDs. The proposed changes are exposed with a capability flag and so SW crypto adapter will not have any perf impact. > I need to have perf data with and without vectorization support to approve. On the cn10k platform, we see nearly 2.5x performance with vectorization. Eth rx adapter already supports vectorization and this spec change is in line with that. Also IPsec gateway update to exercise these APIs. See http://patches.dpdk.org/project/dpdk/patch/20220804103626.102688-6-vfialko@marvell.com/ Command to test on drivers which have this functionality. ./dpdk-ipsec-secgw -c 0xff0000 -a 0002:01:00.1 -a 0002:20:00.1 -a 0002:1e:00.0 -- -P -p 0x1 -P --transfer-mode event -l --event-schedule-type parallel --desc-nb 8192 --event-vector -f simple.conf sample.conf sp ipv4 out esp protect 19 pri 1 dst 192.18.0.0/32 sport 0:65535 dport 0:65535 sa out 19 aead_algo aes-128-gcm aead_key 73:69:78:74:65:65:6e:62:79:74:65:73:20:6b:65:79:64:70:64:6b mode ipv4-tunnel src 2.1.1.1 dst 1.1.1.1 type lookaside-protocol-offload port_id 0 neigh port 0 d0:37:45:02:b0:d3 rt ipv4 dst 1.1.0.0/16 port 0 In order to make forward progress and merge patch in RC1, I would request 1)Review the API specific patch(eventdev: introduce event cryptodev vector type), If spec needs to be changed to adapt any other driver(SW or HW) then the author should address that. 2)If you think, API usage is not enough with dpdk-ipsec-secgw application, I think, author should update the test-eventdev application to support the new mode.Which can be merged after RC1 as it is a test application change. Let us know what you think to make forward progress.