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 E389D41E9F; Wed, 15 Mar 2023 14:47:46 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 68F7D40A7A; Wed, 15 Mar 2023 14:47:46 +0100 (CET) Received: from mail-ua1-f41.google.com (mail-ua1-f41.google.com [209.85.222.41]) by mails.dpdk.org (Postfix) with ESMTP id 70AFA40141; Wed, 15 Mar 2023 14:47:45 +0100 (CET) Received: by mail-ua1-f41.google.com with SMTP id 89so5940966uao.0; Wed, 15 Mar 2023 06:47:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1678888064; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=+9IIwhTMzyCSaUpM0qHuoElHKyQi+GXm29FJLLsT/JY=; b=Vmrc/i47WR8PVVCn55VQDhaZQ8/J0cpcipg7hstkfK/YhAqIkummFdITIsa24FWLpA RhZ4ZeMACjmWyZXtQmexmgTiA0CcsvMeX5uW2QPvrbvodPFWx12Wcutq+xFwStsw16TG xXpWMliATjttesOOFejyZdaD1sCFIcKrB3rpIoVjgEBhPPf0EsFkxePzk3ZIilglsfJ0 6hK+O0kdyqztDFFW7fVv/Mmj9AGNvi7dr64WOvn6+5/P46IYHnZdUbdZvGFNN4zD4z+G hsTuEPmnv+U+zkoDeXOZGjdPRO8BgbGQ5WIVtp+FOSGAlxc1hoLCovmG+DIgB+1FuEkw jKNw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678888064; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=+9IIwhTMzyCSaUpM0qHuoElHKyQi+GXm29FJLLsT/JY=; b=h52ofpptb0tKHicdjXnRAoY0vNN13SN6TlyGVdEkVdo0Sd8mSdMsOttKuqAwvoaEY6 bLwxWB1uS2P/Fjzl0r1ilZbF22iAMhTOGXDJRMgJpx5dPzgm4lFRovq8xjzholSjTa9F 6QqMeZlGdCR/RYMmtP2dWuUqC6ECoVjeo+DG3dbBEUCN9FOrh65yERzZvB3xFdeY39ky JwfW4Z0pTG0KI6+DGhJJRXTZtWMOspokYUKhYKSxs/UEKkfj20ZT3S63bYOfJiu8aPmB 4Tlav79jaUuMqdbGtAjz22z4MjOIGnt2VojnQ7q7re8aV2A+vdT8m6GtJpJIxiSMXwYL dAYg== X-Gm-Message-State: AO0yUKUGw9xIagA7TCerTP/IElIjuSf1rqdRrmwyTQGqXLd40K2GfDjp spTEbBNLGuB3bXmjF+2gKTRVYrbTGrMNE4OZ+fgdUbO/070= X-Google-Smtp-Source: AK7set+vZ2ubDgNZP0T/LWLL1svy/ZqBlOx5lNZxa0nyUXlpZQoE85wdbjPJGNClqZMiNjPOIufXrUQk4zmlNOSRyQw= X-Received: by 2002:a1f:4507:0:b0:42d:5ea6:bd58 with SMTP id s7-20020a1f4507000000b0042d5ea6bd58mr16539148vka.3.1678888064576; Wed, 15 Mar 2023 06:47:44 -0700 (PDT) MIME-Version: 1.0 References: <20230213092616.3589932-1-jerinj@marvell.com> <11338869.jrtcCam0TZ@thomas> In-Reply-To: <11338869.jrtcCam0TZ@thomas> From: Jerin Jacob Date: Wed, 15 Mar 2023 19:17:18 +0530 Message-ID: Subject: Re: [dpdk-web] [RFC PATCH] process: new library approval in principle To: Thomas Monjalon Cc: Jerin Jacob , web@dpdk.org, dev@dpdk.org, techboard@dpdk.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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 On Fri, Mar 3, 2023 at 11:55=E2=80=AFPM Thomas Monjalon wrote: > > Thanks for formalizing our process. Thanks for the review. > > 13/02/2023 10:26, jerinj@marvell.com: > > --- /dev/null > > +++ b/content/process/_index.md > > First question: is the website the best place for this process? > > Inside the code guides, we have a contributing section, > but I'm not sure it is a good fit for the decision process. > > In the website, you are creating a new page "process". > Is it what we want? > What about making it a sub-page of "Technical Board"? Since it is a process, I thought of keeping "process" page. No specific opinion on where to add it. If not other objections, Then I can add at doc/guides/contributing/new_library_policy.rst in DPDK repo. Let me know if you think better name or better place to keep the file > > > @@ -0,0 +1,33 @@ > > ++++ > > +title =3D "Process" > > +weight =3D "9" > > ++++ > > + > > +## Process for new library approval in principle > > + > > +### Rational > > s/Rational/Rationale/ Ack > > > + > > +Adding a new library to DPDK codebase with proper RFC and then full pa= tch-sets is > > +significant work and getting early approval-in-principle that a librar= y help DPDK contributors > > +avoid wasted effort if it is not suitable for various reasons. > > That's a long sentence we could split. OK Changing as: Adding a new library to DPDK codebase with proper RFC and full patch-sets is significant work. Getting early approval-in-principle that a library can help DPDK contributors avoid wasted effort if it is not suitable for various reasons > > > + > > +### Process > > + > > +1. When a contributor would like to add a new library to DPDK code bas= e, the contributor must send > > +the following items to DPDK mailing list for TB approval-in-principle. > > I think we can remove "code base". Ack > > TB should be explained: Technical Board. Ack > > > + > > + - Purpose of the library. > > + - Scope of the library. > > Not sure I understand the difference between Purpose and Scope. Purpose =E2=86=92 The need for the library Scope =E2=86=92 I meant the work scope associated with it. I will change "Scope of the library" to, - Scope of work: Outline the various additional tasks planned for this library, such as developing new test applications, adding new drivers, and updating existing applications. > > > + - Any licensing constraints. > > + - Justification for adding to DPDK. > > + - Any other implementations of the same functionality in other libs= /products and how this version differs. > > libs/products -> libraries/projects Ack > > > + - Public API specification header file as RFC > > + - Optional and good to have. > > You mean providing API is optional at this stage? Yes. I think, TB can request if more clarity is needed as mentioned below. "TB may additionally request this collateral if needed to get more clarity on scope and purpose" > > > + - TB may additionally request this collateral if needed to get = more clarity on scope and purpose. > > + > > +2. TB to schedule discussion on this in upcoming TB meeting along with= author. Based on the TB > > +schedule and/or author availability, TB may need maximum three TB meet= ing slots. > > Better to translate the delay into weeks: 5 weeks? Ack > > > + > > +3. Based on mailing list and TB meeting discussions, TB to vote for ap= proval-in-principle and share > > +the decision in the mailing list. > > I think we should say here that it is safe to start working > on the implementation after this step, > but the patches will need to match usual quality criterias > to be effectively accepted. OK. I will add the following, 4. Once TB approves the library in principle, it is safe to start working on its implementation. However, the patches will need to meet the usual quality criteria in order to be effectively accepted. > >