DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] doc: add PMD known issue
@ 2023-04-20  6:14 Mingjin Ye
  2023-04-20 14:59 ` Stephen Hemminger
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Mingjin Ye @ 2023-04-20  6:14 UTC (permalink / raw)
  To: dev; +Cc: qiming.yang, yidingx.zhou, Mingjin Ye, Wenjun Wu

Add a known issue: ASLR feature causes core dump.

Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
---
 doc/guides/nics/ixgbe.rst | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
index b1d77ab7ab..c346e377e2 100644
--- a/doc/guides/nics/ixgbe.rst
+++ b/doc/guides/nics/ixgbe.rst
@@ -461,3 +461,18 @@ show bypass config
 Show the bypass configuration for a bypass enabled NIC using the lowest port on the NIC::
 
    testpmd> show bypass config (port_id)
+
+ASLR feature causes core dump
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+Core dump may occur when we start secondary processes on the vf port.
+Mainstream Linux distributions have the ASLR feature enabled by default,
+and the text segment of the process's memory space is randomized.
+The secondary process calls the function address shared by the primary
+process, resulting in a core dump.
+
+   .. Note::
+
+	  Support for ASLR features varies by distribution. Redhat and
+	  Centos series distributions work fine. Ubuntu distributions
+	  will core dump, other Linux distributions are unknown.
-- 
2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] doc: add PMD known issue
  2023-04-20  6:14 [PATCH] doc: add PMD known issue Mingjin Ye
@ 2023-04-20 14:59 ` Stephen Hemminger
  2023-04-20 15:22 ` Bruce Richardson
  2023-05-06 10:02 ` [PATCH v2] " Mingjin Ye
  2 siblings, 0 replies; 10+ messages in thread
From: Stephen Hemminger @ 2023-04-20 14:59 UTC (permalink / raw)
  To: Mingjin Ye; +Cc: dev, qiming.yang, yidingx.zhou, Wenjun Wu

On Thu, 20 Apr 2023 06:14:29 +0000
Mingjin Ye <mingjinx.ye@intel.com> wrote:

> Add a known issue: ASLR feature causes core dump.
> 
> Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
> ---

Please provide back trace.
This should be fixable.
Fixing a bug is always better than documenting it.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] doc: add PMD known issue
  2023-04-20  6:14 [PATCH] doc: add PMD known issue Mingjin Ye
  2023-04-20 14:59 ` Stephen Hemminger
@ 2023-04-20 15:22 ` Bruce Richardson
  2023-05-06 10:02 ` [PATCH v2] " Mingjin Ye
  2 siblings, 0 replies; 10+ messages in thread
From: Bruce Richardson @ 2023-04-20 15:22 UTC (permalink / raw)
  To: Mingjin Ye; +Cc: dev, qiming.yang, yidingx.zhou, Wenjun Wu

On Thu, Apr 20, 2023 at 06:14:29AM +0000, Mingjin Ye wrote:
> Add a known issue: ASLR feature causes core dump.
> 
> Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
> ---
>  doc/guides/nics/ixgbe.rst | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
> index b1d77ab7ab..c346e377e2 100644
> --- a/doc/guides/nics/ixgbe.rst
> +++ b/doc/guides/nics/ixgbe.rst
> @@ -461,3 +461,18 @@ show bypass config
>  Show the bypass configuration for a bypass enabled NIC using the lowest port on the NIC::
>  
>     testpmd> show bypass config (port_id)
> +
> +ASLR feature causes core dump
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +Core dump may occur when we start secondary processes on the vf port.
> +Mainstream Linux distributions have the ASLR feature enabled by default,
> +and the text segment of the process's memory space is randomized.
> +The secondary process calls the function address shared by the primary
> +process, resulting in a core dump.
> +
> +   .. Note::
> +
> +	  Support for ASLR features varies by distribution. Redhat and
> +	  Centos series distributions work fine. Ubuntu distributions
> +	  will core dump, other Linux distributions are unknown.
> -- 

I disagree about this description of the bug. ASLR is not the problem;
instead driver is just not multi-process aware and uses the same pointers
in both primary and secondary processes. You will hit this issue even
without ASLR if primary and secondary processes use different static
binaries. Therefore, IMHO, title should be that the VF driver is not
multi-process safe, rather than pinning the blame on ASLR.

/Bruce

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v2] doc: add PMD known issue
  2023-04-20  6:14 [PATCH] doc: add PMD known issue Mingjin Ye
  2023-04-20 14:59 ` Stephen Hemminger
  2023-04-20 15:22 ` Bruce Richardson
@ 2023-05-06 10:02 ` Mingjin Ye
  2023-05-15  5:56   ` Zhang, Qi Z
  2023-05-16 10:22   ` [PATCH v3] doc: comment VF does not support multi-process Mingjin Ye
  2 siblings, 2 replies; 10+ messages in thread
