From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 606B7A0C43; Tue, 28 Sep 2021 21:47:45 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0C419410EF; Tue, 28 Sep 2021 21:47:42 +0200 (CEST) Received: from mail-lf1-f45.google.com (mail-lf1-f45.google.com [209.85.167.45]) by mails.dpdk.org (Postfix) with ESMTP id E871840E3C for ; Tue, 28 Sep 2021 21:47:37 +0200 (CEST) Received: by mail-lf1-f45.google.com with SMTP id y26so682799lfa.11 for ; Tue, 28 Sep 2021 12:47:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=u0zBhMPg2UZxqNoTpQSZS/GYKyQB7o4+MX6FZQGLcQU=; b=pF+5ChVq1jLhPuyCr3N3uN4Jogx8EfJ5sCf12WHFUsgdZzpijBBnTNPjlg6shgR7BI qg5yYsQkjFhue74NWOLzhZ2KZu3g/dt26ihyZ1ABprYMZCFn4qEvO0QrMjpQ2ItsPMJl f6oL96aP7Lc5aKhVo1u1k6AXdkiI09IGNC4kUURWViMEY+Ijk5eaiQVBc01zEaGq3puh cFgpw4QPjMk1ocahaxmWZsjgoU6l2MbmCCGdKD6NgfV+pusvCh+B0hvHTtqYE/xHBEX7 /znkpuicinPhBXqPqS7/arS856gKHGqT5AjYpsfcCo+DpWLfgU/YmFnpf8osV/FyLCYf dFOw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=u0zBhMPg2UZxqNoTpQSZS/GYKyQB7o4+MX6FZQGLcQU=; b=3onMK0s5hEl5hdPGUIM7d3cTqRX47sG3OpsSC+lpwVOKtIAxn3/tqpz74VE9BmjDd2 GqyrYvRRZCuHzpd/VW+YwSpYKWF3tMjPS8syLZDC1ZIbmIPA/vLzyq99l+l/qBJI3LRe B6wDw+u3dxb7JwhR4Jqmf+HQXxe0VwYd2LmNQbafktq+U3/8HcZ8oOGJbHhfb3s/FA82 J45hynZqFoSXJacPiNI+tzFNMESxOzQYTMjD8bCXBuIfreyxvtRSHZXKE9SsfXvJTgZp M6EzzzK3hkAxPEg2I8V1k5d4ArtrBpZ9hiNhrERPhqELZ+him3+HIUEdU2S2h5MoQTfE FGJg== X-Gm-Message-State: AOAM530HwIQZauNtU6KFaceMvLyQL3VgHb36YyyItBHZgadz64gwDmTD MMULE0R3UE9n/pokgAZzxQd8jqHCH0s= X-Google-Smtp-Source: ABdhPJynTsg6XeE35dcHQ4eMMyDWu8SHT0XnmPP3VRokZVr+VVTdW1fViFDSZgIsOyyg5xtiktHd/A== X-Received: by 2002:a05:6512:10c3:: with SMTP id k3mr5714131lfg.72.1632858457261; Tue, 28 Sep 2021 12:47:37 -0700 (PDT) Received: from sovereign.. (broadband-37-110-65-23.ip.moscow.rt.ru. [37.110.65.23]) by smtp.gmail.com with ESMTPSA id 8sm1871ljf.39.2021.09.28.12.47.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Sep 2021 12:47:36 -0700 (PDT) From: Dmitry Kozlyuk To: dev@dpdk.org Cc: David Marchand , Olivier Matz , Dmitry Kozlyuk Date: Tue, 28 Sep 2021 22:47:13 +0300 Message-Id: <20210928194714.365563-2-dmitry.kozliuk@gmail.com> X-Mailer: git-send-email 2.29.3 In-Reply-To: <20210928194714.365563-1-dmitry.kozliuk@gmail.com> References: <20210910231639.789066-2-dmitry.kozliuk@gmail.com> <20210928194714.365563-1-dmitry.kozliuk@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v2 1/2] cmdline: make struct cmdline opaque X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 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" Remove the definition of `struct cmdline` from public header. Deprecation notice: https://mails.dpdk.org/archives/dev/2020-September/183310.html Signed-off-by: Dmitry Kozlyuk Acked-by: David Marchand Acked-by: Olivier Matz --- doc/guides/rel_notes/deprecation.rst | 4 ---- doc/guides/rel_notes/release_21_11.rst | 2 ++ lib/cmdline/cmdline.h | 19 ------------------- lib/cmdline/cmdline_private.h | 8 +++++++- 4 files changed, 9 insertions(+), 24 deletions(-) diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index 76a4abfd6b..a404276fa2 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -275,10 +275,6 @@ Deprecation Notices * metrics: The function ``rte_metrics_init`` will have a non-void return in order to notify errors instead of calling ``rte_exit``. -* cmdline: ``cmdline`` structure will be made opaque to hide platform-specific - content. On Linux and FreeBSD, supported prior to DPDK 20.11, - original structure will be kept until DPDK 21.11. - * security: The functions ``rte_security_set_pkt_metadata`` and ``rte_security_get_userdata`` will be made inline functions and additional flags will be added in structure ``rte_security_ctx`` in DPDK 21.11. diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst index b55900936d..18377e5813 100644 --- a/doc/guides/rel_notes/release_21_11.rst +++ b/doc/guides/rel_notes/release_21_11.rst @@ -101,6 +101,8 @@ API Changes Also, make sure to start the actual text at the margin. ======================================================= +* cmdline: Made ``cmdline`` structure definition hidden on Linux and FreeBSD. + ABI Changes ----------- diff --git a/lib/cmdline/cmdline.h b/lib/cmdline/cmdline.h index c29762ddae..96674dfda2 100644 --- a/lib/cmdline/cmdline.h +++ b/lib/cmdline/cmdline.h @@ -7,10 +7,6 @@ #ifndef _CMDLINE_H_ #define _CMDLINE_H_ -#ifndef RTE_EXEC_ENV_WINDOWS -#include -#endif - #include #include @@ -27,23 +23,8 @@ extern "C" { #endif -#ifndef RTE_EXEC_ENV_WINDOWS - -struct cmdline { - int s_in; - int s_out; - cmdline_parse_ctx_t *ctx; - struct rdline rdl; - char prompt[RDLINE_PROMPT_SIZE]; - struct termios oldterm; -}; - -#else - struct cmdline; -#endif /* RTE_EXEC_ENV_WINDOWS */ - struct cmdline *cmdline_new(cmdline_parse_ctx_t *ctx, const char *prompt, int s_in, int s_out); void cmdline_set_prompt(struct cmdline *cl, const char *prompt); void cmdline_free(struct cmdline *cl); diff --git a/lib/cmdline/cmdline_private.h b/lib/cmdline/cmdline_private.h index a87c45275c..2e93674c66 100644 --- a/lib/cmdline/cmdline_private.h +++ b/lib/cmdline/cmdline_private.h @@ -11,6 +11,8 @@ #include #ifdef RTE_EXEC_ENV_WINDOWS #include +#else +#include #endif #include @@ -22,6 +24,7 @@ struct terminal { int is_console_input; int is_console_output; }; +#endif struct cmdline { int s_in; @@ -29,11 +32,14 @@ struct cmdline { cmdline_parse_ctx_t *ctx; struct rdline rdl; char prompt[RDLINE_PROMPT_SIZE]; +#ifdef RTE_EXEC_ENV_WINDOWS struct terminal oldterm; char repeated_char; WORD repeat_count; -}; +#else + struct termios oldterm; #endif +}; /* Disable buffering and echoing, save previous settings to oldterm. */ void terminal_adjust(struct cmdline *cl); -- 2.29.3