Re: [dpdk-dev] [PATCH v2 1/1] pci: default to whitelist mode

[Bruce Richardson <bruce.richardson@intel.com>]

On Tue, Mar 28, 2017 at 02:01:29PM +0200, Gaetan Rivet wrote:
> Expects all devices to be explicitly defined before being probed.
> 
> The blacklist mode can be prone to errors, coaxing users in capturing
> devices that could be used for management or otherwise.
> The whitelist mode offers users more control and highlight mistakes by
> making them visible on the command line.
> 
> This is more useful to have a clear idea of the state of the system used,
> which is better in the context of standalone / headless applications.
> 
> Using the -b option will revert to the original behavior.
> 
> Signed-off-by: Gaetan Rivet <gaetan.rivet@6wind.com>
> ---
> v2: justify this default behavior evolution.
> ---

I don't have major objections to this patch, though it does make it
mandatory to use port parameters where before it was not. The one
suggestion I will make is that, if we take this approach, we should
probably add a --wl-all (whitelist-all) flag to go back to having all
ports automatically bound, if so desired.

/Bruce