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 E3E47A2EFC for ; Mon, 14 Oct 2019 09:57:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B00701C1DB; Mon, 14 Oct 2019 09:57:08 +0200 (CEST) Received: from dispatch1-us1.ppe-hosted.com (dispatch1-us1.ppe-hosted.com [148.163.129.52]) by dpdk.org (Postfix) with ESMTP id 106A91C1CE for ; Mon, 14 Oct 2019 09:57:07 +0200 (CEST) X-Virus-Scanned: Proofpoint Essentials engine Received: from webmail.solarflare.com (uk.solarflare.com [193.34.186.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by mx1-us4.ppe-hosted.com (PPE Hosted ESMTP Server) with ESMTPS id 8A7FE8005E; Mon, 14 Oct 2019 07:57:05 +0000 (UTC) Received: from [192.168.38.17] (91.220.146.112) by ukex01.SolarFlarecom.com (10.17.10.4) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Mon, 14 Oct 2019 08:57:00 +0100 To: Alfredo Cardigliano , Stephen Hemminger CC: References: <157083994018.11524.11276616720287263690.stgit@devele> <20191012082813.7abe5f4d@hermes.lan> From: Andrew Rybchenko Message-ID: <8bafa882-3d10-4fd4-0af6-0bbe4452945e@solarflare.com> Date: Mon, 14 Oct 2019 10:56:55 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-GB X-Originating-IP: [91.220.146.112] X-ClientProxiedBy: ocex03.SolarFlarecom.com (10.20.40.36) To ukex01.SolarFlarecom.com (10.17.10.4) X-TM-AS-Product-Ver: SMEX-12.5.0.1300-8.5.1010-24974.003 X-TM-AS-Result: No-9.558000-8.000000-10 X-TMASE-MatchedRID: eVEkOcJu0F7sYbGmK/WYxvZvT2zYoYOwC/ExpXrHizxXXbZeMl1WD0w5 MIk9Zic42UJszGAseMGEQMR4E0/LXTaN5j5R2D8i/ccgt/EtX/0A+JHhu0IR5pGhAvBSa2i/wd0 ua4gmnrQbpFu2LL52JpMsbSdapRzTeKZAID3hHVYYxRF+FCQQstZnNOazoZ6SU20Pec0W1lYqvx HAtLsn3b/ookEkN/h0/2jhsEtuxob8XF85VELdKMkFQf0u+7NqsKi4EXb8AIrm5xVqTV9QwbMea pFjC2iRmgaSd8DqI10KsonrqBnRWa/Bl8s56y+QLIrMljt3adtxWv4UB7dQNX5Isu006IGGw/fb fywPSrpQNgaVBskNlj48erx7ecjev1l2Uvx6idpWdFebWIc3VsRB0bsfrpPIfiAqrjYtFiQTZOT +y/yxf2KVDbGtCuwO40ubmgKWF4eWROhZj4B2kX7cGd19dSFd X-TM-AS-User-Approved-Sender: Yes X-TM-AS-User-Blocked-Sender: No X-TMASE-Result: 10--9.558000-8.000000 X-TMASE-Version: SMEX-12.5.0.1300-8.5.1010-24974.003 X-MDID: 1571039826-vBkEkdyJ5xjD Subject: Re: [dpdk-dev] [PATCH 00/17] Series short description 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 10/14/19 10:16 AM, Alfredo Cardigliano wrote: > Hi Stephen > yes, we were thinking of doing a single post for just that reason, > but Thomas recommended that we split it up. I agree with Thomas that it is better to split it up. What I don't understand from the list below is, for example, why RSS goes before Rx/Tx handling.  Ideally, it should be no dead code in patches, the driver should be buildable from patch #1 and subsequent patches should add more and more functionality which is testable. Of course, it is not always doable, for example when a base driver should be added first, but it does not look so in this case. > Thank you > Alfredo > >> On 12 Oct 2019, at 17:28, Stephen Hemminger wrote: >> >> On Sat, 12 Oct 2019 02:26:26 +0200 >> Alfredo Cardigliano wrote: >> >>> The patch series provides an initial version of a >>> poll mode driver for Pensando network adapters. >>> The driver name is ionic. >>> >>> --- >>> >>> Alfredo Cardigliano (17): >>> net/ionic: add skeleton >>> net/ionic: add hardware structures definitions >>> net/ionic: add log >>> net/ionic: register and initialize the adapter >>> net/ionic: add port management commands >>> net/ionic: add basic lif support >>> net/ionic: add doorbells >>> net/ionic: add adminq support >>> net/ionic: add notifyq support >>> net/ionic: add basic port operations >>> net/ionic: add RX filters support >>> net/ionic: net-ionic-add-flow-control-support >>> net/ionic: add RSS support >>> net/ionic: add RX and TX handling >>> net/ionic: add stats >>> net/ionic: add TX checksum support >>> net/ionic: read fw version >>> >> When adding a new driver, could you send it as >> one unit, yes it is a big bit of code to review but >> drippling it out in pieces doesn't make review easier. >>