patch 'raw/ioat: fix build when ioat dmadev enabled' has been queued to stable release 21.11.2
Kevin Traynor
ktraynor at redhat.com
Tue Jun 28 17:19:14 CEST 2022
Hi,
FYI, your patch has been queued to stable release 21.11.2
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/30/22. 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.
Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable
This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable/commit/b72fa6fd7a498744ff5880e2887bf221683ec3d5
Thanks.
Kevin
---
>From b72fa6fd7a498744ff5880e2887bf221683ec3d5 Mon Sep 17 00:00:00 2001
From: Bruce Richardson <bruce.richardson at intel.com>
Date: Thu, 23 Jun 2022 14:49:32 +0100
Subject: [PATCH] raw/ioat: fix build when ioat dmadev enabled
[ upstream commit 7580f35ea7a6f56f3fb65a3fa8601a6c21515805 ]
The build of the raw/ioat driver only occurs when the equivalent dmadev
drivers are disabled. Complications occur when the ioat dmadev is being
built but not the idxd. In this case, only the idxd part of raw/ioat
gets built, but the definition of the logtype is in the ioat part,
causing build errors.
.../raw_ioat_idxd_bus.c.o: In function `idxd_vdev_mmap_wq':
idxd_bus.c:(.text+0x116): undefined reference to `ioat_pmd_logtype'
Fix this by moving the logtype definition to the common C file, and
renaming it to avoid conflicts with a similarly named value in the
dma/ioat driver.
Fixes: ff06fa2cf3ba ("raw/ioat: probe idxd PCI")
Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
---
drivers/raw/ioat/ioat_common.c | 2 ++
drivers/raw/ioat/ioat_private.h | 4 ++--
drivers/raw/ioat/ioat_rawdev.c | 2 --
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/raw/ioat/ioat_common.c b/drivers/raw/ioat/ioat_common.c
index 60de41152a..e44e181e58 100644
--- a/drivers/raw/ioat/ioat_common.c
+++ b/drivers/raw/ioat/ioat_common.c
@@ -10,4 +10,6 @@
#include "ioat_private.h"
+RTE_LOG_REGISTER_DEFAULT(ioat_rawdev_logtype, INFO);
+
static const char * const xstat_names[] = {
"failed_enqueues", "successful_enqueues",
diff --git a/drivers/raw/ioat/ioat_private.h b/drivers/raw/ioat/ioat_private.h
index f032d5fe3d..7fb685a3d7 100644
--- a/drivers/raw/ioat/ioat_private.h
+++ b/drivers/raw/ioat/ioat_private.h
@@ -19,8 +19,8 @@
#include "rte_ioat_rawdev.h"
-extern int ioat_pmd_logtype;
+extern int ioat_rawdev_logtype;
#define IOAT_PMD_LOG(level, fmt, args...) rte_log(RTE_LOG_ ## level, \
- ioat_pmd_logtype, "IOAT: %s(): " fmt "\n", __func__, ##args)
+ ioat_rawdev_logtype, "IOAT: %s(): " fmt "\n", __func__, ##args)
#define IOAT_PMD_DEBUG(fmt, args...) IOAT_PMD_LOG(DEBUG, fmt, ## args)
diff --git a/drivers/raw/ioat/ioat_rawdev.c b/drivers/raw/ioat/ioat_rawdev.c
index 5396671d4f..11341fcf5d 100644
--- a/drivers/raw/ioat/ioat_rawdev.c
+++ b/drivers/raw/ioat/ioat_rawdev.c
@@ -29,6 +29,4 @@ static struct rte_pci_driver ioat_pmd_drv;
#define IOAT_DEVICE_ID_ICX 0x0b00
-RTE_LOG_REGISTER_DEFAULT(ioat_pmd_logtype, INFO);
-
#define DESC_SZ sizeof(struct rte_ioat_generic_hw_desc)
#define COMPLETION_SZ sizeof(__m128i)
--
2.34.3
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2022-06-28 16:18:04.111098127 +0100
+++ 0002-raw-ioat-fix-build-when-ioat-dmadev-enabled.patch 2022-06-28 16:18:03.997387177 +0100
@@ -1 +1 @@
-From 7580f35ea7a6f56f3fb65a3fa8601a6c21515805 Mon Sep 17 00:00:00 2001
+From b72fa6fd7a498744ff5880e2887bf221683ec3d5 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 7580f35ea7a6f56f3fb65a3fa8601a6c21515805 ]
+
@@ -20 +21,0 @@
-Cc: stable at dpdk.org
More information about the stable
mailing list