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 3A26AA00C3; Mon, 8 Jun 2020 11:31:11 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 32B4E1BDAE; Mon, 8 Jun 2020 11:31:10 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id AED2D49E0 for ; Mon, 8 Jun 2020 11:31:08 +0200 (CEST) IronPort-SDR: ZlVQptxyhKBCMM86K+ckkzEkhg9OYEjRSAZh54BnEdbOrIlwlc85NPDMidSgVnqw1ZDOPVRtmX 6XjFVlPTtIyg== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jun 2020 02:31:03 -0700 IronPort-SDR: tn08TqViJ1gQxYsHbgPQBvH1YE2ighDtAFp5FbNwjIwANMzrI46csCt07rO06i+q3y4pW0Qafm 5anwpcSdEPrQ== X-IronPort-AV: E=Sophos;i="5.73,487,1583222400"; d="scan'208";a="446670769" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.25.93]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 08 Jun 2020 02:31:02 -0700 Date: Mon, 8 Jun 2020 10:30:59 +0100 From: Bruce Richardson To: Stephen Hemminger Cc: dev@dpdk.org Message-ID: <20200608093059.GA1560@bricha3-MOBL.ger.corp.intel.com> References: <20200607170127.15694-1-stephen@networkplumber.org> <20200607170127.15694-7-stephen@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200607170127.15694-7-stephen@networkplumber.org> Subject: Re: [dpdk-dev] [PATCH 6/9] eal: replace pci-whitelist/pci-blacklist options 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, Jun 07, 2020 at 10:01:24AM -0700, Stephen Hemminger wrote: > Replace pci-whitelist with pci-allowlist and pci-blacklist with > pci-blocklist. Allow the old options for now, but make sure help > and all tests will use the new options. > > The short flags are more commonly used and changing them > would impact more documentation and applications. > The -b flag can be described as "blocklist" rather than "blacklist. > The -w flag can be thought of as "with" instead "whitelist". > > Signed-off-by: Stephen Hemminger > --- Rather than trying to replace blacklist/whitelist with similar sounding words in blocklist/allowlist, might it be clearer to use more different terms that have a better description. How about: "pci-skip-probe" and "pci-only-probe" or something along those lines?