<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<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: 12pt; color: rgb(0, 0, 0);">
Thanks for clarifying, patch queued to 23.11.3.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Best regards,</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Xueming</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> Jiang, YuX <yux.jiang@intel.com><br>
<b>Sent:</b> Monday, September 2, 2024 10:01 AM<br>
<b>To:</b> Xueming Li <xuemingl@nvidia.com>; Ye, MingjinX <mingjinx.ye@intel.com>; Xu, HailinX <hailinx.xu@intel.com><br>
<b>Cc:</b> dpdk stable <stable@dpdk.org><br>
<b>Subject:</b> RE: patch 'bus/vdev: revert fix devargs in secondary process' has been queued to stable release 23.11.2</font>
<div> </div>
</div>
<style>
<!--
@font-face
        {font-family:"Cambria Math"}
@font-face
        {font-family:DengXian}
@font-face
        {font-family:Calibri}
@font-face
        {font-family:Aptos}
@font-face
        {}
p.x_MsoNormal, li.x_MsoNormal, div.x_MsoNormal
        {margin:0in;
        font-size:12.0pt;
        font-family:"Aptos",sans-serif}
a:link, span.x_MsoHyperlink
        {color:blue;
        text-decoration:underline}
span.x_EmailStyle19
        {font-family:"Aptos",sans-serif;
        color:windowtext}
.x_MsoChpDefault
        {font-size:10.0pt}
@page WordSection1
        {margin:1.0in 1.0in 1.0in 1.0in}
div.x_WordSection1
        {}
