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 2D6DC939E for ; Wed, 21 Oct 2015 11:43:33 +0200 (CEST) Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (Postfix) with ESMTPS id 86CF3A8D; Wed, 21 Oct 2015 09:43:32 +0000 (UTC) Received: from dhcp195.koti.laiskiainen.org (vpn1-6-228.ams2.redhat.com [10.36.6.228]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t9L9hVIg001545; Wed, 21 Oct 2015 05:43:31 -0400 To: Thomas Monjalon References: <0df011be4afdecedb49b44e583c128e8343e621a.1445415982.git.pmatilai@redhat.com> <3437988.28W6rqn9YS@xps13> From: Panu Matilainen Message-ID: <56275E42.1010307@redhat.com> Date: Wed, 21 Oct 2015 12:43:30 +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: <3437988.28W6rqn9YS@xps13> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.27 Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 2/2] eal: add support for driver directory concept 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, 21 Oct 2015 09:43:33 -0000 On 10/21/2015 11:44 AM, Thomas Monjalon wrote: > 2015-10-21 11:29, Panu Matilainen: >> Add a new EAL option -D for loading all drivers from a given directory. >> Additionally a default driver directory can be set in build-time >> configuration, in which case it will be always be used when EAL is >> initialized (but can be overridden or disabled with -D). > [...] >> @@ -955,6 +1004,7 @@ eal_common_usage(void) >> " --"OPT_SYSLOG" Set syslog facility\n" >> " --"OPT_LOG_LEVEL" Set default log level\n" >> " -d LIB.so Add driver (can be used multiple times)\n" >> + " -D DIRECTORY Add driver directory)\n" > > Another idea: instead of adding a new option, why not make -d able to deal > with directories? > If that's what you want, sure. Just wishing you'd come up with this idea a bit earlier :) - Panu -