From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6EA1AA0093 for ; Tue, 19 May 2020 15:00:41 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 664191D5DE; Tue, 19 May 2020 15:00:41 +0200 (CEST) Received: from mail-wr1-f67.google.com (mail-wr1-f67.google.com [209.85.221.67]) by dpdk.org (Postfix) with ESMTP id 739DC1D627 for ; Tue, 19 May 2020 15:00:39 +0200 (CEST) Received: by mail-wr1-f67.google.com with SMTP id v12so15749833wrp.12 for ; Tue, 19 May 2020 06:00:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=KXxIs3JEUuiKGulIsipXBag2wtATAKspV+CrCHoX/3o=; b=CVxEYaeKMgKV4TyDH/rUo/SiNzsEaXmGYkgQ+HW6peD9Cm7yrpr4EEh2b2cseSAo7r HKsCzvrU6AcucZEcpTH7Iic+8PyQ5Issx0FCojSueti3P1OjbG6+xaetXL3kmEepVuM/ c3fSgCDpia0hLCvfxg7LlUP99NrObUOfD5bjzPEmciqAfc+32ioAE/mmes+cbl7rZcyw 9fFNTX3z/2+r8htWLbg4CvPNnmPRnHvWFG1T3ts5EYNW3JG36YtCBHXPENMO5r6TNTC7 U+cG0TMTPUvGNuzqH2mfic1ESQQFVXgq9j/5wK/uhFeCaB4gnrXOJuWrFb6YemYnLQ/P nwkw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=KXxIs3JEUuiKGulIsipXBag2wtATAKspV+CrCHoX/3o=; b=MpLTwuKavQQJF+yT9KHSonY4qSn+9KTnTgOu2CTiLJGr8HDRdq+p6z9oRFaPvoGC8M jdamMh10N5VFaAY0O7aUc7CtFPs6s+x4PfUIsppVx5MnMvRNNOUGQLMwvT0VMfe2r6AQ snt9XnBE0cTThdvhTHKLMyEWuwCqjJFllyhZeC+YtgGZSkFWrdKildbm1nFJHE5KQ+NI 5WKKHWku778Jw37HSyiMTChpxFuQxdBiYR5Cz05rHi9S2alO1MXXl+ZSPRSD0JldK3n+ xrQQxbCyQp576HsfZOiTnlwoWuVN+ap6x0AFEzbw2CgL3vDYts/nwWeE8/5A6x3czc89 fZFQ== X-Gm-Message-State: AOAM5302QstTMb7wrWsWYaV6gJjK5+S49XUHii31FDvSURY+ocGcadV8 kEXABPnpK5i99rtmElW38hU= X-Google-Smtp-Source: ABdhPJwhI763KekutS4KHsRo5sZjsISF4vvJB71RAs5pgJfqVbfDNNHqeDhiMQCriPurv+dxBr++qA== X-Received: by 2002:adf:e5c9:: with SMTP id a9mr27640071wrn.292.1589893239136; Tue, 19 May 2020 06:00:39 -0700 (PDT) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id g184sm4083864wmg.1.2020.05.19.06.00.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 19 May 2020 06:00:38 -0700 (PDT) From: luca.boccassi@gmail.com To: Bruce Richardson Cc: Lukasz Wojciechowski , dpdk stable Date: Tue, 19 May 2020 13:54:25 +0100 Message-Id: <20200519125804.104349-75-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20200519125804.104349-1-luca.boccassi@gmail.com> References: <20200519125804.104349-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'drivers: fix log type variables for -fno-common' has been queued to stable release 19.11.3 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" Hi, FYI, your patch has been queued to stable release 19.11.3 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 05/21/20. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Thanks. Luca Boccassi --- >From efd0dea9c869db269a5f44910598fe286c5814d5 Mon Sep 17 00:00:00 2001 From: Bruce Richardson Date: Fri, 10 Apr 2020 18:27:56 +0100 Subject: [PATCH] drivers: fix log type variables for -fno-common [ upstream commit 3071d471525443978a2ae7e0d068bc3ffcede80c ] The zlib compression driver, as well as the aesni-gcm, aesni-mb and openssl crypto drivers all defined the logtype variable in the header file directly. This gives errors with gcc 10, due to -fno-common being the default, so we need to apply the same fix in all cases: * move the variable definition to a suitable .c file * mark the forward declaration of the variable in the header as "extern" Fixes: 0c4e4c16b004 ("compress/zlib: introduce zlib PMD") Fixes: 90c8a2d02a31 ("crypto/aesni_gcm: add dynamic logging") Fixes: 276624ae2ef6 ("crypto/aesni_mb: add dynamic logging") Fixes: 094b2386f461 ("crypto/openssl: add dynamic logging") Signed-off-by: Bruce Richardson Acked-by: Lukasz Wojciechowski --- drivers/compress/zlib/zlib_pmd.c | 2 ++ drivers/compress/zlib/zlib_pmd_private.h | 2 +- drivers/crypto/aesni_gcm/aesni_gcm_pmd.c | 2 ++ drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h | 2 +- drivers/crypto/aesni_mb/aesni_mb_pmd_private.h | 2 +- drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 2 ++ drivers/crypto/openssl/openssl_pmd_private.h | 2 +- drivers/crypto/openssl/rte_openssl_pmd.c | 1 + 8 files changed, 11 insertions(+), 4 deletions(-) diff --git a/drivers/compress/zlib/zlib_pmd.c b/drivers/compress/zlib/zlib_pmd.c index 19f9200c22..e39be2ed86 100644 --- a/drivers/compress/zlib/zlib_pmd.c +++ b/drivers/compress/zlib/zlib_pmd.c @@ -7,6 +7,8 @@ #include "zlib_pmd_private.h" +int zlib_logtype_driver; + /** Compute next mbuf in the list, assign data buffer and length, * returns 0 if mbuf is NULL */ diff --git a/drivers/compress/zlib/zlib_pmd_private.h b/drivers/compress/zlib/zlib_pmd_private.h index bda49759dc..e36c5dc615 100644 --- a/drivers/compress/zlib/zlib_pmd_private.h +++ b/drivers/compress/zlib/zlib_pmd_private.h @@ -14,7 +14,7 @@ #define DEF_MEM_LEVEL 8 -int zlib_logtype_driver; +extern int zlib_logtype_driver; #define ZLIB_PMD_LOG(level, fmt, args...) \ rte_log(RTE_LOG_ ## level, zlib_logtype_driver, "%s(): "fmt "\n", \ __func__, ##args) diff --git a/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c b/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c index 1a03be31dc..c9c9eb8ca9 100644 --- a/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c +++ b/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c @@ -13,6 +13,8 @@ #include "aesni_gcm_pmd_private.h" +int aesni_gcm_logtype_driver; + static uint8_t cryptodev_driver_id; /** Parse crypto xform chain and set private session parameters */ diff --git a/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h b/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h index 2039adb533..7347c4769f 100644 --- a/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h +++ b/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h @@ -20,7 +20,7 @@ /**< AES-NI GCM PMD device name */ /** AES-NI GCM PMD LOGTYPE DRIVER */ -int aesni_gcm_logtype_driver; +extern int aesni_gcm_logtype_driver; #define AESNI_GCM_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, aesni_gcm_logtype_driver, \ "%s() line %u: "fmt "\n", __func__, __LINE__, \ diff --git a/drivers/crypto/aesni_mb/aesni_mb_pmd_private.h b/drivers/crypto/aesni_mb/aesni_mb_pmd_private.h index 3456693c25..b3cb2f1cf9 100644 --- a/drivers/crypto/aesni_mb/aesni_mb_pmd_private.h +++ b/drivers/crypto/aesni_mb/aesni_mb_pmd_private.h @@ -19,7 +19,7 @@ enum aesni_mb_vector_mode { /**< AES-NI Multi buffer PMD device name */ /** AESNI_MB PMD LOGTYPE DRIVER */ -int aesni_mb_logtype_driver; +extern int aesni_mb_logtype_driver; #define AESNI_MB_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, aesni_mb_logtype_driver, \ diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c index 33f4167452..40feae3c62 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c @@ -14,6 +14,8 @@ #include "aesni_mb_pmd_private.h" +int aesni_mb_logtype_driver; + #define AES_CCM_DIGEST_MIN_LEN 4 #define AES_CCM_DIGEST_MAX_LEN 16 #define HMAC_MAX_BLOCK_SIZE 128 diff --git a/drivers/crypto/openssl/openssl_pmd_private.h b/drivers/crypto/openssl/openssl_pmd_private.h index 43ac3813df..b2054b3754 100644 --- a/drivers/crypto/openssl/openssl_pmd_private.h +++ b/drivers/crypto/openssl/openssl_pmd_private.h @@ -16,7 +16,7 @@ /**< Open SSL Crypto PMD device name */ /** OPENSSL PMD LOGTYPE DRIVER */ -int openssl_logtype_driver; +extern int openssl_logtype_driver; #define OPENSSL_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, openssl_logtype_driver, \ "%s() line %u: " fmt "\n", __func__, __LINE__, \ diff --git a/drivers/crypto/openssl/rte_openssl_pmd.c b/drivers/crypto/openssl/rte_openssl_pmd.c index 199097bf8c..b820f6171d 100644 --- a/drivers/crypto/openssl/rte_openssl_pmd.c +++ b/drivers/crypto/openssl/rte_openssl_pmd.c @@ -18,6 +18,7 @@ #define DES_BLOCK_SIZE 8 +int openssl_logtype_driver; static uint8_t cryptodev_driver_id; #if (OPENSSL_VERSION_NUMBER < 0x10100000L) -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2020-05-19 13:56:21.848933454 +0100 +++ 0075-drivers-fix-log-type-variables-for-fno-common.patch 2020-05-19 13:56:18.307503404 +0100 @@ -1,8 +1,10 @@ -From 3071d471525443978a2ae7e0d068bc3ffcede80c Mon Sep 17 00:00:00 2001 +From efd0dea9c869db269a5f44910598fe286c5814d5 Mon Sep 17 00:00:00 2001 From: Bruce Richardson Date: Fri, 10 Apr 2020 18:27:56 +0100 Subject: [PATCH] drivers: fix log type variables for -fno-common +[ upstream commit 3071d471525443978a2ae7e0d068bc3ffcede80c ] + The zlib compression driver, as well as the aesni-gcm, aesni-mb and openssl crypto drivers all defined the logtype variable in the header file directly. This gives errors with gcc 10, due to -fno-common being the @@ -14,7 +16,6 @@ Fixes: 90c8a2d02a31 ("crypto/aesni_gcm: add dynamic logging") Fixes: 276624ae2ef6 ("crypto/aesni_mb: add dynamic logging") Fixes: 094b2386f461 ("crypto/openssl: add dynamic logging") -Cc: stable@dpdk.org Signed-off-by: Bruce Richardson Acked-by: Lukasz Wojciechowski @@ -56,7 +57,7 @@ rte_log(RTE_LOG_ ## level, zlib_logtype_driver, "%s(): "fmt "\n", \ __func__, ##args) diff --git a/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c b/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c -index a1caab9939..475d523ae1 100644 +index 1a03be31dc..c9c9eb8ca9 100644 --- a/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c +++ b/drivers/crypto/aesni_gcm/aesni_gcm_pmd.c @@ -13,6 +13,8 @@ @@ -67,9 +68,9 @@ + static uint8_t cryptodev_driver_id; - /* setup session handlers */ + /** Parse crypto xform chain and set private session parameters */ diff --git a/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h b/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h -index 080d4f7e49..2763d1c492 100644 +index 2039adb533..7347c4769f 100644 --- a/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h +++ b/drivers/crypto/aesni_gcm/aesni_gcm_pmd_private.h @@ -20,7 +20,7 @@