-->
</style>
<div lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="x_WordSection1">
<p class="x_MsoNormal"><span style="font-size:11.0pt">Hi Xueming,</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">Yes, we need the fix.</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">It should be backporting to 23.11.x, since the previous commit “bus/vdev: fix devargs in secondary process” which is also backporting to 23.11.1 that cause bug
<a href="https://bugs.dpdk.org/show_bug.cgi?id=1450">1450 – Regression: vdev device not setup in secondary process (dpdk.org)</a> exists in 23.11 stable branch.</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">This bug’s resolution is the fix (<a href="http://git.dpdk.org/dpdk/commit/?id=22ce39b3aa">http://git.dpdk.org/dpdk/commit/?id=22ce39b3aa</a>) in main branch, so pls
</span><span style="color:black">schedule </span><span style="color:black">this fix patch</span><span style="color:black"> to 23.11.3</span><span style="color:black">.</span><span style="font-size:11.0pt"></span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">root@virtio-tester-217:~/jiangyu/LTS/dpdk# git show 6148604a43</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">commit 6148604a438aad21e5d74f414976666b370582c1</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">Author: Mingjin Ye <a href="mailto:mingjinx.ye@intel.com">
mingjinx.ye@intel.com</a></span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">Date:   Fri Sep 1 07:24:09 2023 +0000</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    bus/vdev: fix devargs in secondary process</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    [ upstream commit 6666628362c94a0b567a39a0177539c12c97d999 ]</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    When a device is created by a secondary process, an empty devargs is</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    temporarily generated and bound to it. This causes the device to not</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    be associated with the correct devargs, and the empty devargs are not</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    released when the resource is freed.</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    This patch fixes the issue by matching the devargs when inserting a</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    device in secondary process.</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    Fixes: dda987315ca2 ("vdev: make virtual bus use its device struct")</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    Fixes: a16040453968 ("eal: extract vdev infra")</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    Signed-off-by: Mingjin Ye
<a href="mailto:mingjinx.ye@intel.com">mingjinx.ye@intel.com</a></span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">    Acked-by: Anatoly Burakov
<a href="mailto:anatoly.burakov@intel.com">anatoly.burakov@intel.com</a></span></p>
<div>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">Best regards,</span></p>
<p class="x_MsoNormal"><span style="font-size:11.0pt">Yu Jiang</span></p>
</div>
<p class="x_MsoNormal"><span style="font-size:11.0pt"> </span></p>
<div style="border:none; border-left:solid blue 1.5pt; padding:0in 0in 0in 4.0pt">
<div>
<div style="border:none; border-top:solid #E1E1E1 1.0pt; padding:3.0pt 0in 0in 0in">
<p class="x_MsoNormal"><b><span style="font-size:11.0pt; font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt; font-family:"Calibri",sans-serif"> Xueming Li <xuemingl@nvidia.com>
<br>
<b>Sent:</b> Saturday, August 31, 2024 10:51 PM<br>
<b>To:</b> Ye, MingjinX <mingjinx.ye@intel.com>; Xu, HailinX <hailinx.xu@intel.com><br>
<b>Cc:</b> dpdk stable <stable@dpdk.org><br>
<b>Subject:</b> Re: patch 'bus/vdev: revert fix devargs in secondary process' has been queued to stable release 23.11.2</span></p>
</div>
</div>
<p class="x_MsoNormal"> </p>
<div>
<p class="x_MsoNormal"><span style="color:black">Hi Hailin,</span></p>
</div>
<div>
<p class="x_MsoNormal"><span style="color:black"> </span></p>
</div>
<div>
<p class="x_MsoNormal"><span style="color:black">This patch was originally queued to 23.11.2 list, but removed per Mingjin's request. If we do need the fix, let's schedule it to 23.11.3.</span></p>
</div>
<div>
<p class="x_MsoNormal"><span style="color:black"> </span></p>
</div>
<div>
<p class="x_MsoNormal"><span style="color:black">Regards,</span></p>
</div>
<div>
<p class="x_MsoNormal"><span style="color:black">Xueming</span></p>
</div>
<div class="x_MsoNormal" align="center" style="text-align:center">
<hr size="2" width="98%" align="center">
</div>
<div id="x_divRplyFwdMsg">
<p class="x_MsoNormal"><b><span style="font-size:11.0pt; font-family:"Calibri",sans-serif; color:black">From:</span></b><span style="font-size:11.0pt; font-family:"Calibri",sans-serif; color:black"> Ye, MingjinX <</span><a href="mailto:mingjinx.ye@intel.com"><span style="font-size:11.0pt; font-family:"Calibri",sans-serif">mingjinx.ye@intel.com</span></a><span style="font-size:11.0pt; font-family:"Calibri",sans-serif; color:black">><br>
<b>Sent:</b> Wednesday, July 17, 2024 10:21 AM<br>
<b>To:</b> Xueming Li <</span><a href="mailto:xuemingl@nvidia.com"><span style="font-size:11.0pt; font-family:"Calibri",sans-serif">xuemingl@nvidia.com</span></a><span style="font-size:11.0pt; font-family:"Calibri",sans-serif; color:black">><br>
<b>Cc:</b> dpdk stable <</span><a href="mailto:stable@dpdk.org"><span style="font-size:11.0pt; font-family:"Calibri",sans-serif">stable@dpdk.org</span></a><span style="font-size:11.0pt; font-family:"Calibri",sans-serif; color:black">><br>
<b>Subject:</b> RE: patch 'bus/vdev: revert fix devargs in secondary process' has been queued to stable release 23.11.2</span>
</p>
<div>
<p class="x_MsoNormal"> </p>
</div>
</div>
<div>
<div>
<p class="x_MsoNormal"><span style="font-size:11.0pt">Hi,<br>
<br>
Please not backport this revert commit. DPDK 23.11.2 does not need to do anything.<br>
<br>
Thanks,<br>
Mingjin<br>
<br>
> -----Original Message-----<br>
> From: Xueming Li <</span><a href="mailto:xuemingl@nvidia.com"><span style="font-size:11.0pt">xuemingl@nvidia.com</span></a><span style="font-size:11.0pt">><br>
> Sent: Friday, July 12, 2024 7:02 PM<br>
> To: Ye, MingjinX <</span><a href="mailto:mingjinx.ye@intel.com"><span style="font-size:11.0pt">mingjinx.ye@intel.com</span></a><span style="font-size:11.0pt">><br>
> Cc: dpdk stable <</span><a href="mailto:stable@dpdk.org"><span style="font-size:11.0pt">stable@dpdk.org</span></a><span style="font-size:11.0pt">><br>
> Subject: patch 'bus/vdev: revert fix devargs in secondary process' has been<br>
> queued to stable release 23.11.2<br>
> <br>
> Hi,<br>
> <br>
> FYI, your patch has been queued to stable release 23.11.2<br>
> <br>
> Note it hasn't been pushed to </span><a href="http://dpdk.org/browse/dpdk-stable"><span style="font-size:11.0pt">http://dpdk.org/browse/dpdk-stable</span></a><span style="font-size:11.0pt"> yet.<br>
> It will be pushed if I get no objections before 07/14/24. So please shout if<br>
> 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<br>
> 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>
> </span><a href="https://git.dpdk.org/dpdk-stable/log/?h=23.11-staging"><span style="font-size:11.0pt">https://git.dpdk.org/dpdk-stable/log/?h=23.11-staging</span></a><span style="font-size:11.0pt"><br>
> <br>
> This queued commit can be viewed at:<br>
> </span><a href="https://git.dpdk.org/dpdk-stable/commit/?h=23.11-"><span style="font-size:11.0pt">https://git.dpdk.org/dpdk-stable/commit/?h=23.11-</span></a><span style="font-size:11.0pt"><br>
> staging&id=8b75cfb86f90c816bdda28b8696beb18d553fd2b<br>
> <br>
> Thanks.<br>
> <br>
> Xueming Li <</span><a href="mailto:xuemingl@nvidia.com"><span style="font-size:11.0pt">xuemingl@nvidia.com</span></a><span style="font-size:11.0pt">><br>
> <br>
> ---<br>
> From 8b75cfb86f90c816bdda28b8696beb18d553fd2b Mon Sep 17 00:00:00<br>
> 2001<br>
> From: Mingjin Ye <</span><a href="mailto:mingjinx.ye@intel.com"><span style="font-size:11.0pt">mingjinx.ye@intel.com</span></a><span style="font-size:11.0pt">><br>
> Date: Thu, 14 Mar 2024 09:36:28 +0000<br>
> Subject: [PATCH] bus/vdev: revert fix devargs in secondary process<br>
> Cc: Xueming Li <</span><a href="mailto:xuemingl@nvidia.com"><span style="font-size:11.0pt">xuemingl@nvidia.com</span></a><span style="font-size:11.0pt">><br>
> <br>
> [ upstream commit 22ce39b3aadb109cbefabb91aad44c94e8c2a5e6 ]<br>
> <br>
> The ASan tool detected a memory leak in the vdev driver alloc_devargs. The<br>
> previous commit was that when inserting a vdev device, the primary process<br>
> alloc devargs and the secondary process looks for devargs. This causes the<br>
> device to not be created if the secondary process does not initialise the vdev<br>
> device. And, this is not the root cause.<br>
> <br>
> Therefore the following commit was reverted accordingly.<br>
> <br>
> After restoring this commit, the memory leak still exists.<br>
> <br>
> Bugzilla ID: 1450<br>
> Fixes: 6666628362c9 ("bus/vdev: fix devargs in secondary process")<br>
> <br>
> Signed-off-by: Mingjin Ye <</span><a href="mailto:mingjinx.ye@intel.com"><span style="font-size:11.0pt">mingjinx.ye@intel.com</span></a><span style="font-size:11.0pt">><br>
> ---<br>
>  drivers/bus/vdev/vdev.c | 21 +--------------------<br>
>  1 file changed, 1 insertion(+), 20 deletions(-)<br>
> <br>
> diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c index<br>
> 14cf856237..38d05a9fe9 100644<br>
> --- a/drivers/bus/vdev/vdev.c<br>
> +++ b/drivers/bus/vdev/vdev.c<br>
> @@ -263,22 +263,6 @@ alloc_devargs(const char *name, const char *args)<br>
>        return devargs;<br>
>  }<br>
> <br>
> -static struct rte_devargs *<br>
> -vdev_devargs_lookup(const char *name)<br>
> -{<br>
> -     struct rte_devargs *devargs;<br>
> -     char dev_name[32];<br>
> -<br>
> -     RTE_EAL_DEVARGS_FOREACH("vdev", devargs) {<br>
> -             devargs->bus->parse(devargs->name, &dev_name);<br>
> -             if (strcmp(dev_name, name) == 0) {<br>
> -                     VDEV_LOG(INFO, "devargs matched %s", dev_name);<br>
> -                     return devargs;<br>
> -             }<br>
> -     }<br>
> -     return NULL;<br>
> -}<br>
> -<br>
>  static int<br>
>  insert_vdev(const char *name, const char *args,<br>
>                struct rte_vdev_device **p_dev,<br>
> @@ -291,10 +275,7 @@ insert_vdev(const char *name, const char *args,<br>
>        if (name == NULL)<br>
>                return -EINVAL;<br>
> <br>
> -     if (rte_eal_process_type() == RTE_PROC_PRIMARY)<br>
> -             devargs = alloc_devargs(name, args);<br>
> -     else<br>
> -             devargs = vdev_devargs_lookup(name);<br>
> +     devargs = alloc_devargs(name, args);<br>
> <br>
>        if (!devargs)<br>
>                return -ENOMEM;<br>
> --<br>
> 2.34.1<br>
> <br>
> ---<br>
>   Diff of the applied patch vs upstream commit (please double-check if non-<br>
> empty:<br>
> ---<br>
> --- - 2024-07-12 18:40:18.570682691 +0800<br>
> +++ 0100-bus-vdev-revert-fix-devargs-in-secondary-process.patch       2024-<br>
> 07-12 18:40:14.356594197 +0800<br>
> @@ -1 +1 @@<br>
> -From 22ce39b3aadb109cbefabb91aad44c94e8c2a5e6 Mon Sep 17 00:00:00<br>
> 2001<br>
> +From 8b75cfb86f90c816bdda28b8696beb18d553fd2b Mon Sep 17 00:00:00<br>
> 2001<br>
> @@ -4,0 +5,3 @@<br>
> +Cc: Xueming Li <</span><a href="mailto:xuemingl@nvidia.com"><span style="font-size:11.0pt">xuemingl@nvidia.com</span></a><span style="font-size:11.0pt">><br>
> +<br>
> +[ upstream commit 22ce39b3aadb109cbefabb91aad44c94e8c2a5e6 ]<br>
> @@ -20 +22,0 @@<br>
> -Cc: </span><a href="mailto:stable@dpdk.org"><span style="font-size:11.0pt">stable@dpdk.org</span></a><span style="font-size:11.0pt"></span></p>
</div>
</div>
</div>
</div>
</div>
</body>
</html>