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 976B0A0093 for ; Tue, 19 May 2020 15:00:43 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 8EC651D5DE; Tue, 19 May 2020 15:00:43 +0200 (CEST) Received: from mail-wr1-f44.google.com (mail-wr1-f44.google.com [209.85.221.44]) by dpdk.org (Postfix) with ESMTP id 12C6A1D62F for ; Tue, 19 May 2020 15:00:42 +0200 (CEST) Received: by mail-wr1-f44.google.com with SMTP id l11so15879686wru.0 for ; Tue, 19 May 2020 06:00:42 -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=TPdMape56dap++hYbkO1imkHVH2Fa68eYirpMdIzkwM=; b=QUmn0IeIMIBvkXn7O6t0EQaXqc3jtJxPULBwj/JRAfvYtOqH8SNKOt5/oYEUnU5cAW FiBmnrmEBZ2j6CFDqvXO0ppYMADjYAohQU3LJFnwcDshZqmVzQopUHnfgPdkBEjmIZFe J7kHn35Ge32DlxWwb6Opm6guZAPy3znX+L2POZxCpblho3G8ojbnn5qAjOHQP0ANIvZz mGUQvcg0f0yQiNtkydTjoFSrmO5ZWGQmw8tijGgvhX8EfFUnNPr4l+bxEq7I+3N67KE4 X5gRvqPiFVrpSFVcQz+dwaOKsD8uuiX75dvI3rRFvF6hxKAMGbiOf5X4gyD+LXXhIz1a 9mtA== 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=TPdMape56dap++hYbkO1imkHVH2Fa68eYirpMdIzkwM=; b=cJ38CReJurXQnUMm+dmmr9odmtXA4jwonvbryvNNvh7NC1xV6mBDyTqnl5XcPdots6 0qZXx2TwZStpg0wy+lP4fae5XXLTspEZSZfOwAfgPwoaWt3cR0gZJ5NCodIFNXxV3iXD aptoHsnoTBvyeGK1vxaDImvUiyiNWxo/oR2pu/uyPUwqaKY9yadc575X15MOAivrmbM4 tKqhlMF3cYdfVFaEl8C6t1n+fhjAE4BJN0+yxPCAkfp7Zl30b4r3u52so/yF4/f+ndKR Dv6lIW9gU3gS4RZSXSsWoVnUTcMGXORMgfv697qvMJZ3Z/kBkf1LOr/w+N42Xrbq3sP8 eDNQ== X-Gm-Message-State: AOAM5335Gpn976DsoxIncXpcdh7ut52y3Mt3krCMDvGqC4VZEIDGNPVb b0l0MhaCxnfeiZWgyCldbf0= X-Google-Smtp-Source: ABdhPJwdDInQu0WQ+M4B7w6NeUFTrucKR1ti8UT0qE66IqRn7ry6cvh+4HyY62J8D5docg4k050cng== X-Received: by 2002:adf:e64c:: with SMTP id b12mr25083746wrn.131.1589893241772; Tue, 19 May 2020 06:00:41 -0700 (PDT) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id n128sm3875028wmb.47.2020.05.19.06.00.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 19 May 2020 06:00:40 -0700 (PDT) From: luca.boccassi@gmail.com To: Lukasz Wojciechowski Cc: Pablo de Lara , dpdk stable Date: Tue, 19 May 2020 13:54:27 +0100 Message-Id: <20200519125804.104349-77-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/crypto: 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 9aa6caeb5af05c1c440213ee9ff5c40d001b9614 Mon Sep 17 00:00:00 2001 From: Lukasz Wojciechowski Date: Wed, 15 Apr 2020 08:22:16 +0200 Subject: [PATCH] drivers/crypto: fix log type variables for -fno-common [ upstream commit 6da6baf7631b53dd53d3a4c4c7d62d39da4921b6 ] The four crypto drivers: kasumi, mvsam, snow3g and zuc define logtype variables in their header file. As the header files are included in more than one compilation unit, it might cause appearance of multiple instances of the variable and a linker error. Such situation can occur, when no common section is allowed by the compiler settings and tentative definitions are placed in BSS section. Fixes: 2cba3814932e ("crypto/kasumi: add dynamic logging") Fixes: a05a450f42fd ("crypto/mvsam: add dynamic logging") Fixes: f3af5f9d1325 ("crypto/zuc: add dynamic logging") Fixes: a3277ad47feb ("cryptodev: remove crypto device driver name") Signed-off-by: Lukasz Wojciechowski Acked-by: Pablo de Lara --- drivers/crypto/kasumi/kasumi_pmd_private.h | 2 +- drivers/crypto/kasumi/rte_kasumi_pmd.c | 1 + drivers/crypto/mvsam/mrvl_pmd_private.h | 2 +- drivers/crypto/mvsam/rte_mrvl_pmd.c | 1 + drivers/crypto/snow3g/rte_snow3g_pmd.c | 1 + drivers/crypto/snow3g/snow3g_pmd_private.h | 2 +- drivers/crypto/zuc/rte_zuc_pmd.c | 1 + drivers/crypto/zuc/zuc_pmd_private.h | 4 ++-- 8 files changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/crypto/kasumi/kasumi_pmd_private.h b/drivers/crypto/kasumi/kasumi_pmd_private.h index 7ac19c5735..fda696804e 100644 --- a/drivers/crypto/kasumi/kasumi_pmd_private.h +++ b/drivers/crypto/kasumi/kasumi_pmd_private.h @@ -11,7 +11,7 @@ /**< KASUMI PMD device name */ /** KASUMI PMD LOGTYPE DRIVER */ -int kasumi_logtype_driver; +extern int kasumi_logtype_driver; #define KASUMI_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, kasumi_logtype_driver, \ diff --git a/drivers/crypto/kasumi/rte_kasumi_pmd.c b/drivers/crypto/kasumi/rte_kasumi_pmd.c index d0583ef073..c87dca5f48 100644 --- a/drivers/crypto/kasumi/rte_kasumi_pmd.c +++ b/drivers/crypto/kasumi/rte_kasumi_pmd.c @@ -17,6 +17,7 @@ #define KASUMI_MAX_BURST 4 #define BYTE_LEN 8 +int kasumi_logtype_driver; static uint8_t cryptodev_driver_id; /** Get xform chain order. */ diff --git a/drivers/crypto/mvsam/mrvl_pmd_private.h b/drivers/crypto/mvsam/mrvl_pmd_private.h index 09702b9e3e..e575330ef5 100644 --- a/drivers/crypto/mvsam/mrvl_pmd_private.h +++ b/drivers/crypto/mvsam/mrvl_pmd_private.h @@ -13,7 +13,7 @@ /**< Marvell PMD device name */ /** MRVL PMD LOGTYPE DRIVER */ -int mrvl_logtype_driver; +extern int mrvl_logtype_driver; #define MRVL_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, mrvl_logtype_driver, \ diff --git a/drivers/crypto/mvsam/rte_mrvl_pmd.c b/drivers/crypto/mvsam/rte_mrvl_pmd.c index 3c0fe216f0..63782ce974 100644 --- a/drivers/crypto/mvsam/rte_mrvl_pmd.c +++ b/drivers/crypto/mvsam/rte_mrvl_pmd.c @@ -19,6 +19,7 @@ #define MRVL_PMD_MAX_NB_SESS_ARG ("max_nb_sessions") #define MRVL_PMD_DEFAULT_MAX_NB_SESSIONS 2048 +int mrvl_logtype_driver; static uint8_t cryptodev_driver_id; struct mrvl_pmd_init_params { diff --git a/drivers/crypto/snow3g/rte_snow3g_pmd.c b/drivers/crypto/snow3g/rte_snow3g_pmd.c index 9d07e1ab2c..2b1b90b011 100644 --- a/drivers/crypto/snow3g/rte_snow3g_pmd.c +++ b/drivers/crypto/snow3g/rte_snow3g_pmd.c @@ -16,6 +16,7 @@ #define SNOW3G_MAX_BURST 8 #define BYTE_LEN 8 +int snow3g_logtype_driver; static uint8_t cryptodev_driver_id; /** Get xform chain order. */ diff --git a/drivers/crypto/snow3g/snow3g_pmd_private.h b/drivers/crypto/snow3g/snow3g_pmd_private.h index 1fe05eb567..1070800960 100644 --- a/drivers/crypto/snow3g/snow3g_pmd_private.h +++ b/drivers/crypto/snow3g/snow3g_pmd_private.h @@ -11,7 +11,7 @@ /**< SNOW 3G PMD device name */ /** SNOW 3G PMD LOGTYPE DRIVER */ -int snow3g_logtype_driver; +extern int snow3g_logtype_driver; #define SNOW3G_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, snow3g_logtype_driver, \ diff --git a/drivers/crypto/zuc/rte_zuc_pmd.c b/drivers/crypto/zuc/rte_zuc_pmd.c index 8e214cd50e..265aabaebf 100644 --- a/drivers/crypto/zuc/rte_zuc_pmd.c +++ b/drivers/crypto/zuc/rte_zuc_pmd.c @@ -14,6 +14,7 @@ #define ZUC_MAX_BURST 4 #define BYTE_LEN 8 +int zuc_logtype_driver; static uint8_t cryptodev_driver_id; /** Get xform chain order. */ diff --git a/drivers/crypto/zuc/zuc_pmd_private.h b/drivers/crypto/zuc/zuc_pmd_private.h index 428efd4bb5..dc492b1710 100644 --- a/drivers/crypto/zuc/zuc_pmd_private.h +++ b/drivers/crypto/zuc/zuc_pmd_private.h @@ -8,10 +8,10 @@ #include #define CRYPTODEV_NAME_ZUC_PMD crypto_zuc -/**< KASUMI PMD device name */ +/**< ZUC PMD device name */ /** ZUC PMD LOGTYPE DRIVER */ -int zuc_logtype_driver; +extern int zuc_logtype_driver; #define ZUC_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, zuc_logtype_driver, \ "%s()... line %u: " fmt "\n", __func__, __LINE__, \ -- 2.20.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2020-05-19 13:56:21.933619634 +0100 +++ 0077-drivers-crypto-fix-log-type-variables-for-fno-common.patch 2020-05-19 13:56:18.311503476 +0100 @@ -1,8 +1,10 @@ -From 6da6baf7631b53dd53d3a4c4c7d62d39da4921b6 Mon Sep 17 00:00:00 2001 +From 9aa6caeb5af05c1c440213ee9ff5c40d001b9614 Mon Sep 17 00:00:00 2001 From: Lukasz Wojciechowski Date: Wed, 15 Apr 2020 08:22:16 +0200 Subject: [PATCH] drivers/crypto: fix log type variables for -fno-common +[ upstream commit 6da6baf7631b53dd53d3a4c4c7d62d39da4921b6 ] + The four crypto drivers: kasumi, mvsam, snow3g and zuc define logtype variables in their header file. As the header files are included in more than one compilation unit, it might cause appearance @@ -15,7 +17,6 @@ Fixes: a05a450f42fd ("crypto/mvsam: add dynamic logging") Fixes: f3af5f9d1325 ("crypto/zuc: add dynamic logging") Fixes: a3277ad47feb ("cryptodev: remove crypto device driver name") -Cc: stable@dpdk.org Signed-off-by: Lukasz Wojciechowski Acked-by: Pablo de Lara @@ -31,7 +32,7 @@ 8 files changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/crypto/kasumi/kasumi_pmd_private.h b/drivers/crypto/kasumi/kasumi_pmd_private.h -index 3b3a6958fd..b7f1c428b4 100644 +index 7ac19c5735..fda696804e 100644 --- a/drivers/crypto/kasumi/kasumi_pmd_private.h +++ b/drivers/crypto/kasumi/kasumi_pmd_private.h @@ -11,7 +11,7 @@ @@ -44,7 +45,7 @@ #define KASUMI_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, kasumi_logtype_driver, \ diff --git a/drivers/crypto/kasumi/rte_kasumi_pmd.c b/drivers/crypto/kasumi/rte_kasumi_pmd.c -index baad5b4e77..d672648420 100644 +index d0583ef073..c87dca5f48 100644 --- a/drivers/crypto/kasumi/rte_kasumi_pmd.c +++ b/drivers/crypto/kasumi/rte_kasumi_pmd.c @@ -17,6 +17,7 @@ @@ -81,7 +82,7 @@ struct mrvl_pmd_init_params { diff --git a/drivers/crypto/snow3g/rte_snow3g_pmd.c b/drivers/crypto/snow3g/rte_snow3g_pmd.c -index d72112b5f4..8101eaaada 100644 +index 9d07e1ab2c..2b1b90b011 100644 --- a/drivers/crypto/snow3g/rte_snow3g_pmd.c +++ b/drivers/crypto/snow3g/rte_snow3g_pmd.c @@ -16,6 +16,7 @@ @@ -93,7 +94,7 @@ /** Get xform chain order. */ diff --git a/drivers/crypto/snow3g/snow3g_pmd_private.h b/drivers/crypto/snow3g/snow3g_pmd_private.h -index 2074f3d172..23cf078a9c 100644 +index 1fe05eb567..1070800960 100644 --- a/drivers/crypto/snow3g/snow3g_pmd_private.h +++ b/drivers/crypto/snow3g/snow3g_pmd_private.h @@ -11,7 +11,7 @@ @@ -106,11 +107,11 @@ #define SNOW3G_LOG(level, fmt, ...) \ rte_log(RTE_LOG_ ## level, snow3g_logtype_driver, \ diff --git a/drivers/crypto/zuc/rte_zuc_pmd.c b/drivers/crypto/zuc/rte_zuc_pmd.c -index 9e06ab7c95..f47a7880de 100644 +index 8e214cd50e..265aabaebf 100644 --- a/drivers/crypto/zuc/rte_zuc_pmd.c +++ b/drivers/crypto/zuc/rte_zuc_pmd.c @@ -14,6 +14,7 @@ - #define ZUC_MAX_BURST 16 + #define ZUC_MAX_BURST 4 #define BYTE_LEN 8 +int zuc_logtype_driver; @@ -118,11 +119,11 @@ /** Get xform chain order. */ diff --git a/drivers/crypto/zuc/zuc_pmd_private.h b/drivers/crypto/zuc/zuc_pmd_private.h -index 47a8b08dd7..d8684891ee 100644 +index 428efd4bb5..dc492b1710 100644 --- a/drivers/crypto/zuc/zuc_pmd_private.h +++ b/drivers/crypto/zuc/zuc_pmd_private.h @@ -8,10 +8,10 @@ - #include + #include #define CRYPTODEV_NAME_ZUC_PMD crypto_zuc -/**< KASUMI PMD device name */