From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f54.google.com (mail-wm0-f54.google.com [74.125.82.54]) by dpdk.org (Postfix) with ESMTP id 4E4171B208 for ; Thu, 12 Oct 2017 12:55:59 +0200 (CEST) Received: by mail-wm0-f54.google.com with SMTP id u138so12260400wmu.4 for ; Thu, 12 Oct 2017 03:55:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=R5Kf3Q31vQTEBmixX5dHaN84MZcxuznT5gpg+clxBkU=; b=acgCM6bJvFsUMN88jCSr5RJ6E3Ij4OHqvxwPNCMuTvqrIOKbGUGVpAAr7NNho2W2y9 soelvnxn6lziQ9j7fA4+5ly0249D5gTonyyP0USWzaKk8zfDaG+4ahQSz7jUHEDilCwU xXhuvF5LkxYfqQssnmc26iSikV4HvpzCysKnPvnfehzKImjb6kms53ffOcywkeqf/FHb q8TMvRWpJ7E7jSytYsMvc8ux1c9XoA9IkrCmqnWTPKo0mk7VWxJfAt+WDnDukX97x3gf 5bANhUVS2fXMn5FqR+nftGYocdvcIan9NiL4TNxeF8g2CzctnJsIdHGkk3hpdamNUHb1 u+gw== 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:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=R5Kf3Q31vQTEBmixX5dHaN84MZcxuznT5gpg+clxBkU=; b=sUoqlmZUxe1zA22r3xVKJiPzJKsoK9fhy88E719srWNe50HqXWUl7QfkLLm4oTywR5 78okc/+LZX+JLfegi5TZ6RcMIodYS9FOH76c1hJEIgpdFmB3ofe4Y1Sbz/kMjVHnh2Ep cQhbkM97RBinuMGIS5hbL7A3l/5e+oItyJZthSO0DHfslSQSztAvu/fS29SZFY8RUOUd Yxwv8X26r7np8I3tW72nkbvAozNAxHMy659m4Z0heml4VHWyWJLVcTi//nY5dMcnWNOT UZqFD6hjX/vUnexOzsHp0SSmasup+WAu1GDCnIvtLWInUNkwALH57QfRNi3MCaPfh8x4 nRWQ== X-Gm-Message-State: AMCzsaVjPJDKC6RuLoB72GHzkgmwjWoH1zz0JpD7MkPXyZXErXWlROOh ghENnu4CKzp9EuVEw4Wa0FAlyOf4 X-Google-Smtp-Source: AOwi7QByzAb5tsAlJxaUkO8AYuUQ92QwEpeBhM+GyPq1IMNCVOCQ0TQBn2E/PpRbCdKIC9ph+bIj3Q== X-Received: by 10.28.36.212 with SMTP id k203mr1375475wmk.127.1507805758365; Thu, 12 Oct 2017 03:55:58 -0700 (PDT) Received: from bidouze.vm.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id r63sm20301455wmg.13.2017.10.12.03.55.57 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 12 Oct 2017 03:55:57 -0700 (PDT) Date: Thu, 12 Oct 2017 12:55:46 +0200 From: =?iso-8859-1?Q?Ga=EBtan?= Rivet To: dev@dpdk.org Message-ID: <20171012105546.GB3408@bidouze.vm.6wind.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v5 20/20] bus: rename scan policy as probe policy 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: , X-List-Received-Date: Thu, 12 Oct 2017 10:55:59 -0000 This patch is not part of this series, sorry. I can send a v6 if necessary, but here simply ignore this patch. On Thu, Oct 12, 2017 at 12:45:47PM +0200, Gaetan Rivet wrote: > This bus configuration item is misnamed, as it actually refers to the > probing process. > > Signed-off-by: Gaetan Rivet > --- > drivers/bus/pci/pci_common.c | 2 +- > lib/librte_eal/common/eal_common_devargs.c | 6 +++--- > lib/librte_eal/common/include/rte_bus.h | 12 ++++++------ > 3 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/drivers/bus/pci/pci_common.c b/drivers/bus/pci/pci_common.c > index d7a1c05..cc23a39 100644 > --- a/drivers/bus/pci/pci_common.c > +++ b/drivers/bus/pci/pci_common.c > @@ -373,7 +373,7 @@ rte_pci_probe(void) > int probe_all = 0; > int ret = 0; > > - if (rte_pci_bus.bus.conf.scan_mode != RTE_BUS_SCAN_WHITELIST) > + if (rte_pci_bus.bus.conf.probe_mode != RTE_BUS_PROBE_WHITELIST) > probe_all = 1; > > FOREACH_DEVICE_ON_PCIBUS(dev) { > diff --git a/lib/librte_eal/common/eal_common_devargs.c b/lib/librte_eal/common/eal_common_devargs.c > index 6ac88d6..f5ef913 100644 > --- a/lib/librte_eal/common/eal_common_devargs.c > +++ b/lib/librte_eal/common/eal_common_devargs.c > @@ -170,11 +170,11 @@ rte_eal_devargs_add(enum rte_devtype devtype, const char *devargs_str) > bus = devargs->bus; > if (devargs->type == RTE_DEVTYPE_BLACKLISTED_PCI) > devargs->policy = RTE_DEV_BLACKLISTED; > - if (bus->conf.scan_mode == RTE_BUS_SCAN_UNDEFINED) { > + if (bus->conf.probe_mode == RTE_BUS_PROBE_UNDEFINED) { > if (devargs->policy == RTE_DEV_WHITELISTED) > - bus->conf.scan_mode = RTE_BUS_SCAN_WHITELIST; > + bus->conf.probe_mode = RTE_BUS_PROBE_WHITELIST; > else if (devargs->policy == RTE_DEV_BLACKLISTED) > - bus->conf.scan_mode = RTE_BUS_SCAN_BLACKLIST; > + bus->conf.probe_mode = RTE_BUS_PROBE_BLACKLIST; > } > TAILQ_INSERT_TAIL(&devargs_list, devargs, next); > return 0; > diff --git a/lib/librte_eal/common/include/rte_bus.h b/lib/librte_eal/common/include/rte_bus.h > index 6fb0834..331d954 100644 > --- a/lib/librte_eal/common/include/rte_bus.h > +++ b/lib/librte_eal/common/include/rte_bus.h > @@ -168,19 +168,19 @@ typedef int (*rte_bus_unplug_t)(struct rte_device *dev); > typedef int (*rte_bus_parse_t)(const char *name, void *addr); > > /** > - * Bus scan policies > + * Bus probe policies > */ > -enum rte_bus_scan_mode { > - RTE_BUS_SCAN_UNDEFINED, > - RTE_BUS_SCAN_WHITELIST, > - RTE_BUS_SCAN_BLACKLIST, > +enum rte_bus_probe_mode { > + RTE_BUS_PROBE_UNDEFINED, > + RTE_BUS_PROBE_WHITELIST, > + RTE_BUS_PROBE_BLACKLIST, > }; > > /** > * A structure used to configure bus operations. > */ > struct rte_bus_conf { > - enum rte_bus_scan_mode scan_mode; /**< Scan policy. */ > + enum rte_bus_probe_mode probe_mode; /**< Probe policy. */ > }; > > > -- > 2.1.4 > -- Gaëtan Rivet 6WIND