From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 13E358D99 for ; Wed, 14 Oct 2015 12:41:08 +0200 (CEST) Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 4692641; Wed, 14 Oct 2015 10:41:07 +0000 (UTC) Received: from dhcp195.koti.laiskiainen.org (vpn1-7-71.ams2.redhat.com [10.36.7.71]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t9EAf6wK024112; Wed, 14 Oct 2015 06:41:06 -0400 To: David Marchand References: <56054579.9010609@redhat.com> From: Panu Matilainen Message-ID: <561E3141.4080607@redhat.com> Date: Wed, 14 Oct 2015 13:41:05 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <56054579.9010609@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.23 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH 0/2] Add support for driver directories X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 14 Oct 2015 10:41:08 -0000 On 09/25/2015 04:00 PM, Panu Matilainen wrote: > On 09/25/2015 03:35 PM, David Marchand wrote: >> Hello Panu, >> >> On Fri, Sep 25, 2015 at 1:58 PM, Panu Matilainen > > wrote: >> >> This mini-series adds support for driver directory concept as >> described by Thomas Monjalon back in February: >> http://dpdk.org/ml/archives/dev/2015-February/013285.html >> >> Panu Matilainen (2): >> eal: refactor plugin list append from eal_parse_args() to a helper >> function >> eal: add support for driver directory concept >> >> config/common_linuxapp | 3 ++ >> lib/librte_eal/common/eal_common_options.c | 1 + >> lib/librte_eal/linuxapp/eal/eal.c | 76 >> ++++++++++++++++++++++++++---- >> 3 files changed, 71 insertions(+), 9 deletions(-) >> >> >> This patchset looks good to me. >> Is there any reason why we should only have this for linux ? > > I merely followed suite with the related, pre-existing -d option which > also only exists for Linux. And wondered about the same thing :) So... Is this blocking on being Linux-only? I can have a look at moving the initialization to common/ if needed, but I wont be able test on FreeBSD. I'd just like to get this functionality into DPDK 2.2 to remove one of those silly usability obstacles thats all :) - Panu -