From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v11 00/25] Device querying Date: Sun, 15 Jul 2018 23:54:23 +0200 Message-ID: <8148072.TjNI07lPA7@xps> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Neil Horman , Shreyansh Jain , Bruce Richardson , Andrew Rybchenko To: Gaetan Rivet Return-path: Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id F363B98 for ; Sun, 15 Jul 2018 23:54:35 +0200 (CEST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 11/07/2018 23:44, Gaetan Rivet: > Gaetan Rivet (25): > devargs: use rte-log functions > devargs: add non-variadic parsing function > kvargs: remove error logs > kvargs: build before EAL > kvargs: introduce a more flexible parsing function > eal: introduce dtor macros > eal: introduce device class abstraction > devargs: add function to parse device layers > eal/dev: add device iterator interface > eal/dev: implement device iteration initialization > eal/dev: implement device iteration > kvargs: add generic string matching callback > bus/pci: implement device iteration and comparison > bus/pci: add device matching field id > bus/vdev: implement device iteration > bus/vdev: add device matching field driver > ethdev: add private generic device iterator > ethdev: register ether layer as a class > ethdev: add device matching field name > app/testpmd: add show device command > bus/pci: pre-process declarative PCI devargs > bus/vdev: pre-process declarative vdev devargs > bus/pci: process declarative PCI devargs > ethdev: process declarative eth devargs > eal: add generic dev parameter This series did not get a lot of reviews, especially for the last half of patches. As it allows to make some progress to get a generic devargs syntax, I decide to apply the first 12 patches, which brings a base for a generic devargs syntax parsing. The last half of patches are implementing some parsing/iteration for PCI, vdev and ethdev. They need to be discussed and agreed with more reviews. And more importantly, we need to add more properties in the devargs syntax, and we need to address more buses and device classes. Getting the parsing base in 18.08 should help to close the full syntax in 18.11. Thank you Gaetan for the big work.