From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from rcdn-iport-4.cisco.com (rcdn-iport-4.cisco.com [173.37.86.75]) by dpdk.org (Postfix) with ESMTP id B319F5A6C for ; Tue, 17 Nov 2015 14:56:40 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6363; q=dns/txt; s=iport; t=1447768600; x=1448978200; h=subject:to:references:cc:from:message-id:date: mime-version:in-reply-to; bh=a5Fz7N5dobxvLWboE/KYxu38Pkl2rHuroM8V7TuBMVk=; b=j34MbpMvkEBOnWj8d1r6AB4KOPjKFOKUmQNcdG2micOFeAUmj1zSk9Hh zXQXrVKlSSDAbCbXb7zqvy6nNYgqDJ5dRX4OapECmvRHsMXu+rlE5otqJ zfJO/c7YeJa1/kHBlmozonkYSQXWYAOnMuv2X2EsATnKhAp0/m9zvcbV+ 0=; X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0D9AQAVMUtW/40NJK1egzuBQr5hAQ2BZ?= =?us-ascii?q?YYQAoFOOBQBAQEBAQEBgQqENQEBBCNVARALGAkWCAMCAgkDAgECATQRBg0GAgE?= =?us-ascii?q?BiCqrTJBAAQEBAQEBAQEBAQEBAQEBAQEBAQEZhlSEfod1gUQFjhGIOAGNKokdh?= =?us-ascii?q?g+NGR8BAUKEIiA0hQoBAQE?= X-IronPort-AV: E=Sophos;i="5.20,308,1444694400"; d="scan'208,217";a="47510403" Received: from alln-core-8.cisco.com ([173.36.13.141]) by rcdn-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 17 Nov 2015 13:56:37 +0000 Received: from [10.82.250.205] (rtp-vpn6-715.cisco.com [10.82.250.205]) by alln-core-8.cisco.com (8.14.5/8.14.5) with ESMTP id tAHDub1a012505; Tue, 17 Nov 2015 13:56:37 GMT To: David Marchand References: <56475966.2080404@cisco.com> <56489A95.1050706@cisco.com> From: "Roger B. Melton" Message-ID: <564B3215.9020103@cisco.com> Date: Tue, 17 Nov 2015 08:56:37 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: "dev@dpdk.org" , Don Provan Subject: Re: [dpdk-dev] Making rte_eal_pci_probe() in rte_eal_init() optional? 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: Tue, 17 Nov 2015 13:56:41 -0000 Hi David, in-line -Roger On 11/16/15 4:46 AM, David Marchand wrote: > Hello Roger, > > On Sun, Nov 15, 2015 at 3:45 PM, Roger B. Melton > wrote: > > I like the "-b all" and "-w none" idea, but I think it might be > complicated to implement it the way we would need it to work. The > existing -b and -w options persist for the duration of the > application, and we would need the "-b all"/"-w none" to persists > only through rte_eal_init() time. Otherwise our attempt to to > attach a device at a later time would be blocked by the option. > > > I agree, the black/white lists should only apply to initial scan. > I forgot about this problem ... > I had started some cleanup in the pci scan / attach code but this is > too late for 2.2, I will post this in the next merge window. > > > Wouldn't it be simpler to have an option to disable the > rte_eal_init() time the probe. Would that address the issue with > VFIO, prevent automatically attaching to devices while permitting > on demand attach? > > > I suppose we can do this yes (I think Thomas once proposed off-list an > option like --no-pci-scan). > Do you think you can send a patch ? What about --no-pci-init-probe? I know it's long, but it is more descriptive of it's purpose to disable only the init time pci probe. I code and test and have it ready. I'm still working through internal processes to allow me to submit patches, but I hope to have that resolved in the next few weeks and at that time I can submit the patch. > > > -- > David Marchand >