From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shreyansh Jain Subject: Re: [PATCH v2 18/18] doc: remove devargs deprecation notices Date: Fri, 22 Dec 2017 10:29:14 +0530 Message-ID: References: <3930fcee-7ff6-c6f9-273b-14c16bef6ab1@nxp.com> <20171213102552.nmrf73y7yvzexkz7@bidouze.vm.6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit Cc: "dev@dpdk.org" To: =?UTF-8?Q?Ga=c3=abtan_Rivet?= Return-path: Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-sn1nam02on0046.outbound.protection.outlook.com [104.47.36.46]) by dpdk.org (Postfix) with ESMTP id 8B6641B27F for ; Fri, 22 Dec 2017 05:45:19 +0100 (CET) In-Reply-To: Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hello Gaetan, On Wednesday 13 December 2017 04:24 PM, Shreyansh Jain wrote: > Hello Gaetan, > >> -----Original Message----- >> From: Gaƫtan Rivet [mailto:gaetan.rivet@6wind.com] >> Sent: Wednesday, December 13, 2017 3:56 PM >> To: Shreyansh Jain >> Cc: dev@dpdk.org >> Subject: Re: [PATCH v2 18/18] doc: remove devargs deprecation notices >> [...] > > While reading through the code, I also had the same feeling - there can be corner cases in the parsing functions which I can't imagine. Anyways, those need to be runtime-verified - static reviews may not suffice. > >> >> I would certainly appreciate if you are able to fix the pci / vdev >> limitation in rte_eal_dev_attach, as I am starting to be overwhelmed >> with work (trying to finish a lot of things before the holidays). > > OK. > Once you give the devargs a push, I will start work on the PCI removal from rte_eal_dev_attach. Before that, I just want to be sure of devargs with non-PCI bus (non hotplug case). > > And, thanks for tons of work you are handling. I saw the patches and really appreciate how you have split things up in sequential manner per-patch. It is difficult. Have you pushed the new version of the devargs patches? Just wanted to check in case I have missed it. - Shreyansh