From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f176.google.com (mail-wr0-f176.google.com [209.85.128.176]) by dpdk.org (Postfix) with ESMTP id 212A028F3 for ; Wed, 15 Mar 2017 13:43:41 +0100 (CET) Received: by mail-wr0-f176.google.com with SMTP id u108so10152677wrb.3 for ; Wed, 15 Mar 2017 05:43:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:user-agent:in-reply-to :references:mime-version:content-transfer-encoding; bh=j+emdl/tqGm4a6PCbnVLzKrePSHvjA58MAiROM/ff64=; b=zW8yMlWZYoFI0aFImHncP+IhKBZ8fsNfCwEMm0yjGYzzt+FgOR6y206rW9sinUWG5X 4kfxz9AH2YUZoIA/6RExdIsIeNwEjlzjvi/AdSzv9SzVMN0fv2SmmT2ELXO6HEpr2AXp Putj/t62ND7GiefLccs3yZMY7uK/PimlR1kwH1Fs06qEDnfqesrgqyfctVkz7xhj2Shl CfgJnd6DlJ/hfkYXyPKfK37cp2Kgo187EZSv3J7sfHHUQu0Tb+2LW0OiLgMJWac6vHTj YxTH+3X3uKL8gdHm7ynW0QNML/1tCQd4Z3YZS97fEJ/MMwiPG7Ohh7hHRj81qqCRinG3 uaEg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=j+emdl/tqGm4a6PCbnVLzKrePSHvjA58MAiROM/ff64=; b=iQBDyq6emMZGxYphzePTz5l1wk/ehvO4vKwaQ7yu5/fK3tBTPn8xe8mCTy8yc6Mtyf 5Wao3TUWAo7p6RgwWurIcXPDnn9IYbNYpaPvqxaaVEj7zBUO9i/VBN/4TDdwCGa6SYr3 XkM+aeRvS+zeM7EmK0SenbN0JGWevpQptCtaY3ZzRGBYiszn+RzfaXPP7pMSwkpet62L DMa5m+D15Db7SuxGikj0KYZotupYKDe3Nn/RDECGIyHuRV3SUGtHVjxzOczAi11DJZio r9rVPkkXEMEDep56kNVmAOTCdZvXfA+aBc6jJU4+VDObBbQABP9eMKntru7Jro+sEodv 9K8Q== X-Gm-Message-State: AFeK/H2wvjZlNRY8vCDJUMdxEV/GJK+YvVdwW3Kgz/nZGeRQzY+uimVjz3c51diytwZwTfCj X-Received: by 10.223.144.67 with SMTP id h61mr1989983wrh.30.1489581820787; Wed, 15 Mar 2017 05:43:40 -0700 (PDT) Received: from xps13.localnet (184.203.134.77.rev.sfr.net. [77.134.203.184]) by smtp.gmail.com with ESMTPSA id 63sm375298wmp.9.2017.03.15.05.43.40 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 15 Mar 2017 05:43:40 -0700 (PDT) From: Thomas Monjalon To: "Dumitrescu, Cristian" Cc: "O'Driscoll, Tim" , dev@dpdk.org, jerin.jacob@caviumnetworks.com, balasubramanian.manoharan@cavium.com, hemant.agrawal@nxp.com, shreyansh.jain@nxp.com, "Wiles, Keith" , "Richardson, Bruce" Date: Wed, 15 Mar 2017 13:43:39 +0100 Message-ID: <106399437.xFH6ZF6NRJ@xps13> User-Agent: KMail/4.14.10 (Linux/4.5.4-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: <3EB4FA525960D640B5BDFFD6A3D891265275E105@IRSMSX108.ger.corp.intel.com> References: <1488589820-206947-1-git-send-email-cristian.dumitrescu@intel.com> <26FA93C7ED1EAA44AB77D62FBE1D27BA7231952B@IRSMSX108.ger.corp.intel.com> <3EB4FA525960D640B5BDFFD6A3D891265275E105@IRSMSX108.ger.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v3 2/2] ethdev: add hierarchical scheduler API 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: , X-List-Received-Date: Wed, 15 Mar 2017 12:43:41 -0000 2017-03-10 18:37, Dumitrescu, Cristian: > From: O'Driscoll, Tim > > > > OK I better understand now. > > > > You should add this level of explanation in your patch. > > > > > > > > However I am reluctant to add an API if there is no user. > > > > I think we should wait to have at least one existing driver > > > implementing > > > > this API before integrating it. > > > > It was the approach of eventdev which has a dedicated next- tree. > > > > > > The next-tree solution could work, but IMO is not the best for this > > > case, as this is purely driver development. This is just a TX offload > > > feature that is well understood, as opposed to a new library with a huge > > > design effort required like eventdev. > > > > > > I think we are reasonably close to get agreement on the API from Cavium, > > > Intel and NXP. When this is done, how about including it in DPDK with > > > the experimental tag attached to it until several drivers implement it? > > > > > > From Intel side, there are solid plans to implement it for ixgbe and > > > i40e drivers in next DPDK releases, I am CC-ing Tim to confirm this. > > > > That's correct. We plan to add support for this in the ixgbe and i40e drivers in > > 17.08. > > Thomas, given Tim's confirmation of Intel's plans to implement this API for the ixgbe and i40e drivers in DPDK release 17.8, are you in favour of including this API in 17.5 with experimental tag (subject to full API agreement being reached)? I think starting a branch in a dedicated "next" repo is a better approach. rte_flow and eventdev were (and will be) integrated only when at least one hardware device is supported. I suggest to follow the same workflow. > IMO this approach has the advantage of showing that API agreement has been reached and driver development is in progress. Having it in DPDK is also a better way to advertise this API to the developers that would otherwise be unaware about this effort. IMO we can advertise a work in progress in a side branch.