[dpdk-dev] [PATCH v2 1/1] pci: default to whitelist mode
Gaëtan Rivet
gaetan.rivet at 6wind.com
Tue Mar 28 14:44:09 CEST 2017
On Tue, Mar 28, 2017 at 01:20:00PM +0100, Bruce Richardson wrote:
>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 at 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.
>
Are there use cases where the blacklist mode would be used without
blacklisting any device? The current -b option is almost enough for the
same level of functionality.
If there is an actual need to a full PCI probe, adding this option is
certainly possible. I was thinking otherwise of allowing "all" as an
argument to -w, which would have our users using -wall or -w=all, which
seems clear enough. This would essentially be the inverse of the
--no-pci parameter.
Which could probably be removed if this patch is accepted.
--
Gaëtan Rivet
6WIND
More information about the dev
mailing list