DPDK patches and discussions
 help / color / mirror / Atom feed
From: Neil Horman <nhorman@tuxdriver.com>
To: David Marchand <david.marchand@6wind.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH 1/7] eal: remove unused --use-device option
Date: Mon, 22 Sep 2014 08:22:11 -0400
Message-ID: <20140922122211.GB25406@hmsreliant.think-freely.org> (raw)
In-Reply-To: <1411375081-27986-2-git-send-email-david.marchand@6wind.com>

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 <david.marchand@6wind.com>
> ---
>  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

  reply	other threads:[~2014-09-22 12:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-22  8:37 [dpdk-dev] [PATCH 0/7] cleanup option parsing in bsd/linux eal David Marchand
2014-09-22  8:37 ` [dpdk-dev] [PATCH 1/7] eal: remove unused --use-device option David Marchand
2014-09-22 12:22   ` Neil Horman [this message]
2014-09-22 13:49     ` David Marchand
2014-09-22  8:37 ` [dpdk-dev] [PATCH 2/7] eal: factorise unsupported option handling David Marchand
2014-09-22  8:37 ` [dpdk-dev] [PATCH 3/7] eal: remove duplicate handling of white/black list David Marchand
2014-09-22  8:37 ` [dpdk-dev] [PATCH 4/7] eal: fix checkpatch issues before moving code David Marchand
2014-09-22  8:37 ` [dpdk-dev] [PATCH 5/7] eal: merge bsd and linux common options parsing David Marchand
2014-09-22  8:38 ` [dpdk-dev] [PATCH 6/7] eal: rework long " David Marchand
2014-09-22 12:42   ` Neil Horman
2014-09-22  8:38 ` [dpdk-dev] [PATCH 7/7] eal: indent files David Marchand
2014-09-22 12:43 ` [dpdk-dev] [PATCH 0/7] cleanup option parsing in bsd/linux eal Neil Horman
2014-09-23 15:37   ` Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140922122211.GB25406@hmsreliant.think-freely.org \
    --to=nhorman@tuxdriver.com \
    --cc=david.marchand@6wind.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git