From patchwork Sun May 3 20:31:31 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Marchand X-Patchwork-Id: 69682 X-Patchwork-Delegate: david.marchand@redhat.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6D058A04AF; Sun, 3 May 2020 22:32:51 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D41691D532; Sun, 3 May 2020 22:32:25 +0200 (CEST) Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com [205.139.110.120]) by dpdk.org (Postfix) with ESMTP id 6FDBC1D52B for ; Sun, 3 May 2020 22:32:24 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1588537943; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=jwLArlOAImSvqi3uQvKcXQv4IO8pCHM1WmwREFG8XMQ=; b=FvWPDngn/pyU1HLD2EtMEfYEjsm26pfs7DIl+JMPfwb+TWiKLgQgRR0c0dskvWUJKOb/a6 0cjdPRVbAc7AeeWKxr3jaM/1YWOjXb10wi6lriysQE/SK1l1PyFHC7N1LGdQRUuqR3AhH6 nntKKwbcFhYkvh3p+ude3fCxEmIc/D8= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-166-LnhfKoBkO6OPE7Kjmb5sCg-1; Sun, 03 May 2020 16:32:20 -0400 X-MC-Unique: LnhfKoBkO6OPE7Kjmb5sCg-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 755CD1895944; Sun, 3 May 2020 20:32:19 +0000 (UTC) Received: from dmarchan.remote.csb (unknown [10.40.192.236]) by smtp.corp.redhat.com (Postfix) with ESMTP id 9BC3A6F972; Sun, 3 May 2020 20:32:17 +0000 (UTC) From: David Marchand To: dev@dpdk.org Cc: thomas@monjalon.net, Jerin Jacob , Sunil Kumar Kori Date: Sun, 3 May 2020 22:31:31 +0200 Message-Id: <20200503203135.6493-5-david.marchand@redhat.com> In-Reply-To: <20200503203135.6493-1-david.marchand@redhat.com> References: <20200503203135.6493-1-david.marchand@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Subject: [dpdk-dev] [PATCH 4/8] trace: avoid confusion on optarg 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" Prefer a local name to optarg which is a global symbol from the C library. Fixes: 8c8066ea6a7b ("trace: add trace mode configuration parameter") Signed-off-by: David Marchand Acked-by: Sunil Kumar Kori --- .../common/eal_common_trace_utils.c | 32 +++++++++---------- lib/librte_eal/common/eal_trace.h | 8 ++--- 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/lib/librte_eal/common/eal_common_trace_utils.c b/lib/librte_eal/common/eal_common_trace_utils.c index fce8892c38..a7c5893b00 100644 --- a/lib/librte_eal/common/eal_common_trace_utils.c +++ b/lib/librte_eal/common/eal_common_trace_utils.c @@ -135,7 +135,7 @@ trace_dir_update(const char *str) } int -eal_trace_args_save(const char *optarg) +eal_trace_args_save(const char *val) { struct trace *trace = trace_obj_get(); char *trace_args; @@ -144,17 +144,17 @@ eal_trace_args_save(const char *optarg) nb_args = trace->args.nb_args; if (nb_args >= TRACE_MAX_ARGS) { - trace_err("ignoring trace %s as limit exceeds", optarg); + trace_err("ignoring trace %s as limit exceeds", val); return 0; } - trace_args = calloc(1, (strlen(optarg) + 1)); + trace_args = calloc(1, (strlen(val) + 1)); if (trace_args == NULL) { - trace_err("fail to allocate memory for %s", optarg); + trace_err("fail to allocate memory for %s", val); return -ENOMEM; } - memcpy(trace_args, optarg, strlen(optarg)); + memcpy(trace_args, val, strlen(val)); trace->args.args[nb_args++] = trace_args; trace->args.nb_args = nb_args; return 0; @@ -194,17 +194,17 @@ trace_args_apply(const char *arg) } int -eal_trace_bufsz_args_save(char const *optarg) +eal_trace_bufsz_args_save(char const *val) { struct trace *trace = trace_obj_get(); uint64_t bufsz; - if (optarg == NULL) { + if (val == NULL) { trace_err("no optarg is passed"); return -EINVAL; } - bufsz = rte_str_to_size(optarg); + bufsz = rte_str_to_size(val); if (bufsz == 0) { trace_err("buffer size cannot be zero"); return -EINVAL; @@ -224,14 +224,14 @@ trace_bufsz_args_apply(void) } int -eal_trace_mode_args_save(const char *optarg) +eal_trace_mode_args_save(const char *val) { struct trace *trace = trace_obj_get(); - size_t len = strlen(optarg); + size_t len = strlen(val); unsigned long tmp; char *pattern; - if (optarg == NULL) { + if (val == NULL) { trace_err("no optarg is passed"); return -EINVAL; } @@ -247,7 +247,7 @@ eal_trace_mode_args_save(const char *optarg) return -ENOMEM; } - sprintf(pattern, "%s*", optarg); + sprintf(pattern, "%s*", val); if (fnmatch(pattern, "overwrite", 0) == 0) tmp = RTE_TRACE_MODE_OVERWRITE; @@ -264,19 +264,19 @@ eal_trace_mode_args_save(const char *optarg) } int -eal_trace_dir_args_save(char const *optarg) +eal_trace_dir_args_save(char const *val) { struct trace *trace = trace_obj_get(); uint32_t size = sizeof(trace->dir); char *dir_path = NULL; int rc; - if (optarg == NULL) { + if (val == NULL) { trace_err("no optarg is passed"); return -EINVAL; } - if (strlen(optarg) >= size) { + if (strlen(val) >= size) { trace_err("input string is too big"); return -ENAMETOOLONG; } @@ -287,7 +287,7 @@ eal_trace_dir_args_save(char const *optarg) return -ENOMEM; } - sprintf(dir_path, "%s/", optarg); + sprintf(dir_path, "%s/", val); rc = trace_dir_update(dir_path); free(dir_path); diff --git a/lib/librte_eal/common/eal_trace.h b/lib/librte_eal/common/eal_trace.h index f53ccc0d50..7d95bd2aa9 100644 --- a/lib/librte_eal/common/eal_trace.h +++ b/lib/librte_eal/common/eal_trace.h @@ -111,10 +111,10 @@ void trace_mem_per_thread_free(void); /* EAL interface */ int eal_trace_init(void); void eal_trace_fini(void); -int eal_trace_args_save(const char *optarg); +int eal_trace_args_save(const char *val); void eal_trace_args_free(void); -int eal_trace_dir_args_save(const char *optarg); -int eal_trace_mode_args_save(const char *optarg); -int eal_trace_bufsz_args_save(const char *optarg); +int eal_trace_dir_args_save(const char *val); +int eal_trace_mode_args_save(const char *val); +int eal_trace_bufsz_args_save(const char *val); #endif /* __EAL_TRACE_H */