From: Mingjin Ye @ 2023-05-06 10:02 UTC (permalink / raw)
  To: dev; +Cc: qiming.yang, stable, yidingx.zhou, Mingjin Ye, Wenjun Wu

Add a known issue: The ixgbe_vf driver is not multi-process safe.

Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
---
v2: Modify issue description reason.
---
 doc/guides/nics/ixgbe.rst | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
index b1d77ab7ab..9fb3c39bf4 100644
--- a/doc/guides/nics/ixgbe.rst
+++ b/doc/guides/nics/ixgbe.rst
@@ -461,3 +461,18 @@ show bypass config
 Show the bypass configuration for a bypass enabled NIC using the lowest port on the NIC::
 
    testpmd> show bypass config (port_id)
+
+VF driver is not multi-process safe
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+Core dump may occur when we start secondary processes on the VF port.
+Mainstream Linux distributions have the ASLR feature enabled by default,
+and the text segment of the process memory space is randomized.
+The secondary process calls the function address shared by the primary
+process, resulting in a core dump.
+
+   .. Note::
+
+          Support for ASLR features varies by distribution. Redhat and
+          Centos series distributions work fine. Ubuntu distributions
+          will core dump, other Linux distributions are unknown.
-- 
2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* RE: [PATCH v2] doc: add PMD known issue
  2023-05-06 10:02 ` [PATCH v2] " Mingjin Ye
@ 2023-05-15  5:56   ` Zhang, Qi Z
  2023-05-16 10:22   ` [PATCH v3] doc: comment VF does not support multi-process Mingjin Ye
  1 sibling, 0 replies; 10+ messages in thread
From: Zhang, Qi Z @ 2023-05-15  5:56 UTC (permalink / raw)
  To: Ye, MingjinX, dev
  Cc: Yang, Qiming, stable, Zhou, YidingX, Ye, MingjinX, Wu, Wenjun1



> -----Original Message-----
> From: Mingjin Ye <mingjinx.ye@intel.com>
> Sent: Saturday, May 6, 2023 6:03 PM
> To: dev@dpdk.org
> Cc: Yang, Qiming <qiming.yang@intel.com>; stable@dpdk.org; Zhou, YidingX
> <yidingx.zhou@intel.com>; Ye, MingjinX <mingjinx.ye@intel.com>; Wu,
> Wenjun1 <wenjun1.wu@intel.com>
> Subject: [PATCH v2] doc: add PMD known issue
> 
> Add a known issue: The ixgbe_vf driver is not multi-process safe.
> 
> Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
> ---
> v2: Modify issue description reason.
> ---
>  doc/guides/nics/ixgbe.rst | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst index
> b1d77ab7ab..9fb3c39bf4 100644
> --- a/doc/guides/nics/ixgbe.rst
> +++ b/doc/guides/nics/ixgbe.rst
> @@ -461,3 +461,18 @@ show bypass config
>  Show the bypass configuration for a bypass enabled NIC using the lowest
> port on the NIC::
> 
>     testpmd> show bypass config (port_id)
> +
> +VF driver is not multi-process safe
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +Core dump may occur when we start secondary processes on the VF port.
> +Mainstream Linux distributions have the ASLR feature enabled by
> +default, and the text segment of the process memory space is randomized.
> +The secondary process calls the function address shared by the primary
> +process, resulting in a core dump.

Which function did we encounter the issue with? Is this a bug in the PMD? In a multi-process scenario, we should not assume that the addresses of functions are identical across different processes. Additionally, this issue should not be related ASLR because the primary and secondary processes could be two different applications but still share the same DPDK library.

> +
> +   .. Note::
> +
> +          Support for ASLR features varies by distribution. Redhat and
> +          Centos series distributions work fine. Ubuntu distributions
> +          will core dump, other Linux distributions are unknown.
> --
> 2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v3] doc: comment VF does not support multi-process
  2023-05-06 10:02 ` [PATCH v2] " Mingjin Ye
  2023-05-15  5:56   ` Zhang, Qi Z
@ 2023-05-16 10:22   ` Mingjin Ye
  2023-05-16 15:18     ` Stephen Hemminger
  2023-05-17 10:03     ` [PATCH v4] doc: update ixgbe VF features list Mingjin Ye
  1 sibling, 2 replies; 10+ messages in thread
