From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 90315A04B5; Thu, 29 Oct 2020 16:35:38 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 143D2D041; Thu, 29 Oct 2020 16:32:04 +0100 (CET) Received: from mail-pf1-f195.google.com (mail-pf1-f195.google.com [209.85.210.195]) by dpdk.org (Postfix) with ESMTP id 943C4D01C for ; Thu, 29 Oct 2020 16:32:02 +0100 (CET) Received: by mail-pf1-f195.google.com with SMTP id w21so2614555pfc.7 for ; Thu, 29 Oct 2020 08:32:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:subject:message-id:in-reply-to:references:mime-version :content-transfer-encoding; bh=CpDpN/4hQZaVG2fYg+BgeFD2f7XDmAk16oPyeuaSTG8=; b=jb5XlP9T+2cvT8CF2CPa0DxRf8cYs+2H4ie69LDRZQ4ic96J58rz7T+XN/92VXbPmg 7hRcDfVhg5t3cw+yE7bFL/hH8hYNhMoH0HP3cieFyZZSfLsU6wvGBGnvStpQVnUraz8z EfSxYGJaGIz+CDqUs/MuH58tFEV+eOwyVFqMFgPTD74eL80APLtG2AYyTGBLWgIiaqoR PjitFS5Ixil/QkPRUtIMTMn8kCGehfaWU6D2rR+U58Wdx06gApy5+hpyzuHC5Yne5tZJ Mph/ME7M7CRywt9svKTeKnHoPJmrolWt3KjRYNK86CXjv/qpYeU5dRm0fmpawlS5e2NB DlhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=CpDpN/4hQZaVG2fYg+BgeFD2f7XDmAk16oPyeuaSTG8=; b=JoFk9yL+nOrdtdx2DdzOQU55hRsDlT+usXtAK3M3pXplJdgErsh0kTejg7F96zrABR y0tldMCAfyktbo2SpyKgiycYQ5nE8LOWPsxWO4dbtcsGGy7LCZwSm7SJ1WZonHAI1yYt 9oWzt5M+oaIqSB6+ZR2ZaVBwrtsmG4p2a48wawxTDPhvLm9WxpWF6XOJl8zApI4NcqN2 jc6S8eLHrWHnHNjzTFUJQYmXC4259If5aiR2DnAmHhdCaF+X+bNsMj9+fY81djMtwGpj uDwNT0R2S3AwMI0+ENoJADvjZPQ0POUHrdvkbx8S/owvuAVCJUkYRkZZeffFATAI47q6 EFWQ== X-Gm-Message-State: AOAM533EGUrlwMdw3GF9+JyLgiYQklbUppdPpGfEiBtbb81luKtc98iQ zf9xYnk0+GNft9AWx+C0hUoWsWL+F6uiyjrp X-Google-Smtp-Source: ABdhPJyv4+laTQhQ4ldHcnAIHC9JW4Dzvo3j5GYE7rlUOjxHmLUAYtjh44Sv7Fo3RqAKvwsx4kvMgg== X-Received: by 2002:a17:90a:7089:: with SMTP id g9mr384436pjk.4.1603985520349; Thu, 29 Oct 2020 08:32:00 -0700 (PDT) Received: from hermes.local (204-195-22-127.wavecable.com. [204.195.22.127]) by smtp.gmail.com with ESMTPSA id a17sm3415409pfo.108.2020.10.29.08.31.59 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Oct 2020 08:32:00 -0700 (PDT) Date: Thu, 29 Oct 2020 08:31:57 -0700 From: Stephen Hemminger To: dev@dpdk.org Message-ID: <20201029083157.58d84bb8@hermes.local> In-Reply-To: <20201025211518.531-1-stephen@networkplumber.org> References: <20200922143202.8755-1-stephen@networkplumber.org> <20201025211518.531-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v8 0/5] replace blacklist/whitelist with allow/block X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Sun, 25 Oct 2020 14:15:13 -0700 Stephen Hemminger wrote: > This patch series changes the use of the terms blacklist and whitelist > and instead uses block and allow. The change is most visible > in the options passed to DPDK programs for marking which devices > to use. > > The options are also shortened because they can be used for > not just PCI devices. When DPDK was expanded to support other bus > types the -b option was also uses for non-PCI devices. But the > documentation and the name of the long option never changed. > > v8 - incorporate more of the feedback from David Marchand > fix the pci-blacklist option number and reformat the usage > (All the usage message should be fixed format). > > v7 - fix options handling changes in test code. > this should fix the CI failures. > > v6 - fix some confusion in handling options that was introduced > by earlier versions of the patch. > > Stephen Hemminger (5): > eal: replace usage of blacklist/whitelist in enum > drivers: replace references to blacklist > eal: replace pci-whitelist/pci-blacklist options > doc: change references to blacklist and whitelist > app/test: use new allowlist and blocklist > > app/test/autotest.py | 16 ++--- > app/test/autotest_runner.py | 18 ++--- > app/test/test.c | 2 +- > app/test/test_eal_flags.c | 53 +++++++-------- > doc/guides/cryptodevs/dpaa2_sec.rst | 6 +- > doc/guides/cryptodevs/dpaa_sec.rst | 6 +- > doc/guides/cryptodevs/qat.rst | 12 ++-- > doc/guides/eventdevs/octeontx2.rst | 20 +++--- > doc/guides/freebsd_gsg/build_sample_apps.rst | 2 +- > doc/guides/linux_gsg/build_sample_apps.rst | 2 +- > doc/guides/linux_gsg/eal_args.include.rst | 14 ++-- > doc/guides/linux_gsg/linux_drivers.rst | 4 +- > doc/guides/mempool/octeontx2.rst | 4 +- > doc/guides/nics/bnxt.rst | 18 ++--- > doc/guides/nics/cxgbe.rst | 12 ++-- > doc/guides/nics/dpaa.rst | 6 +- > doc/guides/nics/dpaa2.rst | 6 +- > doc/guides/nics/enic.rst | 6 +- > doc/guides/nics/fail_safe.rst | 16 ++--- > doc/guides/nics/features.rst | 2 +- > doc/guides/nics/i40e.rst | 16 ++--- > doc/guides/nics/ice.rst | 28 +++++--- > doc/guides/nics/ixgbe.rst | 4 +- > doc/guides/nics/mlx4.rst | 18 ++--- > doc/guides/nics/mlx5.rst | 14 ++-- > doc/guides/nics/nfb.rst | 2 +- > doc/guides/nics/octeontx2.rst | 23 +++---- > doc/guides/nics/sfc_efx.rst | 2 +- > doc/guides/nics/tap.rst | 2 +- > doc/guides/nics/thunderx.rst | 4 +- > .../prog_guide/env_abstraction_layer.rst | 6 +- > doc/guides/prog_guide/multi_proc_support.rst | 4 +- > doc/guides/prog_guide/poll_mode_drv.rst | 6 +- > .../prog_guide/switch_representation.rst | 6 +- > doc/guides/rel_notes/release_20_11.rst | 5 ++ > doc/guides/sample_app_ug/bbdev_app.rst | 14 ++-- > .../sample_app_ug/eventdev_pipeline.rst | 4 +- > doc/guides/sample_app_ug/ipsec_secgw.rst | 12 ++-- > doc/guides/sample_app_ug/l3_forward.rst | 7 +- > .../sample_app_ug/l3_forward_access_ctrl.rst | 2 +- > .../sample_app_ug/l3_forward_power_man.rst | 3 +- > doc/guides/sample_app_ug/vdpa.rst | 2 +- > doc/guides/tools/cryptoperf.rst | 6 +- > doc/guides/tools/flow-perf.rst | 2 +- > doc/guides/tools/testregex.rst | 2 +- > drivers/bus/dpaa/dpaa_bus.c | 7 +- > drivers/bus/fslmc/fslmc_bus.c | 9 ++- > drivers/bus/fslmc/fslmc_vfio.c | 12 ++-- > drivers/bus/pci/pci_common.c | 24 +++---- > drivers/bus/vmbus/vmbus_common.c | 4 +- > drivers/crypto/virtio/virtio_pci.c | 2 +- > drivers/net/virtio/virtio_pci.c | 2 +- > lib/librte_eal/common/eal_common_devargs.c | 14 ++-- > lib/librte_eal/common/eal_common_options.c | 66 +++++++++++-------- > lib/librte_eal/common/eal_options.h | 13 +++- > lib/librte_eal/include/rte_bus.h | 10 ++- > lib/librte_eal/include/rte_dev.h | 10 ++- > lib/librte_eal/include/rte_devargs.h | 10 ++- > 58 files changed, 329 insertions(+), 273 deletions(-) > Any comments on this? What is timeline for merge?