From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 7FE1EB39F for ; Mon, 22 Sep 2014 14:16:17 +0200 (CEST) Received: from [2001:470:8:a08:18c5:c64e:4bf:67a] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1XW2dA-0002v7-Tk; Mon, 22 Sep 2014 08:22:18 -0400 Date: Mon, 22 Sep 2014 08:22:11 -0400 From: Neil Horman To: David Marchand Message-ID: <20140922122211.GB25406@hmsreliant.think-freely.org> References: <1411375081-27986-1-git-send-email-david.marchand@6wind.com> <1411375081-27986-2-git-send-email-david.marchand@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1411375081-27986-2-git-send-email-david.marchand@6wind.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Spam-Score: -2.9 (--) X-Spam-Status: No Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 1/7] eal: remove unused --use-device option X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 22 Sep 2014 12:16:17 -0000 On Mon, Sep 22, 2014 at 10:37:55AM +0200, David Marchand wrote: > Following commit cac6d08c8bde2fdb57806c49038187cdb54219a8 and > 4bf3fe634a4d9dfce90c4167f3a47d0e2ddf1e64, this option is not available anymore. > > Signed-off-by: David Marchand > --- > lib/librte_eal/bsdapp/eal/eal.c | 7 ------- > lib/librte_eal/linuxapp/eal/eal.c | 7 ------- > 2 files changed, 14 deletions(-) > > diff --git a/lib/librte_eal/bsdapp/eal/eal.c b/lib/librte_eal/bsdapp/eal/eal.c > index 2f84742..ffdc441 100644 > --- a/lib/librte_eal/bsdapp/eal/eal.c > +++ b/lib/librte_eal/bsdapp/eal/eal.c > @@ -89,7 +89,6 @@ > #define OPT_NO_HUGE "no-huge" > #define OPT_FILE_PREFIX "file-prefix" > #define OPT_SOCKET_MEM "socket-mem" > -#define OPT_USE_DEVICE "use-device" > #define OPT_PCI_WHITELIST "pci-whitelist" > #define OPT_PCI_BLACKLIST "pci-blacklist" > #define OPT_VDEV "vdev" > @@ -645,12 +644,6 @@ eal_parse_args(int argc, char **argv) > "FreeBSD\n"); > return -1; > } > - else if (!strcmp(lgopts[option_index].name, OPT_USE_DEVICE)) { > - printf("The --use-device option is deprecated, please use\n" > - "--whitelist or --vdev instead.\n"); > - eal_usage(prgname); > - return -1; > - } > else if (!strcmp(lgopts[option_index].name, OPT_PCI_BLACKLIST)) { > if (rte_eal_devargs_add(RTE_DEVTYPE_BLACKLISTED_PCI, > optarg) < 0) { > diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c > index 38cace6..633e3b8 100644 > --- a/lib/librte_eal/linuxapp/eal/eal.c > +++ b/lib/librte_eal/linuxapp/eal/eal.c > @@ -92,7 +92,6 @@ > #define OPT_NO_HUGE "no-huge" > #define OPT_FILE_PREFIX "file-prefix" > #define OPT_SOCKET_MEM "socket-mem" > -#define OPT_USE_DEVICE "use-device" > #define OPT_PCI_WHITELIST "pci-whitelist" > #define OPT_PCI_BLACKLIST "pci-blacklist" > #define OPT_VDEV "vdev" > @@ -880,12 +879,6 @@ eal_parse_args(int argc, char **argv) > return -1; > } > } > - else if (!strcmp(lgopts[option_index].name, OPT_USE_DEVICE)) { > - printf("The --use-device option is deprecated, please use\n" > - "--whitelist or --vdev instead.\n"); > - eal_usage(prgname); > - return -1; > - } > else if (!strcmp(lgopts[option_index].name, OPT_PCI_BLACKLIST)) { > if (rte_eal_devargs_add(RTE_DEVTYPE_BLACKLISTED_PCI, > optarg) < 0) { > -- > 1.7.10.4 > > just a note here - This usage is great reflection of deprecation policy, we deprecated this back in the 1.6 time frame, left it around for the 1.7 release, and remove it for the 1.8 release. I'd love to add some deprecation functionality to the rte_compat stuff in my patch series on the list so that we can better announe the deprecation of features/flags/API's in a more general fashion. Neil