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 D1FD3A034F; Wed, 10 Nov 2021 09:42:21 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5EABF4014D; Wed, 10 Nov 2021 09:42:21 +0100 (CET) Received: from mail.lysator.liu.se (mail.lysator.liu.se [130.236.254.3]) by mails.dpdk.org (Postfix) with ESMTP id 61BE040142 for ; Wed, 10 Nov 2021 09:42:20 +0100 (CET) Received: from mail.lysator.liu.se (localhost [127.0.0.1]) by mail.lysator.liu.se (Postfix) with ESMTP id C521B40017 for ; Wed, 10 Nov 2021 09:42:19 +0100 (CET) Received: by mail.lysator.liu.se (Postfix, from userid 1004) id AE2E540014; Wed, 10 Nov 2021 09:42:19 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on bernadotte.lysator.liu.se X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=ALL_TRUSTED, AWL, NICE_REPLY_A autolearn=disabled version=3.4.2 X-Spam-Score: -2.0 Received: from [192.168.1.36] (h-62-63-215-114.A163.priv.bahnhof.se [62.63.215.114]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.lysator.liu.se (Postfix) with ESMTPSA id 6653940006; Wed, 10 Nov 2021 09:42:16 +0100 (CET) To: Jerin Jacob , Sunil Kumar Kori Cc: Jerin Jacob , Nikhil Rao , Pavan Nikhilesh , Hemant Agrawal , Nipun Gupta , "Van Haaren, Harry" , Liang Ma , dpdk-dev , Thomas Monjalon , David Marchand , "Gujjar, Abhinandan S" , Erik Gabriel Carrillo , "Jayatheerthan, Jay" , Ferruh Yigit , Akhil Goyal References: <20200227070301.11520-1-skori@marvell.com> <20211022074843.907766-1-skori@marvell.com> From: =?UTF-8?Q?Mattias_R=c3=b6nnblom?= Message-ID: <0c9ce370-51af-9287-9780-eff9dc3b3bcb@ericsson.com> Date: Wed, 10 Nov 2021 09:42:15 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV using ClamSMTP Subject: Re: [dpdk-dev] [PATCH v5 1/4] doc: add skeleton for eventdevs feature matrices 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 Sender: "dev" On 2021-11-09 11:11, Jerin Jacob wrote: > On Fri, Oct 22, 2021 at 1:18 PM wrote: >> >> From: Sunil Kumar Kori >> >> Patch adds skeleton for feature matrices for event dev >> and Rx/Tx adapters. >> >> Along with, .ini files are added for all supported PMDs. >> >> Signed-off-by: Sunil Kumar Kori > > > + Tree maintainers and eventdev adapter maintainers > > 1) Please rebase to dpdk-next-eventdev > 2) Please update devtools/check-doc-vs-code.sh > You can use search _CAP_ and != CAP_REQUIRES_ to detect the capability > change or so. > 3) Update the .ini for all the eventdev drivers, not just Marvell ones. > >> >> # Maximum feature description string length >> -feature_str_len = 30 >> +feature_str_len = 45 > > I hope this OK. I understand that without this it gives errors and > feature-length is not enough > >> diff --git a/doc/guides/eventdevs/features/default.ini b/doc/guides/eventdevs/features/default.ini >> new file mode 100644 >> index 0000000000..d007d7a69e >> --- /dev/null >> +++ b/doc/guides/eventdevs/features/default.ini >> @@ -0,0 +1,30 @@ >> +; >> +; Features of a default eventdev driver. >> +; >> +; This file defines the features that are valid for inclusion in >> +; the other driver files and also the order that they appear in >> +; the features table in the documentation. >> +; >> +[Features] >> +Event queue priority based scheduling = >> +Event priority based scheduling = >> +Event device distributed scheduling = >> +Event device non sequential mode = >> +Event queue all types = >> +Burst mode enqueue/dequeue = >> +Event port implicit release disable = >> +Runtime event queue/port linking = >> +Multi queue linking per port = > > 1) Please sync the doc order with flag definition order in header file[1] > 2) You can remove RTE_EVENT_DEV_CAP_REQUIRES_MAINT from the > list(*CAP_REQUIRES*) as this > is limitation. > > Maybe we should negate RTE_EVENT_DEV_CAP_REQUIRES_MAINT to make it fit better with the other capabilities? To be more in line with how the API has been adapted to cope with various eventdev hardware limitations (e.g., RTE_EVENT_DEV_CAP_CARRY_FLOW_ID). RTE_EVENT_DEV_CAP_MAINTENANCE_FREE All event devices/drivers would set this except DSW. If this make sense and it's not too late to tweak the API, I can create a patch. > [1] > RTE_EVENT_DEV_CAP_QUEUE_QOS > RTE_EVENT_DEV_CAP_EVENT_QOS > RTE_EVENT_DEV_CAP_DISTRIBUTED_SCHED > RTE_EVENT_DEV_CAP_QUEUE_ALL_TYPES > RTE_EVENT_DEV_CAP_BURST_MODE > RTE_EVENT_DEV_CAP_IMPLICIT_RELEASE_DISABLE > RTE_EVENT_DEV_CAP_NONSEQ_MODE > RTE_EVENT_DEV_CAP_RUNTIME_PORT_LINK > RTE_EVENT_DEV_CAP_MULTIPLE_QUEUE_PORT > RTE_EVENT_DEV_CAP_CARRY_FLOW_ID > > > >> + >> +; >> +; Features of a default adapter. >> +; >> +[Adapter Features] > > Please make this as [Eth Rx adapter Features] > >> +Eth Rx/Tx adapter on internal port = > You can remove Eth Rx/Tx adapter as new subject has [Eth Rx adapter] > >> +Multiple event queues per ethdev = >> +Override flow ID = > > Missing RTE_EVENT_ETH_RX_ADAPTER_CAP_EVENT_VECTOR > > > Add new section for Tx adapter > and add > RTE_EVENT_ETH_TX_ADAPTER_CAP_INTERNAL_PORT > RTE_EVENT_ETH_TX_ADAPTER_CAP_EVENT_VECTOR > > > Add new section for Crypto adapter > >> +Crypto adapter new event enqueue = >> +Crypto adapter forwarded event enqueue = >> +Crypto queue pair mapping to event queue = >> +Crypto adapter session data store/retrieve = > > > Add new section for timer adapter > >> +Timer adapter support in H/W = > > Missing RTE_EVENT_TIMER_ADAPTER_CAP_PERIODIC >