07/07/2017 02:04, Gaetan Rivet: > The use of these commands have evolved. > It is now possible to use whitelist and blacklist on any bus. > > Update the source code accordingly. > > Signed-off-by: Gaetan Rivet <gaetan.ri...@6wind.com> > Acked-by: John McNamara <john.mcnam...@intel.com> > --- [...] > --- a/lib/librte_eal/common/eal_options.h > +++ b/lib/librte_eal/common/eal_options.h > @@ -37,10 +37,10 @@ enum { > /* long options mapped to a short option */ > #define OPT_HELP "help" > OPT_HELP_NUM = 'h', > -#define OPT_PCI_BLACKLIST "pci-blacklist" > - OPT_PCI_BLACKLIST_NUM = 'b', > -#define OPT_PCI_WHITELIST "pci-whitelist" > - OPT_PCI_WHITELIST_NUM = 'w', > +#define OPT_BUS_BLACKLIST "bus-blacklist" > + OPT_BUS_BLACKLIST_NUM = 'b', > +#define OPT_BUS_WHITELIST "bus-whitelist" > + OPT_BUS_WHITELIST_NUM = 'w',
No, we cannot drop an existing option without a prior notice. Let's discuss what should be the command line syntax and agree on a good and complete deprecation notice. There will be more to change, like making bus explicit.