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 6D91EA0C41; Tue, 5 Oct 2021 02:55:27 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 27F1941266; Tue, 5 Oct 2021 02:55:24 +0200 (CEST) Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.49]) by mails.dpdk.org (Postfix) with ESMTP id 13D084068F for ; Tue, 5 Oct 2021 02:55:22 +0200 (CEST) Received: by mail-lf1-f49.google.com with SMTP id y26so79426376lfa.11 for ; Mon, 04 Oct 2021 17:55:22 -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=kidtTVROOQYTKOURBLCebRGryZ9kPAa8FsWZuO/msqqH/milmdxU/ZOOCNuOJHzSPg OrfCDz3BTq6oNk9ruaXt6SFSLvpN9Iq1Gwqx3d6rz/IGD80ng9h1Tys6BJPa2VTZkVHL IreUQ3+Og8DbsuENwD8euJk99uiH6IJH2WAe4yRNagtPtyUWAh9D6El9j25qXlyx+rQa 7YZIbcUsXz9J6EV0xdVHQ3RCHcrWxYtND7UZWCOVGoavYrfJrkwIByS/XfjV0p/kMnay C2sWlL4CP71fWsMvQw0v8S0BjoT3YL88XLR+xZ4tkkYqA2d2BOh8OYiJasLjtau9KbOV v5UQ== 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=pmFzusigVFept2Br6pj3PHOdd1A02zlRH9H7fiqNpKuLX869mdzQuMf+4A0s5fAmGN +No+BG4DsK3T+amFpcuHDx3UbAJNWsnTnXsChGJ6NX83UnK1bFCNiRjyGlYgNk5BKIkq CM+hkglnYiURF5PbCGD4tiW7u4LmTl2c40U3zxnTqIzsnwRA/+ykVgE+i1YfF8EJ5eHO 5Fym2vNXnlgmBdJz/vJfbBIbKzgl/MqW16Bbabozkp/KUjaUqnNReYfmPyus7cd1V7BX 0/GJbH0JTu+iUgGxAMJciAETs94ZF0qjN8Ex4lxe68RdB6TI0CMuOsfptBHOAuvpI68K 3Z6g== X-Gm-Message-State: AOAM530GDuag8yYhcFW3TnGziYXY6QNAsMlGIvqhGFIVzWJhh1jfyfpD U7nWV9SrrVdnXUD3J4fHs7ArRyjNJ10= X-Google-Smtp-Source: ABdhPJyyxHUHbd+C6uWqO675kFnpHtOe4PEmRrb+++fy13XkqcPUu4GVzKPI/PRSq3yYkevXFugyow== X-Received: by 2002:a05:6512:39d6:: with SMTP id k22mr347525lfu.258.1633395321563; Mon, 04 Oct 2021 17:55:21 -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 b3sm52840lfe.58.2021.10.04.17.55.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 04 Oct 2021 17:55:21 -0700 (PDT) From: Dmitry Kozlyuk To: dev@dpdk.org Cc: David Marchand , Dmitry Kozlyuk , Olivier Matz , Ray Kinsella Date: Tue, 5 Oct 2021 03:55:15 +0300 Message-Id: <20211005005516.132377-2-dmitry.kozliuk@gmail.com> X-Mailer: git-send-email 2.29.3 In-Reply-To: <20211005005516.132377-1-dmitry.kozliuk@gmail.com> References: <20210928194714.365563-1-dmitry.kozliuk@gmail.com> <20211005005516.132377-1-dmitry.kozliuk@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-dev] [PATCH v3 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