<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
HI Kevin</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
    Thanks.  <a href="mailto:prashant.gupta_3@nxp.com" id="OWAAM593503" class="tWKOu mention ms-bgc-nlr ms-fcl-b">
@Prashant Gupta</a> is working on porting this patch for 25.11 and 24.11 </div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Regards</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Hemant</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Kevin Traynor <ktraynor@redhat.com><br>
<b>Sent:</b> Thursday, July 23, 2026 10:44 PM<br>
<b>To:</b> David Marchand <david.marchand@redhat.com><br>
<b>Cc:</b> Hemant Agrawal <hemant.agrawal@nxp.com>; Maxime Leroy <maxime@leroys.fr>; Kevin Traynor <ktraynor@redhat.com>; Stephen Hemminger <stephen@networkplumber.org>; dpdk stable <stable@dpdk.org><br>
<b>Subject:</b> patch 'bus/fslmc: fix bus cleanup' has been queued to stable release 25.11.3</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hi,<br>
<br>
FYI, your patch has been queued to stable release 25.11.3<br>
<br>
Note it hasn't been pushed to <a href="http://dpdk.org/browse/dpdk-stable">https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.org%2Fbrowse%2Fdpdk-stable&data=05%7C02%7Chemant.agrawal%40nxp.com%7C91eee249d53a40a0081e08dee8de1f08%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C639204237859627434%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=JueGco8XNqrAPMURHwgUgNClmjaA5J4e8eVId4P%2Fy7I%3D&reserved=0</a>
 yet.<br>
It will be pushed if I get no objections before 07/27/26. So please<br>
shout if anyone has objections.<br>
<br>
Also note that after the patch there's a diff of the upstream commit vs the<br>
patch applied to the branch. This will indicate if there was any rebasing<br>
needed to apply to the stable branch. If there were code changes for rebasing<br>
(ie: not only metadata diffs), please double check that the rebase was<br>
correctly done.<br>
<br>
Queued patches are on a temporary branch at:<br>
<a href="https://github.com/kevintraynor/dpdk-stable">https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkevintraynor%2Fdpdk-stable&data=05%7C02%7Chemant.agrawal%40nxp.com%7C91eee249d53a40a0081e08dee8de1f08%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C639204237859654373%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=LtD8CqAgWYBFui%2BmtMktN5jqb88jFichPZiiQVjxynE%3D&reserved=0</a><br>
<br>
This queued commit can be viewed at:<br>
<a href="https://github.com/kevintraynor/dpdk-stable/commit/51882532384fe18094d7357b807a602408b56ef0">https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkevintraynor%2Fdpdk-stable%2Fcommit%2F51882532384fe18094d7357b807a602408b56ef0&data=05%7C02%7Chemant.agrawal%40nxp.com%7C91eee249d53a40a0081e08dee8de1f08%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C639204237859666532%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=XINOyygagjr3dfayFA1YSz%2FZikIKAcqbam4JijFi2%2FY%3D&reserved=0</a><br>
<br>
Thanks.<br>
<br>
Kevin<br>
<br>
---<br>
From 51882532384fe18094d7357b807a602408b56ef0 Mon Sep 17 00:00:00 2001<br>
From: David Marchand <david.marchand@redhat.com><br>
Date: Thu, 26 Feb 2026 11:07:01 +0100<br>
Subject: [PATCH] bus/fslmc: fix bus cleanup<br>
<br>
[ upstream commit 5ea5b0233e806350cdb09aa28ad9a329f5182df0 ]<br>
<br>
The close operation was never closing probed devices.<br>
<br>
Taking a step back, reevaluating the devargs makes no sense during the<br>
close step, as a probed device must have passed the allow/block list<br>
evaluation initially.<br>
<br>
Since the device contains a reference to the driver that probed it,<br>
simply call this driver remove op.<br>
<br>
Fixes: 274fd921ff7f ("bus/fslmc: support close operation")<br>
<br>
Signed-off-by: David Marchand <david.marchand@redhat.com><br>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com><br>
Tested-by: Maxime Leroy <maxime@leroys.fr><br>
Acked-by: Kevin Traynor <ktraynor@redhat.com><br>
Acked-by: Stephen Hemminger <stephen@networkplumber.org><br>
---<br>
 drivers/bus/fslmc/fslmc_vfio.c | 21 ++++-----------------<br>
 1 file changed, 4 insertions(+), 17 deletions(-)<br>
<br>
diff --git a/drivers/bus/fslmc/fslmc_vfio.c b/drivers/bus/fslmc/fslmc_vfio.c<br>
index 550d4e0e8d..7daa18d850 100644<br>
--- a/drivers/bus/fslmc/fslmc_vfio.c<br>
+++ b/drivers/bus/fslmc/fslmc_vfio.c<br>
@@ -1394,5 +1394,5 @@ fslmc_close_iodevices(struct rte_dpaa2_device *dev,<br>
         struct rte_dpaa2_object *object = NULL;<br>
         struct rte_dpaa2_driver *drv;<br>
-       int ret, probe_all;<br>
+       int ret;<br>
 <br>
         switch (dev->dev_type) {<br>
@@ -1412,20 +1412,7 @@ fslmc_close_iodevices(struct rte_dpaa2_device *dev,<br>
         case DPAA2_CRYPTO:<br>
         case DPAA2_QDMA:<br>
-               probe_all = rte_fslmc_bus.bus.conf.scan_mode !=<br>
-                           RTE_BUS_SCAN_ALLOWLIST;<br>
-               TAILQ_FOREACH(drv, &rte_fslmc_bus.driver_list, next) {<br>
-                       if (drv->drv_type != dev->dev_type)<br>
-                               continue;<br>
-                       if (rte_dev_is_probed(&dev->device))<br>
-                               continue;<br>
-                       if (probe_all ||<br>
-                           (dev->device.devargs &&<br>
-                            dev->device.devargs->policy ==<br>
-                            RTE_DEV_ALLOWED)) {<br>
-                               ret = drv->remove(dev);<br>
-                               if (ret)<br>
-                                       DPAA2_BUS_ERR("Unable to remove");<br>
-                       }<br>
-               }<br>
+               drv = dev->driver;<br>
+               if (drv && drv->remove && drv->remove(dev))<br>
+                       DPAA2_BUS_ERR("Unable to remove");<br>
                 break;<br>
         default:<br>
-- <br>
2.55.0<br>
<br>
---<br>
  Diff of the applied patch vs upstream commit (please double-check if non-empty:<br>
---<br>
--- -   2026-07-23 17:57:58.776621965 +0100<br>
+++ 0002-bus-fslmc-fix-bus-cleanup.patch        2026-07-23 17:57:58.622841560 +0100<br>
@@ -1 +1 @@<br>
-From 5ea5b0233e806350cdb09aa28ad9a329f5182df0 Mon Sep 17 00:00:00 2001<br>
+From 51882532384fe18094d7357b807a602408b56ef0 Mon Sep 17 00:00:00 2001<br>
@@ -5,0 +6,2 @@<br>
+[ upstream commit 5ea5b0233e806350cdb09aa28ad9a329f5182df0 ]<br>
+<br>
@@ -16 +17,0 @@<br>
-Cc: stable@dpdk.org<br>
<br>
</div>
</span></font></div>
</body>
</html>