From: Mingjin Ye @ 2023-05-16 10:22 UTC (permalink / raw)
  To: dev; +Cc: qiming.yang, stable, yidingx.zhou, Mingjin Ye, Wenjun Wu

Announcing that multi-process is not supported

Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
---
v2: Modify issue description reason.
---
V3: Modify description.
---
 doc/guides/nics/ixgbe.rst | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
index b1d77ab7ab..816d614c33 100644
--- a/doc/guides/nics/ixgbe.rst
+++ b/doc/guides/nics/ixgbe.rst
@@ -461,3 +461,9 @@ show bypass config
 Show the bypass configuration for a bypass enabled NIC using the lowest port on the NIC::
 
    testpmd> show bypass config (port_id)
+
+VF driver does not support multi-process
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+The VF driver does not support multi-process. And some function pointers
+in the case of multi-process are not set correctly.
-- 
2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v3] doc: comment VF does not support multi-process
  2023-05-16 10:22   ` [PATCH v3] doc: comment VF does not support multi-process Mingjin Ye
@ 2023-05-16 15:18     ` Stephen Hemminger
  2023-05-17  2:29       ` Ye, MingjinX
  2023-05-17 10:03     ` [PATCH v4] doc: update ixgbe VF features list Mingjin Ye
  1 sibling, 1 reply; 10+ messages in thread
From: Stephen Hemminger @ 2023-05-16 15:18 UTC (permalink / raw)
  To: Mingjin Ye; +Cc: dev, qiming.yang, stable, yidingx.zhou, Wenjun Wu

On Tue, 16 May 2023 10:22:05 +0000
Mingjin Ye <mingjinx.ye@intel.com> wrote:

> Announcing that multi-process is not supported
> 
> Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
> ---
> v2: Modify issue description reason.
> ---
> V3: Modify description.
> ---
>  doc/guides/nics/ixgbe.rst | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
> index b1d77ab7ab..816d614c33 100644
> --- a/doc/guides/nics/ixgbe.rst
> +++ b/doc/guides/nics/ixgbe.rst
> @@ -461,3 +461,9 @@ show bypass config
>  Show the bypass configuration for a bypass enabled NIC using the lowest port on the NIC::
>  
>     testpmd> show bypass config (port_id)  
> +
> +VF driver does not support multi-process
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +The VF driver does not support multi-process. And some function pointers
> +in the case of multi-process are not set correctly.

This is best done by updating doc/guides/nic/features/ixgbe_vf.ini

That is enough. Don't need to add note in ixgbe.rst.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* RE: [PATCH v3] doc: comment VF does not support multi-process
  2023-05-16 15:18     ` Stephen Hemminger
@ 2023-05-17  2:29       ` Ye, MingjinX
  0 siblings, 0 replies; 10+ messages in thread
From: Ye, MingjinX @ 2023-05-17  2:29 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: dev, Yang, Qiming, stable, Zhou, YidingX, Wu, Wenjun1



> -----Original Message-----
> From: Stephen Hemminger <stephen@networkplumber.org>
> Sent: 2023年5月16日 23:18
> To: Ye, MingjinX <mingjinx.ye@intel.com>
> Cc: dev@dpdk.org; Yang, Qiming <qiming.yang@intel.com>;
> stable@dpdk.org; Zhou, YidingX <yidingx.zhou@intel.com>; Wu, Wenjun1
> <wenjun1.wu@intel.com>
> Subject: Re: [PATCH v3] doc: comment VF does not support multi-process
> 
> On Tue, 16 May 2023 10:22:05 +0000
> Mingjin Ye <mingjinx.ye@intel.com> wrote:
> 
> > Announcing that multi-process is not supported
> >
> > Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
> > ---
> > v2: Modify issue description reason.
> > ---
> > V3: Modify description.
> > ---
> >  doc/guides/nics/ixgbe.rst | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
> > index b1d77ab7ab..816d614c33 100644
> > --- a/doc/guides/nics/ixgbe.rst
> > +++ b/doc/guides/nics/ixgbe.rst
> > @@ -461,3 +461,9 @@ show bypass config  Show the bypass configuration
> > for a bypass enabled NIC using the lowest port on the NIC::
> >
> >     testpmd> show bypass config (port_id)
> > +
> > +VF driver does not support multi-process
> > +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > +
> > +The VF driver does not support multi-process. And some function
> > +pointers in the case of multi-process are not set correctly.
> 
> This is best done by updating doc/guides/nic/features/ixgbe_vf.ini
> 
> That is enough. Don't need to add note in ixgbe.rst.

