From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 14A59A052B; Thu, 30 Jul 2020 20:48:52 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 8026810A3; Thu, 30 Jul 2020 20:48:50 +0200 (CEST) Received: from mail-io1-f66.google.com (mail-io1-f66.google.com [209.85.166.66]) by dpdk.org (Postfix) with ESMTP id 6D596A69 for ; Thu, 30 Jul 2020 20:48:49 +0200 (CEST) Received: by mail-io1-f66.google.com with SMTP id v6so14040501iow.11 for ; Thu, 30 Jul 2020 11:48:49 -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=1Wa789wb9wUnNQzkSixNWEnrsZPgp2ys6ugFE7yYykw=; b=kWNKBdnpiSjz9MRg9QaP57ITyRNRxF2ehYGhVS6jATgx3Ucftir+zKersxji4EgCMP 4nXEOTc12194reUNFK+UTtKTC3PRHIwGz5n8Mb/0B6JeU3xL0qYx5Xvrg25Kt3mA44Gp ezmvr2RY261P4c2rrE1gW+BGGt8yhMvChLiVsKi4fFkM4GHdrT1kkRZx4nmv97+J3txh qngh9BrhunMq5NtUInQ/Cix6nC7326Bhv4nYMh8xyRlmpO1nxOVFsL+mfHRxjjMUmV/A hbNcKVla71JYVkuhW41Ve2v/7EifdHB1tKsfcc6kOmu7+symjRtKJpkV2LPNRz56luD0 wD4Q== 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=1Wa789wb9wUnNQzkSixNWEnrsZPgp2ys6ugFE7yYykw=; b=pbKQwQP1r8qJ85t4tAE/AZ474RZcna1biUJhwHGojHcxR5iMmBfXbD9Zcpdrvv4ikL DVM2FMKHIqeNIO/djWiPbXDTJFO/dSSpmgeqKkjn4KpgkLGJFyS2TzOwhPcumepg/qQx MIrzBzsIx4NUg67NvHCx4fN26zdbTN5QcJ9bRzjXLtnEZdw8peRlPzt5U4DMDXMlSIWh o/Jc9c0r2KQbG5NPC43ivG7FH27FQ8HMCg6rdtYgq077F3Esr2ochefbwfrnNZwyoEIx EunBFAlR8k9MDuKf2ZXL/sPXLUJZbD+Tjw+7oK8P+iHKmJ86is7gULwSVcjVNlHNAE1E jxqA== X-Gm-Message-State: AOAM530jc3VyqCMN/bBWFXrt2tJJA1lZqbazVa/DZzp99ei+GKaJdJYy +mNlsyRKhn6PMgcukZNbl/oBMwxXC1Eqi5uN/qc= X-Google-Smtp-Source: ABdhPJxAIUNK9Is1U9msMFhb9uaYcn5/G8EQjun8t9/V3joHZ1cjMUZSKDCdiHiNm5RC+QOWbKsEYQUNEfIVY8h3wbU= X-Received: by 2002:a5d:9b05:: with SMTP id y5mr39779465ion.59.1596134928686; Thu, 30 Jul 2020 11:48:48 -0700 (PDT) MIME-Version: 1.0 References: <1593728037-15717-1-git-send-email-timothy.mcdaniel@intel.com> In-Reply-To: From: Jerin Jacob Date: Fri, 31 Jul 2020 00:18:32 +0530 Message-ID: To: "McDaniel, Timothy" Cc: "dev@dpdk.org" , "jerinj@marvell.com" , "Eads, Gage" , "Van Haaren, Harry" , "mdr@ashroe.eu" , "nhorman@tuxdriver.com" , "Rao, Nikhil" , "Carrillo, Erik G" , "Gujjar, Abhinandan S" , "pbhagavatula@marvell.com" , "hemant.agrawal@nxp.com" , "mattias.ronnblom@ericsson.com" , "Mccarthy, Peter" Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH] doc: announce changes to eventdev public data structures 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" On Thu, Jul 30, 2020 at 10:03 PM McDaniel, Timothy wrote: > > > >-----Original Message----- > >From: McDaniel, Timothy > >Sent: Thursday, July 2, 2020 5:14 PM > >To: dev@dpdk.org > >Cc: jerinj@marvell.com; Eads, Gage ; Van Haaren, Harry > >; mdr@ashroe.eu; nhorman@tuxdriver.com; Rao, > >Nikhil ; Carrillo, Erik G ; Gujjar, > >Abhinandan S ; pbhagavatula@marvell.com; > >hemant.agrawal@nxp.com; mattias.ronnblom@ericsson.com; Mccarthy, Peter > >; McDaniel, Timothy > > > >Subject: [PATCH] doc: announce changes to eventdev public data structures > > > >From: "McDaniel, Timothy" > > > >Signed-off-by: McDaniel, Timothy > >--- > > doc/guides/rel_notes/deprecation.rst | 28 ++++++++++++++++++++++++++++ > > 1 file changed, 28 insertions(+) > > > >diff --git a/doc/guides/rel_notes/deprecation.rst > >b/doc/guides/rel_notes/deprecation.rst > >index d1034f6..6af9b40 100644 > >--- a/doc/guides/rel_notes/deprecation.rst > >+++ b/doc/guides/rel_notes/deprecation.rst > >@@ -130,3 +130,31 @@ Deprecation Notices > > Python 2 support will be completely removed in 20.11. > > In 20.08, explicit deprecation warnings will be displayed when running > > scripts with Python 2. > >+ > >+* eventdev: Three public data structures will be updated in 20.11; > >+ ``rte_event_dev_info``, ``rte_event_dev_config``, and > >+ ``rte_event_port_conf``. > >+ Two new members will be added to the ``rte_event_dev_info`` struct. > >+ The first, max_event_port_links, will be a uint8_t, and represents the > >+ maximum number of queues that can be linked to a single event port by > >+ this device. The second, max_single_link_event_port_queue_pairs, will be a > >+ uint8_t, and represents the maximum number of event ports and queues that > >+ are optimized for (and only capable of) single-link configurations > >+ supported by this device. These ports and queues are not accounted for in > >+ max_event_ports or max_event_queues. > >+ One new member will be added to the ``rte_event_dev_config`` struct. The > >+ nb_single_link_event_port_queues member will be a uint8_t, and will > >+ represent the number of event ports and queues that will be singly-linked > >+ to each other. These are a subset of the overall event ports and queues. > >+ This value cannot exceed nb_event_ports or nb_event_queues. If the > >+ device has ports and queues that are optimized for single-link usage, this > >+ field is a hint for how many to allocate; otherwise, regular event ports and > >+ queues can be used. > >+ Finally, the ``rte_event_port_conf`` struct will be > >+ modified as follows. The uint8_t implicit_release_disabled field > >+ will be replaced by a uint32_t event_port_cfg field. The new field will > >+ initially have two bits assigned. RTE_EVENT_PORT_CFG_DISABLE_IMPL_REL > >+ will have the same meaning as implicit_release_disabled. The second bit, > >+ RTE_EVENT_PORT_CFG_SINGLE_LINK will be set if the event port links only > >+ to a single event queue. > >+ > >-- > >1.7.10 > > Hello Jerin and DPDK community. Please review and approve the eventdev interface changes announced in this notice. Changes look good to me in general. Could you reword the description such way or similar like below to accommodate 1) DLB PMD requirements 2) Future extensions[1] I think, We don't need exact mention the data structure member additions, (This is to get the flexibility on additions/deletion on member fields after the patch rework) something like: eventdev: ABI change to support DLB PMD and future extensions The following structures and will be modified to support to DLB PMD and future extension in the eventdev library. And then please enumerate the structures[2] of your patch which needs change, [2]``rte_event_dev_info``, ``rte_event_dev_config``, and ``rte_event_port_conf`` and structures in [1]. Please mention the patches [1] and your spec change patch as a reference in the description. [1] http://patches.dpdk.org/patch/72786/ http://patches.dpdk.org/patch/72787/ > > Thanks, > Tim