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 A5156958A for ; Wed, 21 Oct 2015 14:07:16 +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 8F743C0BEA80; Wed, 21 Oct 2015 12:07:15 +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 t9LC7E4M016575; Wed, 21 Oct 2015 08:07:14 -0400 To: Thomas Monjalon , David Marchand References: <20151021111520.GA21768@bricha3-MOBL3> <3136074.xcMQgt4DKX@xps13> From: Panu Matilainen Message-ID: <56277FF1.5070700@redhat.com> Date: Wed, 21 Oct 2015 15:07:13 +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: <3136074.xcMQgt4DKX@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 1/2] eal: move plugin loading to eal/common 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 12:07:16 -0000 On 10/21/2015 02:53 PM, Thomas Monjalon wrote: > 2015-10-21 12:15, Bruce Richardson: >> On Wed, Oct 21, 2015 at 01:09:24PM +0200, David Marchand wrote: >>> On Wed, Oct 21, 2015 at 12:54 PM, Panu Matilainen >>> wrote: >>>> >>>> Btw, returning an error here would change current behavior of dpdk loading >>>>> drivers. >>>>> Not sure we want this as people may rely on this loading not failing. >>>>> >>>> >>>> Yeah, dpdk currently doesn't fail if you pass garbage to -d, which is >>>> actually fairly questionable behavior. Why would you load drivers with -d >>>> if you dont care about them getting loaded? Well, maybe to handle an >>>> "everything" case but that's much better handled with the driver directory >>>> thing. >>>> >>>> So actually the current patches make things a bit inconsistent, why should >>>> driver directories cause a failure if individual drivers do not? The >>>> question is, which behavior is the one people want: I personally would >>>> rather make -dgiddy.goo fail rather than just warn and chug away but its >>>> not exactly a deal-breaker for me. >>>> >>> >>> Neither to me. >>> I agree on the principle of failing when passing wrong stuff, it is saner. >>> I just want to make sure nobody complains about this change later. >>> >>> Thomas ? Bruce ? >> >> Error early rather than later. If the -d flag doesn't work crash then, rather >> than leaving people having to scroll-back to find why their NIC isn't working. > > Yes, no reason to ignore errors. Okay so we all vigorously agree on this :) Good then, I'll fix the error behavior too in the next version. - Panu -