I will send the v4 version as soon as possible with a better solution.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v4] doc: update ixgbe VF features list
  2023-05-16 10:22   ` [PATCH v3] doc: comment VF does not support multi-process Mingjin Ye
  2023-05-16 15:18     ` Stephen Hemminger
@ 2023-05-17 10:03     ` Mingjin Ye
  2023-05-18  8:25       ` Zhang, Qi Z
  1 sibling, 1 reply; 10+ messages in thread
From: Mingjin Ye @ 2023-05-17 10:03 UTC (permalink / raw)
  To: dev; +Cc: qiming.yang, stable, yidingx.zhou, Mingjin Ye, Wenjun Wu

The ixgbe VF driver is not multi-process aware. Some function pointers are
not set correctly in multi-process situations.

Remove the multiprocess aware feature from the features list.

Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>
---
v2: Modify issue description reason.
---
V3: Modify description.
---
V4: Removed the multiprocess aware feature.
---
 doc/guides/nics/features/ixgbe_vf.ini | 1 -
 1 file changed, 1 deletion(-)

diff --git a/doc/guides/nics/features/ixgbe_vf.ini b/doc/guides/nics/features/ixgbe_vf.ini
index e14325045c..299ccd4b61 100644
--- a/doc/guides/nics/features/ixgbe_vf.ini
+++ b/doc/guides/nics/features/ixgbe_vf.ini
@@ -32,7 +32,6 @@ Tx descriptor status = Y
 Basic stats          = Y
 Extended stats       = Y
 Registers dump       = Y
-Multiprocess aware   = Y
 FreeBSD              = Y
 Linux                = Y
 Windows              = Y
-- 
2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* RE: [PATCH v4] doc: update ixgbe VF features list
  2023-05-17 10:03     ` [PATCH v4] doc: update ixgbe VF features list Mingjin Ye
@ 2023-05-18  8:25       ` Zhang, Qi Z
  0 siblings, 0 replies; 10+ messages in thread
From: Zhang, Qi Z @ 2023-05-18  8:25 UTC (permalink / raw)
  To: Ye, MingjinX, dev
  Cc: Yang, Qiming, stable, Zhou, YidingX, Ye, MingjinX, Wu, Wenjun1



> -----Original Message-----
> From: Mingjin Ye <mingjinx.ye@intel.com>
> Sent: Wednesday, May 17, 2023 6:04 PM
> To: dev@dpdk.org
> Cc: Yang, Qiming <qiming.yang@intel.com>; stable@dpdk.org; Zhou, YidingX
> <yidingx.zhou@intel.com>; Ye, MingjinX <mingjinx.ye@intel.com>; Wu,
> Wenjun1 <wenjun1.wu@intel.com>
> Subject: [PATCH v4] doc: update ixgbe VF features list
> 
> The ixgbe VF driver is not multi-process aware. Some function pointers are
> not set correctly in multi-process situations.
> 
> Remove the multiprocess aware feature from the features list.
> 
> Signed-off-by: Mingjin Ye <mingjinx.ye@intel.com>

Acked-by: Qi Zhang <qi.z.zhang@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2023-05-18  8:25 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-20  6:14 [PATCH] doc: add PMD known issue Mingjin Ye
2023-04-20 14:59 ` Stephen Hemminger
2023-04-20 15:22 ` Bruce Richardson
2023-05-06 10:02 ` [PATCH v2] " Mingjin Ye
2023-05-15  5:56   ` Zhang, Qi Z
2023-05-16 10:22   ` [PATCH v3] doc: comment VF does not support multi-process Mingjin Ye
2023-05-16 15:18     ` Stephen Hemminger
2023-05-17  2:29       ` Ye, MingjinX
2023-05-17 10:03     ` [PATCH v4] doc: update ixgbe VF features list Mingjin Ye
2023-05-18  8:25       ` Zhang, Qi Z

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).